Showing 26 of 32 total issues
File mootiro_importer.rb
has 343 lines of code (exceeds 250 allowed). Consider refactoring. Open
module MootiroImporter
module_function
REDIS_HOST = ENV['MOOTIRO_REDIS_HOST'] || '10.0.2.2'
Method select_tools
has a Cognitive Complexity of 22 (exceeds 5 allowed). Consider refactoring. Open
def select_tools
case type
when 'user'
current_user == object ? [:edit, :settings] : [:star, :flag]
when 'geo_data'
- Read upRead up
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 cleaned_layers_attributes
has a Cognitive Complexity of 21 (exceeds 5 allowed). Consider refactoring. Open
def cleaned_layers_attributes(_params)
_attrs = JSON.parse _params[:layers_attributes]
_attrs.map do |layer|
OpenStruct.new(
id: layer['id'].blank? ? nil : layer['id'].to_i,
- Read upRead up
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 build_map
has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring. Open
def build_map(d, &blk)
Rails.logger.debug "Building Map... "
additional_info = {}
additional_info.merge! short_description: d[:short_description] unless d[:short_description].blank?
- Read upRead up
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 build_user
has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring. Open
def build_user(d, &blk)
Rails.logger.debug "Building User... "
user = User.new(
name: d[:name],
email: d[:email],
- Read upRead up
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 testimonials
has 38 lines of code (exceeds 25 allowed). Consider refactoring. Open
def testimonials
# MOCKED
[
{
:name => 'aldeias-infantis',
Method build_map
has 33 lines of code (exceeds 25 allowed). Consider refactoring. Open
def build_map(d, &blk)
Rails.logger.debug "Building Map... "
additional_info = {}
additional_info.merge! short_description: d[:short_description] unless d[:short_description].blank?
Method use_cases
has 32 lines of code (exceeds 25 allowed). Consider refactoring. Open
def use_cases
# MOCKED
[
{
:href => '#use_cases#jornalismo_de_dados',
Method remove_geometrycollection
has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring. Open
def remove_geometrycollection(location)
Rails.logger.debug "Removing GeometryCollection... "
return nil if location.eql? 'EMPTY GEOMETRYCOLLECTION'
return location unless location.include? "GEOMETRYCOLLECTION"
- Read upRead up
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 remove_geometrycollection
has 28 lines of code (exceeds 25 allowed). Consider refactoring. Open
def remove_geometrycollection(location)
Rails.logger.debug "Removing GeometryCollection... "
return nil if location.eql? 'EMPTY GEOMETRYCOLLECTION'
return location unless location.include? "GEOMETRYCOLLECTION"
Method cleaned_relation_metadata
has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring. Open
def cleaned_relation_metadata(m)
OpenStruct.new(
description: m['description'].blank? ? nil : m['description'],
start_date: m['start_date'].blank? ? nil : Date.parse(m['start_date']),
end_date: m['end_date'].blank? ? nil : Date.parse(m['end_date']),
- Read upRead up
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 build_relation
has 26 lines of code (exceeds 25 allowed). Consider refactoring. Open
def build_relation(d, &blk)
Rails.logger.debug "Building Relation... "
model1 = model_from_oid(d[:oid_1])
model2 = model_from_oid(d[:oid_2])
return false unless model1 && model2
Method build_geo_data
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
def build_geo_data(d, &blk)
Rails.logger.debug "Building GeoData... "
additional_info = {}
additional_info.merge! short_description: d[:short_description] unless d[:short_description].blank?
additional_info.merge! creator: d[:creator] unless d[:creator].blank?
- Read upRead up
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 controls
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
def controls
case control_type_
when :follow_button
return ctx.render 'shared/follow_button', object: object unless object == ctx.current_user
when :remove_button
- Read upRead up
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 selector_option
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def selector_option(label, param, value, default=false, class_name=nil)
Method relations_values
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
def relations_values(splitted_type=false)
relations_with_relateds.map do |r|
return if r.related.nil?
rel = r.relation
{
- Read upRead up
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 parse
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
def parse(geojson, opts={})
return nil if geojson.nil? || geojson.try(:empty?)
geojson_hash = geojson.is_a?(Hash) ? geojson : JSON.parse(geojson)
decoded_geojson = RGeo::GeoJSON.decode(geojson_hash)
decoded_geojson = decoded_geojson.first if decoded_geojson.is_a? Enumerable
- Read upRead up
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
Avoid too many return
statements within this method. Open
return factory.multi_point(points).as_text if points.size > 0
Avoid too many return
statements within this method. Open
return factory.multi_polygon(polygons).as_text if polygons.size > 0
Avoid too many return
statements within this method. Open
return factory.multi_line_string(lines).as_text if lines.size > 0