Changeset 1065 for plugins/apostrophePlugin/trunk/lib/model/doctrine/PluginaPage.class.php

Timestamp:

04/26/10 17:08:58 (2 years ago)

Author:

tboutell

Message:

New default behavior of navigation components is appropriate for use on large sites with many pages. On small sites it may be slightly slower, in which case you can set app_a_many_pages to false to get the old "fetch the entire page tree and reuse it in each navigation component as needed" behavior back. This worked quite well up to about 1,000 pages but became a serious problem on an 8,000-page site.

aPage::getAncestorsInfo now has an optional 'includeSelf' parameter which defaults to false for backwards compatibility.

aPage::getInfo provides a page info array about the current page. Useful when merging information from the Info methods with information about pages you already have as objects.

aPage::getAccordionInfo now has an optional final parameter, 'root', which specifies the slug of the root page of the accordion tree. The page on which getAccordionInfo is called should be a descendant of that page (or the root page itself).

Files:

• plugins/apostrophePlugin/trunk/lib/model/doctrine/PluginaPage.class.php (modified) (6 diffs)

plugins/apostrophePlugin/trunk/lib/model/doctrine/PluginaPage.class.php

@@ -570 +570 @@
   protected $ancestorsInfo;
   
-  public function getAncestorsInfo()
+  public function getAncestorsInfo($includeSelf = false)
   {
     if (!isset($this->ancestorsInfo))
@@ -577 +577 @@
       // Since our presence on an admin page implies we know about it, it's OK to include
       // admin pages in the breadcrumb. It's not OK in other navigation
-      $this->ancestorsInfo = $this->getPagesInfo(false, '( p.lft < ' . $this->lft . ' AND p.rgt > ' . $this->rgt . ' )', true);
+      $less = '<';
+      $greater = '>';
+      if ($includeSelf)
+      {
+        $less = '<=';
+        $greater = '>=';
+      }
+      $this->ancestorsInfo = $this->getPagesInfo(false, "( p.lft $less " . $this->lft . " AND p.rgt $greater " . $this->rgt . ' )', true);
     }
     return $this->ancestorsInfo;
@@ -602 +609 @@
       {
         // It's the home page. Return a stub: the home page is its only peer
-        $this->peerInfo = array(array('id' => $this->id, 'title' => $this->getTitle(), 'slug' => $this->slug, 'view_is_secure' => $this->view_is_secure, 'archived' => $this->archived, 'level' => $this->level, 'lft' => $this->lft, 'rgt' => $this->rgt));
+        $this->peerInfo = array($this->getInfo());
       }
       else
@@ -615 +622 @@
   }
 
+  // Sometimes it is useful to have an info structure describing a page object
+  // (the aNavigation classes exploit this)
+  
+  public function getInfo()
+  {
+    return array('id' => $this->id, 'title' => $this->getTitle(), 'slug' => $this->slug, 'view_is_secure' => $this->view_is_secure, 'archived' => $this->archived, 'level' => $this->level, 'lft' => $this->lft, 'rgt' => $this->rgt);
+  }
+  
   protected $childrenInfo;
   
@@ -709 +724 @@
   //   Two
   
+  // You can now specify the root slug, which defaults to the home page.
+  
   // Note that children of Two, 1a, and 1c are NOT returned. Only the siblings of
   // the current page's ancestors, the current page and its siblings, and the immediate
   // children of the current page are returned. For a full tree use getTreeInfo().
   
-  public function getAccordionInfo($livingOnly = true, $depth = null)
+  public function getAccordionInfo($livingOnly = true, $depth = null, $root = '/')
   {
     // As far as I can tell there is no super-elegant, single-query way to do this
@@ -724 +741 @@
     // an accordion contro. in the first place
     $ancestors = $this->getAncestorsInfo();
+    
+    // Dump ancestors we don't care about
+    for ($i = 0; ($i < count($ancestors)); $i++)
+    {
+      if ($ancestors[$i]['slug'] === $root)
+      {
+        $ancestors = array_slice($ancestors, $i);
+        break;
+      }
+    }
+    if ($i === count($ancestors))
+    {
+      throw new sfException("Root slug $root never found among ancestors in getAccordionInfo");
+    }
     $result = array();
     // Ancestor levels