Changeset 3114

Timestamp:

04/13/08 09:33:59 (3 months ago)

Author:

pmjones

Message:

Solar_Model_Nodes

---

* [DEL] Method _newSelectByTags() is now entirely unnecessary, because the

new Solar_Sql_Model::newSelect() method now adds all types of eager joins
for us via $params.

* [REF] Refactored method fetchAllByTags() to use only fetchAll($params),

instead of building a custom SELECT and running it through the previous
model _fetchAll() support method.

Files:

• trunk/Solar/Model/Nodes.php (modified) (4 diffs)

trunk/Solar/Model/Nodes.php

@@ -173 +173 @@
-     * Fetches a collection of nodes with certain tags.
-     * 
-
+
+
-     * 
-     * @param array $params Added parameters for the SELECT.
@@ -182 +183 @@
-    public function fetchAllByTags($tag_list, $params = null)
-    {
-        // no tags? fetch all to pre-empt errors related to "IN()" not
-        // having a list to work with.
-        $tag_list = $this->_fixTagList($tag_list);
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-        }
-        
-
-
-
+
-    }
-    
@@ -213 +224 @@
-        $tag_list = array_unique($tag_list);
-        
-i
+wa
-        // element in the array.
-$tag_list[0]
+count($tag_list) == 1 && reset($tag_list)
-            $tag_list = array();
-        }
@@ -222 +233 @@
-        return $tag_list;
-    }
-    
-    /**
-     * 
-     * Support method to create a new selection tool based on tag lists.
-     * 
-     * @param array $tag_list The list of tags to select by.
-     * 
-     * @param array &$params A reference to added parameters for the SELECT.
-     * 
-     * @return Solar_Sql_Select
-     * 
-     */
-    protected function _newSelectByTags($tag_list, &$params)
-    {
-        // setup
-        $params = $this->fixSelectParams($params);
-        $select = $this->newSelect($params['eager']);
-        
-        // catalog entries for joining
-        $taggings = $this->_related['taggings'];
-        $tags     = $this->_related['tags'];
-        
-        // primary key on the nodes table as an alias; e.g., "nodes.id"
-        $native_primary = "{$this->_model_name}.{$this->_primary_col}";
-        
-        // http://forge.mysql.com/wiki/TagSchema
-        // build the select differently from other fetchAll() statements
-        $select->distinct($params['distinct'])
-               ->from("{$this->_table_name} AS {$this->_model_name}", $params['cols'])
-               // join taggings on nodes
-               ->join(
-                   "{$taggings->foreign_table} AS {$taggings->foreign_alias}",
-                   "{$taggings->foreign_alias}.node_id = $native_primary"
-               )
-               // join tags on taggings
-               ->join(
-                   "{$tags->foreign_table} AS {$tags->foreign_alias}",
-                   "{$tags->foreign_alias}.id = {$taggings->foreign_alias}.tag_id"
-               )
-               // select for the listed tags
-               ->where("{$tags->foreign_alias}.name IN (?)", $tag_list)
-               // user-provided WHERE
-               ->multiWhere($params['where'])
-               // group by nodes.id to collapse multiple nodes (1 for each tag)
-               ->group($native_primary)
-               // make sure the tag-count matches
-               ->having("COUNT($native_primary) = ?", count($tag_list))
-               // user-provided ORDER, paging, etc
-               ->order($params['order'])
-               ->setPaging($params['paging'])
-               ->limitPage($params['page'])
-               ->bind($params['bind']);
-        
-        // done!
-        return $select;
-    }
-    
-    /**
-     * 
-     * Gets a count of nodes and pages-of-nodes with certain tags.
-     * 
-     * @param array $tag_list Count only nodes with all of these tags.
-     * 
-     * @param array $params Added parameters for the SELECT.
-     * 
-     * @return array An array with elemets 'count' (the number of nodes) and
-     * 'pages' (the number of pages-of-nodes).
-     * 
-     */
-    public function countPagesByTags($tag_list, $params = null)
-    {
-        $tag_list = $this->_fixTagList($tag_list);
-        if (! $tag_list) {
-            return $this->countPages($params);
-        }
-        
-        // we need to select the nodes + tags as an "inner" sub-select;
-        // clear any limits on it.
-        $inner = $this->_newSelectByTags($tag_list, $params);
-        $inner->clear('limit');
-        
-        // set up the outer select, which will wrap the inner sub-select
-        $outer = Solar::factory($this->_select_class, array(
-            'sql' => $this->_sql
-        ));
-        
-        // wrap the sub-select and make sure paging is correct
-        $outer->fromSelect($inner, $this->_model_name);
-        $outer->setPaging($this->_paging);
-        
-        // *now* get the count of pages with the tags requested
-        return $outer->countPages("{$this->_model_name}.{$this->_primary_col}");
-    }
-}