Changeset 52070a1 in sasview for guiframe

Timestamp:

Jan 17, 2010 4:35:43 PM (15 years ago)

Author:

Mathieu Doucet <doucetm@…>

Branches:

master, ESS_GUI, ESS_GUI_Docs, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_iss879, ESS_GUI_iss959, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc, costrafo411, magnetic_scatt, release-4.1.1, release-4.1.2, release-4.2.2, release_4.0.1, ticket-1009, ticket-1094-headless, ticket-1242-2d-resolution, ticket-1243, ticket-1249, ticket885, unittest-saveload

Children:

d68c655

Parents:

aafa962

Message:

gui_frame: updated the version checker to minimize visible effect on performance.

Location:

guiframe

Files:

• gui_manager.py (modified) (2 diffs)
• version.py (added)

guiframe/gui_manager.py

@@ -518 +518 @@
                 n_panels += 1
        
-        if n_panels>5:
+        if n_panels>2:
             viewmenu = wx.Menu()
             for plug in self.plugins:
@@ -689 +689 @@
         """
             Check with the deployment server whether a new version
-            of the application is available
-        """
-        import urllib
-        try: 
-            h = urllib.urlopen(config.__update_URL__)
-            lines = h.readlines()
-            line = ''
-            if len(lines)>0:
-                line = lines[0]
-                
-                toks = line.lstrip().rstrip().split('.')
-                toks_current = config.__version__.split('.')
-                update_available = False
-                for i in range(len(toks)):
-                    if len(toks[i].strip())>0:
-                        if int(toks[i].strip())>int(toks_current[i]):
-                            update_available = True
-                if update_available:
-                    #print "Version %s is available" % line.rstrip().lstrip()
-                    self.SetStatusText("Version %s is available! See the Help menu to download it." % line.rstrip().lstrip())
-                    if event != None:
-                        import webbrowser
-                        webbrowser.open(config.__download_page__)
-                else:
-                    if event != None:
-                        self.SetStatusText("You have the latest version of %s" % config.__appname__)
+            of the application is available.
+            A thread is started for the connecting with the server. The thread calls
+            a call-back method when the current version number has been obtained.
+        """
+        import version
+        checker = version.VersionThread(config.__update_URL__, self._process_version, baggage=event==None)
+        checker.start()  
+    
+    def _process_version(self, version, standalone=True):
+        """
+            Call-back method for the process of checking for updates.
+            This methods is called by a VersionThread object once the current
+            version number has been obtained. If the check is being done in the
+            background, the user will not be notified unless there's an update.
+            
+            @param version: version string
+            @param standalone: True of the update is being checked in the background, False otherwise.
+        """
+        try:
+            if cmp(version, config.__version__)>0:
+                self.SetStatusText("Version %s is available! See the Help menu to download it." % version)
+                if not standalone:
+                    import webbrowser
+                    webbrowser.open(config.__download_page__)
+            else:
+                if not standalone:
+                    self.SetStatusText("You have the latest version of %s" % config.__appname__)
         except:
             logging.error("guiframe: could not get latest application version number\n  %s" % sys.exc_value)
-            if event != None:
-                self.SetStatusText("You have the latest version of %s" % config.__appname__)
-            
-            
+            if not standalone:
+                self.SetStatusText("Could not connect to the application server. Please try again later.")
+                    
+        
     def _onAbout(self, evt):
         """