[4.1] order formdata

[dgrammatiko]

Pull Request for Issue #36346 .

Summary of Changes

Send all the form data (although it's useless as you only need the changed id's for reordering) to the PHP

Testing Instructions

• The core still works as before because it doesn't need any extra data

Actual result BEFORE applying this Pull Request

Works

Expected result AFTER applying this Pull Request

Works

Documentation Changes Required

[dgrammatiko]

impossible to test

Well, we can confirm that the core still works as expected so that the core is not broken but these changes. The extra bits on the payload would be useful for devs doing things out of the core API and that's untestable but also should be ok if we didn't broke the core extensions...

[PhocaCz]

To test it: Use browser with DevTools enabled (e.g. Google Chrome). Create articles and sort them. In DevTools switch to Network tab, to "Fetch/XHR" tab and see the AJAX results (see image):

When the patch is not applied, you will see only cid and order variable.

When the patch is applied, you will see more form data available. When you see them, you can mark this test as successfull.

Jan

[PhocaCz]

I have tested this item ✅ successfully on d88638c

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36353.}

[jwaisner]

@dgrammatiko

Please update the PR to the latest branch.

[dgrammatiko]

@bembelimen is there any interest for this one or should I close it?

[laoneo]

I have tested this item ✅ successfully on 746b970

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36353.}

[laoneo]

[laoneo]

Tested with reordering a module and the last screenshot is the request with all variables.

[laoneo]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36353.}

[dgrammatiko]

@bembelimen any decision here?

[roland-d]

Thanks everybody.

[dgrammatiko]

Nice, that was my last pending PR, so good news for me