Avoid deeply nested control flow statements.

Confirmed

        raise NotMissingError, "Branch call with missingok parameter but branched source (#{params[:project]}/#{params[:package]}) exists." if Package.exists_by_project_and_name(params[:project], params[:package],
                                                                                                                                                                                  allow_remote_packages: true)

Avoid deeply nested control flow statements.

Confirmed

          next if pkg2.nil? || @packages.pluck(:package).include?(pkg2) # avoid double instances

Avoid deeply nested control flow statements.

Confirmed

        self.makeoriginolder = true if tprj.attribs.find_by(attrib_type: at)

Method update_product_list has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def update_product_list
    # short cut to ensure that no products are left over
    unless is_product?
      products.destroy_all
      return

• Read up
  Read up

Method show has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def show
    project_name = params[:project]
    if params.key?(:deleted)
      unless Project.find_by_name(project_name) || Project.is_remote_project?(project_name)
        # project is deleted or not accessible

• Read up
  Read up

Avoid deeply nested control flow statements.

Confirmed

            unless allowed
              render_error status: 403, errorcode: 'execute_cmd_no_permission',
                           message: "No permission to execute command on package #{pack_name}"
              return
            end

Method write_to_backend has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def write_to_backend
    reset_cache
    raise ArgumentError, 'no commit_user set' unless commit_user
    raise InvalidParameterError, 'Project meta file can not be written via package model' if name == '_project'

• Read up
  Read up

Method check_action_permission_source! has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def check_action_permission_source!
    return unless source_project

    sprj = Project.get_by_name(source_project)
    raise UnknownProject, "Unknown source project #{source_project}" unless sprj

• Read up
  Read up

Avoid deeply nested control flow statements.

Confirmed

        raise IllegalRequest, 'Maintenance requests accept only projects as target' if target_package

Method validate_read_access_of_deleted_package has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def validate_read_access_of_deleted_package(project, name)
    prj = Project.get_by_name(project)
    if prj.is_a?(Project)
      raise Project::ReadAccessError, project.to_s if prj.disabled_for?('access', nil, nil)
      raise Package::ReadSourceAccessError, "#{target_project_name}/#{target_package_name}" if prj.disabled_for?('sourceaccess', nil, nil)

• Read up
  Read up

Method update_from_xml has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def update_from_xml(xmlhash, ignore_lock = nil)
    check_write_access!(ignore_lock)

    Package.transaction do
      self.title = xmlhash.value('title')

• Read up
  Read up

Method update_file has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def update_file
    check_permissions_for_file

    raise PutFileNoPermission, "Insufficient permissions to store file in package #{@package_name}, project #{@project_name}" unless @allowed

• Read up
  Read up

Avoid deeply nested control flow statements.

Confirmed

        raise 'We should have expanded a target_project' unless target_project

Method find_assignees has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

    def find_assignees(binary_name)
      projects = @rootproject.expand_all_projects

      # binary search via all projects
      data = Xmlhash.parse(Backend::Api::Search.binary(projects.map(&:name), binary_name))

• Read up
  Read up

Avoid deeply nested control flow statements.

Confirmed

            unless allowed
              render_error status: 403, errorcode: 'execute_cmd_no_permission',
                           message: "No permission to execute command on package #{pack_name} in project #{prj.name}"
              return
            end

Avoid deeply nested control flow statements.

Confirmed

          target = if tpkg
                     Package.find_by_project_and_name(tprj, tpkg)
                   else
                     Project.find_by_name(tprj)
                   end

Method parse_status has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def parse_status(project_name, package)
    comments_to_clear = []
    outs = []
    sortkey = "9-ok-#{package['name']}"
    age = age_from_devel(package['develmtime'].to_i)

• Read up
  Read up

Avoid deeply nested control flow statements.

Confirmed

          target.add_maintainer(@bs_request.creator) if target.can_be_modified_by?(User.possibly_nobody)

Method add_channels has a Cognitive Complexity of 11 (exceeds 8 allowed). Consider refactoring.

Open

  def add_channels(mode = :add_disabled)
    raise InvalidParameterError unless %i[add_disabled skip_disabled enable_all].include?(mode)
    return if is_channel?

    opkg = origin_container(local: false)

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

        raise IncidentHasNoMaintenanceProject, 'incident projects shall only create below maintenance projects' unless prj.kind.in?(%w[maintenance maintenance_incident])