Revision: 2022
          http://mrbs.svn.sourceforge.net/mrbs/?rev=2022&view=rev
Author:   cimorrison
Date:     2011-09-28 13:17:18 +0000 (Wed, 28 Sep 2011)
Log Message:
-----------
Fixed bug preventing the schedule/policy details dialog from being re-opened if 
it had previously been closed

Modified Paths:
--------------
    mrbs/trunk/web/Themes/default/header.inc

Modified: mrbs/trunk/web/Themes/default/header.inc
===================================================================
--- mrbs/trunk/web/Themes/default/header.inc    2011-09-28 12:38:13 UTC (rev 
2021)
+++ mrbs/trunk/web/Themes/default/header.inc    2011-09-28 13:17:18 UTC (rev 
2022)
@@ -1152,6 +1152,8 @@
 
     $('#conflict_check, #policy_check').click(function() {
         var tabId;
+        var checkResults = $('#check_results');
+        var checkTabs = $('#check_tabs');
         <?php 
         // Work out which tab should be selected
         // (Slightly long-winded using a switch, but there may be more tabs in 
future)
@@ -1168,11 +1170,16 @@
         }
         <?php
         // If we've already created the dialog and tabs, then all we have
-        // to do is select the tab corresponding to the div that was clicked
+        // to do is re-open the dialog if it has previously been closed and
+        // select the tab corresponding to the div that was clicked
         ?>
         if (arguments.callee.alreadyExists)
         {
-          $('#check_tabs').tabs("select", tabId);
+          if (!checkResults.dialog("isOpen"))
+          {
+            checkResults.dialog("open");
+          }
+          checkTabs.tabs("select", tabId);
           return;
         }
         <?php
@@ -1180,9 +1187,9 @@
         // with the whole structure being draggable.   Thanks to dbroox at
         // http://forum.jquery.com/topic/combining-ui-dialog-and-tabs for the 
solution.
         ?>
-        $('#check_tabs').tabs();
-        $('#check_tabs').tabs("select", tabId);
-        $('#check_results').dialog({'width':400, 'height':200, 
+        checkTabs.tabs();
+        checkTabs.tabs("select", tabId);
+        checkResults.dialog({'width':400, 'height':200, 
                                     'minWidth':300, 'minHeight':150, 
                                     'draggable':true });
         <?php //steal the close button ?>

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
Mrbs-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mrbs-commits

Reply via email to