Changeset 436

Timestamp:

02/07/05 12:47:05 (8 years ago)

Author:

aprilp

Message:

Modified node update code and added method to update plenty of infos at the same time

Location:

trunk/wifidog-auth/wifidog

Files:

• admin/hotspot.php (modified) (1 diff)
• classes/Node.php (modified) (1 diff)

trunk/wifidog-auth/wifidog/admin/hotspot.php

@@ -114 +114 @@
 
             case 'update_node':
-                $description = $db->EscapeString($description);
-                $home_page_url = $db->EscapeString($home_page_url);
-                $map_url = $db->EscapeString($map_url);
-                $rss_url = $db->EscapeString($rss_url);
-                $public_phone_number = $db->EscapeString($public_phone_number);
-                $public_email = $db->EscapeString($public_email);
-                $mass_transit_info = $db->EscapeString($mass_transit_info);
-                $name = $db->EscapeString($name);
-                /* TODO Use object to update all of this and VALIDATE */
-
                 if ($new_node_id) {
-                    $sql_successful = $db->ExecSqlUpdate("UPDATE nodes SET node_id='$new_node_id',name='$name',rss_url='$rss_url',home_page_url='$home_page_url',description='$description',map_url='$map_url',street_address='$street_address',public_phone_number='$public_phone_number',public_email='$public_email',mass_transit_info='$mass_transit_info',node_deployment_status='$node_deployment_status' WHERE node_id='$node_id'");
+                    $node = Node::getNode($new_node_id);
+                    $node->setInfos( array(
+                            'name'                   => $name,
+                            'rss_url'                => $rss_url,
+                            'home_page_url'          => $home_page_url,
+                            'description'            => $description,
+                            'map_url'                => $map_url,
+                            'street_address'         => $street_address,
+                            'public_phone_number'    => $public_phone_number,
+                            'public_email'           => $public_email,
+                            'mass_transit_info'      => $mass_transit_info,
+                            'node_deployment_status' => $node_deployment_status
+                        ));
                 } else {
                     echo "NO NODE ID, this is a bug";

trunk/wifidog-auth/wifidog/classes/Node.php

@@ -158 +158 @@
   }
 
+  function setInfos($info_array)
+  {
+    global $db;
+
+    $infos_to_add = array();
+    if ($info_array) {
+      foreach ($info_array as $column => $value) {
+        $value = $db->EscapeString($value);
+        array_push($infos_to_add, "$column='$value'");
+      }
+      $sql = "UPDATE nodes SET ";
+      $sql .= implode(",", $infos_to_add);
+      $sql .= " WHERE node_id='{$this->mId}'";
+      if (!$db->ExecSqlUpdate($sql, false)) {
+        throw new Exception(_('Unable to update database!'));
+      }
+    } else {
+      throw new Exception(_('No info to update node with!'));
+    }
+  }
+
   /** Return all the nodes
    */