Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2 for openSUSE:Factory checked 
in at 2021-03-08 15:15:31
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2 (Old)
 and      /work/SRC/openSUSE:Factory/.yast2.new.2378 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2"

Mon Mar  8 15:15:31 2021 rev:505 rq:876738 version:4.3.56

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2/yast2.changes      2021-02-26 
21:58:14.307756903 +0100
+++ /work/SRC/openSUSE:Factory/.yast2.new.2378/yast2.changes    2021-03-08 
15:17:10.213934869 +0100
@@ -1,0 +2,7 @@
+Thu Mar  4 12:41:56 UTC 2021 - Josef Reidinger <jreidin...@suse.com>
+
+- Fix backward compatibility for focus parameter of
+  Report.AnyQuestion/Report.ErrorAnyQuestion (bsc#1183011)
+- 4.3.56
+
+-------------------------------------------------------------------

Old:
----
  yast2-4.3.55.tar.bz2

New:
----
  yast2-4.3.56.tar.bz2

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

Other differences:
------------------
++++++ yast2.spec ++++++
--- /var/tmp/diff_new_pack.57Js2Z/_old  2021-03-08 15:17:11.573935865 +0100
+++ /var/tmp/diff_new_pack.57Js2Z/_new  2021-03-08 15:17:11.577935868 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2
-Version:        4.3.55
+Version:        4.3.56
 Release:        0
 Summary:        YaST2 Main Package
 License:        GPL-2.0-only

++++++ yast2-4.3.55.tar.bz2 -> yast2-4.3.56.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-4.3.55/library/control/src/modules/ProductProfile.rb 
new/yast2-4.3.56/library/control/src/modules/ProductProfile.rb
--- old/yast2-4.3.55/library/control/src/modules/ProductProfile.rb      
2021-02-12 15:32:30.000000000 +0100
+++ new/yast2-4.3.56/library/control/src/modules/ProductProfile.rb      
2021-03-04 14:03:33.000000000 +0100
@@ -212,7 +212,7 @@
         ),
         continue_button,
         cancel_button,
-        :no_button
+        :focus_no
       )
       if !ret && !product_id.nil?
         # canceled adding add-on: remove profile stored before
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-4.3.55/library/general/src/modules/Report.rb 
new/yast2-4.3.56/library/general/src/modules/Report.rb
--- old/yast2-4.3.55/library/general/src/modules/Report.rb      2021-02-12 
15:32:30.000000000 +0100
+++ new/yast2-4.3.56/library/general/src/modules/Report.rb      2021-03-04 
14:03:33.000000000 +0100
@@ -352,16 +352,23 @@
       Builtins.size(@errors)
     end
 
+    BACKWARD_MAPPING = {
+      focus_yes: :yes,
+      focus_no:  :no
+    }.freeze
+
     # Question with headline and Yes/No Buttons
     # @param [String] headline Popup Headline
     # @param [String] message Popup Message
     # @param [String] yes_button_message Yes Button Message
     # @param [String] no_button_message No Button Message
-    # @param [Symbol] focus Which Button has the focus
+    # @param [Symbol] focus Which Button has the focus. Possible values are 
`:yes` and :no`.
+    #   For backward compatibility also `:focus_yes` and `:focus_no` is 
accepted.
     # @return [Boolean] True if Yes is pressed, otherwise false
     def AnyQuestion(headline, message, yes_button_message, no_button_message, 
focus)
       Builtins.y2milestone(1, "%1", message) if @log_yesno_messages
 
+      focus = BACKWARD_MAPPING[focus] || focus
       ret = false
       if @display_yesno_messages
         timeout = (@timeout_yesno_messages.to_s.to_i > 0) ? 
@timeout_yesno_messages : 0
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-4.3.55/library/general/test/report_test.rb 
new/yast2-4.3.56/library/general/test/report_test.rb
--- old/yast2-4.3.55/library/general/test/report_test.rb        2021-02-12 
15:32:30.000000000 +0100
+++ new/yast2-4.3.56/library/general/test/report_test.rb        2021-03-04 
14:03:33.000000000 +0100
@@ -367,5 +367,15 @@
         end
       end
     end
+
+    describe ".AnyQuestion" do
+      it "accepts also :focus_yes/:focus_no as focus parameter" do
+        subject.DisplayYesNoMessages(true, 0)
+        expect(Yast2::Popup).to receive(:show)
+          .with("Message", headline: "test", buttons: { yes: "yes", no: "no" 
}, focus: :no, timeout: 0)
+
+        subject.AnyQuestion("test", "Message", "yes", "no", :focus_no)
+      end
+    end
   end
 end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-4.3.55/package/yast2.changes 
new/yast2-4.3.56/package/yast2.changes
--- old/yast2-4.3.55/package/yast2.changes      2021-02-12 15:32:30.000000000 
+0100
+++ new/yast2-4.3.56/package/yast2.changes      2021-03-04 14:03:33.000000000 
+0100
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Thu Mar  4 12:41:56 UTC 2021 - Josef Reidinger <jreidin...@suse.com>
+
+- Fix backward compatibility for focus parameter of
+  Report.AnyQuestion/Report.ErrorAnyQuestion (bsc#1183011)
+- 4.3.56
+
+-------------------------------------------------------------------
 Fri Feb 12 13:46:19 UTC 2021 - Ancor Gonzalez Sosa <an...@suse.com>
 
 - Fixed bug introduced while adding auto wrapping (bsc#1179893)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-4.3.55/package/yast2.spec 
new/yast2-4.3.56/package/yast2.spec
--- old/yast2-4.3.55/package/yast2.spec 2021-02-12 15:32:30.000000000 +0100
+++ new/yast2-4.3.56/package/yast2.spec 2021-03-04 14:03:33.000000000 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2
-Version:        4.3.55
+Version:        4.3.56
 Release:        0
 Summary:        YaST2 Main Package
 License:        GPL-2.0-only

Reply via email to