NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
3 Jun 2021

Pull Request for Issue #34369.

Summary of Changes

This PR fixes the issue #34369

Testing Instructions

  1. See #34369 to understand the issue.
  2. Install Joomla 4 nightly build. Install Blog sample data so that you have data ready for testing
  3. Create additional categories if needed so that you will have parent-children categories on your site.
  4. Download update package for this PR at https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/34375/downloads/44812/Joomla_4.0.0-rc2-dev+pr.34375-Development-Update_Package.zip , go to System -> Update -> Joomla, upload this package and run the update.
  5. Try to re-order categories (both parent and children categories), articles, modules and make sure ordering of these items are saved when you re-fresh the page.
avatar joomdonation joomdonation - open - 3 Jun 2021
avatar joomdonation joomdonation - change - 3 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2021
Category JavaScript Repository NPM Change
avatar joomdonation
joomdonation - comment - 3 Jun 2021

No, sorry. Not this simple. We should only calculate and update ordering of items in the same group/level only. This won't work. I'm closing this PR and will re-open it if I can come up with a solution.

avatar joomdonation joomdonation - change - 3 Jun 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-06-03 12:33:17
Closed_By joomdonation
Labels Added: NPM Resource Changed ?
avatar joomdonation joomdonation - close - 3 Jun 2021
avatar joomdonation joomdonation - change - 3 Jun 2021
Status Closed New
Closed_Date 2021-06-03 12:33:17
Closed_By joomdonation
avatar joomdonation joomdonation - change - 3 Jun 2021
Status New Pending
avatar joomdonation joomdonation - reopen - 3 Jun 2021
ef956d4 3 Jun 2021 avatar joomdonation CS
avatar joomdonation joomdonation - change - 3 Jun 2021
The description was changed
avatar joomdonation joomdonation - edited - 3 Jun 2021
avatar joomdonation
joomdonation - comment - 3 Jun 2021

OK. I tested it myself and it works. I updated testing instructions, please test. Thanks.

avatar nozaro33
nozaro33 - comment - 4 Jun 2021

I have tested this item successfully, sorry I don't know really know how to test via composer

avatar joomdonation
joomdonation - comment - 4 Jun 2021

Thanks @nozaro33. You do not need to test it using composer. Just follow the testing instructions, download , install the update package, test it and make sure it works.

avatar sandramay0905 sandramay0905 - test_item - 4 Jun 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 4 Jun 2021

I have tested this item successfully on ef956d4

The ordering of parent- and children-categories, articles and modules is saved after refresh of the page.

Question: Drag'n'Drop a parent-category show children-categories only after page-refresh (please see video below). A different Issue?

1


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34375.
avatar joomdonation
joomdonation - comment - 4 Jun 2021

@sandramay0905 Yes. It is different issue. It depends on the script which we are using to handle drag and drop. I asked @dgrammatiko and @Fedik about it #34369 (comment) .

avatar joomdonation joomdonation - alter_testresult - 4 Jun 2021 - nozaro33: Tested successfully
avatar joomdonation
joomdonation - comment - 4 Jun 2021

RTC. Thanks!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34375.

avatar sandramay0905
sandramay0905 - comment - 4 Jun 2021

@joomdonation do i have to open a issue for handle drag and drop or is your comment enough?

avatar joomdonation
joomdonation - comment - 4 Jun 2021

@sandramay0905 Yes. Please open a new issue for that. Thanks !

avatar joomdonation joomdonation - change - 4 Jun 2021
Status Pending Ready to Commit
avatar joomdonation
joomdonation - comment - 4 Jun 2021

RTC again. It was lost when a new comment added.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34375.

avatar wilsonge wilsonge - change - 6 Jun 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-06-06 08:38:11
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 6 Jun 2021
avatar wilsonge wilsonge - merge - 6 Jun 2021
avatar wilsonge
wilsonge - comment - 6 Jun 2021

Thanks! Nice job!

Add a Comment

Login with GitHub to post a comment