Showing 1,992 of 18,382 total issues
Method x_get_tree_region_kids
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def x_get_tree_region_kids(object, count_only)
emses = Rbac.filtered(object.ems_infras)
storages = Rbac.filtered(object.storages)
if count_only
emses.count + storages.count
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method set_display
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def set_display
@display = params[:display]
@display ||= default_display unless pagination_or_gtl_request?
@display ||= 'generic_objects' if role_allows?(:feature => "generic_object_view") && @record.number_of(:generic_objects).positive?
end
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method move_menu_cols_top
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def move_menu_cols_top
if params[:selected_reports].blank? || params[:selected_reports][0] == ""
add_flash(_("No fields were selected to move up") % "", :error)
return
end
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method x_get_tree_kids
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def x_get_tree_kids(parent, count_only, parents)
generator = self.class.kids_generators.detect { |k, v| v if parent.kind_of?(k) }
return nil unless generator
method = generator[1][0]
attributes = generator[1][1..-1].collect do |attribute_name|
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method retrieve_email
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def retrieve_email
assert_privileges("miq_request_edit")
@edit = session[:edit]
begin
method = WORKFLOW_METHOD_WHITELIST[params[:field]]
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method move_menu_cols_up
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def move_menu_cols_up
if params[:selected_reports].blank? || params[:selected_reports][0] == ""
add_flash(_("No fields were selected to move up"), :error)
return
end
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method x_get_tree_hash_kids
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def x_get_tree_hash_kids(parent, count_only)
kids = []
kids.concat(profile_nodes(parent[:profiles])) if parent[:profiles].present?
kids.concat(policy_nodes(parent[:policies])) if parent[:policies].present?
kids.concat(condition_nodes(parent[:conditions])) if parent[:conditions].present?
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method request_edit
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def request_edit
assert_privileges(rbac_feature_id("miq_request_edit"))
provision_request = MiqRequest.find(params[:id])
if provision_request.workflow_class || provision_request.kind_of?(MiqProvisionConfiguredSystemRequest)
request_edit_settings(provision_request)
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method x_get_compliance_kids
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def x_get_compliance_kids(parent, count_only)
kids = []
if parent.compliance_details.empty?
kid = {:id => "#{parent.id}-nopol",
:text => _("No Compliance Policies Found"),
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method find_chart_path
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def find_chart_path
if @fname
# First, if this class specified a chart layout path, use it.
return chart_path(@layout, @perf_record.chart_layout_path) if
@perf_record.respond_to?(:chart_layout_path)
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method validate_user_pre_auth_checks
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def validate_user_pre_auth_checks(user)
# Pre_authenticate checks
return ValidateResult.new(:fail, _("Error: Name is required")) if user.blank? || user[:name].blank?
return ValidateResult.new(:fail, _("Error: New password and verify password must be the same")) if
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method x_build_tree
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def x_build_tree
nodes = x_get_tree_objects(nil, false, []).map do |child|
# already a node? FIXME: make a class for node
if child.kind_of?(Hash) && child.key?(:text) && child.key?(:key) && child.key?(:image)
child
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"