[#8208] - Some useful extra validation added e.g validate-date
- Closed
- 8 May 2016
- Medium
- Build: staging
- # 8208
- Diff
- aniket19491:master
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Pending JTracker/HumanTestResults Human Test Results: 0 Successful 1 Failed. Details
User tests: Successful: Unsuccessful:
Now you can validate your date and numeric fields.
For e.g
If you have a date field in your Joomla's form and the user enters a wrong date then you will get a fatal error on the page. So its better to validate the date field before saving.
Field 1- YYYY/MM/DD can be validate using class "validate-date"
Field 2- YYYY/MM/DD H:I:S can be validate using class "validate-datetime"
These changes have been added to "validate-uncompressed.js". To test this you will need to minify this file and replace the content of "validate.js" with this.
Let me know if any other information is needed.
Votes
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | JavaScript |
I have tested this item 
unsuccessfully on e7f4cf8
If i understand the issue right then it is not fixed yet.
Replication steps:
- Login to the Admin
- Navigate to the Feeds page and create a new Feeds
- For the created date field, try to input some "DD/MM/YYYY", different than current date.
- Save it, it still works.
- Open the Feed again
Observe: The date is automatically transformed to a current date
Expect: We should validate the date format on the inputting steps when user trying to create a new Feed
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8208.
@datredweb Thank you for testing.
Have you added the class "validate-date" or "validate-datetime" to that date input field? It won't work without adding the validate class.
| Category | JavaScript | ⇒ | Fields JavaScript |
I can see what you are doing here but as it is it is not ready for inclusion in the core. If you can look at how to implement validation on the date fields in the core joomla fields we can always re-open it when it is ready for testing again.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8208.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-05-08 13:06:24 |
| Closed_By | ⇒ | brianteeman |
I think it should be kept open. Maybe tested via a different test case. Since this is more like a library fix which allows for new validations ..
Now that it's available it can be used in various parts of the Joomla core ?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8208.
Provide the method for testing then
OK.. Will ask one of the team to Submit another PR to implement this in one of the Standard Joomla forms so that the test case can be provided. And provide that link here as well.
Can you minify the
validate-uncompressed.jsfile intovalidate.jswith your PR? It will be easyier on merge and test ;)