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

Fix issue processwire/processwire-issues#395 where renaming a repeater field didn't rename the related template and fieldgroup

Browse Source
This commit is contained in:
Ryan Cramer
2017-10-12 10:36:53 -04:00
parent 0a790cfb43
commit 7d3dc80798
1 changed files with 37 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
wire/modules/Fieldtype/FieldtypeRepeater/FieldtypeRepeater.module
View File

@@ -1042,6 +1042,32 @@ class FieldtypeRepeater extends Fieldtype implements ConfigurableModule {
return $parent; return $parent;
} }
/**
* Update repeater template and fieldgroup to have same name as field
*
* @param Template $template Template having old name
* @param string $name New name for template
*
*/
protected function updateRepeaterTemplateName(Template $template, $name) {
if($template->name != $name && !$this->wire('templates')->get($name)) {
$this->message("Renamed repeater template from '$template->name' to '$name'", Notice::debug);
$flags = $template->flags;
$template->flags = Template::flagSystemOverride;
$template->flags = 0;
$template->save();
$template->name = $name;
$template->flags = $flags;
$template->save();
}
if($template->fieldgroup && $template->fieldgroup->name != $name && !$this->wire('fieldgroups')->get($name)) {
$template->fieldgroup->name = $name;
$template->fieldgroup->save();
}
}
/** /**
* Return the repeater template used by Field, i.e. repeater_name * Return the repeater template used by Field, i.e. repeater_name
* *
@@ -1056,11 +1082,18 @@ class FieldtypeRepeater extends Fieldtype implements ConfigurableModule {
$template = null; $template = null;
$templateID = (int) $field->get('template_id'); $templateID = (int) $field->get('template_id');
if($templateID) $template = $this->wire('templates')->get($templateID);
if($template) return $template;
// template does not exist, so we create it
$templateName = self::templateNamePrefix . $field->name; $templateName = self::templateNamePrefix . $field->name;
if($templateID) {
$template = $this->wire('templates')->get($templateID);
if($template && $template->name !== $templateName) {
// repeater has been renamed, update the template and fieldgroup names
$this->updateRepeaterTemplateName($template, $templateName);
}
}
// if template already exists, return it now
if($template) return $template;
// make sure the template name isn't already in use, make a unique one if it is // make sure the template name isn't already in use, make a unique one if it is
$n = 0; $n = 0;