Closed (fixed)
Project:
Content locking (anti-concurrent editing)
Version:
8.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Apr 2024 at 13:34 UTC
Updated:
14 May 2024 at 11:29 UTC
Jump to comment: Most recent
Comments
Comment #2
alexpottWe can't do the merge idea because merge is implemented as a select and update or insert... so yet another opportunity for races... so we can consider using upsert or using the lock system.
Comment #4
alexpottComment #5
smustgrave commentedWasn't sure how to trigger the lock but applying the MR on the site we are using on and locks still appear to be working just fine. So going to lean on that.
Comment #6
alexpottI've tried to write a test for this - both low level and and high level - using pcntl_fork() to simultaneously lock via the API and UI neither worked because it is hard to have multiple sessions as the same time in FunctionalTests and separate service instances in kernel tests.
Comment #7
alexpottComment #9
alexpott