[MGNLSLOCK-15] SoftLocking deletes new paragraph after creation - after deleting one at the same position Created: 27/Oct/11  Updated: 09/Nov/11  Resolved: 09/Nov/11

Status: Closed
Project: Magnolia Soft Locking Module
Component/s: None
Affects Version/s: 1.0.3
Fix Version/s: 1.0.4

Type: Bug Priority: Major
Reporter: Florian Fuchs Assignee: Ondrej Chytil
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Magnolia EE 4.4.5


Attachments: File softLocking.js.patch    
Issue Links:
causality
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

If you delete a paragraph and then create a new one at the same position, then the paragraph gets immediately deleted by the softlocking-javascript polling request.

The deletion of a paragraph is done by a GET request eg. http://localhost:8080/untitled.html?mgnlCK=1319700663177&mgnlIntercept=NODE_DELETE&mgnlPath=/untitled/main/00

The softlocking-js invokes a get request on the current page periodically, using the SAME parameters, which had been used to get the page - with the mgnlIntercept=NODE_DELETE parameter in it.
So, the softlocking-js calls this url:
http://localhost:8080/untitled.html?mgnlCK=1319700663177&mgnlIntercept=NODE_DELETE&mgnlPath=/untitled/main/00&isSoftLockingAjaxRequest=true&op=stat and tries to remove the paragraph again.

This bug seems to be related to MGNLSLOCK-13, where location.href (full url, including parameters) gets used to determine the ajax-url.

I hope this can be fixed.

Thank you very much!

Regards,
Florian Fuchs



 Comments   
Comment by Federico Grilli [ 27/Oct/11 ]

Hello Florian, thanks for reporting this bug. I'll try to tackle it as soon as possible.

Comment by Rory Gibson [ 09/Nov/11 ]

Possible fix in JavaScript - please test.

Comment by Ondrej Chytil [ 09/Nov/11 ]

Thanks for patch Rory.

Comment by Federico Grilli [ 09/Nov/11 ]

Thanks for providing the patch. Just applied and tested with different browsers (IE7 in particular which iirc is very important for one of your current projects). It works fine.

Generated at Mon Feb 12 07:14:30 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.