mirror/processwire
1
0
Fork 0
mirror of https://github.com/processwire/processwire.git synced 2025-08-16 03:34:33 +02:00
Code Issues Releases Wiki Activity

Fix issue processwire/processwire-issues#685 where templates with noParents=-1 setting (only one allowed) were not shown as selectable in Template field in page editor

Browse Source
This commit is contained in:
Ryan Cramer
2018-09-13 09:08:44 -04:00
parent 818d9f50a6
commit 272077b1cf
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
wire/modules/Process/ProcessPageEdit/ProcessPageEdit.module
View File

@@ -1454,7 +1454,9 @@ class ProcessPageEdit extends Process implements WirePageEditor, ConfigurableMod
$wrapper->attr('id', $id); $wrapper->attr('id', $id);
$wrapper->attr('title', $restoreLabel); $wrapper->attr('title', $restoreLabel);
$this->addTab($id, $restoreLabel); $this->addTab($id, $restoreLabel);
$newPath = $info['parent']->path() . $info['name'] . '/'; /** @var Page $parent */
$parent = $info['parent'];
$newPath = $parent->path() . $info['name'] . '/';
/** @var InputfieldCheckbox $field */ /** @var InputfieldCheckbox $field */
$field = $this->modules->get('InputfieldCheckbox'); $field = $this->modules->get('InputfieldCheckbox');
@@ -2266,6 +2268,7 @@ class ProcessPageEdit extends Process implements WirePageEditor, ConfigurableMod
$parentEditable = ($parent->id && $parent->editable()); $parentEditable = ($parent->id && $parent->editable());
/** @var Config $config */ /** @var Config $config */
$config = $this->wire('config'); $config = $this->wire('config');
$superAdvanced = $isSuperuser && $config->advanced;
// current page template is assumed, otherwise we wouldn't be here // current page template is assumed, otherwise we wouldn't be here
$templates[$page->template->id] = $page->template; $templates[$page->template->id] = $page->template;
@@ -2302,11 +2305,11 @@ class ProcessPageEdit extends Process implements WirePageEditor, ConfigurableMod
if(!in_array($template->id, $parent->template->childTemplates)) continue; if(!in_array($template->id, $parent->template->childTemplates)) continue;
} }
if($template->noParents == -1 && $template->getNumPages() > 0) { if(!$superAdvanced && $template->noParents < 0 && $template->getNumPages() > 0) {
// only one of these is allowed to exist // only one of these is allowed to exist (noParents=-1)
continue; continue;
} else if($template->noParents) { } else if($template->noParents > 0) {
// user can't change to a template that has been specified as no more instances allowed // user can't change to a template that has been specified as no more instances allowed
continue; continue;