From f8d47edbbdfbb3e45f57de0d90556a780a754e04 Mon Sep 17 00:00:00 2001
From: Andy Miller <rhuk@mac.com>
Date: Wed, 19 Aug 2015 14:58:18 -0600
Subject: [PATCH] refactored the page loop a bit to use _context

---
 admin.php                             |  4 ++--
 themes/grav/templates/pages.html.twig | 18 +++++++++++++-----
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/admin.php b/admin.php
index b9ad32ba..39cb83c8 100644
--- a/admin.php
+++ b/admin.php
@@ -208,9 +208,9 @@ class AdminPlugin extends Plugin
 
         $twig->twig_vars['location'] = $this->template;
         $twig->twig_vars['base_url_relative_frontend'] = $twig->twig_vars['base_url_relative'];
+        $twig->twig_vars['admin_route'] = trim($this->config->get('plugins.admin.route'), '/');
         $twig->twig_vars['base_url_relative'] .=
-            ($twig->twig_vars['base_url_relative'] != '/' ? '/' : '') . trim($this->config->get('plugins.admin.route'),
-                '/');
+            ($twig->twig_vars['base_url_relative'] != '/' ? '/' : '') . $twig->twig_vars['admin_route'];
         $twig->twig_vars['theme_url'] = '/user/plugins/admin/themes/' . $this->theme;
         $twig->twig_vars['base_url'] = $twig->twig_vars['base_url_relative'];
         $twig->twig_vars['admin'] = $this->admin;
diff --git a/themes/grav/templates/pages.html.twig b/themes/grav/templates/pages.html.twig
index 7f176950..b3d83400 100644
--- a/themes/grav/templates/pages.html.twig
+++ b/themes/grav/templates/pages.html.twig
@@ -39,9 +39,16 @@
     {{ parent() }}
 {% endblock %}
 
-{% set preview_link = '<a class="preview" href="'~ base_url_relative_frontend ~ (context.home ? '' : context.route) ~'"><i class="fa fa-fw fa-angle-double-right"></i></a>' %}
+{% set preview_link = '<a class="preview" href="'~ base_url_relative_frontend ~ (context.home ? '' : context.rawRoute) ~'"><i class="fa fa-fw fa-angle-double-right"></i></a>' %}
+
+{% macro loop(page, depth, twig_vars) %}
+    {% set separator = twig_vars['config'].system.param_sep %}
+    {% set base_url = twig_vars['base_url_relative'] %}
+    {% set base_url_simple = twig_vars['base_url_simple'] %}
+    {% set admin_route = twig_vars['admin_route'] %}
+    {% set admin_lang = twig_vars['admin_lang'] %}
+    {% set warn = twig_vars['warn'] %}
 
-{% macro loop(page, base_url, depth, warn, separator, admin_lang, default_site_lang) %}
     {% for p in page.children() %}
         {% set description = (not p.page ? 'Folder &bull; ' : 'Page &bull; ') ~
                              (p.modular ? 'Modular &bull; ' : '') ~
@@ -56,7 +63,7 @@
                 {% if p.language == admin_lang %}
                     <a href="{{ base_url }}/pages/{{ (p.header.routes.default ?: p.rawRoute)|trim('/') }}" class="page-edit">{{ p.menu }}</a>
                 {% else %}
-                    {{ p.menu }} <span class="badge lang">{{p.language ?: default_site_lang}}</span>
+                    <a href="{{ base_url_simple }}/{{ p.language }}/{{ admin_route }}/pages/{{ (p.header.routes.default ?: p.rawRoute)|trim('/') }}" class="page-edit">{{ p.menu }}</a> <span class="badge lang">{{p.language ?: default_site_lang}}</span>
                 {% endif %}
                 <span class="page-home">{{ p.home ? '<i class="fa fa-home"></i>' }}</span>
                 <span class="page-tools">
@@ -73,7 +80,7 @@
             {% if p.children().count > 0 %}
 
             <ul class="depth-{{ depth + 1 }}" style="display:none;">
-                {{ _self.loop(p, base_url, depth + 1, warn, separator, admin_lang, default_site_lang) }}
+                {{ _self.loop(p, depth + 1, twig_vars) }}
             </ul>
             {% endif %}
         </li>
@@ -138,6 +145,7 @@
             </div>
 
             {% if admin.multilang %}
+
                 <div class="button-group">
                     <button type="button" class="button disabled">
                         <i class="fa fa-flag-o"></i> {{ admin.siteLanguages[admin_lang]|capitalize }}
@@ -209,7 +217,7 @@
             </div>
         </form>
         <ul class="pages-list depth-0">
-            {{ _self.loop(pages, base_url_relative, 0, warn, config.system.param_sep, admin.session.admin_lang, default_site_lang) }}
+            {{ _self.loop(pages, 0, _context) }}
         </ul>
     {% endif %}
     </div>