NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
10 Mar 2021

Pull Request for Issue #26711 and #28953.

Thanks @dgrammatiko for the code.

Testing Instructions

Install patch and npm run build:js
or
Download installer package.

Create 2 custom fields: Media and User
Create a subfield consisting these 2 custom fields.
Edit an article.
Go to Fields tab.
Add a row.
Click Select button.
Click User icon.

Actual result BEFORE applying this Pull Request

Modal doesn't open.

Expected result AFTER applying this Pull Request

Modal opens.

avatar Quy Quy - open - 10 Mar 2021
avatar Quy Quy - change - 10 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Mar 2021
Category JavaScript Repository NPM Change
avatar Quy Quy - change - 10 Mar 2021
The description was changed
avatar Quy Quy - edited - 10 Mar 2021
avatar Didldu-Florian Didldu-Florian - test_item - 10 Mar 2021 - Tested successfully
avatar Didldu-Florian
Didldu-Florian - comment - 10 Mar 2021

I have tested this item successfully on aacd7ea


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

Thx

avatar ceford ceford - test_item - 10 Mar 2021 - Tested successfully
avatar ceford
ceford - comment - 10 Mar 2021

I have tested this item successfully on aacd7ea


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

avatar dgrammatiko
dgrammatiko - comment - 10 Mar 2021

@Quy please do the same change in the user modal:

&& window.bootstrap.Modal.getInstance(this.modal) === undefined) {

avatar Quy Quy - change - 10 Mar 2021
Labels Added: NPM Resource Changed ?
avatar brianteeman
brianteeman - comment - 11 Mar 2021

This resolves the issue for media and user fields but the problem still exists for other modals eg article select

avatar dgrammatiko
dgrammatiko - comment - 11 Mar 2021

This resolves the issue for media and user fields but the problem still exists for other modals eg article select

What's the problem with them? This is an issue due to the way subforms work and the fact that some fields have still dependency on Bootstrap modal. The other modals are tightly connected only to editors...

avatar brianteeman
brianteeman - comment - 11 Mar 2021

there is no content in the modal. Just the header and footer

avatar dgrammatiko
dgrammatiko - comment - 11 Mar 2021

URL?

avatar brianteeman
brianteeman - comment - 11 Mar 2021

will send it privately

avatar brianteeman
brianteeman - comment - 11 Mar 2021

scrub - that. I must have had a typo or something. Its working now

avatar brianteeman
brianteeman - comment - 11 Mar 2021

I was wrong - its not working but it is unrelated - sorry for the confusion on this PR

avatar brianteeman brianteeman - test_item - 11 Mar 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 11 Mar 2021

I have tested this item successfully on f49c697

Tested successfully with media and user fields


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

avatar brianteeman
brianteeman - comment - 11 Mar 2021

However it does NOT solve the issue for other modal fields such as administrator/components/com_content/src/Field/Modal/ArticleField.php

I'm guessing there needs to be a patch for modal-field.js

avatar sksuryan sksuryan - test_item - 16 Mar 2021 - Tested successfully
avatar sksuryan
sksuryan - comment - 16 Mar 2021

I have tested this item successfully on f49c697


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

avatar Quy Quy - change - 16 Mar 2021
Status Pending Ready to Commit
avatar Quy
Quy - comment - 16 Mar 2021

RTC


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

avatar chmst chmst - change - 16 Mar 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-03-16 17:55:19
Closed_By chmst
Labels Added: ?
avatar chmst chmst - close - 16 Mar 2021
avatar chmst chmst - merge - 16 Mar 2021
avatar chmst
chmst - comment - 16 Mar 2021

Thanks!

Add a Comment

Login with GitHub to post a comment