joomla-cms
joomla-cms copied to clipboard
[5.1] com_contact remove showon when useglobal is used in the field
Pull Request for Issue #42381 .
Summary of Changes
Removed the showon option in several fields
Testing Instructions
See issue #42381
Actual result BEFORE applying this Pull Request
Showon fields are not displayed in contact and menu item if the "parent" field has useglobal
Expected result AFTER applying this Pull Request
Showon fields are displayed
Link to documentations
Please select:
-
[ ] Documentation link for docs.joomla.org:
-
[x] No documentation changes for docs.joomla.org needed
-
[ ] Pull Request link for manual.joomla.org:
-
[x] No documentation changes for manual.joomla.org needed
maybe it's better to fix the real issue if the issue is true for all global based showOns ?
maybe it's better to fix the real issue if the issue is true for all global based showOns ?
Yes ;-) useglobal inserts a new select option but without value ... do you know where that is created?
I have tested this item :white_check_mark: successfully on 441e2f167ccd0f2d7db4a7d9c992b8ce1a34e2e3
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42386.
I have tested this item :white_check_mark: successfully on 441e2f167ccd0f2d7db4a7d9c992b8ce1a34e2e3
Tested on 5.1.0 RC1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42386.
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42386.
I have tested this item :white_check_mark: successfully on a80245dbdf6d525942a083a494749ff727849ae1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42386.
Back to pending
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42386.
Like Harald said, removing the option does not fix the issue: #42386 (comment)
I was not able to find the code for that ... https://github.com/joomla/joomla-cms/pull/42386#issuecomment-1824108867
This PR doesn't solve the underlying problem of showon fields. We need more coding effort for that.