Rare repeater page creation issue when undergoing heavy writes

[jlahijani]

When my system is undergoing a lot of simultaneous writes from different requests, I seem to get this error very rarely when a repeater page is created:

You may not modify 'name' on page #0 (/admin/repeaters/for-field-219/for-page-1985385/) because it is a system page

#0 /var/www/[example.com/wire/core/Page.php(1543)](http://example.com/wire/core/Page.php(1543)): ProcessWire\Page->set()
#1 /var/www/[example.com/wire/core/PagesNames.php(888)](http://example.com/wire/core/PagesNames.php(888)): ProcessWire\Page->__set()
#2 /var/www/[example.com/wire/core/PagesEditor.php(513)](http://example.com/wire/core/PagesEditor.php(513)): ProcessWire\PagesNames->checkNameConflicts()
#3 /var/www/[example.com/wire/core/Pages.php(856)](http://example.com/wire/core/Pages.php(856)): ProcessWire\PagesEditor->save()
#4 /var/www/[example.com/wire/core/Wire.php(419)](http://example.com/wire/core/Wire.php(419)): ProcessWire\Pages->___save()
#5 /var/www/[example.com/wire/core/WireHooks.php(998)](http://example.com/wire/core/WireHooks.php(998)): ProcessWire\Wire->_callMethod()
#6 /var/www/[example.com/wire/core/Wire.php(484)](http://example.com/wire/core/Wire.php(484)): ProcessWire\WireHooks->runHooks()
#7 /var/www/[example.com/wire/core/Page.php(2418)](http://example.com/wire/core/Page.php(2418)): ProcessWire\Wire->__call()
#8 /var/www/[example.com/wire/modules/Fieldtype/FieldtypeRepeater/FieldtypeRepeater.module(1088)](http://example.com/wire/modules/Fieldtype/FieldtypeRepeater/FieldtypeRepeater.module(1088)): ProcessWire\Page->save()
#9 /var/www/[example.com/wire/modules/Fieldtype/FieldtypeRepeater/FieldtypeRepeater.module(817)](http://example.com/wire/modules/Fieldtype/FieldtypeRepeater/FieldtypeRepeater.module(817)): ProcessWire\FieldtypeRepeater->getRepeaterPageParent()
#10 /var/www/[example.com/wire/modules/Fieldtype/FieldtypeRepeater/RepeaterPageArray.php(178)](http://example.com/wire/modules/Fieldtype/FieldtypeRepeater/RepeaterPageArray.php(178)): ProcessWire\FieldtypeRepeater->getBlankRepeaterPage()

[jlahijani]

This occurred due to a race condition in my system. Not an issue with ProcessWire specifically. Closing.

[ryancramerdesign]

@jlahijani The error is appearing because it's attempting to create the same repeater page parent in 2 different requests, and one of them is the winner. The loser attempts to adapt by incrementing its page name, but that's no good because the system status disallows it (which is good). But I think we can avoid the issue by having it try to find the parent page a second time if it encounters an exception during save. I've just pushed an update to do that.

[jlahijani]

Awesome.

Also see attached image.

[ryancramerdesign]

@jlahijani Oh wow, those look amazing! Where'd you get them?