Feeling Lucky
?
?
[#19771] - [behavior.formvalidator] - pattern attribute behaviour fix
- Fixed in Code Base
- 17 Mar 2018
- Medium
- Build: staging
- # 19771
- Diff
- alikon:patch-104
User tests: Successful: Unsuccessful:
Pull Request for Issue #19751
Summary of Changes
fixed behaviour when pattern attribute is not required and field is empty
Testing Instructions
- add this field declaration to a module manifest mod_wrapper.xml for example
<field
name="test"
type="text"
description="testing field validation"
filter="string"
label="Test"
pattern="[A-Za-z\s0-9]+" />- manage/edit the mod_wrapper from backend
don't fill the test field
- click save
Expected result
The test field to not be marked as invalid
Actual result
The test field is highlighted red and marked as invalid.
- apply pr
- clear cache
Result
module saved as should be
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript |
| Labels |
Added:
?
|
||
Anu1601CS
- comment
- 4 Mar 2018
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19771.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 4 Mar 2018
Ready to Commit after two successful tests.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-03-17 15:17:24 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on cf4731d
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19771.