Changeset 1436 for trunk/wifidog-auth/wifidog/classes/Network.php

Timestamp:

12/17/09 17:02:31 (3 years ago)

Author:

gbastien

Message:

• Code refactoring: Network, NodeGroup?, Node inherit from HotspotGraphElement? (#677)

Files:

• trunk/wifidog-auth/wifidog/classes/Network.php (modified) (17 diffs)

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

@@ -47 +47 @@
  * Load required classes
  */
-require_once('classes/GenericDataObject.php');
 require_once('classes/Content.php');
 require_once('classes/User.php');
 require_once('classes/Node.php');
+require_once('classes/NodeGroup.php');
 require_once('classes/GisPoint.php');
 require_once('classes/Cache.php');
@@ -68 +68 @@
  * @copyright  2006 Max Horváth, Horvath Web Consulting
  */
-class Network extends GenericDataObject
+class Network extends HotspotGraphElement
 {
     /** Object cache for the object factory (getObject())*/
-    private static $instanceArray = array();
-    protected $_hotspotGraphElement;
 
     /**
@@ -88 +86 @@
     public static function &getObject($id)
     {
-        if(!isset(self::$instanceArray[$id]))
-        {
-            self::$instanceArray[$id] = new self($id);
-        }
-        return self::$instanceArray[$id];
+        return HotspotGraphElement::getObject($id, 'Network');
     }
 
@@ -101 +95 @@
     public static function freeObject($id)
     {
-        if(isset(self::$instanceArray[$id]))
-        {
-            unset(self::$instanceArray[$id]);
-        }
+        HotspotGraphElement::freeObject($id, 'Network');
     }
 
@@ -192 +183 @@
     public static function createNewObject($network_id = null)
     {
+        
         $db = AbstractDb::getObject();
         if (empty ($network_id)) {
@@ -380 +372 @@
      * @access private
      */
-    private function __construct($p_network_id)
+    protected function __construct($p_network_id)
     {
         $db = AbstractDb::getObject();
@@ -395 +387 @@
         $this->_row = $row;
         $this->_id = $p_network_id;
-        $this->_hotspotGraphElement = HotspotGraphElement::getObject($this->_id,'Network');
+        parent::__construct($this->_id, 'Network');
     }
 
@@ -1706 +1698 @@
         $data = Content::getLinkedContentUI($name, "network_has_content", "network_id", $this->_id, $display_page = "portal");
         $html .= InterfaceElements::generateAdminSectionContainer("network_content", $title, $data);*/
-        if (!is_null($this->_hotspotGraphElement))
-            $html .= $this->_hotspotGraphElement->getContentAdminUI();
+        
+        $html .= parent::getContentAdminUI();
 
         /*
@@ -1937 +1929 @@
         
         // objects hierarchy
-        if (!is_null($this->_hotspotGraphElement))
-            $html .= $this->_hotspotGraphElement->getGraphAdminUI($this);
+        $html .= parent::getGraphAdminUI($this);
 
         $html .= "</ul>\n";
@@ -1958 +1949 @@
 
         // Content management
-        if (!is_null($this->_hotspotGraphElement))
-            $this->_hotspotGraphElement->processContentAdminUI();
+        parent::processContentAdminUI();
         /* $name = "network_".$this->_id."_content";
         Content :: processLinkedContentUI($name, 'network_has_content', 'network_id', $this->_id);*/
@@ -2079 +2069 @@
         ProfileTemplate :: processLinkedProfileTemplateUI($name, 'network_has_profile_templates', 'network_id', $this->_id);
         
-        if (!is_null($this->_hotspotGraphElement))
-            $this->_hotspotGraphElement->processGraphAdminUI($errMsg, $this);
+        parent::processGraphAdminUI($errMsg, $this);
         if(!empty($errMsg)) {
             echo $errMsg;
@@ -2103 +2092 @@
      * @access public
      */
-    public function addContent(Content $content)
+ /*   public function addContent(Content $content)
     {
         $db = AbstractDb::getObject();
@@ -2110 +2099 @@
         $sql = "INSERT INTO network_has_content (network_id, content_id) VALUES ('$this->_id','$content_id')";
         $db->execSqlUpdate($sql, false);
-    }
+    }*/
 
     /**
@@ -2121 +2110 @@
      * @access public
      */
-    public function removeContent(Content $content)
+ /*   public function removeContent(Content $content)
     {
         $db = AbstractDb::getObject();
@@ -2128 +2117 @@
         $sql = "DELETE FROM network_has_content WHERE network_id='$this->_id' AND content_id='$content_id'";
         $db->execSqlUpdate($sql, false);
-    }
+    }*/
 
 
@@ -2205 +2194 @@
                 $errmsg = _('Could not delete network!');
             } else {
+                parent::_delete($errmsg);
                 $retval = true;
             }
@@ -2274 +2264 @@
         $smarty->assign('networkNumNonMonitoredNodes', $net ? $net->getNumOnlineNodes(true) : 0);
     }
+    
+/**
+     * Get the type of graph element (read-only for now)
+     * 
+     * @return string
+     */
+    protected function getType() {
+        return 'Network';
+    }
+  
+    /**
+     * Return whether this element is a root or has parent (Network is root)
+     * @return boolean
+     */
+    public function isRoot(){
+        return true;
+    }
+    
+                /**
+     * Return whether this element is a leaf or has children (Node is leaf)
+     * @return boolean
+     */
+    public function isLeaf() {
+        return false;
+    }
 }