? NPM Resource Changed bug PR-4.3-dev ? Pending

User tests: Successful: Unsuccessful:

avatar heelc29
heelc29
7 Nov 2022

Pull Request for Issue #39150 .

Summary of Changes

The modal is only closed if the iframe does not contain an element joomla-alert[type="danger"].

Testing Instructions

see #39150

The error also occurs with Single Contact (if name contains only spaces):
image

Actual result BEFORE applying this Pull Request

Modal is closed

Expected result AFTER applying this Pull Request

Modal still open and show error message (like when only Save is clicked)

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar heelc29 heelc29 - open - 7 Nov 2022
avatar heelc29 heelc29 - change - 7 Nov 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Nov 2022
Category JavaScript Repository NPM Change
avatar alikon
alikon - comment - 8 Nov 2022

@heelc29 can you please contact me vicepresident.at.opensourcematters.org

avatar HLeithner
HLeithner - comment - 2 May 2023

This pull request has been automatically rebased to 4.3-dev.

avatar heelc29 heelc29 - change - 2 May 2023
Labels Added: NPM Resource Changed bug ? ?
avatar heelc29 heelc29 - change - 2 May 2023
Title
[4.2] Prevent modal from closing when showing an error
[4.3] Prevent modal from closing when showing an error
avatar heelc29 heelc29 - edited - 2 May 2023
avatar heelc29 heelc29 - change - 20 May 2023
Labels Added: PR-4.3-dev
Removed: ?
avatar chmst chmst - test_item - 26 Aug 2023 - Tested successfully
avatar chmst
chmst - comment - 26 Aug 2023

I have tested this item ✅ successfully on bc5f9c3


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

avatar fancyFranci fancyFranci - test_item - 26 Aug 2023 - Tested successfully
avatar fancyFranci
fancyFranci - comment - 26 Aug 2023

I have tested this item ✅ successfully on bc5f9c3

I tested the case from the issue and the PR does not let the modal close.


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

avatar richard67 richard67 - change - 26 Aug 2023
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 26 Aug 2023

RTC


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

avatar richard67 richard67 - change - 26 Aug 2023
Labels Added: ?
avatar obuisard obuisard - close - 28 Aug 2023
avatar obuisard obuisard - merge - 28 Aug 2023
avatar obuisard obuisard - change - 28 Aug 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-08-28 19:25:25
Closed_By obuisard
avatar obuisard
obuisard - comment - 28 Aug 2023

Thank you @heelc29 for this PR

Add a Comment

Login with GitHub to post a comment