From 87b3bd7d812e247ae9d074784078a4722e6c06e8 Mon Sep 17 00:00:00 2001
From: Starbeamrainbowlabs <sbrl@starbeamrainbowlabs.com>
Date: Wed, 5 Aug 2015 13:12:25 +0100
Subject: [PATCH] sort the sidebar alphabetically. Fixes #17.

---
 index.php                 | 10 +++++++---
 module_index.json         |  2 +-
 modules/extra-sidebar.php | 10 +++++++---
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/index.php b/index.php
index 2b04cd1..a499a44 100644
--- a/index.php
+++ b/index.php
@@ -875,8 +875,12 @@ register_module([
 				// Show the sidebar
 				$exec_start = microtime(true);
 				
+				// Sort the pageindex
+				$sorted_pageindex = get_object_vars($pageindex);
+				ksort($sorted_pageindex, SORT_NATURAL);
+				
 				$sidebar_contents = "";
-				$sidebar_contents .= render_sidebar($pageindex);
+				$sidebar_contents .= render_sidebar($sorted_pageindex);
 				
 				$parts["{body}"] = "<aside class='sidebar'>
 			$sidebar_contents
@@ -894,9 +898,9 @@ register_module([
 ]);
 
 /* 
- * @summary Renders the sidebar for the given pageindex.
+ * @summary Renders the sidebar for a given pageindex.
  * 
- * @param $pageindex {array} - The pageindex to renderthe sidebar for
+ * @param $pageindex {array} - The pageindex to render the sidebar for
  * @param $root_pagename {string} - The pagename that should be considered the root of the rendering. You don't usually need to use this, it is used by the algorithm itself since it is recursive.
  * 
  * @returns {string} A HTML rendering of the sidebar for the given pageindex
diff --git a/module_index.json b/module_index.json
index d9819c9..1b2eecf 100644
--- a/module_index.json
+++ b/module_index.json
@@ -13,7 +13,7 @@
         "author": "Starbeamrainbowlabs",
         "description": "Adds a sidebar to the left hand side of every page. Add '$settings->sidebar_show = true;' to your configuration, or append '&sidebar=yes' to the url to enable. Adding to the url sets a cookie to remember your setting.",
         "id": "extra-sidebar",
-        "lastupdate": 1438775454
+        "lastupdate": 1438776654
     },
     {
         "name": "Credits",
diff --git a/modules/extra-sidebar.php b/modules/extra-sidebar.php
index 3b84742..835e373 100644
--- a/modules/extra-sidebar.php
+++ b/modules/extra-sidebar.php
@@ -42,8 +42,12 @@ register_module([
 				// Show the sidebar
 				$exec_start = microtime(true);
 				
+				// Sort the pageindex
+				$sorted_pageindex = get_object_vars($pageindex);
+				ksort($sorted_pageindex, SORT_NATURAL);
+				
 				$sidebar_contents = "";
-				$sidebar_contents .= render_sidebar($pageindex);
+				$sidebar_contents .= render_sidebar($sorted_pageindex);
 				
 				$parts["{body}"] = "<aside class='sidebar'>
 			$sidebar_contents
@@ -61,9 +65,9 @@ register_module([
 ]);
 
 /* 
- * @summary Renders the sidebar for the given pageindex.
+ * @summary Renders the sidebar for a given pageindex.
  * 
- * @param $pageindex {array} - The pageindex to renderthe sidebar for
+ * @param $pageindex {array} - The pageindex to render the sidebar for
  * @param $root_pagename {string} - The pagename that should be considered the root of the rendering. You don't usually need to use this, it is used by the algorithm itself since it is recursive.
  * 
  * @returns {string} A HTML rendering of the sidebar for the given pageindex