Refactored so that we only retrieve the feature files content if the …

…content is not already on the page.
baphled · Jul 6, 2010 · bd13f9c · bd13f9c
1 parent 8c6dd3f
commit bd13f9c
Showing 1 changed file with 26 additions and 21 deletions.
diff --git a/public/javascripts/jquery.ajax.dialog.js b/public/javascripts/jquery.ajax.dialog.js
@@ -12,27 +12,32 @@ $(function() {
 
   // bind click event to related property on page
   $('a#file-feature').live('click', function() {
-    // make ajax call
-    $.ajax({
-      url: $('a#file-feature').attr('href'),
-      type: 'GET',
-      dataType: 'html',
-      complete: function(xhr, textStatus) {
-      },
-      success: function(data, textStatus, xhr) {
-        // populate dialog box with content
-        $dialog
-          .append(data)
-          .dialog({
-            title: 'Source feature content',
-            width: '75%',
-            buttons: { "Ok": function() { $(this).dialog("close"); } }
-          });
-      },
-      error: function(xhr, textStatus, errorThrown) {
-        // display flash error
-      }
-    });
+    // check to see if we already have the dialog element on the page
+    if ($('div pre').size() == 0) {
+      // make ajax call
+      $.ajax({
+        url: $('a#file-feature').attr('href'),
+        type: 'GET',
+        dataType: 'html',
+        complete: function(xhr, textStatus) {
+        },
+        success: function(data, textStatus, xhr) {
+          // populate dialog box with content
+          $dialog
+            .append(data)
+            .dialog({
+              title: 'Source feature content',
+              width: '75%',
+              buttons: { "Ok": function() { $(this).dialog("close"); } }
+            });
+        },
+        error: function(xhr, textStatus, errorThrown) {
+          // display flash error
+        }
+      });      
+    } else {
+      $('div.dialog').dialog('open');
+    }
 
     return false;
   })