? ? Pending

User tests: Successful: Unsuccessful:

avatar fatmaakay
fatmaakay
1 Aug 2018

Pull Request for Issue #21225 .

Summary of Changes

Parent Item was always empty after changing the menu.
image

This PR fixes this problem.

Testing Instructions

  1. Click on system
  2. Click on global configuration
  3. Click on system
  4. Click on debug system to make it active
  5. Save and close
  6. Create a new menu.
  7. Open the menu item.
  8. Click on the menu.
    image
  9. Check that the parent item is empty
    image
  10. Apply patch
  11. Change the menu item again
  12. Check that the parent item is not empty
    image

Expected result

Expected result is the parent item is not empty

Actual result

Actual result is parent item is empty

Documentation Changes Required

None

avatar fatmaakay fatmaakay - open - 1 Aug 2018
avatar fatmaakay fatmaakay - change - 1 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2018
Category JavaScript
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 1 Aug 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 Aug 2018

I have tested this item successfully on e75c3e3


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

avatar fatmaakay fatmaakay - change - 1 Aug 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2018
Category JavaScript JavaScript Repository
avatar roland-d
roland-d - comment - 2 Aug 2018

@infograf768 Could you test this as well please?

avatar infograf768 infograf768 - test_item - 3 Aug 2018 - Tested successfully
avatar infograf768
infograf768 - comment - 3 Aug 2018

I have tested this item successfully on 8988c5d

Needs to use npm install as node build.js --compile-js is not enough here. (my test site contains #21204

But it works correctly.


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

avatar infograf768 infograf768 - change - 3 Aug 2018
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 3 Aug 2018

Setting this to RTC as it can't be tested by people not using npm...


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

avatar infograf768 infograf768 - change - 3 Aug 2018
Labels Added: ?
avatar wilsonge wilsonge - change - 3 Aug 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-03 09:58:37
Closed_By wilsonge
avatar wilsonge wilsonge - close - 3 Aug 2018
avatar wilsonge wilsonge - merge - 3 Aug 2018

Add a Comment

Login with GitHub to post a comment