Hello community,

here is the log from the commit of package yast2-add-on for openSUSE:Factory 
checked in at 2017-10-23 16:42:34
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-add-on (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-add-on.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-add-on"

Mon Oct 23 16:42:34 2017 rev:84 rq:535191 version:4.0.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-add-on/yast2-add-on.changes        
2017-10-07 17:48:59.231730814 +0200
+++ /work/SRC/openSUSE:Factory/.yast2-add-on.new/yast2-add-on.changes   
2017-10-23 16:42:37.066657501 +0200
@@ -1,0 +2,8 @@
+Thu Oct 19 11:14:52 UTC 2017 - [email protected]
+
+- Allow preselecting the add-on URL schema and hiding the "I would
+  like to install an additional Add On Product" check box from
+  the external code (FATE#323163)
+- 4.0.1
+
+-------------------------------------------------------------------

Old:
----
  yast2-add-on-4.0.0.tar.bz2

New:
----
  yast2-add-on-4.0.1.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-add-on.spec ++++++
--- /var/tmp/diff_new_pack.4nYOas/_old  2017-10-23 16:42:37.794623433 +0200
+++ /var/tmp/diff_new_pack.4nYOas/_new  2017-10-23 16:42:37.794623433 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-add-on
-Version:        4.0.0
+Version:        4.0.1
 Release:        0
 Summary:        YaST2 - Add-On media installation code
 License:        GPL-2.0

++++++ yast2-add-on-4.0.0.tar.bz2 -> yast2-add-on-4.0.1.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-add-on-4.0.0/package/yast2-add-on.changes 
new/yast2-add-on-4.0.1/package/yast2-add-on.changes
--- old/yast2-add-on-4.0.0/package/yast2-add-on.changes 2017-09-21 
14:03:53.926982072 +0200
+++ new/yast2-add-on-4.0.1/package/yast2-add-on.changes 2017-10-19 
13:41:57.610353693 +0200
@@ -1,4 +1,12 @@
 -------------------------------------------------------------------
+Thu Oct 19 11:14:52 UTC 2017 - [email protected]
+
+- Allow preselecting the add-on URL schema and hiding the "I would
+  like to install an additional Add On Product" check box from
+  the external code (FATE#323163)
+- 4.0.1
+
+-------------------------------------------------------------------
 Wed Sep 20 16:46:37 UTC 2017 - [email protected]
 
 - Display also the repository path in the URL column if it is
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-add-on-4.0.0/package/yast2-add-on.spec 
new/yast2-add-on-4.0.1/package/yast2-add-on.spec
--- old/yast2-add-on-4.0.0/package/yast2-add-on.spec    2017-09-21 
14:03:53.926982072 +0200
+++ new/yast2-add-on-4.0.1/package/yast2-add-on.spec    2017-10-19 
13:41:57.614353693 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-add-on
-Version:        4.0.0
+Version:        4.0.1
 Release:        0
 Summary:        YaST2 - Add-On media installation code
 License:        GPL-2.0
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-add-on-4.0.0/src/include/add-on/add-on-workflow.rb 
new/yast2-add-on-4.0.1/src/include/add-on/add-on-workflow.rb
--- old/yast2-add-on-4.0.0/src/include/add-on/add-on-workflow.rb        
2017-09-21 14:03:53.934982072 +0200
+++ new/yast2-add-on-4.0.1/src/include/add-on/add-on-workflow.rb        
2017-10-19 13:41:57.682353693 +0200
@@ -174,7 +174,10 @@
     def MediaSelect
       aliases = {
         "type"  => lambda do
-          ret = TypeDialog()
+          # use the preselected type or the same type as for the previous 
add-on,
+          # if not set it will use the default (the code actually runs
+          # SourceDialogs.SetURL(SourceDialogs.GetURL) internally)
+          ret = TypeDialogOpts(true, SourceDialogs.GetURL)
           log.debug "SourceDialogs.addon_enabled: 
#{SourceDialogs.addon_enabled}"
           # explicitly check for false (nil means the checkbox was not 
displayed)
           ret = :skip if ret == :next && SourceDialogs.addon_enabled == false
@@ -1143,10 +1146,14 @@
 
           # adding new add-on
         elsif ret == :add || ret == :skip_to_add
-          # show checkbox only first time in installation when there is no
-          # other addons, so allow to quickly skip adding addons, otherwise
-          # it make no sense as user explicitelly want add addon
-          SourceDialogs.display_addon_checkbox = ret == :skip_to_add
+          # Show the checkbox only the first time in installation when there 
is no
+          # other addon present, allow to quickly skip adding addons. In the
+          # following runs it makes no sense as user explicitly wants to add 
an addon.
+          # Change the state only if it has the default value (nil),
+          # if the check box state has been already set then keep it unchanged.
+          if SourceDialogs.display_addon_checkbox.nil?
+            SourceDialogs.display_addon_checkbox = (ret == :skip_to_add)
+          end
 
           # bugzilla #293428
           # Release all sources before adding a new one


Reply via email to