NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
22 Sep 2020

Fixes #30664 and #28842.

Summary of Changes

Fixes saving images after editing in media manager.

Testing Instructions

node build.js --compile-js required.

Edit an article.
In editor click CMS Content -> Image.
Edit an image in the modal.
Click Save & Close.

Actual result BEFORE applying this Pull Request

Modified image is not saved and modules appear in the modal after redirect.

Expected result AFTER applying this Pull Request

Modified images is saved and your are redireceted back to media list without modules showing.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 22 Sep 2020
avatar SharkyKZ SharkyKZ - change - 22 Sep 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Sep 2020
Category JavaScript Repository NPM Change
avatar SharkyKZ SharkyKZ - change - 22 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 22 Sep 2020
avatar SharkyKZ SharkyKZ - change - 22 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 22 Sep 2020
ab8c965 22 Sep 2020 avatar SharkyKZ Typo
avatar SharkyKZ SharkyKZ - change - 22 Sep 2020
Labels Added: NPM Resource Changed ?
avatar richard67
richard67 - comment - 22 Sep 2020
b096991 22 Sep 2020 avatar SharkyKZ CS
avatar infograf768
infograf768 - comment - 22 Sep 2020

I always get a correct success message when modifying image title and using Save.
I have no message when modifying image and saving.
If I modify image and Save and Close, the modified image is not yet displayed as it was modified. I have to close the modal and open it again to see the change.
These may be unrelated to this PR.

avatar richard67 richard67 - test_item - 22 Sep 2020 - Tested successfully
avatar richard67
richard67 - comment - 22 Sep 2020

I have tested this item successfully on b096991

Works as described.

Findings by @infograf768 could be fixed with another, future PR.


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

avatar jwaisner jwaisner - test_item - 22 Sep 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 22 Sep 2020

I have tested this item successfully on b096991


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

avatar jwaisner jwaisner - change - 22 Sep 2020
Status Pending Ready to Commit
avatar jwaisner
jwaisner - comment - 22 Sep 2020

RTC


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

avatar infograf768 infograf768 - change - 23 Sep 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-23 07:01:20
Closed_By infograf768
Labels Added: ?
avatar infograf768 infograf768 - close - 23 Sep 2020
avatar infograf768 infograf768 - merge - 23 Sep 2020
avatar infograf768
infograf768 - comment - 23 Sep 2020

Tks.

Add a Comment

Login with GitHub to post a comment