Method initialize has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    def initialize(user, load_admin_abilities = true, pdp_class = nil)
      @pdp_class = pdp_class

      @ability_builders = ability_builder_classes.map do |builder_class|
        builder_class.new(self, user, pdp_class)

• Read up
  Read up

Method update has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

  def update
    alert = nil
    params[:virtual_machine_flavor][:hourly_cost].each do |os_family_id, cost|
      os_family = Atmosphere::OSFamily.find_by(id: os_family_id)
      if cost.strip.empty? &&

• Read up
  Read up

Method add_dnat_for_vm has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    def add_dnat_for_vm(vm, pmts)
      pmt_map = {}
      to_add = pmts.collect {|pmt|
        if not pmt.target_port.in? MIN_PORT_NO..MAX_PORT_NO
          Rails.logger.error "Error when trying to add redirections for VM #{vm.uuid} with IP #{vm.ip}. Requested redirection for forbidden port #{pmt.target_port}"

• Read up
  Read up

Method add_dnat has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    def add_dnat
      return unless ip?
      pmts = nil
      if appliances.first && appliances.first.development?
        pmts = appliances.first.dev_mode_property_set.port_mapping_templates

• Read up
  Read up

Method create_or_update_http_mappings has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

      def create_or_update_http_mappings
        to_update, to_remove = [], []
        port_mapping_templates.each do |pmt|
          to_update << update_for_pmt(pmt, 'http') if pmt.http?
          to_update << update_for_pmt(pmt, 'https') if pmt.https?

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                BillingLog.new(timestamp: Time.now.utc, user: appl.appliance_set.user, appliance: appl.id.to_s+'---'+(appl.name.blank? ? 'unnamed_appliance' : appl.name), fund: appl.fund.name, message: "Error saving appliance following update of billing information.", actor: "bill_appliance", amount_billed: 0).save

Method create_dev_mode_property_set has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def create_dev_mode_property_set(options={})
      unless dev_mode_property_set
        set = DevModePropertySet.create_from(appliance_type)
        set.preference_memory = options[:preference_memory] if options[:preference_memory]
        set.preference_cpu = options[:preference_cpu] if options[:preference_cpu]

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                BillingLog.new(timestamp: Time.now.utc, user: appl.appliance_set.user, appliance: appl.id.to_s+'---'+(appl.name.blank? ? 'unnamed_appliance' : appl.name), fund: appl.fund.name, message: message, actor: "bill_appliance", amount_billed: amount_due).save

Method perform_update! has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def perform_update!
      vm.source_template ||= source_template
      vm.name = server.name || '[unnamed]'
      vm.state = map_saving_state(vm, server.task_state) ||
        map_state(server.state.downcase.to_sym)

• Read up
  Read up

Method ensure_single_default has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    def ensure_single_default
      unless default
        ufs = UserFund.where(user_id: user_id)
        if ufs.count == 0 || (ufs.count == 1 && ufs[0].id == id)
          self.default = true

• Read up
  Read up

Method get_candidate_tenants_for_template has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

        def get_candidate_tenants_for_template(tmpl)
          # Determine which tenants can be used to spawn this specific template in the context
          # of the current @appliance.
          # If @appliance is nil (which may happen) return active tenants:
          if @appliance.blank?

• Read up
  Read up

Method update_custom_proxy has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    def update_custom_proxy
      old_custom_name = custom_name_was

      yield

• Read up
  Read up

Avoid too many return statements within this method.

Open

    return start if start_action?

Avoid too many return statements within this method.

Open

    return suspend if suspend_action?

Method execute has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

      def execute
        if optimization_strategy.can_scale_manually?
          action.log(I18n.t('scale_appliance.start'))
          if quantity > 0
            vms = optimization_strategy.vms_to_start(quantity)

• Read up
  Read up

Method update has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def update
    if @tenant.update(tenant_params)
      if @tenant.proxy_urls_changed?
        Atmosphere::Proxy::TenantUrlUpdater.new(@tenant).update
      end

• Read up
  Read up

Method apply_funding_policy has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    def self.apply_funding_policy
      VirtualMachine.manageable.each do |vm|
        # If this VM has at least one prepaid deployment, it must not be touched.
        # For safety's sake, we will also not touch deployments whose billing state is flagged as erroneous
        if vm.deployments.select {|dep| ['prepaid', 'error'].include? dep.billing_state}.count > 0

• Read up
  Read up