Add new input rule for ShowOn field#41460
Conversation
|
This should be backed up with tests, one reason is that this is some kind of documentation. And further more it needs documentation. |
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
|
I have tested this item ✅ successfully on f402a7b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41460. |
|
I have tested this item ✅ successfully on f402a7b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41460. |
|
Tested successfully. Not sure if this needs documentation as it was a bug. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41460. |
It adds a new validation rule, so it the available validation rules are listed somewhere in our documentation or are even described, this needs to be extended by the new rule. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41460. |
|
Sorry, I should have explained it better, with tests I meant system tests and not manual testing. |
|
thanks, adding a test for this would be great. |
|
Should work: Should fail: |
|
And |
|
Am working on a fix, see #41918 . But I have to test it myself first. So it's still draft. |
|
@magnussinger What I don't understand it the |
|
When fixing my previous comment by using |
|
I've fixed all these issues mentioned above. See #41918 . |
@rdeutz This PR here and my fix are things to be tested by unit tests, not by system tests. |
Pull Request for Issue #41217.
Summary of Changes
Added a new input rule in oder to check if the show on input text is valid. If not, the user is unable to safe the field
Testing Instructions
Create a new custom field and edit the Showon Attribute. Enter something invalid and you should be unable to save. If you enter something valid, the field can be saved.
Actual result BEFORE applying this Pull Request
The user was able to save the field independent on what's in the showon attribute.
Expected result AFTER applying this Pull Request
The user can only save the field when the showon rule is valid
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