Hello community,

here is the log from the commit of package yast2-nfs-client for 
openSUSE:Leap:15.2 checked in at 2020-01-17 11:58:51
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Leap:15.2/yast2-nfs-client (Old)
 and      /work/SRC/openSUSE:Leap:15.2/.yast2-nfs-client.new.26092 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-nfs-client"

Fri Jan 17 11:58:51 2020 rev:39 rq:764550 version:4.2.4

Changes:
--------
--- /work/SRC/openSUSE:Leap:15.2/yast2-nfs-client/yast2-nfs-client.changes      
2020-01-15 16:32:15.452881083 +0100
+++ 
/work/SRC/openSUSE:Leap:15.2/.yast2-nfs-client.new.26092/yast2-nfs-client.changes
   2020-01-17 11:58:52.112436115 +0100
@@ -1,0 +2,7 @@
+Wed Jan  8 22:49:11 UTC 2020 - David Diaz <dgonza...@suse.com>
+
+- Avoids displaying phantom NFS entries adding an action to
+  refresh the UI (related to bsc#1156446).
+- 4.2.4
+
+-------------------------------------------------------------------

Old:
----
  yast2-nfs-client-4.2.3.tar.bz2

New:
----
  yast2-nfs-client-4.2.4.tar.bz2

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

Other differences:
------------------
++++++ yast2-nfs-client.spec ++++++
--- /var/tmp/diff_new_pack.1Ujprq/_old  2020-01-17 11:58:52.480436277 +0100
+++ /var/tmp/diff_new_pack.1Ujprq/_new  2020-01-17 11:58:52.484436278 +0100
@@ -1,7 +1,7 @@
 #
 # spec file for package yast2-nfs-client
 #
-# Copyright (c) 2019 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2020 SUSE LINUX GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-nfs-client
-Version:        4.2.3
+Version:        4.2.4
 Release:        0
 Url:            https://github.com/yast/yast-nfs-client
 Summary:        YaST2 - NFS Configuration

++++++ yast2-nfs-client-4.2.3.tar.bz2 -> yast2-nfs-client-4.2.4.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-nfs-client-4.2.3/package/yast2-nfs-client.changes 
new/yast2-nfs-client-4.2.4/package/yast2-nfs-client.changes
--- old/yast2-nfs-client-4.2.3/package/yast2-nfs-client.changes 2019-10-30 
12:54:23.000000000 +0100
+++ new/yast2-nfs-client-4.2.4/package/yast2-nfs-client.changes 2020-01-09 
16:00:38.000000000 +0100
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Wed Jan  8 22:49:11 UTC 2020 - David Diaz <dgonza...@suse.com>
+
+- Avoids displaying phantom NFS entries adding an action to
+  refresh the UI (related to bsc#1156446).
+- 4.2.4
+
+-------------------------------------------------------------------
 Wed Oct 30 08:31:58 UTC 2019 - Knut Anderssen <kanders...@suse.com>
 
 - Removed nfsidmap hardcoded dependency once libnfsidmap was merged
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-nfs-client-4.2.3/package/yast2-nfs-client.spec 
new/yast2-nfs-client-4.2.4/package/yast2-nfs-client.spec
--- old/yast2-nfs-client-4.2.3/package/yast2-nfs-client.spec    2019-10-30 
12:54:23.000000000 +0100
+++ new/yast2-nfs-client-4.2.4/package/yast2-nfs-client.spec    2020-01-09 
16:00:38.000000000 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-nfs-client
-Version:        4.2.3
+Version:        4.2.4
 Release:        0
 Url:            https://github.com/yast/yast-nfs-client
 Summary:        YaST2 - NFS Configuration
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-nfs-client-4.2.3/src/clients/nfs-client4part.rb 
new/yast2-nfs-client-4.2.4/src/clients/nfs-client4part.rb
--- old/yast2-nfs-client-4.2.3/src/clients/nfs-client4part.rb   2019-10-30 
12:54:23.000000000 +0100
+++ new/yast2-nfs-client-4.2.4/src/clients/nfs-client4part.rb   2020-01-09 
16:00:38.000000000 +0100
@@ -30,16 +30,17 @@
         end
       end
 
-      if @func == "CreateUI"
-        Wizard.SetHelpText(@help_text1)
-        return FstabTab()
-      elsif @func == "FromStorage"
-        shares = Ops.get_list(@param, "shares", [])
+      case @func
+      when "CreateUI"
+        return create_ui
+      when "FromStorage"
+        shares = @param.fetch("shares", [])
         @nfs_entries = Nfs.load_nfs_entries(shares)
-      elsif @func == "Read"
+        refresh_ui
+      when "Read"
         Nfs.skip_fstab = true
         Nfs.Read
-      elsif @func == "HandleEvent"
+      when "HandleEvent"
         @widget_id = Ops.get(@param, "widget_id")
         @w_ids = [:newbut, :editbut, :delbut]
 
@@ -54,6 +55,33 @@
 
       nil
     end
+
+  private
+
+    # Generates the UI that allows to manage the NFS shares entries
+    #
+    # @return [Yast::Term] a term defining the UI
+    def create_ui
+      Wizard.SetHelpText(@help_text1)
+
+      ReplacePoint(ui_id, FstabTab())
+    end
+
+    # Updates the UI that allows to manage the NFS shares entries
+    #
+    # @return [Boolean] true when entries are successfully replaced; false 
otherwise.
+    def refresh_ui
+      # The UI could be not available yet as FromStorage action can be called 
just to sync
+      # the NFS entries before create the interface.
+      return unless UI.WidgetExists(ui_id)
+
+      UI.ReplaceWidget(ui_id, FstabTab())
+    end
+
+    # Returns the id for the NFS shares UI replace point
+    def ui_id
+      @ui_id ||= Id(:fstab_rp)
+    end
   end
 end
 


Reply via email to