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

Update $templates->delete($template) method to also delete fieldgroup (when not used elsewhere)

Browse Source
This commit is contained in:
Ryan Cramer
2022-11-11 14:04:56 -05:00
parent eb94a38f75
commit ca344c838d
1 changed files with 20 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
wire/core/Templates.php
View File

@@ -341,17 +341,35 @@ class Templates extends WireSaveableItems {
*/
public function ___delete(Saveable $item) {
$name = $item->name;
$id = $item->id;
if($item->flags & Template::flagSystem) {
throw new WireException("Can't delete template '{$item->name}' because it is a system template.");
throw new WireException("Can't delete template '$name' because it is a system template.");
}
$cnt = $item->getNumPages();
if($cnt > 0) {
throw new WireException("Can't delete template '{$item->name}' because it is used by $cnt pages.");
throw new WireException("Can't delete template '$name' because it is used by $cnt pages.");
}
$return = parent::___delete($item);
if($return) {
$fieldgroups = $this->wire()->fieldgroups;
$fieldgroup = $fieldgroups->get($name);
if($fieldgroup) {
// also delete fieldgroup, if not used by any other templates
$cnt = 0;
foreach($this as $t) {
/** @var Template $t */
if($t->id != $id && $t->fieldgroup->id == $fieldgroup->id) $cnt++;
}
if(!$cnt) $fieldgroups->delete($fieldgroup);
}
}
$cache = $this->wire()->cache;
$cache->maintenance($item);