None is equal to zero for php. use === operator to check the type

diff --git a/planetlab/sites/join_request.php b/planetlab/sites/join_request.php
index ce7b309..815c328 100644 (file)
--- a/planetlab/sites/join_request.php
+++ b/planetlab/sites/join_request.php
@@ -68,7 +68,7 @@ function render_join_request_review($api, $site_id) {
       return;
   }
   $site = $sites[0];
-  if ($site['enabled'] && $site['ext_consortium_id'] == $PENDING_CONSORTIUM_ID) {
+  if ($site['enabled'] && $site['ext_consortium_id'] === $PENDING_CONSORTIUM_ID) {
     print("<p class='plc-warning'> This site is already enabled </p>");
     return;
   }

diff --git a/planetlab/sites/site.php b/planetlab/sites/site.php
index 807479d..d80dc5f 100644 (file)
--- a/planetlab/sites/site.php
+++ b/planetlab/sites/site.php
@@ -155,7 +155,7 @@ $peers->block_start ($peer_id);
 if ( $local_peer ) {
   // pending site
   global $PENDING_CONSORTIUM_ID;
-  if ( $ext_consortium_id == $PENDING_CONSORTIUM_ID) {
+  if ( $ext_consortium_id === $PENDING_CONSORTIUM_ID) {
     if ( ! $enabled ) 
       plc_warning ("This site is under pending registration - Please visit " . 
                   href (l_sites_pending(),"this page") . 

diff --git a/planetlab/sites/sites.php b/planetlab/sites/sites.php
index 8eb5a03..9b188b9 100644 (file)
--- a/planetlab/sites/sites.php
+++ b/planetlab/sites/sites.php
@@ -51,7 +51,7 @@ function site_status ($site) {
     if ( ! $site['enabled']) 
       $messages [] = "Not enabled";
     global $PENDING_CONSORTIUM_ID;
-    if ( $site['ext_consortium_id'] == $PENDING_CONSORTIUM_ID )
+    if ( $site['ext_consortium_id'] === $PENDING_CONSORTIUM_ID )
       $messages [] = "Pending registration";
   
     // check that site has at least a PI and a tech