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

Open

    def run
      guarded_run
    rescue ::Bootloader::NoRoot
      Yast::Report.Error(
        _("YaST cannot configure the bootloader because it failed to find the root file system.")

• Read up
  Read up

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

Open

    def merge_attributes(default, other)
      # string attributes
      [:serial_console, :timeout, :hidden_timeout, :distributor,
       :gfxmode, :theme, :default].each do |attr|
        val = other.public_send(attr)

• Read up
  Read up

Consider simplifying this complex logical expression.

Open

      if Arch.i386 || Arch.x86_64 || Arch.aarch64 || Arch.arm || Arch.ppc || Arch.riscv64
        ret = kernel_cmdline
        ret << " resume=#{resume}" unless resume.empty?
        ret << " #{features}" unless features.empty?
        ret << propose_cpu_mitigations

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

Open

      def export_grub2(res, bootloader)
        GRUB2_BOOLEAN_MAPPING.each do |key, method|
          val = bootloader.public_send(method)
          res[key] = val ? "true" : "false" unless val.nil?
        end

• Read up
  Read up

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

Open

      def import_kernel_params(data, default)
        DEFAULT_KERNEL_PARAMS_MAPPING.each do |key, method|
          val = data.global.public_send(key)
          next unless val

• Read up
  Read up

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

Open

    def correct_loader_type(type)
      return true if type == "none"

      # grub2 is sooo cool...
      return true if type == "grub2" && !::Bootloader::Systeminfo.efi_mandatory?

• Read up
  Read up

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

Open

    def available_locations
      case Yast::Arch.architecture
      when "i386", "x86_64"
        res = [:mbr]
        return res unless can_use_boot?

• Read up
  Read up

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

Open

      def export_grub2efi(res, bootloader)
        GRUB2EFI_BOOLEAN_MAPPING.each do |key, method|
          val = bootloader.public_send(method)
          res[key] = val ? "true" : "false" unless val.nil?
        end

• Read up
  Read up

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

Open

    def write(etc_only: false)
      # super have to called as first as grub install require some config writen in ancestor
      super

      device_map.write if (Yast::Arch.x86_64 || Yast::Arch.i386) && !etc_only

• Read up
  Read up

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

Open

    def ask_user(param)
      chosen_id = param["chosen_id"]
      result = :next
      log.info "ask user called with #{chosen_id}"

• Read up
  Read up

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

Open

    def self.load_from_kernel_args(kernel_params)
      console_parameters = kernel_params.parameter("console")
      return nil unless console_parameters

      console_parameters = Array(console_parameters)

• Read up
  Read up

Avoid too many return statements within this method.

Open

        return false

Avoid too many return statements within this method.

Open

        return false

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

Open

    def reset_needed?(force_reset, storage_changed)
      log.info "reset_needed? force_reset: #{force_reset} storage_changed: #{storage_changed}" \
               "auto mode: #{Yast::Mode.auto} cfg_changed #{Yast::Bootloader.proposed_cfg_changed}"
      return true if storage_changed
      return false if Yast::Mode.autoinst || Yast::Mode.autoupgrade

• Read up
  Read up

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

Open

      def bootloader_by_name(name)
        # needed to be able to store settings when moving between bootloaders
        @cached_bootloaders ||= {}
        case name
        when "grub2"

• Read up
  Read up

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

Open

      def export(config)
        log.info "exporting config #{config.inspect}"

        bootloader_type = config.name
        res = { "loader_type" => bootloader_type }

• Read up
  Read up

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

Open

      def import_grub2efi(data, bootloader)
        return unless bootloader.name == "grub2-efi"

        GRUB2EFI_BOOLEAN_MAPPING.each do |key, method|
          val = data.global.public_send(key)

• Read up
  Read up

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

Open

    def propose_cpu_mitigations
      linuxrc_value = Yast::Linuxrc.value_for("mitigations")
      log.info "linuxrc mitigations #{linuxrc_value.inspect}"
      return "" unless linuxrc_value.nil? # linuxrc already has mitigations

• Read up
  Read up

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

Open

    def assign_bootloader_device(selected_location)
      log.info "assign bootloader device '#{selected_location.inspect}'"
      # first, default to all off:
      stage1.clear_devices

• Read up
  Read up

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

Open

    def write
      log.info "Saving /etc/sysconfig/bootloader for #{bootloader}"

      write_option(:bootloader, bootloader)

• Read up
  Read up