Changeset 765 for branches/0.4/web/addons/job_monarch/libtoga.php

Timestamp:

03/28/13 12:21:53 (11 years ago)

Author:

ramonb

Message:

• speedup: XML was parsed 2x for ClusterImage?: now only 1x

File:

• branches/0.4/web/addons/job_monarch/libtoga.php (modified) (3 diffs)

branches/0.4/web/addons/job_monarch/libtoga.php

@@ -1362 +1362 @@
     function ClusterImage( $data, $clustername )
     {
-        $this->dataget        = new DataGatherer( $clustername );
-        $this->data        = $data;
-        $this->clustername    = $clustername;
-        $this->filters        = array();
-        $this->size        = 's';
+        $this->dataget      = new DataGatherer( $clustername );
+        $this->data         = $data;
+        $this->clustername  = $clustername;
+        $this->filters      = array();
+        $this->size         = 's';
         $this->width        = 0;
-        $this->height        = 0;
-        $this->output        = 1;
+        $this->height       = 0;
+        $this->output       = 1;
         $this->jobs         = null;
         $this->nodes        = null;
+        $this->parsed       = false;
     }
 
@@ -1394 +1395 @@
         $this->output    = 0;
     }
+    function checkParse()
+    {
+        if( $this->parsed == false )
+        {
+            $this->dataget->parseXML( $this->data );
+            $this->parsed = true;
+        }
+    }
     function getJobs()
     {
         if( $this->jobs == null )
         {
+            $this->checkParse();
             $mydatag = $this->dataget;
-            $mydatag->parseXML( $this->data );
             $this->jobs = $mydatag->getJobs();
         }
@@ -1413 +1422 @@
         if( $this->nodes== null )
         {
+            $this->checkParse();
             $mydatag = $this->dataget;
-            $mydatag->parseXML( $this->data );
             $this->nodes = $mydatag->getNodes();
         }