- 🇺🇸United States smustgrave
A number of changes seemed to fix several previous issues with things not locking
Can someone verify if this is still a problem?
- Merge request !43Issue #3167103: Content lock not correctly checking for valid link template → (Closed) created by smustgrave
- 🇺🇸United States smustgrave
Not sure how to test this one but the additional check makes sense I believe.
- Status changed to Needs review
about 1 year ago 10:42pm 27 March 2024 - 🇺🇸United States smustgrave
Hiding patch as I converted to an MR. Didn't need the == TRUE part.
- First commit to issue fork.
- Status changed to Needs work
about 1 year ago 8:55am 28 March 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
It would be great to add a test for this using the modules mentioned in the issue summary.
Also I think the fix is wrong. I pushed up what I think the fix should look like.
- 🇺🇸United States jamesini Tacoma, Washington
I applied the patches provided by Merge Request 43 from Comment #10 and it resolved the issue -- i.e. after making an edit to a Paragraph using the Paragraphs Edit module, the Content Lock module correctly redirects the user back to the ultimate parent node.
Thanks, and well done @alexpott!
-James
-
astonvictor →
committed 44eacd49 on 3.x
Issue #3167103 - Fix redirect
-
astonvictor →
committed 44eacd49 on 3.x
- 🇺🇦Ukraine AstonVictor
Created a new MR.
Added a new validation to check if the entity type has the canonical route.
The fix doesn't change the behavior for default entity types (node, user, taxonomy, etc). It should be the same.If entity type doesn't have a canonical route then the user will be redirect to the front page.
thanks
Automatically closed - issue fixed for 2 weeks with no activity.