Allow in autoyast only supported bootloaders(bnc#882210)

src/clients/bootloader_auto.rb

@@ -52,11 +52,12 @@ def main
       Builtins.y2debug("param=%1", @param)
 
       if @func == "Import"
-        @ret = Bootloader.Import(
-          AI2Export(
-            Convert.convert(@param, :from => "map", :to => "map <string, any>")
-          )
-        )
+        data = AI2Export(@param)
+        if data
+          @ret = Bootloader.Import(data)
+        else
+          @ret = false
+        end
       # Create a summary
       # return string
       elsif @func == "Summary"

src/include/bootloader/routines/autoinstall.rb

@@ -109,6 +109,7 @@ def initialize_bootloader_routines_autoinstall(include_target)
       Yast.import "Initrd"
       Yast.import "Kernel"
       Yast.import "Mode"
+      Yast.import "Popup"
     end
 
     # Add missing data (eg. root filesystem) to sections imported from profile
@@ -413,6 +414,14 @@ def AI2Export(ai)
     def Export2AI(exp)
       exp = deep_copy(exp)
       # bootloader type and location stuff
+      unsupported_bootloaders = ["grub", "zipl", "plilo", "lilo", "elilo"]
+      if unsupported_bootloaders.include?(exp["loader_location"])
+        # FIXME this should be better handled by exception and show it properly, but it require too big change now
+        Popup.Error(_("Unsupported bootloader '%s'. Please adapt your autoyast profile according."),
+          exp["loader_location"])
+        return nil
+      end
+
       ai = { "loader_type" => Ops.get_string(exp, "loader_type", "default") }
       glob = Builtins.filter(Ops.get_map(exp, ["specific", "global"], {})) do |k, v|
         Builtins.substring(k, 0, 2) != "__"