From 0284bd28a1cfadebec9dbd1c1fb8cba4909058da Mon Sep 17 00:00:00 2001 From: Ryan Cramer Date: Thu, 13 Sep 2018 11:17:21 -0400 Subject: [PATCH] Fix issue processwire/processwire-issues#695 change order of listable() call in ProcessPageList::renderNavJSON so that one could make an viewable/editable page non-listable in the tree nav dropdown --- wire/modules/Process/ProcessPageList/ProcessPageList.module | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/wire/modules/Process/ProcessPageList/ProcessPageList.module b/wire/modules/Process/ProcessPageList/ProcessPageList.module index 546775cf..8934e1be 100644 --- a/wire/modules/Process/ProcessPageList/ProcessPageList.module +++ b/wire/modules/Process/ProcessPageList/ProcessPageList.module @@ -530,13 +530,13 @@ class ProcessPageList extends Process implements ConfigurableModule { $url = ''; $editable = false; - if($page->editable()) { + if(!$page->listable()) { + continue; + } else if($page->editable()) { $url = $page->editUrl(); $editable = true; } else if($page->viewable()) { $url = $page->url(); - } else if(!$page->listable()) { - continue; } $numChildren = $id > 1 ? $page->numChildren : 0;