Changeset 518 for trunk/web

Timestamp:

03/08/08 21:51:57 (16 years ago)

Author:

bastiaans

Message:

libtoga.php:

• dont run na code if info unavailable

File:

• trunk/web/addons/job_monarch/libtoga.php (modified) (2 diffs)

trunk/web/addons/job_monarch/libtoga.php

@@ -889 +889 @@
                 if( $name == "GANGLIA_XML" )
                 {
-                        foreach( $this->down_nodes as $reported => $dnodes )
-                        {
-
-                                if( $reported == $this->heartbeat['time'] )
+                        if( sizeof( $this->down_nodes ) > 0 )
+                        {
+                                foreach( $this->down_nodes as $reported => $dnodes )
                                 {
-                                        $domain = $dnodes[1];
-
-                                        foreach( $dnodes[0] as $downhost )
+
+                                        if( $reported == $this->heartbeat['time'] )
                                         {
-                                                $downhost = $this->makeHostname( $downhost, $domain );
-
-                                                if( isset( $nodes[$downhost] ) )
+                                                $domain = $dnodes[1];
+
+                                                foreach( $dnodes[0] as $downhost )
                                                 {
-                                                        // OMG PHP4 is fking stupid!
-                                                        // $nodes[$downhost]->setDown( 1 ) won't work here..
-                                                        //
-                                                        $mynode = $nodes[$downhost];
-                                                        $mynode->setDown( 1 );
-                                                        $nodes[$downhost] = $mynode;
+                                                        $downhost = $this->makeHostname( $downhost, $domain );
+
+                                                        if( isset( $nodes[$downhost] ) )
+                                                        {
+                                                                // OMG PHP4 is fking stupid!
+                                                                // $nodes[$downhost]->setDown( 1 ) won't work here..
+                                                                //
+                                                                $mynode = $nodes[$downhost];
+                                                                $mynode->setDown( 1 );
+                                                                $nodes[$downhost] = $mynode;
+                                                        }
                                                 }
                                         }
@@ -913 +916 @@
                         }
 
-                        foreach( $this->offline_nodes as $reported => $onodes )
-                        {
-                                if( $reported == $this->heartbeat['time'] )
+                        if( sizeof( $this->offline_nodes ) > 0 )
+                        {
+                                foreach( $this->offline_nodes as $reported => $onodes )
                                 {
-                                        $domain = $onodes[1];
-
-                                        foreach( $onodes[0] as $offlinehost )
+                                        if( $reported == $this->heartbeat['time'] )
                                         {
-                                                $offlinehost = $this->makeHostname( $offlinehost, $domain );
-
-                                                if( isset( $nodes[$offlinehost] ) )
+                                                $domain = $onodes[1];
+
+                                                foreach( $onodes[0] as $offlinehost )
                                                 {
-                                                        // OMG PHP4 is fking stupid!
-                                                        // $nodes[$offlinehost]->setDown( 1 ) won't work here..
-                                                        //
-                                                        $mynode = $nodes[$offlinehost];
-                                                        $mynode->setOffline( 1 );
-                                                        $nodes[$offlinehost] = $mynode;
+                                                        $offlinehost = $this->makeHostname( $offlinehost, $domain );
+
+                                                        if( isset( $nodes[$offlinehost] ) )
+                                                        {
+                                                                // OMG PHP4 is fking stupid!
+                                                                // $nodes[$offlinehost]->setDown( 1 ) won't work here..
+                                                                //
+                                                                $mynode = $nodes[$offlinehost];
+                                                                $mynode->setOffline( 1 );
+                                                                $nodes[$offlinehost] = $mynode;
+                                                        }
                                                 }
                                         }