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