From 781460a6ed98db6b765f674d128b204e702671f5 Mon Sep 17 00:00:00 2001
From: nliautaud <nicolas.liautaud@gmail.com>
Date: Mon, 6 Nov 2017 11:55:44 +0100
Subject: [PATCH 1/5] skip hidden pages

---
 PicoPagesList.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/PicoPagesList.php b/PicoPagesList.php
index 809e499..62cd08e 100644
--- a/PicoPagesList.php
+++ b/PicoPagesList.php
@@ -210,6 +210,8 @@ private function output($pages, $currentPath = '')
         $html = '<ul>';
         foreach ($pages as $pageID => $page)
         {
+            if (!empty($page['hidden'])) continue;
+
             $childPath = $currentPath ? $currentPath.'/'.$pageID : $pageID;
 
             $childsOutput = '';

From 7cc03eac2a17ea47978579719d02755e99715095 Mon Sep 17 00:00:00 2001
From: nliautaud <nicolas.liautaud@gmail.com>
Date: Mon, 6 Nov 2017 15:22:37 +0100
Subject: [PATCH 2/5] update config documentation for Pico 2.0 YML

---
 README.md | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 1978f30..3e30a72 100644
--- a/README.md
+++ b/README.md
@@ -142,7 +142,7 @@ You may want a recursive Twig template or macro to walk trough it, for example :
 
 The lists are sorted according to the default settings in Pico `config.php`.
 
-```php
-$config['pages_order_by'] = 'date';
-$config['pages_order'] = 'desc';
+```yml
+pages_order_by: date
+pages_order: desc
 ```

From 1539998a9260f16485c18e507c63bf98d48b8a40 Mon Sep 17 00:00:00 2001
From: Johannes Heimbach <heimbachjohannes@gmail.com>
Date: Mon, 13 Nov 2017 23:22:48 +0100
Subject: [PATCH 3/5] change onPageRendering for Pico 2.0

---
 PicoPagesList.php | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/PicoPagesList.php b/PicoPagesList.php
index d03679b..7b728ee 100644
--- a/PicoPagesList.php
+++ b/PicoPagesList.php
@@ -20,6 +20,7 @@
  */
 class PicoPagesList extends AbstractPicoPlugin
 {
+    const API_VERSION = 2;
     public $items;
     private $currentPagePath;
 
@@ -59,13 +60,13 @@ public function onPagesLoaded(
      *
      * @see    Pico::getTwig()
      * @see    DummyPlugin::onPageRendered()
-     * @param  Twig_Environment &$twig          twig template engine
-     * @param  array            &$twigVariables template variables
      * @param  string           &$templateName  file name of the template
+     * @param  array            &$twigVariables template variables
      * @return void
      */
-    public function onPageRendering(Twig_Environment &$twig, array &$twigVariables, &$templateName)
+    public function onPageRendering(string &$templateName, array &$twigVariables)
     {
+        $twig = $this->getPico()->getTwig();
         $twigVariables['nested_pages'] = $this->items;
         
         $twig->addFilter(new Twig_SimpleFilter('navigation', function($pages) {
@@ -124,7 +125,7 @@ private function nested_path($page)
                 $parent = $value;
                 break;
             }
-            
+
             $parent['_childs'][$part] = $value;
             $parent = &$parent['_childs'][$part];
         }
@@ -143,7 +144,7 @@ private static function rtrim($str, $substr)
         $length = strlen($substr);
         return (substr($str, -$length) === $substr) ? substr($str, 0, -$length) : $str;
     }
-    
+
     /**
      * Filter the pages array according to given paths, as exclusive or inclusive.
      *
@@ -233,4 +234,3 @@ private function output($pages)
         return $html;
     }
 }
-?>

From 47f2a8d953c0b8df36811799d8f7774f895c84d4 Mon Sep 17 00:00:00 2001
From: johannes Heimbach <heimbachjohannes@gmail.com>
Date: Wed, 11 Apr 2018 16:48:53 +0200
Subject: [PATCH 4/5] split onPagesLoaded in onPagesLoaded and
 onCurrentPageDiscovered

---
 PicoPagesList.php | 28 +++++++++++++++++++++-------
 1 file changed, 21 insertions(+), 7 deletions(-)

diff --git a/PicoPagesList.php b/PicoPagesList.php
index a10a5ab..3510a92 100644
--- a/PicoPagesList.php
+++ b/PicoPagesList.php
@@ -25,7 +25,7 @@ class PicoPagesList extends AbstractPicoPlugin
     private $currentPagePath;
 
     /**
-     * Store the current url and construct the nested pages array.
+     * Construct the nested pages array.
      *
      * Triggered after Pico has read all known pages
      *
@@ -33,24 +33,38 @@ class PicoPagesList extends AbstractPicoPlugin
      * structure of the page data.
      *
      * @see    Pico::getPages()
+     * @param  array[] &$pages data of all known pages
+     * @return void
+     */
+    public function onPagesLoaded(array &$pages)
+    {
+        $this->items = $this->nestedPages($pages);
+    }
+
+
+    /**
+     * Store the current url.
+     *
+     * Triggered after Pico has found the current page and possible siblings
+     *
+     * See {@link DummyPlugin::onSinglePageLoaded()} for details about the
+     * structure of the page data.
+     *
      * @see    Pico::getCurrentPage()
      * @see    Pico::getPreviousPage()
      * @see    Pico::getNextPage()
-     * @param  array[]    &$pages        data of all known pages
      * @param  array|null &$currentPage  data of the page being served
      * @param  array|null &$previousPage data of the previous page
      * @param  array|null &$nextPage     data of the next page
      * @return void
      */
-    public function onPagesLoaded(
-        array &$pages,
+    protected function onCurrentPageDiscovered(
         array &$currentPage = null,
         array &$previousPage = null,
-        array &$nextPage = null
-    ) {
+        array &$nextPage = null)
+    {
         $base_url = $this->getConfig('base_url');
         $this->currentPagePath = str_replace(array('?', $base_url), '', urldecode($currentPage['url']));
-        $this->items = $this->nestedPages($pages);
     }
 
     /**

From ab4b82a91b5e41f6b69c1e3ca90f41c0f2cdb44e Mon Sep 17 00:00:00 2001
From: johannes Heimbach <heimbachjohannes@gmail.com>
Date: Wed, 11 Apr 2018 16:48:53 +0200
Subject: [PATCH 5/5] split onPagesLoaded in onPagesLoaded and
 onCurrentPageDiscovered

---
 PicoPagesList.php | 35 ++++++++++++++++++++++++-----------
 1 file changed, 24 insertions(+), 11 deletions(-)

diff --git a/PicoPagesList.php b/PicoPagesList.php
index a10a5ab..968b308 100644
--- a/PicoPagesList.php
+++ b/PicoPagesList.php
@@ -1,13 +1,13 @@
 <?php
 /**
  * Flat and nested pages list navigation for Pico CMS.
- * 
+ *
  * - Adds twig global `{{ nested_pages }}` in addition to `{{ pages }}`
  * - Render flat or nested HTML navigation tree with `navigation` twig filter
  * - Filter pages and nested pages by paths with `exclude()` and `only()` twig filters
- * 
+ *
  * Examples :
- * 
+ *
  *     {{ pages | navigation }} // output a flat pages list
  *     {{ nested_pages | navigation }} // output a nested pages list
  *     {{ nested_pages | exclude('sub/page') | navigation }} // filtered nested pages list
@@ -25,7 +25,7 @@ class PicoPagesList extends AbstractPicoPlugin
     private $currentPagePath;
 
     /**
-     * Store the current url and construct the nested pages array.
+     * Construct the nested pages array.
      *
      * Triggered after Pico has read all known pages
      *
@@ -33,24 +33,37 @@ class PicoPagesList extends AbstractPicoPlugin
      * structure of the page data.
      *
      * @see    Pico::getPages()
+     * @param  array[] &$pages data of all known pages
+     * @return void
+     */
+    public function onPagesLoaded(array &$pages)
+    {
+        $this->items = $this->nestedPages($pages);
+    }
+
+
+    /**
+     * Store the current url.
+     *
+     * Triggered after Pico has found the current page and possible siblings
+     *
+     * See {@link DummyPlugin::onSinglePageLoaded()} for details about the
+     * structure of the page data.
+     *
      * @see    Pico::getCurrentPage()
      * @see    Pico::getPreviousPage()
      * @see    Pico::getNextPage()
-     * @param  array[]    &$pages        data of all known pages
      * @param  array|null &$currentPage  data of the page being served
      * @param  array|null &$previousPage data of the previous page
      * @param  array|null &$nextPage     data of the next page
      * @return void
      */
-    public function onPagesLoaded(
-        array &$pages,
+    protected function onCurrentPageDiscovered(
         array &$currentPage = null,
         array &$previousPage = null,
-        array &$nextPage = null
-    ) {
+        array &$nextPage = null) {
         $base_url = $this->getConfig('base_url');
         $this->currentPagePath = str_replace(array('?', $base_url), '', urldecode($currentPage['url']));
-        $this->items = $this->nestedPages($pages);
     }
 
     /**
@@ -68,7 +81,7 @@ public function onPageRendering(string &$templateName, array &$twigVariables)
     {
         $twig = $this->getPico()->getTwig();
         $twigVariables['nested_pages'] = $this->items;
-        
+
         $twig->addFilter(new Twig_SimpleFilter('navigation', function($pages) {
             return $this->output($pages);
         }));