Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- AccountController:
- Rating: A
- Churn: 0
- Complexity: 5.95
- Duplication: 0
- Smells: 2
- * (InstanceVariableAssumption) AccountController assumes too much for instance variable '@title'
- - app/controllers/account_controller.rb:1
- * (IrresponsibleModule) AccountController has no descriptive comment
- - app/controllers/account_controller.rb:1
- ActiveModel::OneTimePassword::ClassMethods:
- Rating: A
- Churn: 0
- Complexity: 28.71
- Duplication: 0
- Smells: 4
- * (HighComplexity) ActiveModel::OneTimePassword::ClassMethods#has_one_time_password has a flog score of 25
- - lib/modules/active_model_otp.rb:3
- * (IrresponsibleModule) ActiveModel::OneTimePassword::ClassMethods has no descriptive comment
- - lib/modules/active_model_otp.rb:2
- * (ManualDispatch) ActiveModel::OneTimePassword::ClassMethods#has_one_time_password manually dispatches method call
- - lib/modules/active_model_otp.rb:22
- * (TooManyStatements) ActiveModel::OneTimePassword::ClassMethods#has_one_time_password has approx 11 statements
- - lib/modules/active_model_otp.rb:3
- AddAddons:
- Rating: A
- Churn: 0
- Complexity: 6.68
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) AddAddons has no descriptive comment
- - db/migrate/20170918132352_add_addons.rb:1
- * (TooManyStatements) AddAddons#change has approx 6 statements
- - db/migrate/20170918132352_add_addons.rb:2
- * (UncommunicativeVariableName) AddAddons#change has the variable name 't'
- - db/migrate/20170918132352_add_addons.rb:10
- AddAdminToUsers:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddAdminToUsers has no descriptive comment
- - db/migrate/20180205181042_add_admin_to_users.rb:1
- AddBrandAndCurrencyToUser:
- Rating: A
- Churn: 0
- Complexity: 2.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddBrandAndCurrencyToUser has no descriptive comment
- - db/migrate/20161122224539_add_brand_and_currency_to_user.rb:1
- AddBrandIdToProducts:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddBrandIdToProducts has no descriptive comment
- - db/migrate/20160630205805_add_brand_id_to_products.rb:1
- AddColumnsToUser:
- Rating: A
- Churn: 0
- Complexity: 10.0
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) AddColumnsToUser has no descriptive comment
- - db/migrate/20161127225505_add_columns_to_user.rb:1
- * (TooManyStatements) AddColumnsToUser#change has approx 10 statements
- - db/migrate/20161127225505_add_columns_to_user.rb:2
- AddCompanyPaymentMethod:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddCompanyPaymentMethod has no descriptive comment
- - db/migrate/20170419100623_add_company_payment_method.rb:1
- AddContactsToUser:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddContactsToUser has no descriptive comment
- - db/migrate/20161127230454_add_contacts_to_user.rb:1
- AddDescriptionsToUpgradeItems:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddDescriptionsToUpgradeItems has no descriptive comment
- - db/migrate/20180514114555_add_descriptions_to_upgrade_items.rb:1
- AddFirstNameToUsers:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddFirstNameToUsers has no descriptive comment
- - db/migrate/20161127221330_add_first_name_to_users.rb:1
- AddInfoToUpgrades:
- Rating: A
- Churn: 0
- Complexity: 2.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddInfoToUpgrades has no descriptive comment
- - db/migrate/20180228164334_add_info_to_upgrades.rb:1
- AddInvoiceIdToChartMogulDeadLetter:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddInvoiceIdToChartMogulDeadLetter has no descriptive comment
- - db/migrate/20180809192525_add_invoice_id_to_chart_mogul_dead_letter.rb:1
- AddLangToUsers:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddLangToUsers has no descriptive comment
- - db/migrate/20161224143312_add_lang_to_users.rb:1
- AddMetadataToOrderItem:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddMetadataToOrderItem has no descriptive comment
- - db/migrate/20170731150623_add_metadata_to_order_item.rb:1
- AddOrderCoupon:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddOrderCoupon has no descriptive comment
- - db/migrate/20170119212945_add_order_coupon.rb:1
- AddOrderItemDiscountValue:
- Rating: A
- Churn: 0
- Complexity: 1.6
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddOrderItemDiscountValue has no descriptive comment
- - db/migrate/20170126223119_add_order_item_discount_value.rb:1
- AddOtpSecretKeyToUsers:
- Rating: A
- Churn: 0
- Complexity: 2.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddOtpSecretKeyToUsers has no descriptive comment
- - db/migrate/20170817024114_add_otp_secret_key_to_users.rb:1
- AddSetupToProduct:
- Rating: A
- Churn: 0
- Complexity: 1.6
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddSetupToProduct has no descriptive comment
- - db/migrate/20171009153119_add_setup_to_product.rb:1
- AddSignupStepToCompanies:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddSignupStepToCompanies has no descriptive comment
- - db/migrate/20181029131555_add_signup_step_to_companies.rb:1
- AddUpgradeItemGroupsNames:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradeItemGroupsNames has no descriptive comment
- - db/migrate/20160802204436_add_upgrade_item_groups_names.rb:1
- AddUpgradeItemOrder:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradeItemOrder has no descriptive comment
- - db/migrate/20170320145811_add_upgrade_item_order.rb:1
- AddUpgradeItemPrices:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradeItemPrices has no descriptive comment
- - db/migrate/20160711215135_add_upgrade_item_prices.rb:1
- AddUpgradeItemsNames:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradeItemsNames has no descriptive comment
- - db/migrate/20160802193255_add_upgrade_items_names.rb:1
- AddUpgradeItemsToOrderItems:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradeItemsToOrderItems has no descriptive comment
- - db/migrate/20160615201311_add_upgrade_items_to_order_items.rb:1
- AddUpgradeNames:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradeNames has no descriptive comment
- - db/migrate/20160728200611_add_upgrade_names.rb:1
- AddUpgradePugNames:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradePugNames has no descriptive comment
- - db/migrate/20160817223016_add_upgrade_pug_names.rb:1
- AddUpgradesToProducts:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUpgradesToProducts has no descriptive comment
- - db/migrate/20160605215905_add_upgrades_to_products.rb:1
- AddUserClientId:
- Rating: A
- Churn: 0
- Complexity: 2.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AddUserClientId has no descriptive comment
- - db/migrate/20161116172008_add_user_client_id.rb:1
- AlertComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) AlertComponent has no descriptive comment
- - frontend/components/alert/alert_component.rb:3
- AnalyticsHelper:
- Rating: A
- Churn: 0
- Complexity: 43.78
- Duplication: 0
- Smells: 7
- * (DuplicateMethodCall) AnalyticsHelper#group_traits calls 'user.company' 7 times
- - app/helpers/analytics_helper.rb:40
- - app/helpers/analytics_helper.rb:43
- - app/helpers/analytics_helper.rb:44
- - app/helpers/analytics_helper.rb:45
- - app/helpers/analytics_helper.rb:46
- - app/helpers/analytics_helper.rb:47
- - app/helpers/analytics_helper.rb:48
- * (FeatureEnvy) AnalyticsHelper#segment_group refers to 'user' more than self (maybe move it to another class?)
- - app/helpers/analytics_helper.rb:11
- - app/helpers/analytics_helper.rb:12
- * (FeatureEnvy) AnalyticsHelper#segment_identify refers to 'user' more than self (maybe move it to another class?)
- - app/helpers/analytics_helper.rb:2
- - app/helpers/analytics_helper.rb:4
- * (FeatureEnvy) AnalyticsHelper#track_event refers to 'user' more than self (maybe move it to another class?)
- - app/helpers/analytics_helper.rb:17
- - app/helpers/analytics_helper.rb:19
- * (IrresponsibleModule) AnalyticsHelper has no descriptive comment
- - app/helpers/analytics_helper.rb:1
- * (UtilityFunction) AnalyticsHelper#group_traits doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/analytics_helper.rb:38
- * (UtilityFunction) AnalyticsHelper#traits doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/analytics_helper.rb:27
- AnalyticsRuby:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ApplicationCable::Channel:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ApplicationCable::Channel has no descriptive comment
- - app/channels/application_cable/channel.rb:2
- ApplicationCable::Connection:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ApplicationCable::Connection has no descriptive comment
- - app/channels/application_cable/connection.rb:2
- ApplicationJob:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ApplicationJob has no descriptive comment
- - app/jobs/application_job.rb:1
- ApplicationMailer:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ApplicationMailer has no descriptive comment
- - app/mailers/application_mailer.rb:1
- ApplicationRecord:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ApplicationRecord has no descriptive comment
- - app/models/application_record.rb:1
- Assets:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Asyncs::BaseController:
- Rating: A
- Churn: 0
- Complexity: 3.6
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) Asyncs::BaseController has no descriptive comment
- - app/controllers/asyncs/base_controller.rb:1
- * (UnusedParameters) Asyncs::BaseController#default_render has unused parameter 'args'
- - app/controllers/asyncs/base_controller.rb:5
- Auth0:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Auth0Controller:
- Rating: A
- Churn: 0
- Complexity: 26.2
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) Auth0Controller has no descriptive comment
- - app/controllers/auth0_controller.rb:1
- Auth0ControllerSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- AvatarComponent:
- Rating: A
- Churn: 0
- Complexity: 13.05
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) AvatarComponent has no descriptive comment
- - frontend/components/avatar/avatar_component.rb:3
- * (TooManyStatements) AvatarComponent#colorize has approx 12 statements
- - frontend/components/avatar/avatar_component.rb:11
- * (UtilityFunction) AvatarComponent#colorize doesn't depend on instance state (maybe move it to another class?)
- - frontend/components/avatar/avatar_component.rb:11
- BadgeComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) BadgeComponent has no descriptive comment
- - frontend/components/badge/badge_component.rb:3
- BillingOrdersHelper:
- Rating: A
- Churn: 0
- Complexity: 13.23
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) BillingOrdersHelper has no descriptive comment
- - app/helpers/billing_orders_helper.rb:1
- * (UtilityFunction) BillingOrdersHelper#formatted_ordered_servers doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/billing_orders_helper.rb:2
- BillingRoutingSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- BlankStateComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) BlankStateComponent has no descriptive comment
- - frontend/components/blank_state/blank_state_component.rb:3
- Boot:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- BoxComponent:
- Rating: A
- Churn: 0
- Complexity: 5.81
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) BoxComponent has no descriptive comment
- - frontend/components/box/box_component.rb:3
- * (UtilityFunction) BoxComponent#attached? doesn't depend on instance state (maybe move it to another class?)
- - frontend/components/box/box_component.rb:36
- Bugsnag:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ButtonComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ButtonComponent has no descriptive comment
- - frontend/components/button/button_component.rb:3
- ChangeContactsColumnType:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ChangeContactsColumnType has no descriptive comment
- - db/migrate/20161227043853_change_contacts_column_type.rb:1
- ChartMogul::Client:
- Rating: A
- Churn: 0
- Complexity: 34.76
- Duplication: 0
- Smells: 3
- * (DuplicateMethodCall) ChartMogul::Client#parse_response calls 'req.body' 2 times
- - app/adapters/chart_mogul/client.rb:52
- - app/adapters/chart_mogul/client.rb:54
- * (IrresponsibleModule) ChartMogul::Client has no descriptive comment
- - app/adapters/chart_mogul/client.rb:2
- * (TooManyStatements) ChartMogul::Client#build_api_request has approx 6 statements
- - app/adapters/chart_mogul/client.rb:39
- ChartMogul::DeadLetter::MaxRetriesError:
- Rating: A
- Churn: 0
- Complexity: 35.07
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) ChartMogul::DeadLetter has no descriptive comment
- - app/adapters/chart_mogul/dead_letter.rb:2
- * (IrresponsibleModule) ChartMogul::DeadLetter::MaxRetriesError has no descriptive comment
- - app/adapters/chart_mogul/dead_letter.rb:5
- ChartMogul::Errors:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 4
- * (IrresponsibleModule) ChartMogul::Errors::BaseError has no descriptive comment
- - app/adapters/chart_mogul/errors.rb:3
- * (IrresponsibleModule) ChartMogul::Errors::MogulRequestError has no descriptive comment
- - app/adapters/chart_mogul/errors.rb:5
- * (IrresponsibleModule) ChartMogul::Errors::ParameterError has no descriptive comment
- - app/adapters/chart_mogul/errors.rb:7
- * (IrresponsibleModule) ChartMogul::Errors::SmithRequestError has no descriptive comment
- - app/adapters/chart_mogul/errors.rb:6
- ChartMogul::Logging::Methods:
- Rating: A
- Churn: 0
- Complexity: 8.1
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) ChartMogul::Logging has no descriptive comment
- - app/adapters/chart_mogul/logging.rb:2
- * (IrresponsibleModule) ChartMogul::Logging::Methods has no descriptive comment
- - app/adapters/chart_mogul/logging.rb:10
- * (UtilityFunction) ChartMogul::Logging::Methods#logger doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/chart_mogul/logging.rb:26
- ChartMogul::Models::CustomerInvoice:
- Rating: A
- Churn: 0
- Complexity: 31.92
- Duplication: 0
- Smells: 4
- * (HighComplexity) ChartMogul::Models::CustomerInvoice::create has a flog score of 28
- - app/adapters/chart_mogul/models/customer_invoice.rb:13
- * (IrresponsibleModule) ChartMogul::Models::CustomerInvoice has no descriptive comment
- - app/adapters/chart_mogul/models/customer_invoice.rb:3
- * (TooManyStatements) ChartMogul::Models::CustomerInvoice#create has approx 10 statements
- - app/adapters/chart_mogul/models/customer_invoice.rb:13
- * (UncommunicativeVariableName) ChartMogul::Models::CustomerInvoice#create has the variable name 'i'
- - app/adapters/chart_mogul/models/customer_invoice.rb:21
- ChartMogul::ResponseValidator:
- Rating: A
- Churn: 0
- Complexity: 12.76
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) ChartMogul::ResponseValidator has no descriptive comment
- - app/adapters/chart_mogul/response_validator.rb:2
- * (MissingSafeMethod) ChartMogul::ResponseValidator has missing safe method 'validate!'
- - app/adapters/chart_mogul/response_validator.rb:16
- ChartMogulDeadLetter:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ChartMogulDeadLetter has no descriptive comment
- - app/models/chart_mogul_dead_letter.rb:1
- ChartMogulDeadLetterSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ChartMogulDeadLetters:
- Rating: A
- Churn: 0
- Complexity: 4.52
- Duplication: 0
- Smells: 0
- Companies:
- Rating: A
- Churn: 0
- Complexity: 4.52
- Duplication: 0
- Smells: 0
- CompaniesController:
- Rating: A
- Churn: 0
- Complexity: 18.67
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) CompaniesController has no descriptive comment
- - app/controllers/companies_controller.rb:1
- Company:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) Company has no descriptive comment
- - app/models/company.rb:1
- ComponentGenerator:
- Rating: A
- Churn: 0
- Complexity: 12.41
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ComponentGenerator has no descriptive comment
- - lib/generators/component_generator.rb:1
- ComponentsHelper:
- Rating: A
- Churn: 0
- Complexity: 15.9
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) ComponentsHelper has no descriptive comment
- - app/helpers/components_helper.rb:1
- * (UtilityFunction) ComponentsHelper#parse_component_classes doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/components_helper.rb:3
- ContactsRoutingSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Control::Application:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) Control::Application has no descriptive comment
- - config/application.rb:20
- ControllerMacros:
- Rating: A
- Churn: 0
- Complexity: 8.96
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ControllerMacros has no descriptive comment
- - spec/auth0_custom_helper.rb:1
- CookiesSerializer:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- CreateChartMogulDeadLetters:
- Rating: A
- Churn: 0
- Complexity: 5.89
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) CreateChartMogulDeadLetters#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20180808205954_create_chart_mogul_dead_letters.rb:4
- - db/migrate/20180808205954_create_chart_mogul_dead_letters.rb:5
- - db/migrate/20180808205954_create_chart_mogul_dead_letters.rb:6
- - db/migrate/20180808205954_create_chart_mogul_dead_letters.rb:8
- * (IrresponsibleModule) CreateChartMogulDeadLetters has no descriptive comment
- - db/migrate/20180808205954_create_chart_mogul_dead_letters.rb:1
- * (UncommunicativeVariableName) CreateChartMogulDeadLetters#change has the variable name 't'
- - db/migrate/20180808205954_create_chart_mogul_dead_letters.rb:3
- CreateCompanies:
- Rating: A
- Churn: 0
- Complexity: 17.83
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateCompanies#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20161227025614_create_companies.rb:4
- - db/migrate/20161227025614_create_companies.rb:5
- - db/migrate/20161227025614_create_companies.rb:6
- - db/migrate/20161227025614_create_companies.rb:7
- - db/migrate/20161227025614_create_companies.rb:8
- - db/migrate/20161227025614_create_companies.rb:9
- - db/migrate/20161227025614_create_companies.rb:10
- - db/migrate/20161227025614_create_companies.rb:11
- - db/migrate/20161227025614_create_companies.rb:12
- - db/migrate/20161227025614_create_companies.rb:13
- - db/migrate/20161227025614_create_companies.rb:14
- - db/migrate/20161227025614_create_companies.rb:15
- - db/migrate/20161227025614_create_companies.rb:16
- - db/migrate/20161227025614_create_companies.rb:18
- * (IrresponsibleModule) CreateCompanies has no descriptive comment
- - db/migrate/20161227025614_create_companies.rb:1
- * (TooManyStatements) CreateCompanies#change has approx 15 statements
- - db/migrate/20161227025614_create_companies.rb:2
- * (UncommunicativeVariableName) CreateCompanies#change has the variable name 't'
- - db/migrate/20161227025614_create_companies.rb:3
- CreateDelayedJobs:
- Rating: A
- Churn: 0
- Complexity: 15.04
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) CreateDelayedJobs has no descriptive comment
- - db/migrate/20161127220958_create_delayed_jobs.rb:1
- * (TooManyStatements) CreateDelayedJobs#self.up has approx 12 statements
- - db/migrate/20161127220958_create_delayed_jobs.rb:2
- CreateDeviceComponents:
- Rating: A
- Churn: 0
- Complexity: 3.54
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) CreateDeviceComponents#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20170710160511_create_device_components.rb:4
- - db/migrate/20170710160511_create_device_components.rb:5
- * (IrresponsibleModule) CreateDeviceComponents has no descriptive comment
- - db/migrate/20170710160511_create_device_components.rb:1
- * (UncommunicativeVariableName) CreateDeviceComponents#change has the variable name 't'
- - db/migrate/20170710160511_create_device_components.rb:3
- CreateInvoiceVia:
- Rating: A
- Churn: 0
- Complexity: 5.89
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) CreateInvoiceVia#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20161216211536_create_invoice_via.rb:4
- - db/migrate/20161216211536_create_invoice_via.rb:5
- - db/migrate/20161216211536_create_invoice_via.rb:6
- - db/migrate/20161216211536_create_invoice_via.rb:7
- * (IrresponsibleModule) CreateInvoiceVia has no descriptive comment
- - db/migrate/20161216211536_create_invoice_via.rb:1
- * (UncommunicativeVariableName) CreateInvoiceVia#change has the variable name 't'
- - db/migrate/20161216211536_create_invoice_via.rb:3
- CreateOrderItems:
- Rating: A
- Churn: 0
- Complexity: 9.65
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateOrderItems#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160604191955_create_order_items.rb:4
- - db/migrate/20160604191955_create_order_items.rb:5
- - db/migrate/20160604191955_create_order_items.rb:6
- - db/migrate/20160604191955_create_order_items.rb:7
- - db/migrate/20160604191955_create_order_items.rb:8
- - db/migrate/20160604191955_create_order_items.rb:10
- * (IrresponsibleModule) CreateOrderItems has no descriptive comment
- - db/migrate/20160604191955_create_order_items.rb:1
- * (TooManyStatements) CreateOrderItems#change has approx 7 statements
- - db/migrate/20160604191955_create_order_items.rb:2
- * (UncommunicativeVariableName) CreateOrderItems#change has the variable name 't'
- - db/migrate/20160604191955_create_order_items.rb:3
- CreateOrderStatuses:
- Rating: A
- Churn: 0
- Complexity: 3.54
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) CreateOrderStatuses#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160604191816_create_order_statuses.rb:4
- - db/migrate/20160604191816_create_order_statuses.rb:6
- * (IrresponsibleModule) CreateOrderStatuses has no descriptive comment
- - db/migrate/20160604191816_create_order_statuses.rb:1
- * (UncommunicativeVariableName) CreateOrderStatuses#change has the variable name 't'
- - db/migrate/20160604191816_create_order_statuses.rb:3
- CreateOrders:
- Rating: A
- Churn: 0
- Complexity: 9.15
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateOrders#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160604191917_create_orders.rb:4
- - db/migrate/20160604191917_create_orders.rb:5
- - db/migrate/20160604191917_create_orders.rb:6
- - db/migrate/20160604191917_create_orders.rb:7
- - db/migrate/20160604191917_create_orders.rb:9
- * (IrresponsibleModule) CreateOrders has no descriptive comment
- - db/migrate/20160604191917_create_orders.rb:1
- * (TooManyStatements) CreateOrders#change has approx 6 statements
- - db/migrate/20160604191917_create_orders.rb:2
- * (UncommunicativeVariableName) CreateOrders#change has the variable name 't'
- - db/migrate/20160604191917_create_orders.rb:3
- CreateProductUpgradeItems:
- Rating: A
- Churn: 0
- Complexity: 7.07
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateProductUpgradeItems#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160701203211_create_product_upgrade_items.rb:4
- - db/migrate/20160701203211_create_product_upgrade_items.rb:5
- - db/migrate/20160701203211_create_product_upgrade_items.rb:6
- - db/migrate/20160701203211_create_product_upgrade_items.rb:7
- - db/migrate/20160701203211_create_product_upgrade_items.rb:8
- * (IrresponsibleModule) CreateProductUpgradeItems has no descriptive comment
- - db/migrate/20160701203211_create_product_upgrade_items.rb:1
- * (TooManyStatements) CreateProductUpgradeItems#change has approx 6 statements
- - db/migrate/20160701203211_create_product_upgrade_items.rb:2
- * (UncommunicativeVariableName) CreateProductUpgradeItems#change has the variable name 't'
- - db/migrate/20160701203211_create_product_upgrade_items.rb:3
- CreateProducts:
- Rating: A
- Churn: 0
- Complexity: 8.96
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateProducts#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160604191631_create_products.rb:4
- - db/migrate/20160604191631_create_products.rb:5
- - db/migrate/20160604191631_create_products.rb:6
- - db/migrate/20160604191631_create_products.rb:7
- - db/migrate/20160604191631_create_products.rb:8
- - db/migrate/20160604191631_create_products.rb:10
- * (IrresponsibleModule) CreateProducts has no descriptive comment
- - db/migrate/20160604191631_create_products.rb:1
- * (TooManyStatements) CreateProducts#change has approx 7 statements
- - db/migrate/20160604191631_create_products.rb:2
- * (UncommunicativeVariableName) CreateProducts#change has the variable name 't'
- - db/migrate/20160604191631_create_products.rb:3
- CreateTaskQueues:
- Rating: A
- Churn: 0
- Complexity: 4.71
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) CreateTaskQueues#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20180718123333_create_task_queues.rb:4
- - db/migrate/20180718123333_create_task_queues.rb:5
- - db/migrate/20180718123333_create_task_queues.rb:6
- * (IrresponsibleModule) CreateTaskQueues has no descriptive comment
- - db/migrate/20180718123333_create_task_queues.rb:1
- * (UncommunicativeVariableName) CreateTaskQueues#change has the variable name 't'
- - db/migrate/20180718123333_create_task_queues.rb:3
- CreateTokens:
- Rating: A
- Churn: 0
- Complexity: 4.71
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) CreateTokens#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20180613193346_create_tokens.rb:4
- - db/migrate/20180613193346_create_tokens.rb:5
- - db/migrate/20180613193346_create_tokens.rb:7
- * (IrresponsibleModule) CreateTokens has no descriptive comment
- - db/migrate/20180613193346_create_tokens.rb:1
- * (UncommunicativeVariableName) CreateTokens#change has the variable name 't'
- - db/migrate/20180613193346_create_tokens.rb:3
- CreateUpgradeItem:
- Rating: A
- Churn: 0
- Complexity: 18.23
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateUpgradeItem#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160613223316_create_upgrade_item.rb:4
- - db/migrate/20160613223316_create_upgrade_item.rb:5
- - db/migrate/20160613223316_create_upgrade_item.rb:6
- - db/migrate/20160613223316_create_upgrade_item.rb:7
- - db/migrate/20160613223316_create_upgrade_item.rb:8
- - db/migrate/20160613223316_create_upgrade_item.rb:9
- - db/migrate/20160613223316_create_upgrade_item.rb:10
- - db/migrate/20160613223316_create_upgrade_item.rb:11
- - db/migrate/20160613223316_create_upgrade_item.rb:12
- - db/migrate/20160613223316_create_upgrade_item.rb:13
- - db/migrate/20160613223316_create_upgrade_item.rb:14
- - db/migrate/20160613223316_create_upgrade_item.rb:15
- * (IrresponsibleModule) CreateUpgradeItem has no descriptive comment
- - db/migrate/20160613223316_create_upgrade_item.rb:1
- * (TooManyStatements) CreateUpgradeItem#change has approx 13 statements
- - db/migrate/20160613223316_create_upgrade_item.rb:2
- * (UncommunicativeVariableName) CreateUpgradeItem#change has the variable name 't'
- - db/migrate/20160613223316_create_upgrade_item.rb:3
- CreateUpgradeItemGroups:
- Rating: A
- Churn: 0
- Complexity: 7.07
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateUpgradeItemGroups#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:4
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:5
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:6
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:7
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:8
- * (IrresponsibleModule) CreateUpgradeItemGroups has no descriptive comment
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:1
- * (TooManyStatements) CreateUpgradeItemGroups#change has approx 6 statements
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:2
- * (UncommunicativeVariableName) CreateUpgradeItemGroups#change has the variable name 't'
- - db/migrate/20160629211522_create_upgrade_item_groups.rb:3
- CreateUpgrades:
- Rating: A
- Churn: 0
- Complexity: 11.84
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) CreateUpgrades#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20160605205616_create_upgrades.rb:4
- - db/migrate/20160605205616_create_upgrades.rb:5
- - db/migrate/20160605205616_create_upgrades.rb:6
- - db/migrate/20160605205616_create_upgrades.rb:7
- - db/migrate/20160605205616_create_upgrades.rb:8
- - db/migrate/20160605205616_create_upgrades.rb:9
- - db/migrate/20160605205616_create_upgrades.rb:10
- - db/migrate/20160605205616_create_upgrades.rb:11
- - db/migrate/20160605205616_create_upgrades.rb:13
- * (IrresponsibleModule) CreateUpgrades has no descriptive comment
- - db/migrate/20160605205616_create_upgrades.rb:1
- * (TooManyStatements) CreateUpgrades#change has approx 10 statements
- - db/migrate/20160605205616_create_upgrades.rb:2
- * (UncommunicativeVariableName) CreateUpgrades#change has the variable name 't'
- - db/migrate/20160605205616_create_upgrades.rb:3
- CreditInvoiceUbersmithService:
- Rating: A
- Churn: 0
- Complexity: 19.93
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) CreditInvoiceUbersmithService has no descriptive comment
- - app/services/credit_invoice_ubersmith_service.rb:1
- * (UncommunicativeVariableName) CreditInvoiceUbersmithService#call has the variable name 'e'
- - app/services/credit_invoice_ubersmith_service.rb:9
- * (UtilityFunction) CreditInvoiceUbersmithService#filters doesn't depend on instance state (maybe move it to another class?)
- - app/services/credit_invoice_ubersmith_service.rb:17
- CurrentUser::Token:
- Rating: A
- Churn: 0
- Complexity: 2.49
- Duplication: 0
- Smells: 2
- * (ControlParameter) CurrentUser::Token#initialize is controlled by argument 'created_at'
- - app/adapters/current_user/token.rb:7
- * (IrresponsibleModule) CurrentUser::Token has no descriptive comment
- - app/adapters/current_user/token.rb:2
- Development:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- DevelopmentMailer:
- Rating: A
- Churn: 0
- Complexity: 2.2
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) DevelopmentMailer has no descriptive comment
- - app/mailers/development_mailer.rb:1
- * (LongParameterList) DevelopmentMailer#send_welcome_email has 4 parameters
- - app/mailers/development_mailer.rb:2
- * (UnusedParameters) DevelopmentMailer#send_welcome_email has unused parameter 'brand_id'
- - app/mailers/development_mailer.rb:2
- DeviseCreateUsers:
- Rating: A
- Churn: 0
- Complexity: 16.23
- Duplication: 0
- Smells: 4
- * (FeatureEnvy) DeviseCreateUsers#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20161026133150_devise_create_users.rb:5
- - db/migrate/20161026133150_devise_create_users.rb:6
- - db/migrate/20161026133150_devise_create_users.rb:9
- - db/migrate/20161026133150_devise_create_users.rb:10
- - db/migrate/20161026133150_devise_create_users.rb:13
- - db/migrate/20161026133150_devise_create_users.rb:16
- - db/migrate/20161026133150_devise_create_users.rb:17
- - db/migrate/20161026133150_devise_create_users.rb:18
- - db/migrate/20161026133150_devise_create_users.rb:19
- - db/migrate/20161026133150_devise_create_users.rb:20
- - db/migrate/20161026133150_devise_create_users.rb:34
- * (IrresponsibleModule) DeviseCreateUsers has no descriptive comment
- - db/migrate/20161026133150_devise_create_users.rb:1
- * (TooManyStatements) DeviseCreateUsers#change has approx 14 statements
- - db/migrate/20161026133150_devise_create_users.rb:2
- * (UncommunicativeVariableName) DeviseCreateUsers#change has the variable name 't'
- - db/migrate/20161026133150_devise_create_users.rb:3
- DeviseUpdateUsers:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) DeviseUpdateUsers has no descriptive comment
- - db/migrate/20161110220433_devise_update_users.rb:1
- Edit.html.erbSpec:
- Rating: A
- Churn: 0
- Complexity: 17.98
- Duplication: 0
- Smells: 0
- Edit.html.erbSpec:
- Rating: A
- Churn: 0
- Complexity: 1.9
- Duplication: 0
- Smells: 0
- Edit.html.erbSpec:
- Rating: A
- Churn: 0
- Complexity: 23.9
- Duplication: 0
- Smells: 0
- EmailHelper:
- Rating: A
- Churn: 0
- Complexity: 16.15
- Duplication: 0
- Smells: 5
- * (DuplicateMethodCall) EmailHelper#send_welcome_email calls 'Rails.env' 2 times
- - app/helpers/email_helper.rb:3
- * (DuplicateMethodCall) EmailHelper#send_welcome_email calls 'email.downcase' 2 times
- - app/helpers/email_helper.rb:10
- - app/helpers/email_helper.rb:14
- * (IrresponsibleModule) EmailHelper has no descriptive comment
- - app/helpers/email_helper.rb:1
- * (LongParameterList) EmailHelper#send_welcome_email has 4 parameters
- - app/helpers/email_helper.rb:2
- * (UtilityFunction) EmailHelper#send_welcome_email doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/email_helper.rb:2
- Environment:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ErrorHelper:
- Rating: A
- Churn: 0
- Complexity: 28.77
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) ErrorHelper has no descriptive comment
- - app/helpers/error_helper.rb:1
- * (TooManyStatements) ErrorHelper#process_error has approx 6 statements
- - app/helpers/error_helper.rb:3
- Errors:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ErrorsController:
- Rating: A
- Churn: 0
- Complexity: 10.35
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ErrorsController has no descriptive comment
- - app/controllers/errors_controller.rb:1
- Extconf:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- FastGettext:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- FeedbackWidgetController:
- Rating: A
- Churn: 0
- Complexity: 30.53
- Duplication: 0
- Smells: 3
- * (DuplicateMethodCall) FeedbackWidgetController#email_body calls 'current_user.email' 2 times
- - app/controllers/feedback_widget_controller.rb:26
- - app/controllers/feedback_widget_controller.rb:27
- * (DuplicateMethodCall) FeedbackWidgetController#email_body calls 'params[:page_feedback]' 2 times
- - app/controllers/feedback_widget_controller.rb:28
- - app/controllers/feedback_widget_controller.rb:29
- * (IrresponsibleModule) FeedbackWidgetController has no descriptive comment
- - app/controllers/feedback_widget_controller.rb:3
- FilterParameterLogging:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- FlashesHelper:
- Rating: A
- Churn: 0
- Complexity: 3.6
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) FlashesHelper has no descriptive comment
- - app/helpers/flashes_helper.rb:1
- FormFieldComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) FormFieldComponent has no descriptive comment
- - frontend/components/form/field/form_field_component.rb:3
- GridComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) GridComponent has no descriptive comment
- - frontend/components/grid/grid_component.rb:3
- GridItemComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) GridItemComponent has no descriptive comment
- - frontend/components/grid/item/grid_item_component.rb:3
- HelpController:
- Rating: A
- Churn: 0
- Complexity: 2.24
- Duplication: 0
- Smells: 2
- * (InstanceVariableAssumption) HelpController assumes too much for instance variable '@title'
- - app/controllers/help_controller.rb:1
- * (IrresponsibleModule) HelpController has no descriptive comment
- - app/controllers/help_controller.rb:1
- HelpControllerSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- I18n::Config:
- Rating: A
- Churn: 0
- Complexity: 4.99
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) I18n::Base has no descriptive comment
- - lib/modules/i18n.rb:9
- * (IrresponsibleModule) I18n::Config has no descriptive comment
- - lib/modules/i18n.rb:3
- * (UtilityFunction) I18n::Config#locale doesn't depend on instance state (maybe move it to another class?)
- - lib/modules/i18n.rb:4
- I18nHelper:
- Rating: A
- Churn: 0
- Complexity: 3.35
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) I18nHelper has no descriptive comment
- - app/helpers/i18n_helper.rb:1
- * (UtilityFunction) I18nHelper#error doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/i18n_helper.rb:2
- IconComponent:
- Rating: A
- Churn: 0
- Complexity: 6.94
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) IconComponent has no descriptive comment
- - frontend/components/icon/icon_component.rb:3
- IncidentPresenter:
- Rating: A
- Churn: 0
- Complexity: 25.73
- Duplication: 0
- Smells: 4
- * (IrresponsibleModule) IncidentPresenter has no descriptive comment
- - app/presenters/incident_presenter.rb:1
- * (TooManyStatements) IncidentPresenter#self.retrieve_incidents has approx 8 statements
- - app/presenters/incident_presenter.rb:6
- * (UncommunicativeVariableName) IncidentPresenter#self.last_incident has the variable name 'k'
- - app/presenters/incident_presenter.rb:26
- * (UncommunicativeVariableName) IncidentPresenter#self.last_incident has the variable name 'v'
- - app/presenters/incident_presenter.rb:26
- Index.html.erbSpec:
- Rating: A
- Churn: 0
- Complexity: 7.4
- Duplication: 0
- Smells: 0
- Inflections:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- InvoicePresenter:
- Rating: A
- Churn: 0
- Complexity: 49.92
- Duplication: 0
- Smells: 3
- * (HighComplexity) InvoicePresenter#label has a flog score of 36
- - app/presenters/invoice_presenter.rb:10
- * (DuplicateMethodCall) InvoicePresenter#label calls 'invoice['amount_unpaid']' 2 times
- - app/presenters/invoice_presenter.rb:13
- - app/presenters/invoice_presenter.rb:16
- * (IrresponsibleModule) InvoicePresenter has no descriptive comment
- - app/presenters/invoice_presenter.rb:1
- InvoicesRoutingSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- IpsController:
- Rating: A
- Churn: 0
- Complexity: 2.24
- Duplication: 0
- Smells: 2
- * (InstanceVariableAssumption) IpsController assumes too much for instance variable '@title'
- - app/controllers/ips_controller.rb:1
- * (IrresponsibleModule) IpsController has no descriptive comment
- - app/controllers/ips_controller.rb:1
- JsonEncoding:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- JsonResponseHelper:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) JsonResponseHelper has no descriptive comment
- - app/helpers/json_response_helper.rb:1
- Komponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ListHorizontalComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ListHorizontalComponent has no descriptive comment
- - frontend/components/list/horizontal/list_horizontal_component.rb:3
- ListPropertiesComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ListPropertiesComponent has no descriptive comment
- - frontend/components/list/properties/list_properties_component.rb:3
- ListVerticalComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) ListVerticalComponent has no descriptive comment
- - frontend/components/list/vertical/list_vertical_component.rb:3
- MaasProvisionJob:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) MaasProvisionJob has no descriptive comment
- - app/jobs/maas_provision_job.rb:1
- * (UtilityFunction) MaasProvisionJob#perform doesn't depend on instance state (maybe move it to another class?)
- - app/jobs/maas_provision_job.rb:4
- MaxihostApi::Account:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Account has no descriptive comment
- - app/models/maxihost_api/account.rb:2
- MaxihostApi::Account::Key:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Account::Key has no descriptive comment
- - app/models/maxihost_api/account/key.rb:3
- MaxihostApi::Auth::Token:
- Rating: A
- Churn: 0
- Complexity: 16.44
- Duplication: 0
- Smells: 4
- * (InstanceVariableAssumption) MaxihostApi::Auth::Token assumes too much for instance variable '@login'
- - app/models/maxihost_api/auth/token.rb:3
- * (IrresponsibleModule) MaxihostApi::Auth::Token has no descriptive comment
- - app/models/maxihost_api/auth/token.rb:3
- * (MissingSafeMethod) MaxihostApi::Auth::Token has missing safe method 'retrieve!'
- - app/models/maxihost_api/auth/token.rb:15
- * (MissingSafeMethod) MaxihostApi::Auth::Token has missing safe method 'retrieve_for!'
- - app/models/maxihost_api/auth/token.rb:9
- MaxihostApi::Authorised:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Authorised has no descriptive comment
- - app/models/maxihost_api/authorised.rb:2
- MaxihostApi::Base:
- Rating: A
- Churn: 0
- Complexity: 6.71
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Base has no descriptive comment
- - app/models/maxihost_api/base.rb:2
- MaxihostApi::Contacts:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Contacts has no descriptive comment
- - app/models/maxihost_api/contacts.rb:2
- MaxihostApi::Device:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Device has no descriptive comment
- - app/models/maxihost_api/device.rb:4
- MaxihostApi::Login:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Login has no descriptive comment
- - app/models/maxihost_api/login.rb:2
- MaxihostApi::OperatingSystem:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::OperatingSystem has no descriptive comment
- - app/models/maxihost_api/operating_system.rb:2
- MaxihostApi::Plan:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) MaxihostApi::Plan has no descriptive comment
- - app/models/maxihost_api/plan.rb:2
- New.html.erbSpec:
- Rating: A
- Churn: 0
- Complexity: 20.9
- Duplication: 0
- Smells: 0
- PaginationComponent:
- Rating: A
- Churn: 0
- Complexity: 32.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) PaginationComponent has no descriptive comment
- - frontend/components/pagination/pagination_component.rb:3
- ParseAirtableDataService:
- Rating: A
- Churn: 0
- Complexity: 19.18
- Duplication: 0
- Smells: 3
- * (DuplicateMethodCall) ParseAirtableDataService#call calls 'data['fields']' 2 times
- - app/services/parse_airtable_data_service.rb:12
- * (FeatureEnvy) ParseAirtableDataService#call refers to 'data' more than self (maybe move it to another class?)
- - app/services/parse_airtable_data_service.rb:12
- * (IrresponsibleModule) ParseAirtableDataService has no descriptive comment
- - app/services/parse_airtable_data_service.rb:1
- ParseAirtableDataServiceSpec:
- Rating: A
- Churn: 0
- Complexity: 25.65
- Duplication: 0
- Smells: 0
- Puma:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- RackMiniProfiler:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- RailsHelper:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- RedisAdapter:
- Rating: A
- Churn: 0
- Complexity: 1.41
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RedisAdapter has no descriptive comment
- - app/adapters/redis_adapter.rb:1
- RegistrationsRoutingSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- RemoveColumnsFromUsers:
- Rating: A
- Churn: 0
- Complexity: 12.0
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) RemoveColumnsFromUsers has no descriptive comment
- - db/migrate/20161227031922_remove_columns_from_users.rb:1
- * (TooManyStatements) RemoveColumnsFromUsers#change has approx 12 statements
- - db/migrate/20161227031922_remove_columns_from_users.rb:2
- RemoveDeviceComponents:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RemoveDeviceComponents has no descriptive comment
- - db/migrate/20181105161625_remove_device_components.rb:1
- RemovePaymentMethodFromCompanies:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RemovePaymentMethodFromCompanies has no descriptive comment
- - db/migrate/20180919175533_remove_payment_method_from_companies.rb:1
- RenameAndRemoveCompanyColumns:
- Rating: A
- Churn: 0
- Complexity: 2.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RenameAndRemoveCompanyColumns has no descriptive comment
- - db/migrate/20161227035959_rename_and_remove_company_columns.rb:1
- RenameClientIdToCompanyId:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RenameClientIdToCompanyId has no descriptive comment
- - db/migrate/20161227035017_rename_client_id_to_company_id.rb:1
- RenameUserRealId:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RenameUserRealId has no descriptive comment
- - db/migrate/20161113222051_rename_user_real_id.rb:1
- RootCardComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RootCardComponent has no descriptive comment
- - frontend/components/root/card/root_card_component.rb:3
- RootLoaderComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) RootLoaderComponent has no descriptive comment
- - frontend/components/root/loader/root_loader_component.rb:3
- Routes:
- Rating: A
- Churn: 0
- Complexity: 2.4
- Duplication: 0
- Smells: 0
- SectionContentComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) SectionContentComponent has no descriptive comment
- - frontend/components/section/content/section_content_component.rb:3
- SectionFooterComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) SectionFooterComponent has no descriptive comment
- - frontend/components/section/footer/section_footer_component.rb:3
- SectionHeaderComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) SectionHeaderComponent has no descriptive comment
- - frontend/components/section/header/section_header_component.rb:3
- Seeds:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ServersAccessPresenter:
- Rating: A
- Churn: 0
- Complexity: 18.25
- Duplication: 0
- Smells: 3
- * (DuplicateMethodCall) ServersAccessPresenter#self.parse_message calls 'access_values[0]' 2 times
- - app/presenters/servers_access_presenter.rb:17
- - app/presenters/servers_access_presenter.rb:20
- * (IrresponsibleModule) ServersAccessPresenter has no descriptive comment
- - app/presenters/servers_access_presenter.rb:1
- * (TooManyStatements) ServersAccessPresenter#self.parse_message has approx 6 statements
- - app/presenters/servers_access_presenter.rb:16
- ServersAccessPresenterSpec:
- Rating: A
- Churn: 0
- Complexity: 49.8
- Duplication: 0
- Smells: 0
- ServersControllerSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ServiceHelper:
- Rating: A
- Churn: 0
- Complexity: 35.36
- Duplication: 0
- Smells: 5
- * (ControlParameter) ServiceHelper#get_status is controlled by argument 'service_status'
- - app/helpers/service_helper.rb:20
- * (DuplicateMethodCall) ServiceHelper#service_discount calls '@service['discount']' 2 times
- - app/helpers/service_helper.rb:39
- - app/helpers/service_helper.rb:41
- * (DuplicateMethodCall) ServiceHelper#service_discount calls '@service['discount_type']' 2 times
- - app/helpers/service_helper.rb:38
- - app/helpers/service_helper.rb:40
- * (IrresponsibleModule) ServiceHelper has no descriptive comment
- - app/helpers/service_helper.rb:1
- * (TooManyStatements) ServiceHelper#get_status has approx 6 statements
- - app/helpers/service_helper.rb:19
- SessionsController:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) SessionsController has no descriptive comment
- - app/controllers/sessions_controller.rb:1
- ShouldaMatchersHelper:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Show.html.erbSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- SidenavComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) SidenavComponent has no descriptive comment
- - frontend/components/sidenav/sidenav_component.rb:3
- SidenavNavigationComponent:
- Rating: A
- Churn: 0
- Complexity: 27.47
- Duplication: 0
- Smells: 3
- * (ControlParameter) SidenavNavigationComponent#primary_current_class? is controlled by argument 'path'
- - frontend/components/sidenav/navigation/sidenav_navigation_component.rb:32
- * (ControlParameter) SidenavNavigationComponent#secondary_current_class? is controlled by argument 'path'
- - frontend/components/sidenav/navigation/sidenav_navigation_component.rb:36
- * (IrresponsibleModule) SidenavNavigationComponent has no descriptive comment
- - frontend/components/sidenav/navigation/sidenav_navigation_component.rb:3
- SidenavThumbComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) SidenavThumbComponent has no descriptive comment
- - frontend/components/sidenav/thumb/sidenav_thumb_component.rb:3
- SimpleForm:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Smtp:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- SpecHelper:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Spring:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Status::Incident:
- Rating: A
- Churn: 0
- Complexity: 1.0
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) Status::Incident has no descriptive comment
- - app/models/status/incident.rb:2
- * (UnusedParameters) Status::Incident#self.default_message has unused parameter 'args'
- - app/models/status/incident.rb:7
- Status::ScheduledMaintenance:
- Rating: A
- Churn: 0
- Complexity: 13.74
- Duplication: 0
- Smells: 2
- * (IrresponsibleModule) Status::ScheduledMaintenance has no descriptive comment
- - app/models/status/scheduled_maintenance.rb:2
- * (TooManyStatements) Status::ScheduledMaintenance#self.default_message has approx 8 statements
- - app/models/status/scheduled_maintenance.rb:7
- StatusCode:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (TooManyConstants) StatusCode has 10 constants
- - lib/modules/status_code.rb:1
- TableBodyComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableBodyComponent has no descriptive comment
- - frontend/components/table/body/table_body_component.rb:3
- TableCellComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableCellComponent has no descriptive comment
- - frontend/components/table/cell/table_cell_component.rb:3
- TableComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableComponent has no descriptive comment
- - frontend/components/table/table_component.rb:3
- TableHeadComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableHeadComponent has no descriptive comment
- - frontend/components/table/head/table_head_component.rb:3
- TableRowComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableRowComponent has no descriptive comment
- - frontend/components/table/row/table_row_component.rb:3
- TableTextCellComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableTextCellComponent has no descriptive comment
- - frontend/components/table/text_cell/table_text_cell_component.rb:3
- TableTextHeaderCellComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableTextHeaderCellComponent has no descriptive comment
- - frontend/components/table/text_header_cell/table_text_header_cell_component.rb:3
- TableTextSubCellComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TableTextSubCellComponent has no descriptive comment
- - frontend/components/table/text_sub_cell/table_text_sub_cell_component.rb:3
- TabsComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TabsComponent has no descriptive comment
- - frontend/components/tabs/tabs_component.rb:3
- TabsTabComponent:
- Rating: A
- Churn: 0
- Complexity: 5.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TabsTabComponent has no descriptive comment
- - frontend/components/tabs/tab/tabs_tab_component.rb:3
- TaskMailer:
- Rating: A
- Churn: 0
- Complexity: 35.41
- Duplication: 0
- Smells: 6
- * (HighComplexity) TaskMailer::send has a flog score of 31
- - lib/task_helpers/mailer.rb:17
- * (DuplicateMethodCall) TaskMailer#self.send calls 'Rails.env' 2 times
- - lib/task_helpers/mailer.rb:18
- * (DuplicateMethodCall) TaskMailer#self.send calls 'params.has_key?(:body)' 2 times
- - lib/task_helpers/mailer.rb:20
- - lib/task_helpers/mailer.rb:25
- * (IrresponsibleModule) TaskMailer has no descriptive comment
- - lib/task_helpers/mailer.rb:1
- * (LongParameterList) TaskMailer#self.send_generic_email has 4 parameters
- - lib/task_helpers/mailer.rb:4
- * (TooManyStatements) TaskMailer#self.send has approx 9 statements
- - lib/task_helpers/mailer.rb:17
- TaskQueue:
- Rating: A
- Churn: 0
- Complexity: 22.0
- Duplication: 0
- Smells: 4
- * (IrresponsibleModule) TaskQueue has no descriptive comment
- - app/models/task_queue.rb:1
- * (NestedIterators) TaskQueue#self.validate contains iterators nested 2 deep
- - app/models/task_queue.rb:43
- * (NilCheck) TaskQueue#self.validate performs a nil-check
- - app/models/task_queue.rb:43
- * (TooManyStatements) TaskQueue#self.add_to_maas_reinstall has approx 6 statements
- - app/models/task_queue.rb:22
- TechnicalVisit:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TechnicalVisit has no descriptive comment
- - app/models/technical_visit.rb:1
- Test:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- TextClickToCopyComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TextClickToCopyComponent has no descriptive comment
- - frontend/components/text/click_to_copy/text_click_to_copy_component.rb:3
- TextSpanComponent:
- Rating: A
- Churn: 0
- Complexity: 3.4
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TextSpanComponent has no descriptive comment
- - frontend/components/text/span/text_span_component.rb:3
- Token:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) Token has no descriptive comment
- - app/models/token.rb:1
- TokenController:
- Rating: A
- Churn: 0
- Complexity: 11.39
- Duplication: 0
- Smells: 2
- * (InstanceVariableAssumption) TokenController assumes too much for instance variable '@title'
- - app/controllers/token_controller.rb:1
- * (IrresponsibleModule) TokenController has no descriptive comment
- - app/controllers/token_controller.rb:1
- Tokens:
- Rating: A
- Churn: 0
- Complexity: 8.62
- Duplication: 0
- Smells: 0
- TopcontentComponent:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) TopcontentComponent has no descriptive comment
- - frontend/components/topcontent/topcontent_component.rb:3
- UpdateCompanyDataJob:
- Rating: A
- Churn: 0
- Complexity: 19.74
- Duplication: 0
- Smells: 7
- * (DuplicateMethodCall) UpdateCompanyDataJob#perform calls 'api.client' 2 times
- - app/jobs/update_company_data_job.rb:6
- - app/jobs/update_company_data_job.rb:20
- * (FeatureEnvy) UpdateCompanyDataJob#perform refers to 'active_client' more than self (maybe move it to another class?)
- - app/jobs/update_company_data_job.rb:7
- * (FeatureEnvy) UpdateCompanyDataJob#perform refers to 'api' more than self (maybe move it to another class?)
- - app/jobs/update_company_data_job.rb:6
- - app/jobs/update_company_data_job.rb:20
- * (FeatureEnvy) UpdateCompanyDataJob#perform refers to 'update_result' more than self (maybe move it to another class?)
- - app/jobs/update_company_data_job.rb:22
- - app/jobs/update_company_data_job.rb:23
- * (IrresponsibleModule) UpdateCompanyDataJob has no descriptive comment
- - app/jobs/update_company_data_job.rb:1
- * (LongParameterList) UpdateCompanyDataJob#perform has 9 parameters
- - app/jobs/update_company_data_job.rb:4
- * (TooManyStatements) UpdateCompanyDataJob#perform has approx 6 statements
- - app/jobs/update_company_data_job.rb:4
- UpdateStatusService:
- Rating: A
- Churn: 0
- Complexity: 45.77
- Duplication: 0
- Smells: 3
- * (FeatureEnvy) UpdateStatusService#body refers to 'payload' more than self (maybe move it to another class?)
- - app/services/update_status_service.rb:27
- - app/services/update_status_service.rb:30
- - app/services/update_status_service.rb:31
- - app/services/update_status_service.rb:32
- - app/services/update_status_service.rb:33
- * (IrresponsibleModule) UpdateStatusService has no descriptive comment
- - app/services/update_status_service.rb:1
- * (UtilityFunction) UpdateStatusService#last_update doesn't depend on instance state (maybe move it to another class?)
- - app/services/update_status_service.rb:37
- UpdateUserDataJob:
- Rating: A
- Churn: 0
- Complexity: 12.31
- Duplication: 0
- Smells: 5
- * (ControlParameter) UpdateUserDataJob#perform is controlled by argument 'lang'
- - app/jobs/update_user_data_job.rb:7
- * (DuplicateMethodCall) UpdateUserDataJob#perform calls 'api.client' 2 times
- - app/jobs/update_user_data_job.rb:12
- - app/jobs/update_user_data_job.rb:22
- * (IrresponsibleModule) UpdateUserDataJob has no descriptive comment
- - app/jobs/update_user_data_job.rb:1
- * (LongParameterList) UpdateUserDataJob#perform has 5 parameters
- - app/jobs/update_user_data_job.rb:4
- * (UtilityFunction) UpdateUserDataJob#perform doesn't depend on instance state (maybe move it to another class?)
- - app/jobs/update_user_data_job.rb:4
- User:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 1
- * (IrresponsibleModule) User has no descriptive comment
- - app/models/user.rb:1
- Users:
- Rating: A
- Churn: 0
- Complexity: 8.51
- Duplication: 0
- Smells: 0
- WebhooksController:
- Rating: A
- Churn: 0
- Complexity: 45.09
- Duplication: 0
- Smells: 5
- * (HighComplexity) WebhooksController#order_automation has a flog score of 45
- - app/controllers/webhooks_controller.rb:5
- * (DuplicateMethodCall) WebhooksController#order_automation calls 'params[:order_action_id]' 2 times
- - app/controllers/webhooks_controller.rb:8
- * (DuplicateMethodCall) WebhooksController#order_automation calls 'params[:order_id]' 2 times
- - app/controllers/webhooks_controller.rb:6
- - app/controllers/webhooks_controller.rb:10
- * (IrresponsibleModule) WebhooksController has no descriptive comment
- - app/controllers/webhooks_controller.rb:1
- * (TooManyStatements) WebhooksController#order_automation has approx 7 statements
- - app/controllers/webhooks_controller.rb:5
- WebhooksRoutingSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- WhatsNewController:
- Rating: A
- Churn: 0
- Complexity: 31.72
- Duplication: 0
- Smells: 3
- * (IrresponsibleModule) WhatsNewController has no descriptive comment
- - app/controllers/whats_new_controller.rb:1
- * (TooManyStatements) WhatsNewController#index has approx 6 statements
- - app/controllers/whats_new_controller.rb:8
- * (UtilityFunction) WhatsNewController#redis doesn't depend on instance state (maybe move it to another class?)
- - app/controllers/whats_new_controller.rb:34
- WrapParameters:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- ZendeskSsoController:
- Rating: A
- Churn: 0
- Complexity: 39.37
- Duplication: 0
- Smells: 3
- * (DuplicateMethodCall) ZendeskSsoController#zendesk_sso_url calls 'params["return_to"]' 2 times
- - app/controllers/zendesk_sso_controller.rb:49
- * (IrresponsibleModule) ZendeskSsoController has no descriptive comment
- - app/controllers/zendesk_sso_controller.rb:7
- * (UnusedParameters) ZendeskSsoController#sign_into_zendesk has unused parameter 'user'
- - app/controllers/zendesk_sso_controller.rb:31
- ZendeskSsoControllerSpec:
- Rating: A
- Churn: 0
- Complexity: 0.0
- Duplication: 0
- Smells: 0
- Account::BillingOrdersController:
- Rating: B
- Churn: 0
- Complexity: 60.53
- Duplication: 0
- Smells: 7
- * (VeryHighComplexity) Account::BillingOrdersController#index has a flog score of 61
- - app/controllers/account/billing_orders_controller.rb:5
- * (DuplicateMethodCall) Account::BillingOrdersController#index calls 'api.order' 2 times
- - app/controllers/account/billing_orders_controller.rb:10
- - app/controllers/account/billing_orders_controller.rb:16
- * (DuplicateMethodCall) Account::BillingOrdersController#index calls 'current_user.company_id' 2 times
- - app/controllers/account/billing_orders_controller.rb:11
- - app/controllers/account/billing_orders_controller.rb:17
- * (DuplicateMethodCall) Account::BillingOrdersController#index calls 'params[:locale]' 2 times
- - app/controllers/account/billing_orders_controller.rb:8
- - app/controllers/account/billing_orders_controller.rb:32
- * (InstanceVariableAssumption) Account::BillingOrdersController assumes too much for instance variable '@orders'
- - app/controllers/account/billing_orders_controller.rb:1
- * (IrresponsibleModule) Account::BillingOrdersController has no descriptive comment
- - app/controllers/account/billing_orders_controller.rb:1
- * (TooManyStatements) Account::BillingOrdersController#index has approx 20 statements
- - app/controllers/account/billing_orders_controller.rb:5
- Account::SecurityController:
- Rating: B
- Churn: 0
- Complexity: 74.69
- Duplication: 0
- Smells: 9
- * (HighComplexity) Account::SecurityController#update_otp has a flog score of 25
- - app/controllers/account/security_controller.rb:12
- * (DuplicateMethodCall) Account::SecurityController#update_otp calls 'current_user.id' 2 times
- - app/controllers/account/security_controller.rb:19
- - app/controllers/account/security_controller.rb:26
- * (DuplicateMethodCall) Account::SecurityController#update_otp calls 'redirect_to account_security_index_path' 2 times
- - app/controllers/account/security_controller.rb:31
- - app/controllers/account/security_controller.rb:34
- * (IrresponsibleModule) Account::SecurityController has no descriptive comment
- - app/controllers/account/security_controller.rb:1
- * (MissingSafeMethod) Account::SecurityController has missing safe method 'disable_two_factor!'
- - app/controllers/account/security_controller.rb:74
- * (MissingSafeMethod) Account::SecurityController has missing safe method 'enable_two_factor!'
- - app/controllers/account/security_controller.rb:69
- * (MissingSafeMethod) Account::SecurityController has missing safe method 'ensure_otp_code_exists!'
- - app/controllers/account/security_controller.rb:62
- * (TooManyStatements) Account::SecurityController#update_otp has approx 9 statements
- - app/controllers/account/security_controller.rb:12
- * (UncommunicativeVariableName) Account::SecurityController#update_otp has the variable name 'e'
- - app/controllers/account/security_controller.rb:32
- Asyncs::BillingController:
- Rating: B
- Churn: 0
- Complexity: 65.32
- Duplication: 0
- Smells: 13
- * (HighComplexity) Asyncs::BillingController#payment_method_list has a flog score of 39
- - app/controllers/asyncs/billing_controller.rb:5
- * (HighComplexity) Asyncs::BillingController#unpaid_invoices_list has a flog score of 27
- - app/controllers/asyncs/billing_controller.rb:22
- * (DuplicateMethodCall) Asyncs::BillingController#payment_method_list calls '@payment_method_list.select' 2 times
- - app/controllers/asyncs/billing_controller.rb:12
- - app/controllers/asyncs/billing_controller.rb:13
- * (DuplicateMethodCall) Asyncs::BillingController#payment_method_list calls 'params[:locale]' 2 times
- - app/controllers/asyncs/billing_controller.rb:16
- - app/controllers/asyncs/billing_controller.rb:17
- * (DuplicateMethodCall) Asyncs::BillingController#unpaid_invoices_list calls 'params[:locale]' 2 times
- - app/controllers/asyncs/billing_controller.rb:25
- - app/controllers/asyncs/billing_controller.rb:26
- * (InstanceVariableAssumption) Asyncs::BillingController assumes too much for instance variable '@client_id'
- - app/controllers/asyncs/billing_controller.rb:1
- * (InstanceVariableAssumption) Asyncs::BillingController assumes too much for instance variable '@payment_method_list'
- - app/controllers/asyncs/billing_controller.rb:1
- * (IrresponsibleModule) Asyncs::BillingController has no descriptive comment
- - app/controllers/asyncs/billing_controller.rb:1
- * (NilCheck) Asyncs::BillingController#unpaid_invoices_list performs a nil-check
- - app/controllers/asyncs/billing_controller.rb:33
- * (TooManyInstanceVariables) Asyncs::BillingController has at least 7 instance variables
- - app/controllers/asyncs/billing_controller.rb:1
- * (TooManyStatements) Asyncs::BillingController#payment_method_list has approx 13 statements
- - app/controllers/asyncs/billing_controller.rb:5
- * (TooManyStatements) Asyncs::BillingController#unpaid_invoices_list has approx 11 statements
- - app/controllers/asyncs/billing_controller.rb:22
- * (UncommunicativeVariableName) Asyncs::BillingController#unpaid_invoices_list has the variable name 'v'
- - app/controllers/asyncs/billing_controller.rb:30
- AuthenticationHelper:
- Rating: B
- Churn: 0
- Complexity: 78.03
- Duplication: 0
- Smells: 15
- * (HighComplexity) AuthenticationHelper#remote_registration has a flog score of 36
- - app/helpers/authentication_helper.rb:3
- * (HighComplexity) AuthenticationHelper#create_user has a flog score of 25
- - app/helpers/authentication_helper.rb:35
- * (DuplicateMethodCall) AuthenticationHelper#authenticate calls 'response['credentials']' 3 times
- - app/helpers/authentication_helper.rb:78
- - app/helpers/authentication_helper.rb:84
- * (DuplicateMethodCall) AuthenticationHelper#authenticate calls 'response['credentials']['error_description']' 2 times
- - app/helpers/authentication_helper.rb:78
- * (DuplicateMethodCall) AuthenticationHelper#create_user calls 'params[:email]' 2 times
- - app/helpers/authentication_helper.rb:40
- - app/helpers/authentication_helper.rb:48
- * (DuplicateMethodCall) AuthenticationHelper#remote_registration calls 'fields[:password]' 2 times
- - app/helpers/authentication_helper.rb:6
- - app/helpers/authentication_helper.rb:21
- * (DuplicateMethodCall) AuthenticationHelper#remote_registration calls 'result.error?' 3 times
- - app/helpers/authentication_helper.rb:10
- - app/helpers/authentication_helper.rb:11
- - app/helpers/authentication_helper.rb:12
- * (FeatureEnvy) AuthenticationHelper#authenticate refers to 'response' more than self (maybe move it to another class?)
- - app/helpers/authentication_helper.rb:78
- - app/helpers/authentication_helper.rb:80
- - app/helpers/authentication_helper.rb:84
- * (IrresponsibleModule) AuthenticationHelper has no descriptive comment
- - app/helpers/authentication_helper.rb:1
- * (NilCheck) AuthenticationHelper#remote_registration performs a nil-check
- - app/helpers/authentication_helper.rb:10
- - app/helpers/authentication_helper.rb:17
- * (TooManyStatements) AuthenticationHelper#authenticate has approx 6 statements
- - app/helpers/authentication_helper.rb:76
- * (TooManyStatements) AuthenticationHelper#create_user has approx 7 statements
- - app/helpers/authentication_helper.rb:35
- * (TooManyStatements) AuthenticationHelper#remote_registration has approx 13 statements
- - app/helpers/authentication_helper.rb:3
- * (UncommunicativeVariableName) AuthenticationHelper#create_user has the variable name 'p'
- - app/helpers/authentication_helper.rb:36
- * (UtilityFunction) AuthenticationHelper#create_user doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/authentication_helper.rb:35
- BillingHelper:
- Rating: B
- Churn: 0
- Complexity: 86.02
- Duplication: 0
- Smells: 19
- * (HighComplexity) BillingHelper#get_credit_card_data has a flog score of 26
- - app/helpers/billing_helper.rb:4
- * (DuplicateMethodCall) BillingHelper#find_payment_method calls 'payment['payment_type']' 2 times
- - app/helpers/billing_helper.rb:48
- * (DuplicateMethodCall) BillingHelper#format_card_values calls 'params['cc_num']' 2 times
- - app/helpers/billing_helper.rb:66
- * (DuplicateMethodCall) BillingHelper#get_credit_card_data calls 'card_result.data' 3 times
- - app/helpers/billing_helper.rb:9
- - app/helpers/billing_helper.rb:12
- - app/helpers/billing_helper.rb:14
- * (DuplicateMethodCall) BillingHelper#get_credit_card_data calls 'card_result.data.empty?' 2 times
- - app/helpers/billing_helper.rb:9
- - app/helpers/billing_helper.rb:14
- * (DuplicateMethodCall) BillingHelper#get_credit_card_data calls 'card_result.error?' 2 times
- - app/helpers/billing_helper.rb:9
- - app/helpers/billing_helper.rb:14
- * (DuplicateMethodCall) BillingHelper#get_new_card_data calls 'user.company' 5 times
- - app/helpers/billing_helper.rb:27
- - app/helpers/billing_helper.rb:28
- - app/helpers/billing_helper.rb:29
- - app/helpers/billing_helper.rb:30
- - app/helpers/billing_helper.rb:31
- * (FeatureEnvy) BillingHelper#default_payment_method refers to 'list' more than self (maybe move it to another class?)
- - app/helpers/billing_helper.rb:37
- - app/helpers/billing_helper.rb:43
- * (FeatureEnvy) BillingHelper#default_payment_method refers to 'list_result' more than self (maybe move it to another class?)
- - app/helpers/billing_helper.rb:40
- * (FeatureEnvy) BillingHelper#get_credit_card_data refers to 'card_result' more than self (maybe move it to another class?)
- - app/helpers/billing_helper.rb:9
- - app/helpers/billing_helper.rb:12
- - app/helpers/billing_helper.rb:14
- * (IrresponsibleModule) BillingHelper has no descriptive comment
- - app/helpers/billing_helper.rb:1
- * (NilCheck) BillingHelper#default_payment_method performs a nil-check
- - app/helpers/billing_helper.rb:37
- * (NilCheck) BillingHelper#get_credit_card_data performs a nil-check
- - app/helpers/billing_helper.rb:6
- * (TooManyStatements) BillingHelper#credit_balance has approx 6 statements
- - app/helpers/billing_helper.rb:53
- * (TooManyStatements) BillingHelper#default_payment_method has approx 7 statements
- - app/helpers/billing_helper.rb:36
- * (TooManyStatements) BillingHelper#get_credit_card_data has approx 7 statements
- - app/helpers/billing_helper.rb:4
- * (UtilityFunction) BillingHelper#find_payment_method doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/billing_helper.rb:47
- * (UtilityFunction) BillingHelper#format_card_values doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/billing_helper.rb:65
- * (UtilityFunction) BillingHelper#get_new_card_data doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/billing_helper.rb:20
- CartHelper::CheckoutError:
- Rating: B
- Churn: 0
- Complexity: 90.05
- Duplication: 0
- Smells: 14
- * (VeryHighComplexity) CartHelper#run_checkout has a flog score of 90
- - app/helpers/cart_helper.rb:4
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'Bugsnag.notify(CartHelper::CheckoutError.new(order.error_message))' 2 times
- - app/helpers/cart_helper.rb:15
- - app/helpers/cart_helper.rb:21
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'Environment::Maas.order_queue(brand_id)' 2 times
- - app/helpers/cart_helper.rb:9
- - app/helpers/cart_helper.rb:40
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'api.order' 4 times
- - app/helpers/cart_helper.rb:12
- - app/helpers/cart_helper.rb:18
- - app/helpers/cart_helper.rb:47
- - app/helpers/cart_helper.rb:56
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'button[0]' 2 times
- - app/helpers/cart_helper.rb:30
- - app/helpers/cart_helper.rb:31
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'order.error_message' 4 times
- - app/helpers/cart_helper.rb:14
- - app/helpers/cart_helper.rb:15
- - app/helpers/cart_helper.rb:20
- - app/helpers/cart_helper.rb:21
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'order['order_id']' 3 times
- - app/helpers/cart_helper.rb:18
- - app/helpers/cart_helper.rb:47
- - app/helpers/cart_helper.rb:56
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'process.error?' 2 times
- - app/helpers/cart_helper.rb:19
- - app/helpers/cart_helper.rb:49
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'process.error_message' 2 times
- - app/helpers/cart_helper.rb:50
- - app/helpers/cart_helper.rb:51
- * (DuplicateMethodCall) CartHelper#run_checkout calls 'result[:error] = order.error_message' 2 times
- - app/helpers/cart_helper.rb:14
- - app/helpers/cart_helper.rb:20
- * (FeatureEnvy) CartHelper#run_checkout refers to 'order' more than self (maybe move it to another class?)
- - app/helpers/cart_helper.rb:13
- - app/helpers/cart_helper.rb:14
- - app/helpers/cart_helper.rb:15
- - app/helpers/cart_helper.rb:18
- - app/helpers/cart_helper.rb:20
- - app/helpers/cart_helper.rb:21
- - app/helpers/cart_helper.rb:47
- - app/helpers/cart_helper.rb:56
- * (IrresponsibleModule) CartHelper has no descriptive comment
- - app/helpers/cart_helper.rb:1
- * (NilCheck) CartHelper#run_checkout performs a nil-check
- - app/helpers/cart_helper.rb:46
- * (TooManyStatements) CartHelper#run_checkout has approx 30 statements
- - app/helpers/cart_helper.rb:4
- ChartMogul::Models::Customer:
- Rating: B
- Churn: 0
- Complexity: 65.83
- Duplication: 16
- Smells: 9
- * (DuplicateCode) Similar code found in 2 nodes
- - app/adapters/chart_mogul/models/customer.rb:51
- - app/adapters/chart_mogul/models/plan.rb:85
- * (DuplicateMethodCall) ChartMogul::Models::Customer#body calls 'resp_body.entries' 2 times
- - app/adapters/chart_mogul/models/customer.rb:34
- * (DuplicateMethodCall) ChartMogul::Models::Customer#create_with_validation calls '@resp.body' 2 times
- - app/adapters/chart_mogul/models/customer.rb:40
- - app/adapters/chart_mogul/models/customer.rb:44
- * (DuplicateMethodCall) ChartMogul::Models::Customer#create_with_validation calls '@resp.body.entries' 2 times
- - app/adapters/chart_mogul/models/customer.rb:40
- - app/adapters/chart_mogul/models/customer.rb:44
- * (FeatureEnvy) ChartMogul::Models::Customer#body refers to 'resp_body' more than self (maybe move it to another class?)
- - app/adapters/chart_mogul/models/customer.rb:34
- * (InstanceVariableAssumption) ChartMogul::Models::Customer assumes too much for instance variable '@resp'
- - app/adapters/chart_mogul/models/customer.rb:5
- * (IrresponsibleModule) ChartMogul::Models::Customer has no descriptive comment
- - app/adapters/chart_mogul/models/customer.rb:5
- * (UtilityFunction) ChartMogul::Models::Customer#data_source_uuid doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/chart_mogul/models/customer.rb:76
- * (UtilityFunction) ChartMogul::Models::Customer#to_iso_date doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/chart_mogul/models/customer.rb:71
- ChartMogul::Models::Plan:
- Rating: B
- Churn: 0
- Complexity: 61.75
- Duplication: 16
- Smells: 6
- * (DuplicateCode) Similar code found in 2 nodes
- - app/adapters/chart_mogul/models/customer.rb:51
- - app/adapters/chart_mogul/models/plan.rb:85
- * (DuplicateMethodCall) ChartMogul::Models::Plan#response_body calls 'body.plans' 2 times
- - app/adapters/chart_mogul/models/plan.rb:41
- * (FeatureEnvy) ChartMogul::Models::Plan#response_body refers to 'body' more than self (maybe move it to another class?)
- - app/adapters/chart_mogul/models/plan.rb:41
- * (InstanceVariableAssumption) ChartMogul::Models::Plan assumes too much for instance variable '@resp'
- - app/adapters/chart_mogul/models/plan.rb:5
- * (IrresponsibleModule) ChartMogul::Models::Plan has no descriptive comment
- - app/adapters/chart_mogul/models/plan.rb:5
- * (TooManyInstanceVariables) ChartMogul::Models::Plan has at least 5 instance variables
- - app/adapters/chart_mogul/models/plan.rb:5
- CurrentUser::Company:
- Rating: B
- Churn: 0
- Complexity: 54.14
- Duplication: 0
- Smells: 5
- * (InstanceVariableAssumption) CurrentUser::Company assumes too much for instance variable '@primary_user'
- - app/adapters/current_user/company.rb:2
- * (IrresponsibleModule) CurrentUser::Company has no descriptive comment
- - app/adapters/current_user/company.rb:2
- * (NilCheck) CurrentUser::Company#primary_user performs a nil-check
- - app/adapters/current_user/company.rb:60
- - app/adapters/current_user/company.rb:63
- * (TooManyInstanceVariables) CurrentUser::Company has at least 11 instance variables
- - app/adapters/current_user/company.rb:2
- * (TooManyStatements) CurrentUser::Company#primary_user has approx 6 statements
- - app/adapters/current_user/company.rb:59
- DeviceGraphGenerator:
- Rating: B
- Churn: 0
- Complexity: 50.29
- Duplication: 0
- Smells: 12
- * (DuplicateMethodCall) DeviceGraphGenerator#load_bandwidth_graph calls '@device.id' 2 times
- - app/services/device_graph_generator.rb:45
- - app/services/device_graph_generator.rb:47
- * (InstanceVariableAssumption) DeviceGraphGenerator assumes too much for instance variable '@end_time'
- - app/services/device_graph_generator.rb:3
- * (InstanceVariableAssumption) DeviceGraphGenerator assumes too much for instance variable '@period'
- - app/services/device_graph_generator.rb:3
- * (InstanceVariableAssumption) DeviceGraphGenerator assumes too much for instance variable '@start_time'
- - app/services/device_graph_generator.rb:3
- * (IrresponsibleModule) DeviceGraphGenerator has no descriptive comment
- - app/services/device_graph_generator.rb:3
- * (NilCheck) DeviceGraphGenerator#call performs a nil-check
- - app/services/device_graph_generator.rb:22
- * (RepeatedConditional) DeviceGraphGenerator tests 'custom_period?' at least 3 times
- - app/services/device_graph_generator.rb:35
- - app/services/device_graph_generator.rb:60
- - app/services/device_graph_generator.rb:61
- * (TooManyInstanceVariables) DeviceGraphGenerator has at least 8 instance variables
- - app/services/device_graph_generator.rb:3
- * (TooManyStatements) DeviceGraphGenerator#load_bandwidth_graph has approx 7 statements
- - app/services/device_graph_generator.rb:44
- * (UncommunicativeVariableName) DeviceGraphGenerator#bandwidth_period_params has the variable name 'h'
- - app/services/device_graph_generator.rb:58
- * (UncommunicativeVariableName) DeviceGraphGenerator#load_bandwidth_graph has the variable name 'e'
- - app/services/device_graph_generator.rb:52
- * (UtilityFunction) DeviceGraphGenerator#date_to_year doesn't depend on instance state (maybe move it to another class?)
- - app/services/device_graph_generator.rb:40
- FetchAuth0MetadataService:
- Rating: B
- Churn: 0
- Complexity: 68.39
- Duplication: 0
- Smells: 13
- * (HighComplexity) FetchAuth0MetadataService#fetch_client has a flog score of 31
- - app/services/fetch_auth0_metadata_service.rb:58
- * (DuplicateMethodCall) FetchAuth0MetadataService#fetch_client calls '@data['created']' 2 times
- - app/services/fetch_auth0_metadata_service.rb:60
- - app/services/fetch_auth0_metadata_service.rb:61
- * (DuplicateMethodCall) FetchAuth0MetadataService#fetch_client calls '@result[:app_metadata]' 2 times
- - app/services/fetch_auth0_metadata_service.rb:60
- - app/services/fetch_auth0_metadata_service.rb:61
- * (DuplicateMethodCall) FetchAuth0MetadataService#fetch_client calls '@result[:user_metadata]' 8 times
- - app/services/fetch_auth0_metadata_service.rb:62
- - app/services/fetch_auth0_metadata_service.rb:63
- - app/services/fetch_auth0_metadata_service.rb:64
- - app/services/fetch_auth0_metadata_service.rb:65
- - app/services/fetch_auth0_metadata_service.rb:66
- - app/services/fetch_auth0_metadata_service.rb:67
- - app/services/fetch_auth0_metadata_service.rb:70
- - app/services/fetch_auth0_metadata_service.rb:71
- * (DuplicateMethodCall) FetchAuth0MetadataService#request! calls '@api.client' 2 times
- - app/services/fetch_auth0_metadata_service.rb:29
- - app/services/fetch_auth0_metadata_service.rb:31
- * (InstanceVariableAssumption) FetchAuth0MetadataService assumes too much for instance variable '@client_id'
- - app/services/fetch_auth0_metadata_service.rb:3
- * (InstanceVariableAssumption) FetchAuth0MetadataService assumes too much for instance variable '@contact_id'
- - app/services/fetch_auth0_metadata_service.rb:3
- * (InstanceVariableAssumption) FetchAuth0MetadataService assumes too much for instance variable '@data'
- - app/services/fetch_auth0_metadata_service.rb:3
- * (InstanceVariableAssumption) FetchAuth0MetadataService assumes too much for instance variable '@result'
- - app/services/fetch_auth0_metadata_service.rb:3
- * (IrresponsibleModule) FetchAuth0MetadataService has no descriptive comment
- - app/services/fetch_auth0_metadata_service.rb:3
- * (MissingSafeMethod) FetchAuth0MetadataService has missing safe method 'request!'
- - app/services/fetch_auth0_metadata_service.rb:25
- * (TooManyInstanceVariables) FetchAuth0MetadataService has at least 7 instance variables
- - app/services/fetch_auth0_metadata_service.rb:3
- * (TooManyStatements) FetchAuth0MetadataService#fetch_client has approx 11 statements
- - app/services/fetch_auth0_metadata_service.rb:58
- Index.html.erbSpec:
- Rating: B
- Churn: 0
- Complexity: 5.2
- Duplication: 23
- Smells: 1
- * (DuplicateCode) Identical code found in 2 nodes
- - spec/views/invoices/index.html.erb_spec.rb:7
- - spec/views/invoices/show.html.erb_spec.rb:8
- Production:
- Rating: B
- Churn: 0
- Complexity: 0.0
- Duplication: 21
- Smells: 1
- * (DuplicateCode) Identical code found in 2 nodes
- - config/environments/production.rb:93
- - config/environments/staging.rb:90
- Schema:
- Rating: B
- Churn: 0
- Complexity: 90.4
- Duplication: 0
- Smells: 1
- * (HighComplexity) create_table#users has a flog score of 30
- - db/schema.rb:75
- SecuredHelper:
- Rating: B
- Churn: 0
- Complexity: 57.62
- Duplication: 0
- Smells: 8
- * (IrresponsibleModule) SecuredHelper has no descriptive comment
- - app/helpers/secured_helper.rb:1
- * (NilCheck) SecuredHelper#logout_url performs a nil-check
- - app/helpers/secured_helper.rb:54
- * (TooManyStatements) SecuredHelper#logout_url has approx 6 statements
- - app/helpers/secured_helper.rb:46
- * (UncommunicativeVariableName) SecuredHelper#logout_url has the variable name 'k'
- - app/helpers/secured_helper.rb:54
- * (UncommunicativeVariableName) SecuredHelper#logout_url has the variable name 'v'
- - app/helpers/secured_helper.rb:54
- * (UncommunicativeVariableName) SecuredHelper#raw_user has the variable name 'k'
- - app/helpers/secured_helper.rb:60
- * (UncommunicativeVariableName) SecuredHelper#raw_user has the variable name 'v'
- - app/helpers/secured_helper.rb:60
- * (UtilityFunction) SecuredHelper#raw_user doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/secured_helper.rb:58
- Servers::SshController:
- Rating: B
- Churn: 0
- Complexity: 53.74
- Duplication: 0
- Smells: 2
- * (DuplicateMethodCall) Servers::SshController#validate_target calls 'params[:render_action]' 2 times
- - app/controllers/servers/ssh_controller.rb:59
- - app/controllers/servers/ssh_controller.rb:62
- * (IrresponsibleModule) Servers::SshController has no descriptive comment
- - app/controllers/servers/ssh_controller.rb:1
- Show.html.erbSpec:
- Rating: B
- Churn: 0
- Complexity: 5.2
- Duplication: 23
- Smells: 2
- * (DuplicateCode) Identical code found in 2 nodes
- - spec/views/invoices/index.html.erb_spec.rb:7
- - spec/views/invoices/show.html.erb_spec.rb:8
- * (UnusedParameters) find_device has unused parameter 'service_id'
- - spec/views/invoices/show.html.erb_spec.rb:15
- Staging:
- Rating: B
- Churn: 0
- Complexity: 0.0
- Duplication: 21
- Smells: 1
- * (DuplicateCode) Identical code found in 2 nodes
- - config/environments/production.rb:93
- - config/environments/staging.rb:90
- TokenControllerSpec:
- Rating: B
- Churn: 0
- Complexity: 75.67
- Duplication: 0
- Smells: 0
- Ubersmith:
- Rating: B
- Churn: 0
- Complexity: 91.7
- Duplication: 0
- Smells: 17
- * (HighComplexity) main#call has a flog score of 55
- - lib/modules/ubersmith.rb:41
- * (ControlParameter) call_with_cache is controlled by argument 'is_retry'
- - lib/modules/ubersmith.rb:25
- * (DuplicateMethodCall) call calls '!page.content.nil?' 2 times
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:55
- * (DuplicateMethodCall) call calls '!page.nil?' 2 times
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:55
- * (DuplicateMethodCall) call calls 'page.content' 7 times
- - lib/modules/ubersmith.rb:51
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:54
- - lib/modules/ubersmith.rb:55
- - lib/modules/ubersmith.rb:56
- * (DuplicateMethodCall) call calls 'page.content.nil?' 2 times
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:55
- * (DuplicateMethodCall) call calls 'page.nil?' 2 times
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:55
- * (DuplicateMethodCall) use_as_cache calls 'args[0]' 3 times
- - lib/modules/ubersmith.rb:38
- * (FeatureEnvy) call refers to 'page' more than self (maybe move it to another class?)
- - lib/modules/ubersmith.rb:51
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:54
- - lib/modules/ubersmith.rb:55
- - lib/modules/ubersmith.rb:56
- * (NilCheck) call performs a nil-check
- - lib/modules/ubersmith.rb:49
- - lib/modules/ubersmith.rb:53
- - lib/modules/ubersmith.rb:55
- * (NilCheck) use_as_cache performs a nil-check
- - lib/modules/ubersmith.rb:38
- * (TooManyStatements) call has approx 14 statements
- - lib/modules/ubersmith.rb:41
- * (TooManyStatements) call_with_cache has approx 8 statements
- - lib/modules/ubersmith.rb:17
- * (UncommunicativeVariableName) call has the variable name 'a'
- - lib/modules/ubersmith.rb:47
- * (UncommunicativeVariableName) call has the variable name 'e'
- - lib/modules/ubersmith.rb:52
- * (UncommunicativeVariableName) call_with_cache has the variable name 'e'
- - lib/modules/ubersmith.rb:24
- * (UtilityFunction) use_as_cache doesn't depend on instance state (maybe move it to another class?)
- - lib/modules/ubersmith.rb:37
- Account::ContactsController:
- Rating: C
- Churn: 0
- Complexity: 140.16
- Duplication: 0
- Smells: 16
- * (VeryHighComplexity) Account::ContactsController#create has a flog score of 63
- - app/controllers/account/contacts_controller.rb:14
- * (HighComplexity) Account::ContactsController#edit has a flog score of 32
- - app/controllers/account/contacts_controller.rb:49
- * (DuplicateMethodCall) Account::ContactsController#create calls 'contact_result.error?' 2 times
- - app/controllers/account/contacts_controller.rb:32
- - app/controllers/account/contacts_controller.rb:33
- * (DuplicateMethodCall) Account::ContactsController#create calls 'contact_result.error_message' 2 times
- - app/controllers/account/contacts_controller.rb:35
- - app/controllers/account/contacts_controller.rb:36
- * (DuplicateMethodCall) Account::ContactsController#create calls 'params[:email]' 3 times
- - app/controllers/account/contacts_controller.rb:24
- - app/controllers/account/contacts_controller.rb:26
- - app/controllers/account/contacts_controller.rb:44
- * (DuplicateMethodCall) Account::ContactsController#create calls 'params[:real_name]' 2 times
- - app/controllers/account/contacts_controller.rb:23
- - app/controllers/account/contacts_controller.rb:44
- * (DuplicateMethodCall) Account::ContactsController#create calls 'redirect_to account_contacts_path' 2 times
- - app/controllers/account/contacts_controller.rb:37
- - app/controllers/account/contacts_controller.rb:45
- * (InstanceVariableAssumption) Account::ContactsController assumes too much for instance variable '@contact'
- - app/controllers/account/contacts_controller.rb:1
- * (InstanceVariableAssumption) Account::ContactsController assumes too much for instance variable '@title'
- - app/controllers/account/contacts_controller.rb:1
- * (InstanceVariableAssumption) Account::ContactsController assumes too much for instance variable '@user'
- - app/controllers/account/contacts_controller.rb:1
- * (IrresponsibleModule) Account::ContactsController has no descriptive comment
- - app/controllers/account/contacts_controller.rb:1
- * (NilCheck) Account::ContactsController#edit performs a nil-check
- - app/controllers/account/contacts_controller.rb:52
- * (TooManyStatements) Account::ContactsController#create has approx 16 statements
- - app/controllers/account/contacts_controller.rb:14
- * (TooManyStatements) Account::ContactsController#edit has approx 8 statements
- - app/controllers/account/contacts_controller.rb:49
- * (UncommunicativeVariableName) Account::ContactsController#assign_changed_contact_values has the variable name 'k'
- - app/controllers/account/contacts_controller.rb:78
- * (UncommunicativeVariableName) Account::ContactsController#assign_changed_contact_values has the variable name 'v'
- - app/controllers/account/contacts_controller.rb:78
- ApplicationController:
- Rating: C
- Churn: 0
- Complexity: 116.11
- Duplication: 0
- Smells: 8
- * (HighComplexity) ApplicationController#validate_user_status has a flog score of 27
- - app/controllers/application_controller.rb:79
- * (DuplicateMethodCall) ApplicationController#set_locale calls 'I18n.default_locale' 2 times
- - app/controllers/application_controller.rb:68
- - app/controllers/application_controller.rb:73
- * (DuplicateMethodCall) ApplicationController#set_locale calls 'params[:locale]' 4 times
- - app/controllers/application_controller.rb:70
- - app/controllers/application_controller.rb:71
- * (DuplicateMethodCall) ApplicationController#validate_user_status calls 'current_user.company' 3 times
- - app/controllers/application_controller.rb:80
- - app/controllers/application_controller.rb:85
- - app/controllers/application_controller.rb:87
- * (DuplicateMethodCall) ApplicationController#validate_user_status calls 'current_user.company.signup_step' 3 times
- - app/controllers/application_controller.rb:80
- - app/controllers/application_controller.rb:85
- - app/controllers/application_controller.rb:87
- * (IrresponsibleModule) ApplicationController has no descriptive comment
- - app/controllers/application_controller.rb:1
- * (NilCheck) ApplicationController#check_user_data performs a nil-check
- - app/controllers/application_controller.rb:47
- * (UtilityFunction) ApplicationController#client doesn't depend on instance state (maybe move it to another class?)
- - app/controllers/application_controller.rb:126
- ApplicationHelper:
- Rating: C
- Churn: 0
- Complexity: 172.18
- Duplication: 0
- Smells: 24
- * (HighComplexity) ApplicationHelper#show_avatar has a flog score of 45
- - app/helpers/application_helper.rb:125
- * (HighComplexity) ApplicationHelper#format_pagination has a flog score of 39
- - app/helpers/application_helper.rb:84
- * (ControlParameter) ApplicationHelper#get_url_by_brand is controlled by argument 'brand'
- - app/helpers/application_helper.rb:45
- * (ControlParameter) ApplicationHelper#search_page is controlled by argument 'results_by_page'
- - app/helpers/application_helper.rb:115
- * (ControlParameter) ApplicationHelper#search_page is controlled by argument 'value'
- - app/helpers/application_helper.rb:113
- * (DuplicateMethodCall) ApplicationHelper#format_pagination calls '@current_page*by_page' 2 times
- - app/helpers/application_helper.rb:95
- - app/helpers/application_helper.rb:96
- * (DuplicateMethodCall) ApplicationHelper#format_pagination calls 'value.nil?' 2 times
- - app/helpers/application_helper.rb:85
- - app/helpers/application_helper.rb:87
- * (DuplicateMethodCall) ApplicationHelper#show_avatar calls 'user["real_name"]' 3 times
- - app/helpers/application_helper.rb:129
- - app/helpers/application_helper.rb:130
- * (DuplicateMethodCall) ApplicationHelper#show_avatar calls 'user["real_name"].split(" ")' 2 times
- - app/helpers/application_helper.rb:130
- * (IrresponsibleModule) ApplicationHelper has no descriptive comment
- - app/helpers/application_helper.rb:1
- * (LongParameterList) ApplicationHelper#format_pagination has 4 parameters
- - app/helpers/application_helper.rb:84
- * (LongParameterList) ApplicationHelper#search_page has 4 parameters
- - app/helpers/application_helper.rb:108
- * (NilCheck) ApplicationHelper#format_pagination performs a nil-check
- - app/helpers/application_helper.rb:85
- - app/helpers/application_helper.rb:87
- * (TooManyStatements) ApplicationHelper#format_pagination has approx 15 statements
- - app/helpers/application_helper.rb:84
- * (TooManyStatements) ApplicationHelper#get_currency_by_brand has approx 8 statements
- - app/helpers/application_helper.rb:62
- * (TooManyStatements) ApplicationHelper#search_page has approx 9 statements
- - app/helpers/application_helper.rb:108
- * (TooManyStatements) ApplicationHelper#show_avatar has approx 8 statements
- - app/helpers/application_helper.rb:125
- * (UncommunicativeVariableName) ApplicationHelper#format_pagination has the variable name 't'
- - app/helpers/application_helper.rb:99
- * (UncommunicativeVariableName) ApplicationHelper#search_page has the variable name 't'
- - app/helpers/application_helper.rb:111
- * (UtilityFunction) ApplicationHelper#convert_locale doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/application_helper.rb:35
- * (UtilityFunction) ApplicationHelper#format_date doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/application_helper.rb:142
- * (UtilityFunction) ApplicationHelper#get_currency_by_brand_id doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/application_helper.rb:50
- * (UtilityFunction) ApplicationHelper#get_currency_value_by doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/application_helper.rb:54
- * (UtilityFunction) ApplicationHelper#get_lang_by_brand doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/application_helper.rb:79
- Asyncs::IpsController:
- Rating: C
- Churn: 0
- Complexity: 131.56
- Duplication: 0
- Smells: 25
- * (VeryHighComplexity) Asyncs::IpsController#addon_checkout has a flog score of 75
- - app/controllers/asyncs/ips_controller.rb:16
- * (HighComplexity) Asyncs::IpsController#list has a flog score of 28
- - app/controllers/asyncs/ips_controller.rb:5
- * (HighComplexity) Asyncs::IpsController#group_ips_by_device has a flog score of 25
- - app/controllers/asyncs/ips_controller.rb:58
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'addon[:desserv]' 2 times
- - app/controllers/asyncs/ips_controller.rb:32
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'addon[:did]' 4 times
- - app/controllers/asyncs/ips_controller.rb:34
- - app/controllers/asyncs/ips_controller.rb:37
- - app/controllers/asyncs/ips_controller.rb:38
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'addon[:paymentmethod]' 2 times
- - app/controllers/asyncs/ips_controller.rb:30
- - app/controllers/asyncs/ips_controller.rb:44
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'addon[:reason]' 2 times
- - app/controllers/asyncs/ips_controller.rb:33
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'addon[:return_to]' 2 times
- - app/controllers/asyncs/ips_controller.rb:51
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'device.service_id' 2 times
- - app/controllers/asyncs/ips_controller.rb:39
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'params[:addon]' 2 times
- - app/controllers/asyncs/ips_controller.rb:18
- - app/controllers/asyncs/ips_controller.rb:20
- * (DuplicateMethodCall) Asyncs::IpsController#addon_checkout calls 'result[:error]' 3 times
- - app/controllers/asyncs/ips_controller.rb:46
- - app/controllers/asyncs/ips_controller.rb:49
- * (DuplicateMethodCall) Asyncs::IpsController#group_ips_by_device calls 'ip['device_id']' 4 times
- - app/controllers/asyncs/ips_controller.rb:62
- - app/controllers/asyncs/ips_controller.rb:65
- - app/controllers/asyncs/ips_controller.rb:66
- * (DuplicateMethodCall) Asyncs::IpsController#group_ips_by_device calls 'ip['device_id'].to_i' 3 times
- - app/controllers/asyncs/ips_controller.rb:62
- - app/controllers/asyncs/ips_controller.rb:65
- - app/controllers/asyncs/ips_controller.rb:66
- * (DuplicateMethodCall) Asyncs::IpsController#list calls 'current_user.company_id' 2 times
- - app/controllers/asyncs/ips_controller.rb:6
- - app/controllers/asyncs/ips_controller.rb:10
- * (DuplicateMethodCall) Asyncs::IpsController#list calls 'ips_result.data' 2 times
- - app/controllers/asyncs/ips_controller.rb:7
- * (DuplicateMethodCall) Asyncs::IpsController#list calls 'ips_result.error?' 2 times
- - app/controllers/asyncs/ips_controller.rb:7
- - app/controllers/asyncs/ips_controller.rb:8
- * (FeatureEnvy) Asyncs::IpsController#addon_checkout refers to 'addon' more than self (maybe move it to another class?)
- - app/controllers/asyncs/ips_controller.rb:22
- - app/controllers/asyncs/ips_controller.rb:23
- - app/controllers/asyncs/ips_controller.rb:30
- - app/controllers/asyncs/ips_controller.rb:32
- - app/controllers/asyncs/ips_controller.rb:33
- - app/controllers/asyncs/ips_controller.rb:34
- - app/controllers/asyncs/ips_controller.rb:36
- - app/controllers/asyncs/ips_controller.rb:37
- - app/controllers/asyncs/ips_controller.rb:38
- - app/controllers/asyncs/ips_controller.rb:44
- - app/controllers/asyncs/ips_controller.rb:51
- * (IrresponsibleModule) Asyncs::IpsController has no descriptive comment
- - app/controllers/asyncs/ips_controller.rb:1
- * (NilCheck) Asyncs::IpsController#addon_checkout performs a nil-check
- - app/controllers/asyncs/ips_controller.rb:18
- - app/controllers/asyncs/ips_controller.rb:32
- - app/controllers/asyncs/ips_controller.rb:33
- - app/controllers/asyncs/ips_controller.rb:34
- - app/controllers/asyncs/ips_controller.rb:41
- - app/controllers/asyncs/ips_controller.rb:46
- * (TooManyInstanceVariables) Asyncs::IpsController has at least 5 instance variables
- - app/controllers/asyncs/ips_controller.rb:1
- * (TooManyStatements) Asyncs::IpsController#addon_checkout has approx 18 statements
- - app/controllers/asyncs/ips_controller.rb:16
- * (TooManyStatements) Asyncs::IpsController#group_ips_by_device has approx 10 statements
- - app/controllers/asyncs/ips_controller.rb:58
- * (TooManyStatements) Asyncs::IpsController#list has approx 8 statements
- - app/controllers/asyncs/ips_controller.rb:5
- * (UncommunicativeVariableName) Asyncs::IpsController#group_ips_by_device has the variable name 'v'
- - app/controllers/asyncs/ips_controller.rb:70
- * (UtilityFunction) Asyncs::IpsController#group_ips_by_device doesn't depend on instance state (maybe move it to another class?)
- - app/controllers/asyncs/ips_controller.rb:58
- Asyncs::ServersController:
- Rating: C
- Churn: 0
- Complexity: 107.64
- Duplication: 0
- Smells: 17
- * (HighComplexity) Asyncs::ServersController#reinstall has a flog score of 56
- - app/controllers/asyncs/servers_controller.rb:37
- * (DuplicateMethodCall) Asyncs::ServersController#overview calls '@device.service_id' 2 times
- - app/controllers/asyncs/servers_controller.rb:18
- - app/controllers/asyncs/servers_controller.rb:19
- * (DuplicateMethodCall) Asyncs::ServersController#overview calls '@service['modules']' 3 times
- - app/controllers/asyncs/servers_controller.rb:23
- - app/controllers/asyncs/servers_controller.rb:25
- * (DuplicateMethodCall) Asyncs::ServersController#reinstall calls 'task.data' 2 times
- - app/controllers/asyncs/servers_controller.rb:60
- * (DuplicateMethodCall) Asyncs::ServersController#reinstall calls 'task.data['last_status']' 2 times
- - app/controllers/asyncs/servers_controller.rb:60
- * (InstanceVariableAssumption) Asyncs::ServersController assumes too much for instance variable '@device'
- - app/controllers/asyncs/servers_controller.rb:1
- * (InstanceVariableAssumption) Asyncs::ServersController assumes too much for instance variable '@distro_label_maps'
- - app/controllers/asyncs/servers_controller.rb:1
- * (InstanceVariableAssumption) Asyncs::ServersController assumes too much for instance variable '@graph'
- - app/controllers/asyncs/servers_controller.rb:1
- * (InstanceVariableAssumption) Asyncs::ServersController assumes too much for instance variable '@service'
- - app/controllers/asyncs/servers_controller.rb:1
- * (InstanceVariableAssumption) Asyncs::ServersController assumes too much for instance variable '@system_id'
- - app/controllers/asyncs/servers_controller.rb:1
- * (IrresponsibleModule) Asyncs::ServersController has no descriptive comment
- - app/controllers/asyncs/servers_controller.rb:1
- * (MissingSafeMethod) Asyncs::ServersController has missing safe method 'ensure_graph_present!'
- - app/controllers/asyncs/servers_controller.rb:79
- * (NilCheck) Asyncs::ServersController#ensure_graph_present! performs a nil-check
- - app/controllers/asyncs/servers_controller.rb:80
- * (NilCheck) Asyncs::ServersController#reinstall performs a nil-check
- - app/controllers/asyncs/servers_controller.rb:42
- - app/controllers/asyncs/servers_controller.rb:45
- - app/controllers/asyncs/servers_controller.rb:60
- * (NilCheck) Asyncs::ServersController#set_main_ip performs a nil-check
- - app/controllers/asyncs/servers_controller.rb:93
- * (TooManyInstanceVariables) Asyncs::ServersController has at least 11 instance variables
- - app/controllers/asyncs/servers_controller.rb:1
- * (TooManyStatements) Asyncs::ServersController#reinstall has approx 20 statements
- - app/controllers/asyncs/servers_controller.rb:37
- ChartMogul::Models::InvoicePlan:
- Rating: C
- Churn: 0
- Complexity: 153.65
- Duplication: 0
- Smells: 27
- * (HighComplexity) ChartMogul::Models::InvoicePlan#ensure_plan_exists has a flog score of 31
- - app/adapters/chart_mogul/models/invoice_plan.rb:78
- * (DuplicateMethodCall) ChartMogul::Models::InvoicePlan#as_iso_date calls 'epoch.to_i' 2 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:152
- - app/adapters/chart_mogul/models/invoice_plan.rb:153
- * (DuplicateMethodCall) ChartMogul::Models::InvoicePlan#ensure_plan_exists calls 'payment.has_key?(:refund_id)' 4 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:80
- - app/adapters/chart_mogul/models/invoice_plan.rb:85
- - app/adapters/chart_mogul/models/invoice_plan.rb:88
- - app/adapters/chart_mogul/models/invoice_plan.rb:95
- * (DuplicateMethodCall) ChartMogul::Models::InvoicePlan#line_item_payload calls 'plan.package' 2 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:115
- - app/adapters/chart_mogul/models/invoice_plan.rb:117
- * (DuplicateMethodCall) ChartMogul::Models::InvoicePlan#line_item_payload calls 'plan.recurrent?' 2 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:113
- - app/adapters/chart_mogul/models/invoice_plan.rb:117
- * (DuplicateMethodCall) ChartMogul::Models::InvoicePlan#line_item_recurrent_plan_payload calls 'plan.pack_id' 2 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:124
- - app/adapters/chart_mogul/models/invoice_plan.rb:126
- * (FeatureEnvy) ChartMogul::Models::InvoicePlan#line_item_payload refers to 'h' more than self (maybe move it to another class?)
- - app/adapters/chart_mogul/models/invoice_plan.rb:113
- - app/adapters/chart_mogul/models/invoice_plan.rb:114
- - app/adapters/chart_mogul/models/invoice_plan.rb:115
- - app/adapters/chart_mogul/models/invoice_plan.rb:116
- - app/adapters/chart_mogul/models/invoice_plan.rb:117
- * (FeatureEnvy) ChartMogul::Models::InvoicePlan#line_item_recurrent_plan_payload refers to 'plan' more than self (maybe move it to another class?)
- - app/adapters/chart_mogul/models/invoice_plan.rb:123
- - app/adapters/chart_mogul/models/invoice_plan.rb:124
- - app/adapters/chart_mogul/models/invoice_plan.rb:126
- - app/adapters/chart_mogul/models/invoice_plan.rb:127
- * (InstanceVariableAssumption) ChartMogul::Models::InvoicePlan assumes too much for instance variable '@external_id'
- - app/adapters/chart_mogul/models/invoice_plan.rb:5
- * (InstanceVariableAssumption) ChartMogul::Models::InvoicePlan assumes too much for instance variable '@line_items'
- - app/adapters/chart_mogul/models/invoice_plan.rb:5
- * (IrresponsibleModule) ChartMogul::Models::InvoicePlan has no descriptive comment
- - app/adapters/chart_mogul/models/invoice_plan.rb:5
- * (NilCheck) ChartMogul::Models::InvoicePlan#billing_period_for performs a nil-check
- - app/adapters/chart_mogul/models/invoice_plan.rb:136
- * (NilCheck) ChartMogul::Models::InvoicePlan#ensure_plan_exists performs a nil-check
- - app/adapters/chart_mogul/models/invoice_plan.rb:89
- * (NilCheck) ChartMogul::Models::InvoicePlan#exists? performs a nil-check
- - app/adapters/chart_mogul/models/invoice_plan.rb:157
- * (NilCheck) ChartMogul::Models::InvoicePlan#line_item_payload performs a nil-check
- - app/adapters/chart_mogul/models/invoice_plan.rb:115
- * (NilCheck) ChartMogul::Models::InvoicePlan#push_to_line_items performs a nil-check
- - app/adapters/chart_mogul/models/invoice_plan.rb:103
- * (RepeatedConditional) ChartMogul::Models::InvoicePlan tests 'payment.has_key?(:refund_id)' at least 3 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:85
- - app/adapters/chart_mogul/models/invoice_plan.rb:88
- - app/adapters/chart_mogul/models/invoice_plan.rb:95
- * (RepeatedConditional) ChartMogul::Models::InvoicePlan tests 'plan.recurrent?' at least 3 times
- - app/adapters/chart_mogul/models/invoice_plan.rb:113
- - app/adapters/chart_mogul/models/invoice_plan.rb:117
- - app/adapters/chart_mogul/models/invoice_plan.rb:123
- * (TooManyInstanceVariables) ChartMogul::Models::InvoicePlan has at least 6 instance variables
- - app/adapters/chart_mogul/models/invoice_plan.rb:5
- * (TooManyStatements) ChartMogul::Models::InvoicePlan#billing_period_for has approx 8 statements
- - app/adapters/chart_mogul/models/invoice_plan.rb:133
- * (TooManyStatements) ChartMogul::Models::InvoicePlan#ensure_plan_exists has approx 11 statements
- - app/adapters/chart_mogul/models/invoice_plan.rb:78
- * (TooManyStatements) ChartMogul::Models::InvoicePlan#invoice_payload has approx 6 statements
- - app/adapters/chart_mogul/models/invoice_plan.rb:23
- * (TooManyStatements) ChartMogul::Models::InvoicePlan#line_item_payload has approx 8 statements
- - app/adapters/chart_mogul/models/invoice_plan.rb:111
- * (UncommunicativeVariableName) ChartMogul::Models::InvoicePlan#billing_period_for has the variable name 'h'
- - app/adapters/chart_mogul/models/invoice_plan.rb:134
- * (UncommunicativeVariableName) ChartMogul::Models::InvoicePlan#line_item_payload has the variable name 'h'
- - app/adapters/chart_mogul/models/invoice_plan.rb:112
- * (UtilityFunction) ChartMogul::Models::InvoicePlan#as_iso_date doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/chart_mogul/models/invoice_plan.rb:150
- * (UtilityFunction) ChartMogul::Models::InvoicePlan#exists? doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/chart_mogul/models/invoice_plan.rb:156
- ChartMogul::Smith::BadRequest:
- Rating: C
- Churn: 0
- Complexity: 93.44
- Duplication: 17
- Smells: 7
- * (DuplicateCode) Identical code found in 2 nodes
- - app/adapters/chart_mogul/smith.rb:35
- - app/adapters/current_user/smith.rb:4
- * (HighComplexity) ChartMogul::Smith::parse_invoice has a flog score of 57
- - app/adapters/chart_mogul/smith.rb:14
- * (DuplicateMethodCall) ChartMogul::Smith#parse_invoice calls 'invoices[payment['invid']]' 3 times
- - app/adapters/chart_mogul/smith.rb:28
- - app/adapters/chart_mogul/smith.rb:29
- * (DuplicateMethodCall) ChartMogul::Smith#parse_invoice calls 'payment['invid']' 7 times
- - app/adapters/chart_mogul/smith.rb:17
- - app/adapters/chart_mogul/smith.rb:19
- - app/adapters/chart_mogul/smith.rb:20
- - app/adapters/chart_mogul/smith.rb:24
- - app/adapters/chart_mogul/smith.rb:28
- - app/adapters/chart_mogul/smith.rb:29
- * (IrresponsibleModule) ChartMogul::Smith has no descriptive comment
- - app/adapters/chart_mogul/smith.rb:4
- * (IrresponsibleModule) ChartMogul::Smith::BadRequest has no descriptive comment
- - app/adapters/chart_mogul/smith.rb:59
- * (TooManyStatements) ChartMogul::Smith#parse_invoice has approx 8 statements
- - app/adapters/chart_mogul/smith.rb:14
- CreditInvoiceUbersmithServiceSpec:
- Rating: C
- Churn: 0
- Complexity: 100.89
- Duplication: 62
- Smells: 4
- * (DuplicateCode) Similar code found in 2 nodes
- - spec/services/credit_invoice_ubersmith_service_spec.rb:9
- - spec/services/credit_invoice_ubersmith_service_spec.rb:18
- * (HighComplexity) describe(#call)::context#when had paid by credit has a flog score of 28
- - spec/services/credit_invoice_ubersmith_service_spec.rb:9
- * (HighComplexity) describe(#call)::context#when had not paid by credit has a flog score of 28
- - spec/services/credit_invoice_ubersmith_service_spec.rb:18
- * (HighComplexity) describe(#call)::context#when had error fetching payments has a flog score of 26
- - spec/services/credit_invoice_ubersmith_service_spec.rb:27
- CurrentUser::Client:
- Rating: C
- Churn: 0
- Complexity: 121.0
- Duplication: 0
- Smells: 24
- * (ControlParameter) CurrentUser::Client#method_missing is controlled by argument 'method'
- - app/adapters/current_user/client.rb:100
- * (DuplicateMethodCall) CurrentUser::Client#associate_two_factor calls '@session[:inactive_otp]' 2 times
- - app/adapters/current_user/client.rb:137
- * (DuplicateMethodCall) CurrentUser::Client#token calls '@session[:api_token]' 2 times
- - app/adapters/current_user/client.rb:21
- - app/adapters/current_user/client.rb:22
- * (InstanceVariableAssumption) CurrentUser::Client assumes too much for instance variable '@company_id'
- - app/adapters/current_user/client.rb:2
- * (InstanceVariableAssumption) CurrentUser::Client assumes too much for instance variable '@contact_id'
- - app/adapters/current_user/client.rb:2
- * (IrresponsibleModule) CurrentUser::Client has no descriptive comment
- - app/adapters/current_user/client.rb:2
- * (MissingSafeMethod) CurrentUser::Client has missing safe method 'update!'
- - app/adapters/current_user/client.rb:37
- * (NilCheck) CurrentUser::Client#associate_two_factor performs a nil-check
- - app/adapters/current_user/client.rb:137
- * (NilCheck) CurrentUser::Client#enable_two_factor performs a nil-check
- - app/adapters/current_user/client.rb:69
- * (NilCheck) CurrentUser::Client#provisioning_uri performs a nil-check
- - app/adapters/current_user/client.rb:57
- * (NilCheck) CurrentUser::Client#token performs a nil-check
- - app/adapters/current_user/client.rb:21
- * (RepeatedConditional) CurrentUser::Client tests 'otp_enabled?' at least 3 times
- - app/adapters/current_user/client.rb:54
- - app/adapters/current_user/client.rb:75
- - app/adapters/current_user/client.rb:135
- * (TooManyInstanceVariables) CurrentUser::Client has at least 12 instance variables
- - app/adapters/current_user/client.rb:2
- * (TooManyMethods) CurrentUser::Client has at least 18 methods
- - app/adapters/current_user/client.rb:2
- * (TooManyStatements) CurrentUser::Client#associate_two_factor has approx 6 statements
- - app/adapters/current_user/client.rb:134
- * (TooManyStatements) CurrentUser::Client#disable_two_factor has approx 7 statements
- - app/adapters/current_user/client.rb:74
- * (TooManyStatements) CurrentUser::Client#set_attributes has approx 9 statements
- - app/adapters/current_user/client.rb:105
- * (TooManyStatements) CurrentUser::Client#update! has approx 6 statements
- - app/adapters/current_user/client.rb:37
- * (TooManyStatements) CurrentUser::Client#update_session has approx 6 statements
- - app/adapters/current_user/client.rb:121
- * (UncommunicativeMethodName) CurrentUser::Client#mfa_auth0 has the name 'mfa_auth0'
- - app/adapters/current_user/client.rb:146
- * (UncommunicativeVariableName) CurrentUser::Client has the variable name '@mfa_auth0'
- - app/adapters/current_user/client.rb:147
- * (UncommunicativeVariableName) CurrentUser::Client#update_session has the variable name 'k'
- - app/adapters/current_user/client.rb:122
- * (UtilityFunction) CurrentUser::Client#has_new_password? doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/current_user/client.rb:90
- * (UtilityFunction) CurrentUser::Client#is_new_password_match? doesn't depend on instance state (maybe move it to another class?)
- - app/adapters/current_user/client.rb:94
- CurrentUser::Smith:
- Rating: C
- Churn: 0
- Complexity: 108.54
- Duplication: 17
- Smells: 12
- * (DuplicateCode) Identical code found in 2 nodes
- - app/adapters/chart_mogul/smith.rb:35
- - app/adapters/current_user/smith.rb:4
- * (HighComplexity) CurrentUser::Smith::client_sync has a flog score of 30
- - app/adapters/current_user/smith.rb:42
- * (ControlParameter) CurrentUser::Smith#language is controlled by argument 'text'
- - app/adapters/current_user/smith.rb:95
- * (ControlParameter) CurrentUser::Smith#sync! is controlled by argument 'type'
- - app/adapters/current_user/smith.rb:9
- * (DuplicateMethodCall) CurrentUser::Smith#client_sync calls 'data[:company]' 2 times
- - app/adapters/current_user/smith.rb:56
- - app/adapters/current_user/smith.rb:57
- * (DuplicateMethodCall) CurrentUser::Smith#sync! calls 'client_sync(current_user.company_id, data)' 2 times
- - app/adapters/current_user/smith.rb:11
- - app/adapters/current_user/smith.rb:14
- * (DuplicateMethodCall) CurrentUser::Smith#sync! calls 'current_user.company_id' 2 times
- - app/adapters/current_user/smith.rb:11
- - app/adapters/current_user/smith.rb:14
- * (DuplicateMethodCall) CurrentUser::Smith#sync! calls 'current_user.contact_id' 2 times
- - app/adapters/current_user/smith.rb:13
- - app/adapters/current_user/smith.rb:15
- * (IrresponsibleModule) CurrentUser::Smith has no descriptive comment
- - app/adapters/current_user/smith.rb:2
- * (TooManyStatements) CurrentUser::Smith#client_sync has approx 9 statements
- - app/adapters/current_user/smith.rb:42
- * (TooManyStatements) CurrentUser::Smith#contact_sync has approx 11 statements
- - app/adapters/current_user/smith.rb:64
- * (UncommunicativeVariableName) CurrentUser::Smith#contact_sync has the variable name 'i'
- - app/adapters/current_user/smith.rb:70
- Environment::Ubersmith:
- Rating: C
- Churn: 0
- Complexity: 54.99
- Duplication: 160
- Smells: 9
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/environment.rb:8
- - lib/modules/environment.rb:10
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/environment.rb:18
- - lib/modules/environment.rb:22
- * (DuplicateCode) Similar code found in 6 nodes
- - lib/modules/environment.rb:28
- - lib/modules/environment.rb:32
- - lib/modules/environment.rb:54
- - lib/modules/environment.rb:58
- - lib/modules/environment.rb:62
- - lib/modules/environment.rb:66
- * (DuplicateMethodCall) Environment#self.choice calls 'brand != 0' 2 times
- - lib/modules/environment.rb:9
- - lib/modules/environment.rb:11
- * (DuplicateMethodCall) Environment#self.choice calls 'brand.to_i' 2 times
- - lib/modules/environment.rb:9
- - lib/modules/environment.rb:11
- * (IrresponsibleModule) Environment has no descriptive comment
- - lib/modules/environment.rb:1
- * (IrresponsibleModule) Environment::Addon has no descriptive comment
- - lib/modules/environment.rb:53
- * (IrresponsibleModule) Environment::Maas has no descriptive comment
- - lib/modules/environment.rb:27
- * (IrresponsibleModule) Environment::Ubersmith has no descriptive comment
- - lib/modules/environment.rb:17
- InvoicePresenterSpec:
- Rating: C
- Churn: 0
- Complexity: 121.83
- Duplication: 48
- Smells: 1
- * (DuplicateCode) Similar code found in 3 nodes
- - spec/presenters/invoice_presenter_spec.rb:6
- - spec/presenters/invoice_presenter_spec.rb:16
- - spec/presenters/invoice_presenter_spec.rb:26
- PaypalHelper:
- Rating: C
- Churn: 0
- Complexity: 97.96
- Duplication: 36
- Smells: 17
- * (DuplicateCode) Identical code found in 2 nodes
- - app/helpers/paypal_helper.rb:10
- - app/helpers/paypal_helper.rb:33
- * (HighComplexity) PaypalHelper#call_token has a flog score of 35
- - app/helpers/paypal_helper.rb:3
- * (HighComplexity) PaypalHelper#approve_billing_agreement has a flog score of 32
- - app/helpers/paypal_helper.rb:26
- * (HighComplexity) PaypalHelper#find_paypal_billing_agreements has a flog score of 25
- - app/helpers/paypal_helper.rb:54
- * (DuplicateMethodCall) PaypalHelper#approve_billing_agreement calls 'api.client' 3 times
- - app/helpers/paypal_helper.rb:29
- - app/helpers/paypal_helper.rb:35
- - app/helpers/paypal_helper.rb:36
- * (DuplicateMethodCall) PaypalHelper#approve_billing_agreement calls 'ba['billing_info_id']' 2 times
- - app/helpers/paypal_helper.rb:35
- - app/helpers/paypal_helper.rb:36
- * (DuplicateMethodCall) PaypalHelper#approve_billing_agreement calls 'result.error?' 2 times
- - app/helpers/paypal_helper.rb:30
- - app/helpers/paypal_helper.rb:36
- * (DuplicateMethodCall) PaypalHelper#call_token calls 'api.client' 3 times
- - app/helpers/paypal_helper.rb:9
- - app/helpers/paypal_helper.rb:13
- - app/helpers/paypal_helper.rb:17
- * (DuplicateMethodCall) PaypalHelper#call_token calls 'request.original_url' 2 times
- - app/helpers/paypal_helper.rb:20
- - app/helpers/paypal_helper.rb:21
- * (FeatureEnvy) PaypalHelper#call_token refers to 'api' more than self (maybe move it to another class?)
- - app/helpers/paypal_helper.rb:9
- - app/helpers/paypal_helper.rb:13
- - app/helpers/paypal_helper.rb:17
- * (FeatureEnvy) PaypalHelper#call_token refers to 'ba' more than self (maybe move it to another class?)
- - app/helpers/paypal_helper.rb:10
- - app/helpers/paypal_helper.rb:13
- * (IrresponsibleModule) PaypalHelper has no descriptive comment
- - app/helpers/paypal_helper.rb:1
- * (NilCheck) PaypalHelper#approve_billing_agreement performs a nil-check
- - app/helpers/paypal_helper.rb:34
- * (TooManyStatements) PaypalHelper#approve_billing_agreement has approx 9 statements
- - app/helpers/paypal_helper.rb:26
- * (TooManyStatements) PaypalHelper#call_token has approx 8 statements
- - app/helpers/paypal_helper.rb:3
- * (UtilityFunction) PaypalHelper#approve_billing_agreement doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/paypal_helper.rb:26
- * (UtilityFunction) PaypalHelper#find_paypal_billing_agreements doesn't depend on instance state (maybe move it to another class?)
- - app/helpers/paypal_helper.rb:54
- RegistrationsController:
- Rating: C
- Churn: 0
- Complexity: 188.71
- Duplication: 0
- Smells: 28
- * (HighComplexity) RegistrationsController#create has a flog score of 36
- - app/controllers/registrations_controller.rb:19
- * (HighComplexity) RegistrationsController#run_risk_check has a flog score of 36
- - app/controllers/registrations_controller.rb:87
- * (DuplicateMethodCall) RegistrationsController#create calls 'redirect_to new_user_registration_path' 2 times
- - app/controllers/registrations_controller.rb:33
- - app/controllers/registrations_controller.rb:37
- * (DuplicateMethodCall) RegistrationsController#run_risk_check calls 'current_user.company' 4 times
- - app/controllers/registrations_controller.rb:91
- - app/controllers/registrations_controller.rb:92
- - app/controllers/registrations_controller.rb:97
- - app/controllers/registrations_controller.rb:112
- * (DuplicateMethodCall) RegistrationsController#sign_up_params calls 'user[:company]' 3 times
- - app/controllers/registrations_controller.rb:154
- - app/controllers/registrations_controller.rb:155
- * (DuplicateMethodCall) RegistrationsController#update calls 'redirect_to edit_user_registration_path' 2 times
- - app/controllers/registrations_controller.rb:62
- - app/controllers/registrations_controller.rb:65
- * (FeatureEnvy) RegistrationsController#sign_up_params refers to 'user' more than self (maybe move it to another class?)
- - app/controllers/registrations_controller.rb:153
- - app/controllers/registrations_controller.rb:154
- - app/controllers/registrations_controller.rb:155
- * (InstanceVariableAssumption) RegistrationsController assumes too much for instance variable '@ip_data'
- - app/controllers/registrations_controller.rb:1
- * (IrresponsibleModule) RegistrationsController has no descriptive comment
- - app/controllers/registrations_controller.rb:1
- * (MissingSafeMethod) RegistrationsController has missing safe method 'ensure_has_valid_new_password!'
- - app/controllers/registrations_controller.rb:136
- * (MissingSafeMethod) RegistrationsController has missing safe method 'ensure_has_valid_password!'
- - app/controllers/registrations_controller.rb:131
- * (MissingSafeMethod) RegistrationsController has missing safe method 'update_company!'
- - app/controllers/registrations_controller.rb:80
- * (MissingSafeMethod) RegistrationsController has missing safe method 'validate_signup_data!'
- - app/controllers/registrations_controller.rb:118
- * (NilCheck) RegistrationsController#ip_data performs a nil-check
- - app/controllers/registrations_controller.rb:71
- * (NilCheck) RegistrationsController#validate_signup_data! performs a nil-check
- - app/controllers/registrations_controller.rb:121
- * (TooManyInstanceVariables) RegistrationsController has at least 5 instance variables
- - app/controllers/registrations_controller.rb:1
- * (TooManyMethods) RegistrationsController has at least 17 methods
- - app/controllers/registrations_controller.rb:1
- * (TooManyStatements) RegistrationsController#create has approx 14 statements
- - app/controllers/registrations_controller.rb:19
- * (TooManyStatements) RegistrationsController#ip_data has approx 6 statements
- - app/controllers/registrations_controller.rb:70
- * (TooManyStatements) RegistrationsController#risk_check has approx 7 statements
- - app/controllers/registrations_controller.rb:40
- * (TooManyStatements) RegistrationsController#run_risk_check has approx 6 statements
- - app/controllers/registrations_controller.rb:87
- * (TooManyStatements) RegistrationsController#sign_up_params has approx 6 statements
- - app/controllers/registrations_controller.rb:150
- * (TooManyStatements) RegistrationsController#update has approx 8 statements
- - app/controllers/registrations_controller.rb:55
- * (UncommunicativeVariableName) RegistrationsController#create has the variable name 'e'
- - app/controllers/registrations_controller.rb:35
- * (UncommunicativeVariableName) RegistrationsController#ip_data has the variable name 'e'
- - app/controllers/registrations_controller.rb:76
- * (UncommunicativeVariableName) RegistrationsController#risk_check has the variable name 'e'
- - app/controllers/registrations_controller.rb:46
- * (UncommunicativeVariableName) RegistrationsController#update has the variable name 'e'
- - app/controllers/registrations_controller.rb:63
- * (UtilityFunction) RegistrationsController#error_message doesn't depend on instance state (maybe move it to another class?)
- - app/controllers/registrations_controller.rb:143
- RemoveCart:
- Rating: C
- Churn: 0
- Complexity: 115.63
- Duplication: 0
- Smells: 13
- * (VeryHighComplexity) RemoveCart#change has a flog score of 116
- - db/migrate/20181219175613_remove_cart.rb:2
- * (DuplicateMethodCall) RemoveCart#change calls 't.datetime "created_at", null: false' 8 times
- - db/migrate/20181219175613_remove_cart.rb:9
- - db/migrate/20181219175613_remove_cart.rb:23
- - db/migrate/20181219175613_remove_cart.rb:31
- - db/migrate/20181219175613_remove_cart.rb:41
- - db/migrate/20181219175613_remove_cart.rb:53
- - db/migrate/20181219175613_remove_cart.rb:63
- - db/migrate/20181219175613_remove_cart.rb:78
- - db/migrate/20181219175613_remove_cart.rb:95
- * (DuplicateMethodCall) RemoveCart#change calls 't.datetime "updated_at", null: false' 8 times
- - db/migrate/20181219175613_remove_cart.rb:10
- - db/migrate/20181219175613_remove_cart.rb:24
- - db/migrate/20181219175613_remove_cart.rb:32
- - db/migrate/20181219175613_remove_cart.rb:42
- - db/migrate/20181219175613_remove_cart.rb:54
- - db/migrate/20181219175613_remove_cart.rb:64
- - db/migrate/20181219175613_remove_cart.rb:79
- - db/migrate/20181219175613_remove_cart.rb:96
- * (DuplicateMethodCall) RemoveCart#change calls 't.integer "plan_id"' 3 times
- - db/migrate/20181219175613_remove_cart.rb:21
- - db/migrate/20181219175613_remove_cart.rb:48
- - db/migrate/20181219175613_remove_cart.rb:104
- * (DuplicateMethodCall) RemoveCart#change calls 't.integer "pog_id"' 2 times
- - db/migrate/20181219175613_remove_cart.rb:28
- - db/migrate/20181219175613_remove_cart.rb:75
- * (DuplicateMethodCall) RemoveCart#change calls 't.integer "pu_id"' 4 times
- - db/migrate/20181219175613_remove_cart.rb:20
- - db/migrate/20181219175613_remove_cart.rb:30
- - db/migrate/20181219175613_remove_cart.rb:70
- - db/migrate/20181219175613_remove_cart.rb:87
- * (DuplicateMethodCall) RemoveCart#change calls 't.string "name"' 2 times
- - db/migrate/20181219175613_remove_cart.rb:49
- - db/migrate/20181219175613_remove_cart.rb:62
- * (DuplicateMethodCall) RemoveCart#change calls 't.string "pog_name"' 2 times
- - db/migrate/20181219175613_remove_cart.rb:29
- - db/migrate/20181219175613_remove_cart.rb:76
- * (DuplicateMethodCall) RemoveCart#change calls 't.text "names", default: [], array: true' 3 times
- - db/migrate/20181219175613_remove_cart.rb:33
- - db/migrate/20181219175613_remove_cart.rb:81
- - db/migrate/20181219175613_remove_cart.rb:97
- * (FeatureEnvy) RemoveCart#change refers to 't' more than self (maybe move it to another class?)
- - db/migrate/20181219175613_remove_cart.rb:4
- - db/migrate/20181219175613_remove_cart.rb:5
- - db/migrate/20181219175613_remove_cart.rb:6
- - db/migrate/20181219175613_remove_cart.rb:7
- - db/migrate/20181219175613_remove_cart.rb:8
- - db/migrate/20181219175613_remove_cart.rb:9
- - db/migrate/20181219175613_remove_cart.rb:10
- - db/migrate/20181219175613_remove_cart.rb:11
- - db/migrate/20181219175613_remove_cart.rb:12
- - db/migrate/20181219175613_remove_cart.rb:13
- - db/migrate/20181219175613_remove_cart.rb:14
- - db/migrate/20181219175613_remove_cart.rb:15
- - db/migrate/20181219175613_remove_cart.rb:16
- - db/migrate/20181219175613_remove_cart.rb:20
- - db/migrate/20181219175613_remove_cart.rb:21
- - db/migrate/20181219175613_remove_cart.rb:22
- - db/migrate/20181219175613_remove_cart.rb:23
- - db/migrate/20181219175613_remove_cart.rb:24
- - db/migrate/20181219175613_remove_cart.rb:28
- - db/migrate/20181219175613_remove_cart.rb:29
- - db/migrate/20181219175613_remove_cart.rb:30
- - db/migrate/20181219175613_remove_cart.rb:31
- - db/migrate/20181219175613_remove_cart.rb:32
- - db/migrate/20181219175613_remove_cart.rb:33
- - db/migrate/20181219175613_remove_cart.rb:37
- - db/migrate/20181219175613_remove_cart.rb:38
- - db/migrate/20181219175613_remove_cart.rb:39
- - db/migrate/20181219175613_remove_cart.rb:40
- - db/migrate/20181219175613_remove_cart.rb:41
- - db/migrate/20181219175613_remove_cart.rb:42
- - db/migrate/20181219175613_remove_cart.rb:43
- - db/migrate/20181219175613_remove_cart.rb:44
- - db/migrate/20181219175613_remove_cart.rb:48
- - db/migrate/20181219175613_remove_cart.rb:49
- - db/migrate/20181219175613_remove_cart.rb:50
- - db/migrate/20181219175613_remove_cart.rb:51
- - db/migrate/20181219175613_remove_cart.rb:52
- - db/migrate/20181219175613_remove_cart.rb:53
- - db/migrate/20181219175613_remove_cart.rb:54
- - db/migrate/20181219175613_remove_cart.rb:55
- - db/migrate/20181219175613_remove_cart.rb:56
- - db/migrate/20181219175613_remove_cart.rb:57
- - db/migrate/20181219175613_remove_cart.rb:58
- - db/migrate/20181219175613_remove_cart.rb:62
- - db/migrate/20181219175613_remove_cart.rb:63
- - db/migrate/20181219175613_remove_cart.rb:64
- - db/migrate/20181219175613_remove_cart.rb:68
- - db/migrate/20181219175613_remove_cart.rb:69
- - db/migrate/20181219175613_remove_cart.rb:70
- - db/migrate/20181219175613_remove_cart.rb:71
- - db/migrate/20181219175613_remove_cart.rb:72
- - db/migrate/20181219175613_remove_cart.rb:73
- - db/migrate/20181219175613_remove_cart.rb:74
- - db/migrate/20181219175613_remove_cart.rb:75
- - db/migrate/20181219175613_remove_cart.rb:76
- - db/migrate/20181219175613_remove_cart.rb:77
- - db/migrate/20181219175613_remove_cart.rb:78
- - db/migrate/20181219175613_remove_cart.rb:79
- - db/migrate/20181219175613_remove_cart.rb:80
- - db/migrate/20181219175613_remove_cart.rb:81
- - db/migrate/20181219175613_remove_cart.rb:82
- - db/migrate/20181219175613_remove_cart.rb:83
- - db/migrate/20181219175613_remove_cart.rb:87
- - db/migrate/20181219175613_remove_cart.rb:88
- - db/migrate/20181219175613_remove_cart.rb:89
- - db/migrate/20181219175613_remove_cart.rb:90
- - db/migrate/20181219175613_remove_cart.rb:91
- - db/migrate/20181219175613_remove_cart.rb:92
- - db/migrate/20181219175613_remove_cart.rb:93
- - db/migrate/20181219175613_remove_cart.rb:94
- - db/migrate/20181219175613_remove_cart.rb:95
- - db/migrate/20181219175613_remove_cart.rb:96
- - db/migrate/20181219175613_remove_cart.rb:97
- - db/migrate/20181219175613_remove_cart.rb:98
- - db/migrate/20181219175613_remove_cart.rb:99
- - db/migrate/20181219175613_remove_cart.rb:100
- - db/migrate/20181219175613_remove_cart.rb:104
- - db/migrate/20181219175613_remove_cart.rb:105
- - db/migrate/20181219175613_remove_cart.rb:106
- - db/migrate/20181219175613_remove_cart.rb:110
- - db/migrate/20181219175613_remove_cart.rb:111
- - db/migrate/20181219175613_remove_cart.rb:112
- - db/migrate/20181219175613_remove_cart.rb:113
- * (IrresponsibleModule) RemoveCart has no descriptive comment
- - db/migrate/20181219175613_remove_cart.rb:1
- * (TooManyStatements) RemoveCart#change has approx 93 statements
- - db/migrate/20181219175613_remove_cart.rb:2
- * (UncommunicativeVariableName) RemoveCart#change has the variable name 't'
- - db/migrate/20181219175613_remove_cart.rb:3
- - db/migrate/20181219175613_remove_cart.rb:19
- - db/migrate/20181219175613_remove_cart.rb:27
- - db/migrate/20181219175613_remove_cart.rb:36
- - db/migrate/20181219175613_remove_cart.rb:47
- - db/migrate/20181219175613_remove_cart.rb:61
- - db/migrate/20181219175613_remove_cart.rb:67
- - db/migrate/20181219175613_remove_cart.rb:86
- - db/migrate/20181219175613_remove_cart.rb:103
- - db/migrate/20181219175613_remove_cart.rb:109
- ServerHelper:
- Rating: C
- Churn: 0
- Complexity: 164.01
- Duplication: 0
- Smells: 25
- * (VeryHighComplexity) ServerHelper#format_plans has a flog score of 130
- - app/helpers/server_helper.rb:31
- * (DuplicateMethodCall) ServerHelper#device_specs calls 'device.type_id' 2 times
- - app/helpers/server_helper.rb:17
- - app/helpers/server_helper.rb:19
- * (DuplicateMethodCall) ServerHelper#device_specs calls 'device.type_id.to_i' 2 times
- - app/helpers/server_helper.rb:17
- - app/helpers/server_helper.rb:19
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'Environment::Maas.order_queue(brand_id)' 2 times
- - app/helpers/server_helper.rb:83
- - app/helpers/server_helper.rb:86
- * (DuplicateMethodCall) ServerHelper#format_plans calls '_('minutes')' 2 times
- - app/helpers/server_helper.rb:84
- - app/helpers/server_helper.rb:87
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'brand_id.to_i' 2 times
- - app/helpers/server_helper.rb:39
- - app/helpers/server_helper.rb:52
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'descriptions << text' 3 times
- - app/helpers/server_helper.rb:64
- - app/helpers/server_helper.rb:71
- - app/helpers/server_helper.rb:77
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'drive[:count]' 2 times
- - app/helpers/server_helper.rb:75
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'plan[:deploy_type] == 'automated'' 2 times
- - app/helpers/server_helper.rb:83
- - app/helpers/server_helper.rb:84
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'plan[:deploy_type]' 2 times
- - app/helpers/server_helper.rb:83
- - app/helpers/server_helper.rb:84
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'plan[:in_stock]' 3 times
- - app/helpers/server_helper.rb:86
- - app/helpers/server_helper.rb:87
- - app/helpers/server_helper.rb:99
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'plan[:name]' 2 times
- - app/helpers/server_helper.rb:32
- - app/helpers/server_helper.rb:92
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'plan[:pricing]' 2 times
- - app/helpers/server_helper.rb:53
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'plan[:slug]' 3 times
- - app/helpers/server_helper.rb:45
- - app/helpers/server_helper.rb:91
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'server[:options]' 2 times
- - app/helpers/server_helper.rb:47
- - app/helpers/server_helper.rb:102
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'value['cores']' 2 times
- - app/helpers/server_helper.rb:66
- - app/helpers/server_helper.rb:67
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'value['count']' 3 times
- - app/helpers/server_helper.rb:60
- - app/helpers/server_helper.rb:61
- * (DuplicateMethodCall) ServerHelper#format_plans calls 'value['type']' 2 times
- - app/helpers/server_helper.rb:63
- * (FeatureEnvy) ServerHelper#device_specs refers to 'device' more than self (maybe move it to another class?)
- - app/helpers/server_helper.rb:16
- - app/helpers/server_helper.rb:17
- - app/helpers/server_helper.rb:18
- - app/helpers/server_helper.rb:19
- * (FeatureEnvy) ServerHelper#format_plans refers to 'plan' more than self (maybe move it to another class?)
- - app/helpers/server_helper.rb:32
- - app/helpers/server_helper.rb:45
- - app/helpers/server_helper.rb:53
- - app/helpers/server_helper.rb:56
- - app/helpers/server_helper.rb:83
- - app/helpers/server_helper.rb:84
- - app/helpers/server_helper.rb:86
- - app/helpers/server_helper.rb:87
- - app/helpers/server_helper.rb:91
- - app/helpers/server_helper.rb:92
- - app/helpers/server_helper.rb:99
- * (IrresponsibleModule) ServerHelper has no descriptive comment
- - app/helpers/server_helper.rb:1
- * (NestedIterators) ServerHelper#format_plans contains iterators nested 4 deep
- - app/helpers/server_helper.rb:73
- * (NilCheck) ServerHelper#device_specs performs a nil-check
- - app/helpers/server_helper.rb:20
- * (NilCheck) ServerHelper#format_plans performs a nil-check
- - app/helpers/server_helper.rb:47
- - app/helpers/server_helper.rb:53
- * (TooManyStatements) ServerHelper#format_plans has approx 43 statements
- - app/helpers/server_helper.rb:31
- ServicesController:
- Rating: C
- Churn: 0
- Complexity: 189.2
- Duplication: 0
- Smells: 32
- * (VeryHighComplexity) ServicesController#show has a flog score of 157
- - app/controllers/services_controller.rb:29
- * (HighComplexity) ServicesController#index has a flog score of 33
- - app/controllers/services_controller.rb:5
- * (DuplicateMethodCall) ServicesController#show calls '@children[key]' 7 times
- - app/controllers/services_controller.rb:85
- - app/controllers/services_controller.rb:86
- - app/controllers/services_controller.rb:90
- - app/controllers/services_controller.rb:92
- - app/controllers/services_controller.rb:94
- - app/controllers/services_controller.rb:96
- - app/controllers/services_controller.rb:98
- * (DuplicateMethodCall) ServicesController#show calls '@options[pug_name]' 2 times
- - app/controllers/services_controller.rb:49
- - app/controllers/services_controller.rb:50
- * (DuplicateMethodCall) ServicesController#show calls 'ENV['UBERSMITH_DOMAIN']' 2 times
- - app/controllers/services_controller.rb:30
- - app/controllers/services_controller.rb:72
- * (DuplicateMethodCall) ServicesController#show calls 'ENV['UBERSMITH_PASSWORD']' 2 times
- - app/controllers/services_controller.rb:30
- - app/controllers/services_controller.rb:72
- * (DuplicateMethodCall) ServicesController#show calls 'ENV['UBERSMITH_USERNAME']' 2 times
- - app/controllers/services_controller.rb:30
- - app/controllers/services_controller.rb:72
- * (DuplicateMethodCall) ServicesController#show calls '_("Annually")' 2 times
- - app/controllers/services_controller.rb:68
- - app/controllers/services_controller.rb:98
- * (DuplicateMethodCall) ServicesController#show calls '_("Monthly")' 2 times
- - app/controllers/services_controller.rb:62
- - app/controllers/services_controller.rb:92
- * (DuplicateMethodCall) ServicesController#show calls '_("One Time")' 2 times
- - app/controllers/services_controller.rb:60
- - app/controllers/services_controller.rb:90
- * (DuplicateMethodCall) ServicesController#show calls '_("Quarterly")' 2 times
- - app/controllers/services_controller.rb:64
- - app/controllers/services_controller.rb:94
- * (DuplicateMethodCall) ServicesController#show calls '_("Semi-Annually")' 2 times
- - app/controllers/services_controller.rb:66
- - app/controllers/services_controller.rb:96
- * (DuplicateMethodCall) ServicesController#show calls 'api.client' 2 times
- - app/controllers/services_controller.rb:31
- - app/controllers/services_controller.rb:73
- * (DuplicateMethodCall) ServicesController#show calls 'children.data' 2 times
- - app/controllers/services_controller.rb:78
- - app/controllers/services_controller.rb:82
- * (DuplicateMethodCall) ServicesController#show calls 'children.error?' 2 times
- - app/controllers/services_controller.rb:78
- - app/controllers/services_controller.rb:81
- * (DuplicateMethodCall) ServicesController#show calls 'current_user.company_id' 2 times
- - app/controllers/services_controller.rb:38
- - app/controllers/services_controller.rb:74
- * (DuplicateMethodCall) ServicesController#show calls 'get_currency_by_brand(params[:locale])' 2 times
- - app/controllers/services_controller.rb:35
- - app/controllers/services_controller.rb:79
- * (DuplicateMethodCall) ServicesController#show calls 'option['pu_id']' 2 times
- - app/controllers/services_controller.rb:45
- - app/controllers/services_controller.rb:46
- * (DuplicateMethodCall) ServicesController#show calls 'params[:id]' 2 times
- - app/controllers/services_controller.rb:31
- - app/controllers/services_controller.rb:75
- * (DuplicateMethodCall) ServicesController#show calls 'params[:locale]' 2 times
- - app/controllers/services_controller.rb:35
- - app/controllers/services_controller.rb:79
- * (DuplicateMethodCall) ServicesController#show calls 'service_result.error?' 2 times
- - app/controllers/services_controller.rb:33
- - app/controllers/services_controller.rb:38
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@children'
- - app/controllers/services_controller.rb:1
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@client_id'
- - app/controllers/services_controller.rb:1
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@options'
- - app/controllers/services_controller.rb:1
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@service'
- - app/controllers/services_controller.rb:1
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@status_color'
- - app/controllers/services_controller.rb:1
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@status_name'
- - app/controllers/services_controller.rb:1
- * (InstanceVariableAssumption) ServicesController assumes too much for instance variable '@title'
- - app/controllers/services_controller.rb:1
- * (IrresponsibleModule) ServicesController has no descriptive comment
- - app/controllers/services_controller.rb:1
- * (TooManyInstanceVariables) ServicesController has at least 9 instance variables
- - app/controllers/services_controller.rb:1
- * (TooManyStatements) ServicesController#index has approx 15 statements
- - app/controllers/services_controller.rb:5
- * (TooManyStatements) ServicesController#show has approx 43 statements
- - app/controllers/services_controller.rb:29
- TechnicalVisitsController:
- Rating: C
- Churn: 0
- Complexity: 107.12
- Duplication: 0
- Smells: 20
- * (HighComplexity) TechnicalVisitsController#prepare_to_render_index has a flog score of 57
- - app/controllers/technical_visits_controller.rb:54
- * (DuplicateMethodCall) TechnicalVisitsController#allow_iframe calls 'response.headers' 2 times
- - app/controllers/technical_visits_controller.rb:50
- - app/controllers/technical_visits_controller.rb:51
- * (DuplicateMethodCall) TechnicalVisitsController#create calls 'params[:technical_visit]' 2 times
- - app/controllers/technical_visits_controller.rb:14
- - app/controllers/technical_visits_controller.rb:15
- * (DuplicateMethodCall) TechnicalVisitsController#prepare_to_render_index calls 'current_user.company_id' 2 times
- - app/controllers/technical_visits_controller.rb:63
- - app/controllers/technical_visits_controller.rb:70
- * (DuplicateMethodCall) TechnicalVisitsController#prepare_to_render_index calls 'data['dev_desc']' 2 times
- - app/controllers/technical_visits_controller.rb:74
- * (DuplicateMethodCall) TechnicalVisitsController#prepare_to_render_index calls 'data['location']' 2 times
- - app/controllers/technical_visits_controller.rb:74
- * (DuplicateMethodCall) TechnicalVisitsController#prepare_to_render_index calls 'params.has_key?(:technical_visit)' 2 times
- - app/controllers/technical_visits_controller.rb:55
- - app/controllers/technical_visits_controller.rb:56
- * (InstanceVariableAssumption) TechnicalVisitsController assumes too much for instance variable '@email'
- - app/controllers/technical_visits_controller.rb:1
- * (InstanceVariableAssumption) TechnicalVisitsController assumes too much for instance variable '@id'
- - app/controllers/technical_visits_controller.rb:1
- * (InstanceVariableAssumption) TechnicalVisitsController assumes too much for instance variable '@is_internal'
- - app/controllers/technical_visits_controller.rb:1
- * (IrresponsibleModule) TechnicalVisitsController has no descriptive comment
- - app/controllers/technical_visits_controller.rb:1
- * (TooManyInstanceVariables) TechnicalVisitsController has at least 10 instance variables
- - app/controllers/technical_visits_controller.rb:1
- * (TooManyStatements) TechnicalVisitsController#create has approx 7 statements
- - app/controllers/technical_visits_controller.rb:13
- * (TooManyStatements) TechnicalVisitsController#prepare_to_render_index has approx 23 statements
- - app/controllers/technical_visits_controller.rb:54
- * (TooManyStatements) TechnicalVisitsController#send_params has approx 8 statements
- - app/controllers/technical_visits_controller.rb:30
- * (UncommunicativeVariableName) TechnicalVisitsController has the variable name '@conditionIsChecked'
- - app/controllers/technical_visits_controller.rb:55
- * (UncommunicativeVariableName) TechnicalVisitsController#send_params has the variable name 'c'
- - app/controllers/technical_visits_controller.rb:33
- * (UncommunicativeVariableName) TechnicalVisitsController#send_params has the variable name 'e'
- - app/controllers/technical_visits_controller.rb:38
- * (UncommunicativeVariableName) TechnicalVisitsController#send_params has the variable name 'r'
- - app/controllers/technical_visits_controller.rb:33
- * (UtilityFunction) TechnicalVisitsController#send_params doesn't depend on instance state (maybe move it to another class?)
- - app/controllers/technical_visits_controller.rb:30
- UpdateStatusServiceSpec:
- Rating: C
- Churn: 0
- Complexity: 136.64
- Duplication: 60
- Smells: 2
- * (DuplicateCode) Similar code found in 2 nodes
- - spec/services/update_status_service_spec.rb:6
- - spec/services/update_status_service_spec.rb:25
- * (HighComplexity) describe(#call)::context(when raise an error)::it#log this error has a flog score of 28
- - spec/services/update_status_service_spec.rb:79
- Account::BillingController:
- Rating: D
- Churn: 0
- Complexity: 282.34
- Duplication: 88
- Smells: 41
- * (DuplicateCode) Similar code found in 2 nodes
- - app/controllers/account/billing_controller.rb:42
- - app/controllers/account/billing_controller.rb:45
- * (DuplicateCode) Similar code found in 2 nodes
- - app/controllers/account/billing_controller.rb:154
- - app/controllers/account/billing_controller.rb:181
- * (VeryHighComplexity) Account::BillingController#paypal has a flog score of 62
- - app/controllers/account/billing_controller.rb:62
- * (HighComplexity) Account::BillingController#save has a flog score of 56
- - app/controllers/account/billing_controller.rb:11
- * (HighComplexity) Account::BillingController#remove has a flog score of 47
- - app/controllers/account/billing_controller.rb:35
- * (HighComplexity) Account::BillingController#update has a flog score of 43
- - app/controllers/account/billing_controller.rb:164
- * (HighComplexity) Account::BillingController#edit has a flog score of 34
- - app/controllers/account/billing_controller.rb:125
- * (HighComplexity) Account::BillingController#create has a flog score of 30
- - app/controllers/account/billing_controller.rb:144
- * (DuplicateMethodCall) Account::BillingController#create calls 'params[:billing_index]' 2 times
- - app/controllers/account/billing_controller.rb:145
- - app/controllers/account/billing_controller.rb:147
- * (DuplicateMethodCall) Account::BillingController#edit calls 'card_result.data' 3 times
- - app/controllers/account/billing_controller.rb:129
- - app/controllers/account/billing_controller.rb:133
- - app/controllers/account/billing_controller.rb:136
- * (DuplicateMethodCall) Account::BillingController#edit calls 'card_result.error?' 2 times
- - app/controllers/account/billing_controller.rb:129
- - app/controllers/account/billing_controller.rb:130
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'current_user.company_id' 3 times
- - app/controllers/account/billing_controller.rb:63
- - app/controllers/account/billing_controller.rb:75
- - app/controllers/account/billing_controller.rb:89
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'flash['error'] = response.error_message' 2 times
- - app/controllers/account/billing_controller.rb:80
- - app/controllers/account/billing_controller.rb:91
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'params.has_key?(:token)' 3 times
- - app/controllers/account/billing_controller.rb:73
- - app/controllers/account/billing_controller.rb:102
- - app/controllers/account/billing_controller.rb:110
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'redirect_to return_url' 2 times
- - app/controllers/account/billing_controller.rb:86
- - app/controllers/account/billing_controller.rb:104
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'response.error?' 2 times
- - app/controllers/account/billing_controller.rb:79
- - app/controllers/account/billing_controller.rb:90
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'response.error_message' 3 times
- - app/controllers/account/billing_controller.rb:80
- - app/controllers/account/billing_controller.rb:91
- - app/controllers/account/billing_controller.rb:92
- * (DuplicateMethodCall) Account::BillingController#paypal calls 'service_response[:status] = StatusCode::SUCCESS' 2 times
- - app/controllers/account/billing_controller.rb:82
- - app/controllers/account/billing_controller.rb:94
- * (DuplicateMethodCall) Account::BillingController#remove calls 'api.client' 3 times
- - app/controllers/account/billing_controller.rb:37
- - app/controllers/account/billing_controller.rb:44
- - app/controllers/account/billing_controller.rb:47
- * (DuplicateMethodCall) Account::BillingController#remove calls 'payment.nil?' 2 times
- - app/controllers/account/billing_controller.rb:44
- - app/controllers/account/billing_controller.rb:47
- * (DuplicateMethodCall) Account::BillingController#remove calls 'payment['billing_info_id']' 2 times
- - app/controllers/account/billing_controller.rb:44
- - app/controllers/account/billing_controller.rb:47
- * (DuplicateMethodCall) Account::BillingController#remove calls 'payment_method_list.select' 2 times
- - app/controllers/account/billing_controller.rb:43
- - app/controllers/account/billing_controller.rb:46
- * (DuplicateMethodCall) Account::BillingController#save calls 'api.client' 2 times
- - app/controllers/account/billing_controller.rb:15
- - app/controllers/account/billing_controller.rb:22
- * (DuplicateMethodCall) Account::BillingController#save calls 'current_user.company_id' 2 times
- - app/controllers/account/billing_controller.rb:15
- - app/controllers/account/billing_controller.rb:22
- * (DuplicateMethodCall) Account::BillingController#save calls 'params[:payment_method]' 3 times
- - app/controllers/account/billing_controller.rb:12
- - app/controllers/account/billing_controller.rb:17
- * (DuplicateMethodCall) Account::BillingController#update calls 'params[:account_billing_edit]' 5 times
- - app/controllers/account/billing_controller.rb:167
- - app/controllers/account/billing_controller.rb:168
- - app/controllers/account/billing_controller.rb:170
- - app/controllers/account/billing_controller.rb:171
- - app/controllers/account/billing_controller.rb:173
- * (InstanceVariableAssumption) Account::BillingController assumes too much for instance variable '@credit'
- - app/controllers/account/billing_controller.rb:1
- * (InstanceVariableAssumption) Account::BillingController assumes too much for instance variable '@title'
- - app/controllers/account/billing_controller.rb:1
- * (IrresponsibleModule) Account::BillingController has no descriptive comment
- - app/controllers/account/billing_controller.rb:1
- * (NilCheck) Account::BillingController#paypal performs a nil-check
- - app/controllers/account/billing_controller.rb:77
- * (NilCheck) Account::BillingController#remove performs a nil-check
- - app/controllers/account/billing_controller.rb:44
- - app/controllers/account/billing_controller.rb:47
- - app/controllers/account/billing_controller.rb:52
- * (NilCheck) Account::BillingController#save performs a nil-check
- - app/controllers/account/billing_controller.rb:19
- * (RepeatedConditional) Account::BillingController tests 'card_result.error?' at least 3 times
- - app/controllers/account/billing_controller.rb:130
- - app/controllers/account/billing_controller.rb:150
- - app/controllers/account/billing_controller.rb:176
- * (RepeatedConditional) Account::BillingController tests 'params.has_key?(:token)' at least 3 times
- - app/controllers/account/billing_controller.rb:73
- - app/controllers/account/billing_controller.rb:102
- - app/controllers/account/billing_controller.rb:110
- * (RepeatedConditional) Account::BillingController tests 'payment.nil?' at least 3 times
- - app/controllers/account/billing_controller.rb:19
- - app/controllers/account/billing_controller.rb:44
- - app/controllers/account/billing_controller.rb:47
- * (TooManyStatements) Account::BillingController#create has approx 9 statements
- - app/controllers/account/billing_controller.rb:144
- * (TooManyStatements) Account::BillingController#edit has approx 11 statements
- - app/controllers/account/billing_controller.rb:125
- * (TooManyStatements) Account::BillingController#paypal has approx 23 statements
- - app/controllers/account/billing_controller.rb:62
- * (TooManyStatements) Account::BillingController#remove has approx 15 statements
- - app/controllers/account/billing_controller.rb:35
- * (TooManyStatements) Account::BillingController#save has approx 12 statements
- - app/controllers/account/billing_controller.rb:11
- * (TooManyStatements) Account::BillingController#update has approx 10 statements
- - app/controllers/account/billing_controller.rb:164
- BillingInflectionsSpec:
- Rating: D
- Churn: 0
- Complexity: 0.0
- Duplication: 59
- Smells: 1
- * (DuplicateCode) Similar code found in 5 nodes
- - spec/inflections/billing_inflections_spec.rb:3
- - spec/inflections/company_inflections_spec.rb:3
- - spec/inflections/delayed_job_inflections_spec.rb:3
- - spec/inflections/invoice_via_inflections_spec.rb:3
- - spec/inflections/user_inflections_spec.rb:3
- ChartMogul::Interface:
- Rating: D
- Churn: 0
- Complexity: 190.35
- Duplication: 56
- Smells: 30
- * (DuplicateCode) Similar code found in 2 nodes
- - app/adapters/chart_mogul/interface.rb:127
- - app/adapters/chart_mogul/interface.rb:136
- * (HighComplexity) ChartMogul::Interface::ensure_customers_plans_exist has a flog score of 36
- - app/adapters/chart_mogul/interface.rb:74
- * (HighComplexity) ChartMogul::Interface::update_offset has a flog score of 27
- - app/adapters/chart_mogul/interface.rb:149
- * (DuplicateMethodCall) ChartMogul::Interface#push_to_dead_letter_queue calls 'invoice[:invid]' 2 times
- - app/adapters/chart_mogul/interface.rb:97
- - app/adapters/chart_mogul/interface.rb:99
- * (DuplicateMethodCall) ChartMogul::Interface#update_offset calls 'model.data' 4 times
- - app/adapters/chart_mogul/interface.rb:160
- - app/adapters/chart_mogul/interface.rb:161
- - app/adapters/chart_mogul/interface.rb:163
- - app/adapters/chart_mogul/interface.rb:164
- * (InstanceVariableAssumption) ChartMogul::Interface assumes too much for instance variable '@customer_invoices'
- - app/adapters/chart_mogul/interface.rb:4
- * (InstanceVariableAssumption) ChartMogul::Interface assumes too much for instance variable '@dead_letters'
- - app/adapters/chart_mogul/interface.rb:4
- * (InstanceVariableAssumption) ChartMogul::Interface assumes too much for instance variable '@invoices'
- - app/adapters/chart_mogul/interface.rb:4
- * (InstanceVariableAssumption) ChartMogul::Interface assumes too much for instance variable '@new_payment_offset'
- - app/adapters/chart_mogul/interface.rb:4
- * (InstanceVariableAssumption) ChartMogul::Interface assumes too much for instance variable '@new_refund_offset'
- - app/adapters/chart_mogul/interface.rb:4
- * (InstanceVariableAssumption) ChartMogul::Interface assumes too much for instance variable '@pending_invoices'
- - app/adapters/chart_mogul/interface.rb:4
- * (IrresponsibleModule) ChartMogul::Interface has no descriptive comment
- - app/adapters/chart_mogul/interface.rb:4
- * (MissingSafeMethod) ChartMogul::Interface has missing safe method 'run!'
- - app/adapters/chart_mogul/interface.rb:11
- * (NilCheck) ChartMogul::Interface#ensure_customers_plans_exist performs a nil-check
- - app/adapters/chart_mogul/interface.rb:83
- * (NilCheck) ChartMogul::Interface#push_to_dead_letter_queue performs a nil-check
- - app/adapters/chart_mogul/interface.rb:97
- * (NilCheck) ChartMogul::Interface#recently_paid_invoices performs a nil-check
- - app/adapters/chart_mogul/interface.rb:129
- * (NilCheck) ChartMogul::Interface#recently_refunded_invoices performs a nil-check
- - app/adapters/chart_mogul/interface.rb:138
- * (NilCheck) ChartMogul::Interface#update_offset performs a nil-check
- - app/adapters/chart_mogul/interface.rb:151
- * (RepeatedConditional) ChartMogul::Interface tests 'dead_letters?' at least 3 times
- - app/adapters/chart_mogul/interface.rb:21
- - app/adapters/chart_mogul/interface.rb:29
- - app/adapters/chart_mogul/interface.rb:61
- * (RepeatedConditional) ChartMogul::Interface tests 'model.nil?' at least 3 times
- - app/adapters/chart_mogul/interface.rb:129
- - app/adapters/chart_mogul/interface.rb:138
- - app/adapters/chart_mogul/interface.rb:151
- * (TooManyInstanceVariables) ChartMogul::Interface has at least 6 instance variables
- - app/adapters/chart_mogul/interface.rb:4
- * (TooManyMethods) ChartMogul::Interface has at least 16 methods
- - app/adapters/chart_mogul/interface.rb:4
- * (TooManyStatements) ChartMogul::Interface#ensure_customers_plans_exist has approx 11 statements
- - app/adapters/chart_mogul/interface.rb:74
- * (TooManyStatements) ChartMogul::Interface#fetch_invoices has approx 10 statements
- - app/adapters/chart_mogul/interface.rb:44
- * (TooManyStatements) ChartMogul::Interface#recently_paid_invoices has approx 6 statements
- - app/adapters/chart_mogul/interface.rb:127
- * (TooManyStatements) ChartMogul::Interface#recently_refunded_invoices has approx 6 statements
- - app/adapters/chart_mogul/interface.rb:136
- * (TooManyStatements) ChartMogul::Interface#run_invoices has approx 6 statements
- - app/adapters/chart_mogul/interface.rb:16
- * (UncommunicativeVariableName) ChartMogul::Interface#init_customer_invoices_store has the variable name 'h'
- - app/adapters/chart_mogul/interface.rb:124
- * (UncommunicativeVariableName) ChartMogul::Interface#init_customer_invoices_store has the variable name 'k'
- - app/adapters/chart_mogul/interface.rb:124
- * (UncommunicativeVariableName) ChartMogul::Interface#push_to_dead_letter_queue has the variable name 'i'
- - app/adapters/chart_mogul/interface.rb:97
- CompanyInflectionsSpec:
- Rating: D
- Churn: 0
- Complexity: 0.0
- Duplication: 59
- Smells: 1
- * (DuplicateCode) Similar code found in 5 nodes
- - spec/inflections/billing_inflections_spec.rb:3
- - spec/inflections/company_inflections_spec.rb:3
- - spec/inflections/delayed_job_inflections_spec.rb:3
- - spec/inflections/invoice_via_inflections_spec.rb:3
- - spec/inflections/user_inflections_spec.rb:3
- CurrentUser::Auth0:
- Rating: D
- Churn: 0
- Complexity: 207.59
- Duplication: 0
- Smells: 20
- * (HighComplexity) CurrentUser::Auth0::mfa_request has a flog score of 32
- - app/adapters/current_user/auth0.rb:177
- * (HighComplexity) CurrentUser::Auth0::sync! has a flog score of 27
- - app/adapters/current_user/auth0.rb:6
- * (HighComplexity) CurrentUser::Auth0::mfa_token has a flog score of 25
- - app/adapters/current_user/auth0.rb:139
- * (ControlParameter) CurrentUser::Auth0#mfa_request is controlled by argument 'as'
- - app/adapters/current_user/auth0.rb:182
- * (DuplicateMethodCall) CurrentUser::Auth0#mfa_request calls 'req.body' 3 times
- - app/adapters/current_user/auth0.rb:191
- - app/adapters/current_user/auth0.rb:193
- * (DuplicateMethodCall) CurrentUser::Auth0#mfa_token calls '@mfa_token.nil?' 2 times
- - app/adapters/current_user/auth0.rb:140
- - app/adapters/current_user/auth0.rb:150
- * (DuplicateMethodCall) CurrentUser::Auth0#mfa_token calls 'options[:refresh_token]' 2 times
- - app/adapters/current_user/auth0.rb:146
- - app/adapters/current_user/auth0.rb:151
- * (DuplicateMethodCall) CurrentUser::Auth0#otp_authenticate calls 'response['access_token']' 2 times
- - app/adapters/current_user/auth0.rb:54
- * (DuplicateMethodCall) CurrentUser::Auth0#sync! calls 'data[:password]' 2 times
- - app/adapters/current_user/auth0.rb:16
- * (DuplicateMethodCall) CurrentUser::Auth0#user_authenticate calls 'ENV['AUTH0_ORIGINAL_DOMAIN']' 2 times
- - app/adapters/current_user/auth0.rb:22
- - app/adapters/current_user/auth0.rb:28
- * (DuplicateMethodCall) CurrentUser::Auth0#user_authenticate calls 'credentials['access_token']' 2 times
- - app/adapters/current_user/auth0.rb:35
- - app/adapters/current_user/auth0.rb:36
- * (IrresponsibleModule) CurrentUser::Auth0 has no descriptive comment
- - app/adapters/current_user/auth0.rb:2
- * (NilCheck) CurrentUser::Auth0#basic_token performs a nil-check
- - app/adapters/current_user/auth0.rb:127
- * (NilCheck) CurrentUser::Auth0#mfa_token performs a nil-check
- - app/adapters/current_user/auth0.rb:140
- - app/adapters/current_user/auth0.rb:150
- * (NilCheck) CurrentUser::Auth0#otp_authenticate performs a nil-check
- - app/adapters/current_user/auth0.rb:56
- * (NilCheck) CurrentUser::Auth0#otp_remove performs a nil-check
- - app/adapters/current_user/auth0.rb:80
- * (TooManyStatements) CurrentUser::Auth0#mfa_request has approx 10 statements
- - app/adapters/current_user/auth0.rb:177
- * (TooManyStatements) CurrentUser::Auth0#mfa_token has approx 6 statements
- - app/adapters/current_user/auth0.rb:139
- * (TooManyStatements) CurrentUser::Auth0#sync! has approx 6 statements
- - app/adapters/current_user/auth0.rb:6
- * (UncommunicativeModuleName) CurrentUser::Auth0 has the name 'Auth0'
- - app/adapters/current_user/auth0.rb:2
- DelayedJobInflectionsSpec:
- Rating: D
- Churn: 0
- Complexity: 0.0
- Duplication: 59
- Smells: 1
- * (DuplicateCode) Similar code found in 5 nodes
- - spec/inflections/billing_inflections_spec.rb:3
- - spec/inflections/company_inflections_spec.rb:3
- - spec/inflections/delayed_job_inflections_spec.rb:3
- - spec/inflections/invoice_via_inflections_spec.rb:3
- - spec/inflections/user_inflections_spec.rb:3
- GetUserDataJob:
- Rating: D
- Churn: 0
- Complexity: 224.04
- Duplication: 42
- Smells: 27
- * (DuplicateCode) Similar code found in 2 nodes
- - app/jobs/get_user_data_job.rb:58
- - app/jobs/get_user_data_job.rb:60
- * (VeryHighComplexity) GetUserDataJob#perform has a flog score of 224
- - app/jobs/get_user_data_job.rb:6
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@client["class_id"]' 2 times
- - app/jobs/get_user_data_job.rb:42
- - app/jobs/get_user_data_job.rb:43
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@client["metadata"]' 6 times
- - app/jobs/get_user_data_job.rb:58
- - app/jobs/get_user_data_job.rb:59
- - app/jobs/get_user_data_job.rb:60
- - app/jobs/get_user_data_job.rb:61
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@client["metadata"].is_a?(Hash)' 2 times
- - app/jobs/get_user_data_job.rb:58
- - app/jobs/get_user_data_job.rb:60
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.company' 21 times
- - app/jobs/get_user_data_job.rb:34
- - app/jobs/get_user_data_job.rb:35
- - app/jobs/get_user_data_job.rb:36
- - app/jobs/get_user_data_job.rb:37
- - app/jobs/get_user_data_job.rb:38
- - app/jobs/get_user_data_job.rb:39
- - app/jobs/get_user_data_job.rb:40
- - app/jobs/get_user_data_job.rb:41
- - app/jobs/get_user_data_job.rb:42
- - app/jobs/get_user_data_job.rb:43
- - app/jobs/get_user_data_job.rb:46
- - app/jobs/get_user_data_job.rb:47
- - app/jobs/get_user_data_job.rb:55
- - app/jobs/get_user_data_job.rb:58
- - app/jobs/get_user_data_job.rb:60
- - app/jobs/get_user_data_job.rb:62
- - app/jobs/get_user_data_job.rb:63
- - app/jobs/get_user_data_job.rb:68
- - app/jobs/get_user_data_job.rb:73
- - app/jobs/get_user_data_job.rb:75
- - app/jobs/get_user_data_job.rb:81
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.company.brand_id' 2 times
- - app/jobs/get_user_data_job.rb:73
- - app/jobs/get_user_data_job.rb:75
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.company.contacts' 5 times
- - app/jobs/get_user_data_job.rb:46
- - app/jobs/get_user_data_job.rb:47
- - app/jobs/get_user_data_job.rb:62
- - app/jobs/get_user_data_job.rb:63
- - app/jobs/get_user_data_job.rb:68
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.company.contacts.has_key?("#{@user.contact_id}")' 2 times
- - app/jobs/get_user_data_job.rb:46
- - app/jobs/get_user_data_job.rb:62
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.company.contacts["#{@user.contact_id}"]' 2 times
- - app/jobs/get_user_data_job.rb:47
- - app/jobs/get_user_data_job.rb:63
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.company_id' 4 times
- - app/jobs/get_user_data_job.rb:10
- - app/jobs/get_user_data_job.rb:12
- - app/jobs/get_user_data_job.rb:34
- - app/jobs/get_user_data_job.rb:68
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.contact_id' 7 times
- - app/jobs/get_user_data_job.rb:46
- - app/jobs/get_user_data_job.rb:47
- - app/jobs/get_user_data_job.rb:51
- - app/jobs/get_user_data_job.rb:62
- - app/jobs/get_user_data_job.rb:63
- - app/jobs/get_user_data_job.rb:68
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.lang' 2 times
- - app/jobs/get_user_data_job.rb:73
- - app/jobs/get_user_data_job.rb:75
- * (DuplicateMethodCall) GetUserDataJob#perform calls '@user.lang.nil?' 2 times
- - app/jobs/get_user_data_job.rb:73
- - app/jobs/get_user_data_job.rb:75
- * (DuplicateMethodCall) GetUserDataJob#perform calls 'Bugsnag.notify(GetUserDataJobNoUser.new(error))' 2 times
- - app/jobs/get_user_data_job.rb:13
- - app/jobs/get_user_data_job.rb:70
- * (DuplicateMethodCall) GetUserDataJob#perform calls 'api.client' 2 times
- - app/jobs/get_user_data_job.rb:10
- - app/jobs/get_user_data_job.rb:18
- * (DuplicateMethodCall) GetUserDataJob#perform calls 'client_result.error?' 2 times
- - app/jobs/get_user_data_job.rb:11
- - app/jobs/get_user_data_job.rb:16
- * (DuplicateMethodCall) GetUserDataJob#perform calls 'contact_user['real_name']' 2 times
- - app/jobs/get_user_data_job.rb:64
- - app/jobs/get_user_data_job.rb:65
- * (DuplicateMethodCall) GetUserDataJob#perform calls 'contact_user['real_name'].split(' ')' 2 times
- - app/jobs/get_user_data_job.rb:64
- - app/jobs/get_user_data_job.rb:65
- * (InstanceVariableAssumption) GetUserDataJob assumes too much for instance variable '@client'
- - app/jobs/get_user_data_job.rb:1
- * (InstanceVariableAssumption) GetUserDataJob assumes too much for instance variable '@contacts'
- - app/jobs/get_user_data_job.rb:1
- * (InstanceVariableAssumption) GetUserDataJob assumes too much for instance variable '@user'
- - app/jobs/get_user_data_job.rb:1
- * (IrresponsibleModule) GetUserDataJob has no descriptive comment
- - app/jobs/get_user_data_job.rb:1
- * (IrresponsibleModule) GetUserDataJobNoUser has no descriptive comment
- - app/jobs/get_user_data_job.rb:88
- * (NilCheck) GetUserDataJob#perform performs a nil-check
- - app/jobs/get_user_data_job.rb:34
- - app/jobs/get_user_data_job.rb:69
- - app/jobs/get_user_data_job.rb:73
- - app/jobs/get_user_data_job.rb:75
- * (TooManyStatements) GetUserDataJob#perform has approx 47 statements
- - app/jobs/get_user_data_job.rb:6
- * (UncommunicativeVariableName) GetUserDataJob#perform has the variable name 'e'
- - app/jobs/get_user_data_job.rb:83
- InvoiceViaInflectionsSpec:
- Rating: D
- Churn: 0
- Complexity: 0.0
- Duplication: 59
- Smells: 1
- * (DuplicateCode) Similar code found in 5 nodes
- - spec/inflections/billing_inflections_spec.rb:3
- - spec/inflections/company_inflections_spec.rb:3
- - spec/inflections/delayed_job_inflections_spec.rb:3
- - spec/inflections/invoice_via_inflections_spec.rb:3
- - spec/inflections/user_inflections_spec.rb:3
- InvoicesController:
- Rating: D
- Churn: 0
- Complexity: 235.69
- Duplication: 0
- Smells: 34
- * (VeryHighComplexity) InvoicesController#show has a flog score of 146
- - app/controllers/invoices_controller.rb:29
- * (HighComplexity) InvoicesController#execute_charge has a flog score of 58
- - app/controllers/invoices_controller.rb:82
- * (HighComplexity) InvoicesController#index has a flog score of 30
- - app/controllers/invoices_controller.rb:5
- * (DuplicateMethodCall) InvoicesController#execute_charge calls 'api.client' 2 times
- - app/controllers/invoices_controller.rb:97
- - app/controllers/invoices_controller.rb:111
- * (DuplicateMethodCall) InvoicesController#execute_charge calls 'card_result.data' 2 times
- - app/controllers/invoices_controller.rb:100
- - app/controllers/invoices_controller.rb:103
- * (DuplicateMethodCall) InvoicesController#execute_charge calls 'current_user.company_id' 2 times
- - app/controllers/invoices_controller.rb:89
- - app/controllers/invoices_controller.rb:97
- * (DuplicateMethodCall) InvoicesController#execute_charge calls 'params[:id]' 2 times
- - app/controllers/invoices_controller.rb:112
- - app/controllers/invoices_controller.rb:124
- * (DuplicateMethodCall) InvoicesController#execute_charge calls 'params[:payment_method]' 2 times
- - app/controllers/invoices_controller.rb:84
- - app/controllers/invoices_controller.rb:87
- * (DuplicateMethodCall) InvoicesController#index calls '_("Invoices")' 2 times
- - app/controllers/invoices_controller.rb:20
- - app/controllers/invoices_controller.rb:26
- * (DuplicateMethodCall) InvoicesController#index calls 'current_user.company_id' 2 times
- - app/controllers/invoices_controller.rb:7
- - app/controllers/invoices_controller.rb:25
- * (DuplicateMethodCall) InvoicesController#show calls '@invoice['current_packs']' 2 times
- - app/controllers/invoices_controller.rb:66
- * (DuplicateMethodCall) InvoicesController#show calls '@invoice['pre_billed_packs']' 2 times
- - app/controllers/invoices_controller.rb:67
- * (DuplicateMethodCall) InvoicesController#show calls 'api.client' 3 times
- - app/controllers/invoices_controller.rb:38
- - app/controllers/invoices_controller.rb:46
- - app/controllers/invoices_controller.rb:62
- * (DuplicateMethodCall) InvoicesController#show calls 'current_user.company' 2 times
- - app/controllers/invoices_controller.rb:45
- - app/controllers/invoices_controller.rb:56
- * (DuplicateMethodCall) InvoicesController#show calls 'current_user.company.brand_id' 2 times
- - app/controllers/invoices_controller.rb:45
- - app/controllers/invoices_controller.rb:56
- * (DuplicateMethodCall) InvoicesController#show calls 'current_user.company_id' 3 times
- - app/controllers/invoices_controller.rb:41
- - app/controllers/invoices_controller.rb:60
- - app/controllers/invoices_controller.rb:62
- * (DuplicateMethodCall) InvoicesController#show calls 'invoice_result.data' 3 times
- - app/controllers/invoices_controller.rb:39
- - app/controllers/invoices_controller.rb:51
- - app/controllers/invoices_controller.rb:52
- * (DuplicateMethodCall) InvoicesController#show calls 'invoice_result.error?' 3 times
- - app/controllers/invoices_controller.rb:39
- - app/controllers/invoices_controller.rb:41
- - app/controllers/invoices_controller.rb:47
- * (DuplicateMethodCall) InvoicesController#show calls 'params[:format]' 6 times
- - app/controllers/invoices_controller.rb:30
- - app/controllers/invoices_controller.rb:44
- - app/controllers/invoices_controller.rb:46
- - app/controllers/invoices_controller.rb:52
- * (DuplicateMethodCall) InvoicesController#show calls 'params[:format].present?' 2 times
- - app/controllers/invoices_controller.rb:30
- - app/controllers/invoices_controller.rb:44
- * (DuplicateMethodCall) InvoicesController#show calls 'params[:id]' 5 times
- - app/controllers/invoices_controller.rb:33
- - app/controllers/invoices_controller.rb:38
- - app/controllers/invoices_controller.rb:46
- - app/controllers/invoices_controller.rb:52
- - app/controllers/invoices_controller.rb:73
- * (DuplicateMethodCall) InvoicesController#show calls 'redirect_to invoices_path' 2 times
- - app/controllers/invoices_controller.rb:43
- - app/controllers/invoices_controller.rb:49
- * (InstanceVariableAssumption) InvoicesController assumes too much for instance variable '@client_id'
- - app/controllers/invoices_controller.rb:1
- * (InstanceVariableAssumption) InvoicesController assumes too much for instance variable '@invoice'
- - app/controllers/invoices_controller.rb:1
- * (InstanceVariableAssumption) InvoicesController assumes too much for instance variable '@payment_methods'
- - app/controllers/invoices_controller.rb:1
- * (InstanceVariableAssumption) InvoicesController assumes too much for instance variable '@title'
- - app/controllers/invoices_controller.rb:1
- * (IrresponsibleModule) InvoicesController has no descriptive comment
- - app/controllers/invoices_controller.rb:1
- * (NilCheck) InvoicesController#execute_charge performs a nil-check
- - app/controllers/invoices_controller.rb:90
- - app/controllers/invoices_controller.rb:110
- * (TooManyInstanceVariables) InvoicesController has at least 8 instance variables
- - app/controllers/invoices_controller.rb:1
- * (TooManyStatements) InvoicesController#execute_charge has approx 17 statements
- - app/controllers/invoices_controller.rb:82
- * (TooManyStatements) InvoicesController#index has approx 16 statements
- - app/controllers/invoices_controller.rb:5
- * (TooManyStatements) InvoicesController#show has approx 28 statements
- - app/controllers/invoices_controller.rb:29
- * (UncommunicativeVariableName) InvoicesController#index has the variable name 'k'
- - app/controllers/invoices_controller.rb:13
- * (UncommunicativeVariableName) InvoicesController#index has the variable name 'v'
- - app/controllers/invoices_controller.rb:13
- ServersController:
- Rating: D
- Churn: 0
- Complexity: 310.45
- Duplication: 0
- Smells: 35
- * (VeryHighComplexity) ServersController#update has a flog score of 113
- - app/controllers/servers_controller.rb:84
- * (VeryHighComplexity) ServersController#reinstall has a flog score of 73
- - app/controllers/servers_controller.rb:139
- * (HighComplexity) ServersController#new has a flog score of 38
- - app/controllers/servers_controller.rb:12
- * (HighComplexity) ServersController#show has a flog score of 37
- - app/controllers/servers_controller.rb:37
- * (HighComplexity) ServersController#create has a flog score of 35
- - app/controllers/servers_controller.rb:57
- * (DuplicateMethodCall) ServersController#index calls '_('Servers')' 2 times
- - app/controllers/servers_controller.rb:8
- - app/controllers/servers_controller.rb:9
- * (DuplicateMethodCall) ServersController#new calls 'params[:locale]' 2 times
- - app/controllers/servers_controller.rb:13
- - app/controllers/servers_controller.rb:14
- * (DuplicateMethodCall) ServersController#reinstall calls 'metadata[:ssh]' 2 times
- - app/controllers/servers_controller.rb:157
- - app/controllers/servers_controller.rb:158
- * (DuplicateMethodCall) ServersController#reinstall calls 'params[:id]' 3 times
- - app/controllers/servers_controller.rb:148
- - app/controllers/servers_controller.rb:154
- - app/controllers/servers_controller.rb:179
- * (DuplicateMethodCall) ServersController#reinstall calls 'params[:reinstall]' 2 times
- - app/controllers/servers_controller.rb:143
- - app/controllers/servers_controller.rb:153
- * (DuplicateMethodCall) ServersController#reinstall calls 'reinstall[:label]' 3 times
- - app/controllers/servers_controller.rb:146
- - app/controllers/servers_controller.rb:149
- * (DuplicateMethodCall) ServersController#show calls 'params[:server_bandwidth]' 2 times
- - app/controllers/servers_controller.rb:45
- - app/controllers/servers_controller.rb:47
- * (DuplicateMethodCall) ServersController#show calls 'params[:tab]' 2 times
- - app/controllers/servers_controller.rb:41
- * (DuplicateMethodCall) ServersController#update calls 'ENV['UBERSMITH_DOMAIN']' 2 times
- - app/controllers/servers_controller.rb:91
- - app/controllers/servers_controller.rb:108
- * (DuplicateMethodCall) ServersController#update calls 'ENV['UBERSMITH_PASSWORD']' 2 times
- - app/controllers/servers_controller.rb:91
- - app/controllers/servers_controller.rb:108
- * (DuplicateMethodCall) ServersController#update calls 'ENV['UBERSMITH_USERNAME']' 2 times
- - app/controllers/servers_controller.rb:91
- - app/controllers/servers_controller.rb:108
- * (DuplicateMethodCall) ServersController#update calls 'api.device' 2 times
- - app/controllers/servers_controller.rb:92
- - app/controllers/servers_controller.rb:109
- * (DuplicateMethodCall) ServersController#update calls 'params[:bandwidth]' 2 times
- - app/controllers/servers_controller.rb:125
- - app/controllers/servers_controller.rb:126
- * (DuplicateMethodCall) ServersController#update calls 'params[:id]' 6 times
- - app/controllers/servers_controller.rb:92
- - app/controllers/servers_controller.rb:109
- - app/controllers/servers_controller.rb:124
- - app/controllers/servers_controller.rb:129
- - app/controllers/servers_controller.rb:132
- - app/controllers/servers_controller.rb:137
- * (DuplicateMethodCall) ServersController#update calls 'reboot_result.error_message' 2 times
- - app/controllers/servers_controller.rb:111
- - app/controllers/servers_controller.rb:112
- * (InstanceVariableAssumption) ServersController assumes too much for instance variable '@bandwidth_params'
- - app/controllers/servers_controller.rb:1
- * (InstanceVariableAssumption) ServersController assumes too much for instance variable '@brand_id'
- - app/controllers/servers_controller.rb:1
- * (InstanceVariableAssumption) ServersController assumes too much for instance variable '@current_tab'
- - app/controllers/servers_controller.rb:1
- * (InstanceVariableAssumption) ServersController assumes too much for instance variable '@device'
- - app/controllers/servers_controller.rb:1
- * (InstanceVariableAssumption) ServersController assumes too much for instance variable '@title'
- - app/controllers/servers_controller.rb:1
- * (IrresponsibleModule) ServerRebootError has no descriptive comment
- - app/controllers/servers_controller.rb:195
- * (IrresponsibleModule) ServersController has no descriptive comment
- - app/controllers/servers_controller.rb:1
- * (TooManyInstanceVariables) ServersController has at least 10 instance variables
- - app/controllers/servers_controller.rb:1
- * (TooManyStatements) ServersController#create has approx 12 statements
- - app/controllers/servers_controller.rb:57
- * (TooManyStatements) ServersController#new has approx 13 statements
- - app/controllers/servers_controller.rb:12
- * (TooManyStatements) ServersController#reinstall has approx 17 statements
- - app/controllers/servers_controller.rb:141
- * (TooManyStatements) ServersController#show has approx 13 statements
- - app/controllers/servers_controller.rb:37
- * (TooManyStatements) ServersController#update has approx 23 statements
- - app/controllers/servers_controller.rb:84
- * (UncommunicativeVariableName) ServersController#show has the variable name 'k'
- - app/controllers/servers_controller.rb:47
- * (UncommunicativeVariableName) ServersController#show has the variable name 'v'
- - app/controllers/servers_controller.rb:47
- UbersmithOrder:
- Rating: D
- Churn: 0
- Complexity: 244.63
- Duplication: 50
- Smells: 30
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/ubersmith_order.rb:99
- - lib/modules/ubersmith_order.rb:104
- * (VeryHighComplexity) UbersmithOrder::verify_payment has a flog score of 65
- - lib/modules/ubersmith_order.rb:7
- * (VeryHighComplexity) UbersmithOrder::validate_invoice has a flog score of 64
- - lib/modules/ubersmith_order.rb:135
- * (HighComplexity) UbersmithOrder::apply_credit has a flog score of 38
- - lib/modules/ubersmith_order.rb:183
- * (DuplicateMethodCall) UbersmithOrder#self.apply_credit calls 'api.client' 2 times
- - lib/modules/ubersmith_order.rb:185
- - lib/modules/ubersmith_order.rb:200
- * (DuplicateMethodCall) UbersmithOrder#self.apply_credit calls 'invoice['invid']' 2 times
- - lib/modules/ubersmith_order.rb:187
- - lib/modules/ubersmith_order.rb:202
- * (DuplicateMethodCall) UbersmithOrder#self.get_order_list calls 'api.order' 6 times
- - lib/modules/ubersmith_order.rb:96
- - lib/modules/ubersmith_order.rb:97
- - lib/modules/ubersmith_order.rb:98
- - lib/modules/ubersmith_order.rb:101
- - lib/modules/ubersmith_order.rb:102
- - lib/modules/ubersmith_order.rb:103
- * (DuplicateMethodCall) UbersmithOrder#self.parse_paid calls 'services[key]' 2 times
- - lib/modules/ubersmith_order.rb:228
- - lib/modules/ubersmith_order.rb:229
- * (DuplicateMethodCall) UbersmithOrder#self.send_paid_email calls 'order["info"]' 4 times
- - lib/modules/ubersmith_order.rb:70
- - lib/modules/ubersmith_order.rb:71
- - lib/modules/ubersmith_order.rb:72
- * (DuplicateMethodCall) UbersmithOrder#self.validate_invoice calls 'Bugsnag.notify(OrderVerifyPaymentError.new(message))' 2 times
- - lib/modules/ubersmith_order.rb:146
- - lib/modules/ubersmith_order.rb:153
- * (DuplicateMethodCall) UbersmithOrder#self.validate_invoice calls 'api.client' 2 times
- - lib/modules/ubersmith_order.rb:141
- - lib/modules/ubersmith_order.rb:162
- * (DuplicateMethodCall) UbersmithOrder#self.validate_invoice calls 'invoice_result.data' 2 times
- - lib/modules/ubersmith_order.rb:147
- - lib/modules/ubersmith_order.rb:158
- * (DuplicateMethodCall) UbersmithOrder#self.validate_invoice calls 'pp message' 3 times
- - lib/modules/ubersmith_order.rb:145
- - lib/modules/ubersmith_order.rb:152
- - lib/modules/ubersmith_order.rb:175
- * (DuplicateMethodCall) UbersmithOrder#self.verify_payment calls 'action['status']' 2 times
- - lib/modules/ubersmith_order.rb:34
- - lib/modules/ubersmith_order.rb:40
- * (DuplicateMethodCall) UbersmithOrder#self.verify_payment calls 'key.to_i' 4 times
- - lib/modules/ubersmith_order.rb:34
- - lib/modules/ubersmith_order.rb:35
- - lib/modules/ubersmith_order.rb:38
- - lib/modules/ubersmith_order.rb:40
- * (DuplicateMethodCall) UbersmithOrder#self.verify_payment calls 'order_result.error_message' 2 times
- - lib/modules/ubersmith_order.rb:20
- - lib/modules/ubersmith_order.rb:22
- * (DuplicateMethodCall) UbersmithOrder#self.verify_payment calls 'pp message' 2 times
- - lib/modules/ubersmith_order.rb:24
- - lib/modules/ubersmith_order.rb:49
- * (DuplicateMethodCall) UbersmithOrder#self.verify_payment calls 'verify_payment_id == 0' 2 times
- - lib/modules/ubersmith_order.rb:43
- - lib/modules/ubersmith_order.rb:47
- * (IrresponsibleModule) UbersmithOrder has no descriptive comment
- - lib/modules/ubersmith_order.rb:4
- * (NestedIterators) UbersmithOrder#self.parse_paid contains iterators nested 2 deep
- - lib/modules/ubersmith_order.rb:229
- * (NestedIterators) UbersmithOrder#self.verify_payment contains iterators nested 4 deep
- - lib/modules/ubersmith_order.rb:32
- * (TooManyStatements) UbersmithOrder#self.apply_credit has approx 16 statements
- - lib/modules/ubersmith_order.rb:183
- * (TooManyStatements) UbersmithOrder#self.parse_paid has approx 7 statements
- - lib/modules/ubersmith_order.rb:225
- * (TooManyStatements) UbersmithOrder#self.send_paid_email has approx 9 statements
- - lib/modules/ubersmith_order.rb:69
- * (TooManyStatements) UbersmithOrder#self.validate_invoice has approx 21 statements
- - lib/modules/ubersmith_order.rb:135
- * (TooManyStatements) UbersmithOrder#self.verify_payment has approx 23 statements
- - lib/modules/ubersmith_order.rb:7
- * (UncommunicativeVariableName) UbersmithOrder#self.send_paid_email has the variable name 'k'
- - lib/modules/ubersmith_order.rb:70
- * (UncommunicativeVariableName) UbersmithOrder#self.send_paid_email has the variable name 'v'
- - lib/modules/ubersmith_order.rb:70
- * (UnusedParameters) UbersmithOrder#self.on_after_paid has unused parameter 'brand_id'
- - lib/modules/ubersmith_order.rb:61
- * (UnusedParameters) UbersmithOrder#self.on_after_paid has unused parameter 'verify_payment_id'
- - lib/modules/ubersmith_order.rb:61
- UserInflectionsSpec:
- Rating: D
- Churn: 0
- Complexity: 0.0
- Duplication: 59
- Smells: 1
- * (DuplicateCode) Similar code found in 5 nodes
- - spec/inflections/billing_inflections_spec.rb:3
- - spec/inflections/company_inflections_spec.rb:3
- - spec/inflections/delayed_job_inflections_spec.rb:3
- - spec/inflections/invoice_via_inflections_spec.rb:3
- - spec/inflections/user_inflections_spec.rb:3
- WhatsNewControllerSpec:
- Rating: D
- Churn: 0
- Complexity: 227.52
- Duplication: 0
- Smells: 2
- * (HighComplexity) describe(GET #index)::context#when request is done with success has a flog score of 49
- - spec/controllers/whats_new_controller_spec.rb:26
- * (HighComplexity) describe(GET #index)::context(when request is done with success)::it#return status ok with a message has a flog score of 26
- - spec/controllers/whats_new_controller_spec.rb:37
- Maas::ProvisionError:
- Rating: F
- Churn: 0
- Complexity: 1053.21
- Duplication: 240
- Smells: 112
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/maas.rb:91
- - lib/modules/maas.rb:98
- * (DuplicateCode) Similar code found in 3 nodes
- - lib/modules/maas.rb:158
- - lib/modules/maas.rb:169
- - lib/modules/maas.rb:221
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/maas.rb:304
- - lib/modules/maas.rb:616
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/maas.rb:459
- - lib/modules/maas.rb:535
- * (DuplicateCode) Similar code found in 2 nodes
- - lib/modules/maas.rb:676
- - lib/modules/maas.rb:694
- * (VeryHighComplexity) Maas::choice_machine has a flog score of 307
- - lib/modules/maas.rb:120
- * (VeryHighComplexity) Maas::reinstall has a flog score of 143
- - lib/modules/maas.rb:522
- * (VeryHighComplexity) Maas::start_deploy has a flog score of 117
- - lib/modules/maas.rb:342
- * (VeryHighComplexity) Maas::check has a flog score of 95
- - lib/modules/maas.rb:36
- * (VeryHighComplexity) Maas::deploy has a flog score of 87
- - lib/modules/maas.rb:253
- * (VeryHighComplexity) Maas::send_access has a flog score of 65
- - lib/modules/maas.rb:465
- * (HighComplexity) Maas::find_paid_order has a flog score of 53
- - lib/modules/maas.rb:85
- * (HighComplexity) Maas::validate_order has a flog score of 29
- - lib/modules/maas.rb:11
- * (ControlParameter) Maas#self.brand_id_by_queue is controlled by argument 'queue_id'
- - lib/modules/maas.rb:736
- * (DuplicateMethodCall) Maas#self.check calls 'device['label']' 2 times
- - lib/modules/maas.rb:54
- - lib/modules/maas.rb:57
- * (DuplicateMethodCall) Maas#self.check calls 'device['tags']' 2 times
- - lib/modules/maas.rb:51
- * (DuplicateMethodCall) Maas#self.check calls 'devices_result.data' 2 times
- - lib/modules/maas.rb:42
- - lib/modules/maas.rb:50
- * (DuplicateMethodCall) Maas#self.check calls 'message.join("\n")' 2 times
- - lib/modules/maas.rb:69
- - lib/modules/maas.rb:73
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'Bugsnag.notify(Maas::ProvisionError.new(error))' 5 times
- - lib/modules/maas.rb:160
- - lib/modules/maas.rb:171
- - lib/modules/maas.rb:179
- - lib/modules/maas.rb:196
- - lib/modules/maas.rb:223
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'api.client' 2 times
- - lib/modules/maas.rb:200
- - lib/modules/maas.rb:218
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'api.device' 3 times
- - lib/modules/maas.rb:143
- - lib/modules/maas.rb:166
- - lib/modules/maas.rb:191
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device['dev']' 2 times
- - lib/modules/maas.rb:190
- - lib/modules/maas.rb:226
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device['label']' 2 times
- - lib/modules/maas.rb:206
- - lib/modules/maas.rb:225
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device['label'].split(' ')' 2 times
- - lib/modules/maas.rb:206
- - lib/modules/maas.rb:225
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device['label'].split(' ').first' 2 times
- - lib/modules/maas.rb:206
- - lib/modules/maas.rb:225
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device_result.data' 3 times
- - lib/modules/maas.rb:161
- - lib/modules/maas.rb:173
- - lib/modules/maas.rb:215
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device_result.data.values' 2 times
- - lib/modules/maas.rb:173
- - lib/modules/maas.rb:215
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device_result.error?' 2 times
- - lib/modules/maas.rb:156
- - lib/modules/maas.rb:167
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'device_result.error_message' 2 times
- - lib/modules/maas.rb:158
- - lib/modules/maas.rb:169
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'did.nil?' 3 times
- - lib/modules/maas.rb:141
- - lib/modules/maas.rb:165
- - lib/modules/maas.rb:177
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'key.include?('pack')' 2 times
- - lib/modules/maas.rb:127
- - lib/modules/maas.rb:237
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'order['info']' 3 times
- - lib/modules/maas.rb:125
- - lib/modules/maas.rb:126
- - lib/modules/maas.rb:236
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'order['order_id']' 11 times
- - lib/modules/maas.rb:121
- - lib/modules/maas.rb:158
- - lib/modules/maas.rb:169
- - lib/modules/maas.rb:176
- - lib/modules/maas.rb:184
- - lib/modules/maas.rb:194
- - lib/modules/maas.rb:208
- - lib/modules/maas.rb:221
- - lib/modules/maas.rb:228
- - lib/modules/maas.rb:242
- - lib/modules/maas.rb:243
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'params[:device_id] = device['dev']' 2 times
- - lib/modules/maas.rb:190
- - lib/modules/maas.rb:226
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'params[:order_id] = order['order_id']' 2 times
- - lib/modules/maas.rb:208
- - lib/modules/maas.rb:228
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'pp error' 5 times
- - lib/modules/maas.rb:159
- - lib/modules/maas.rb:170
- - lib/modules/maas.rb:178
- - lib/modules/maas.rb:195
- - lib/modules/maas.rb:222
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'self.deploy(hostname, distro, params)' 2 times
- - lib/modules/maas.rb:209
- - lib/modules/maas.rb:229
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value.is_a?(Hash)' 2 times
- - lib/modules/maas.rb:127
- - lib/modules/maas.rb:237
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['hostname']' 2 times
- - lib/modules/maas.rb:152
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['packid']' 11 times
- - lib/modules/maas.rb:128
- - lib/modules/maas.rb:129
- - lib/modules/maas.rb:143
- - lib/modules/maas.rb:147
- - lib/modules/maas.rb:158
- - lib/modules/maas.rb:169
- - lib/modules/maas.rb:176
- - lib/modules/maas.rb:194
- - lib/modules/maas.rb:201
- - lib/modules/maas.rb:218
- - lib/modules/maas.rb:221
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['parentpack']' 2 times
- - lib/modules/maas.rb:127
- - lib/modules/maas.rb:237
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['parentpack'].present? == false' 2 times
- - lib/modules/maas.rb:127
- - lib/modules/maas.rb:237
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['parentpack'].present?' 2 times
- - lib/modules/maas.rb:127
- - lib/modules/maas.rb:237
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['ssh']' 2 times
- - lib/modules/maas.rb:153
- * (DuplicateMethodCall) Maas#self.choice_machine calls 'value['ssh_name']' 2 times
- - lib/modules/maas.rb:154
- * (DuplicateMethodCall) Maas#self.deploy calls 'Bugsnag.notify(Maas::ProvisionError.new(error))' 2 times
- - lib/modules/maas.rb:306
- - lib/modules/maas.rb:328
- * (DuplicateMethodCall) Maas#self.deploy calls 'ENV['MAAS_TEST_DEVICE']' 2 times
- - lib/modules/maas.rb:256
- - lib/modules/maas.rb:257
- * (DuplicateMethodCall) Maas#self.deploy calls 'TaskMailer.send_generic_email( ... )' 2 times
- - lib/modules/maas.rb:308
- - lib/modules/maas.rb:330
- * (DuplicateMethodCall) Maas#self.deploy calls 'params.to_json' 2 times
- - lib/modules/maas.rb:286
- - lib/modules/maas.rb:326
- * (DuplicateMethodCall) Maas#self.deploy calls 'pp error' 2 times
- - lib/modules/maas.rb:305
- - lib/modules/maas.rb:327
- * (DuplicateMethodCall) Maas#self.distro_series calls 'option.to_i' 3 times
- - lib/modules/maas.rb:670
- * (DuplicateMethodCall) Maas#self.execute_request calls 'params[:header]' 2 times
- - lib/modules/maas.rb:645
- - lib/modules/maas.rb:648
- * (DuplicateMethodCall) Maas#self.find_machine_by_label calls 'request.body' 2 times
- - lib/modules/maas.rb:779
- - lib/modules/maas.rb:781
- * (DuplicateMethodCall) Maas#self.find_paid_order calls 'api.order' 2 times
- - lib/modules/maas.rb:87
- - lib/modules/maas.rb:88
- * (DuplicateMethodCall) Maas#self.find_paid_order calls 'orders_result_1.data' 2 times
- - lib/modules/maas.rb:93
- - lib/modules/maas.rb:94
- * (DuplicateMethodCall) Maas#self.find_paid_order calls 'orders_result_2.data' 2 times
- - lib/modules/maas.rb:100
- - lib/modules/maas.rb:101
- * (DuplicateMethodCall) Maas#self.reinstall calls 'Bugsnag.notify(Maas::ActionError.new(error))' 2 times
- - lib/modules/maas.rb:563
- - lib/modules/maas.rb:618
- * (DuplicateMethodCall) Maas#self.reinstall calls 'api.client' 2 times
- - lib/modules/maas.rb:559
- - lib/modules/maas.rb:589
- * (DuplicateMethodCall) Maas#self.reinstall calls 'args['ssh']' 2 times
- - lib/modules/maas.rb:608
- * (DuplicateMethodCall) Maas#self.reinstall calls 'args['ssh_name']' 2 times
- - lib/modules/maas.rb:578
- * (DuplicateMethodCall) Maas#self.reinstall calls 'device['dev_desc']' 3 times
- - lib/modules/maas.rb:572
- - lib/modules/maas.rb:602
- * (DuplicateMethodCall) Maas#self.reinstall calls 'device['label']' 3 times
- - lib/modules/maas.rb:539
- - lib/modules/maas.rb:572
- - lib/modules/maas.rb:602
- * (DuplicateMethodCall) Maas#self.reinstall calls 'device['packid']' 2 times
- - lib/modules/maas.rb:559
- - lib/modules/maas.rb:590
- * (DuplicateMethodCall) Maas#self.reinstall calls 'device_result.error_message' 2 times
- - lib/modules/maas.rb:535
- - lib/modules/maas.rb:536
- * (DuplicateMethodCall) Maas#self.reinstall calls 'pp error' 2 times
- - lib/modules/maas.rb:562
- - lib/modules/maas.rb:617
- * (DuplicateMethodCall) Maas#self.reinstall calls 'service_result.error?' 2 times
- - lib/modules/maas.rb:560
- - lib/modules/maas.rb:596
- * (DuplicateMethodCall) Maas#self.send_access calls 'params[:client_id]' 2 times
- - lib/modules/maas.rb:468
- - lib/modules/maas.rb:504
- * (DuplicateMethodCall) Maas#self.send_access calls 'params[:distro]' 2 times
- - lib/modules/maas.rb:484
- - lib/modules/maas.rb:488
- * (DuplicateMethodCall) Maas#self.send_access calls 'params[:ssh_name]' 3 times
- - lib/modules/maas.rb:488
- - lib/modules/maas.rb:489
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'order_item[:dev_desc]' 2 times
- - lib/modules/maas.rb:427
- - lib/modules/maas.rb:430
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'order_item[:mac_address]' 2 times
- - lib/modules/maas.rb:405
- - lib/modules/maas.rb:415
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'order_item[:option]' 2 times
- - lib/modules/maas.rb:377
- - lib/modules/maas.rb:387
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'order_item[:ssh]' 4 times
- - lib/modules/maas.rb:361
- - lib/modules/maas.rb:438
- - lib/modules/maas.rb:440
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'order_item[:ssh].present?' 2 times
- - lib/modules/maas.rb:361
- - lib/modules/maas.rb:438
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'pp response.body' 2 times
- - lib/modules/maas.rb:456
- - lib/modules/maas.rb:459
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'response.body' 3 times
- - lib/modules/maas.rb:456
- - lib/modules/maas.rb:459
- - lib/modules/maas.rb:460
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'user_data_file.include?('windows')' 2 times
- - lib/modules/maas.rb:425
- - lib/modules/maas.rb:438
- * (DuplicateMethodCall) Maas#self.start_deploy calls 'wan_ip.nil?' 2 times
- - lib/modules/maas.rb:400
- - lib/modules/maas.rb:405
- * (DuplicateMethodCall) Maas#self.type calls 'option.to_i' 2 times
- - lib/modules/maas.rb:661
- * (DuplicateMethodCall) Maas#self.validate_order calls 'order_item.metadata' 2 times
- - lib/modules/maas.rb:18
- * (DuplicateMethodCall) Maas#self.validate_order calls 'order_item.upgrade_items' 2 times
- - lib/modules/maas.rb:16
- - lib/modules/maas.rb:17
- * (IrresponsibleModule) Maas has no descriptive comment
- - lib/modules/maas.rb:6
- * (LongParameterList) Maas#self.show_access_on_control has 4 parameters
- - lib/modules/maas.rb:799
- * (LongParameterList) Maas#self.start_deploy has 4 parameters
- - lib/modules/maas.rb:342
- * (LongParameterList) Maas#self.validate_order has 4 parameters
- - lib/modules/maas.rb:11
- * (NestedIterators) Maas#self.check contains iterators nested 2 deep
- - lib/modules/maas.rb:52
- - lib/modules/maas.rb:54
- * (NestedIterators) Maas#self.deploy contains iterators nested 2 deep
- - lib/modules/maas.rb:272
- * (NestedIterators) Maas#self.find_mac_address contains iterators nested 2 deep
- - lib/modules/maas.rb:788
- * (NilCheck) Maas#self.check performs a nil-check
- - lib/modules/maas.rb:53
- - lib/modules/maas.rb:55
- * (NilCheck) Maas#self.choice_machine performs a nil-check
- - lib/modules/maas.rb:141
- - lib/modules/maas.rb:165
- - lib/modules/maas.rb:177
- * (NilCheck) Maas#self.deploy performs a nil-check
- - lib/modules/maas.rb:284
- - lib/modules/maas.rb:302
- * (NilCheck) Maas#self.finish_service_process performs a nil-check
- - lib/modules/maas.rb:511
- * (NilCheck) Maas#self.reinstall performs a nil-check
- - lib/modules/maas.rb:548
- * (NilCheck) Maas#self.start_deploy performs a nil-check
- - lib/modules/maas.rb:400
- - lib/modules/maas.rb:405
- * (NilCheck) Maas#self.validate_order performs a nil-check
- - lib/modules/maas.rb:20
- * (TooManyStatements) Maas#self.check has approx 23 statements
- - lib/modules/maas.rb:36
- * (TooManyStatements) Maas#self.choice_machine has approx 60 statements
- - lib/modules/maas.rb:120
- * (TooManyStatements) Maas#self.deploy has approx 35 statements
- - lib/modules/maas.rb:253
- * (TooManyStatements) Maas#self.execute_request has approx 6 statements
- - lib/modules/maas.rb:635
- * (TooManyStatements) Maas#self.find_device_ip has approx 7 statements
- - lib/modules/maas.rb:761
- * (TooManyStatements) Maas#self.find_mac_address has approx 6 statements
- - lib/modules/maas.rb:785
- * (TooManyStatements) Maas#self.find_paid_order has approx 12 statements
- - lib/modules/maas.rb:85
- * (TooManyStatements) Maas#self.reinstall has approx 41 statements
- - lib/modules/maas.rb:522
- * (TooManyStatements) Maas#self.send_access has approx 21 statements
- - lib/modules/maas.rb:465
- * (TooManyStatements) Maas#self.start_deploy has approx 46 statements
- - lib/modules/maas.rb:342
- * (TooManyStatements) Maas#self.validate_order has approx 9 statements
- - lib/modules/maas.rb:11
- * (UncommunicativeVariableName) Maas#self.check has the variable name 'n'
- - lib/modules/maas.rb:54
- * (UncommunicativeVariableName) Maas#self.check has the variable name 't'
- - lib/modules/maas.rb:52
- * (UncommunicativeVariableName) Maas#self.find_paid_order has the variable name 'orders_result_1'
- - lib/modules/maas.rb:87
- * (UncommunicativeVariableName) Maas#self.find_paid_order has the variable name 'orders_result_2'
- - lib/modules/maas.rb:88
- * (UncommunicativeVariableName) Maas#self.random_password has the variable name 'i'
- - lib/modules/maas.rb:745
- * (UncommunicativeVariableName) Maas#self.start_deploy has the variable name 'f'
- - lib/modules/maas.rb:370
- Score: 90.31
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement