Author: david
Date: Thu Nov 18 12:39:15 2010
New Revision: 8855

Log:
Merge -r8837:8853 from trunk

Modified:
   branches/dcb/apps/qubit/modules/default/templates/_updateCheck.php
   branches/dcb/js/updateCheck.js

Modified: branches/dcb/apps/qubit/modules/default/templates/_updateCheck.php
==============================================================================
--- branches/dcb/apps/qubit/modules/default/templates/_updateCheck.php  Thu Nov 
18 12:38:47 2010        (r8854)
+++ branches/dcb/apps/qubit/modules/default/templates/_updateCheck.php  Thu Nov 
18 12:39:15 2010        (r8855)
@@ -1,7 +1,8 @@
+<?php $notificationMessage = __('A release %1% upgrade is available.') ?>
+
 <?php if ($sf_request->getCookie('has_js')): ?>
 
   <?php $data = json_encode($sf_data->getRaw('updateCheckData')) ?>
-  <?php $notificationMessage = __('A release %1% upgrade is available.') ?>
 
   <?php use_helper('Javascript') ?>
 

Modified: branches/dcb/js/updateCheck.js
==============================================================================
--- branches/dcb/js/updateCheck.js      Thu Nov 18 12:38:47 2010        (r8854)
+++ branches/dcb/js/updateCheck.js      Thu Nov 18 12:39:15 2010        (r8855)
@@ -9,7 +9,8 @@
         {
           var showNotification = function (version)
             {
-              if (1 > version_compare(Qubit.updateCheck.currentVersion, 
version))
+              // Show notification only when server version is greater
+              if (-1 == version_compare(Qubit.updateCheck.currentVersion, 
version))
               {
                 $('<div id="update-check"><span>' + 
Qubit.updateCheck.notificationMessage.replace('%1%', version) + 
'</span></div>').prependTo('#page');
               }

-- 
You received this message because you are subscribed to the Google Groups 
"Qubit Toolkit Commits" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/qubit-commits?hl=en.

Reply via email to