- π¬π§United Kingdom philipnorton42 Cheshire
I found the same problem with the module and after some debugging I identified the same line of code being at fault.
The $form_op is the lock that needs to be broken, so if this is called from the content admin page then it contains a "*", which means to "break all locks". This string makes no sense to the database query since we are trying to delete any operations that are called "*" for this entity.
I think this solution to detect for the presence of the "*" and ignore the condition works and should be merged. The patch also passes coding standards checks.
- First commit to issue fork.
- Merge request !27Issue #3328108: route content_lock.break_lock.XYZ not working? Wrong form_op? β (Open) created by smustgrave
- πΊπΈUnited States smustgrave
Does appear #3073565: Unable to release the lock on /admin/content/locked-content view β was marked fixed without a commit. Turned #2 to an MR.
- First commit to issue fork.
- Status changed to Needs work
about 1 year ago 4:05pm 27 March 2024 - π¬π§United Kingdom alexpott πͺπΊπ
It'd be great to add an automated test for this. It's a condition so easy to break in the future.
- πΊπΈUnited States smustgrave
Actually wonder if this could be covered in β¨ Option to lock all form operations together without the black listed ones Needs work ?
- Status changed to Postponed: needs info
about 1 year ago 7:20pm 27 March 2024 - πΊπΈUnited States smustgrave
Wonder if still a bug actually. Following the steps but the lock appears to be deleted.