[#5116] - Introduce jquery formvalidator for com_tags ::frontend
- Closed
- 2 Dec 2014
- Medium
- Build: staging
- # 5116
- Diff
- dgt41:_form_jq_com_tags_frontend
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Executive summary
This PR converts the form validation on com_tags to use plain jquery (no mootools call on every form).
Also NO MORE INLINE SCRIPTS!
Testing
Make sure you have some tags
- Apply first PR #4888 (!important)
- Apply this PR
- In the admin area create new menu items for tags (all three options)
- go to the new menus and test that everything is still OK
If no javascript errors are logged in your browser’s console and the functionality remains the same, your test is passing in any other case please report the errors here
Please also check these:
administrator/index.php?option=com_checkin should demonstrate multiselect without mt
administrator/index.php?option=com_users&view=mail should demonstrate form sent and validate without mt
administrator/index.php?option=com_modules should demonstrate multiselect and combobox without mt
Logout and log in to demonstrate the use of noframes without mt.
jissues-bot
- | Labels |
Added:
?
|
||
zero-24
- | Category | ⇒ | Front End JavaScript |
@smanzi Would you mind testing this one also? Thank you.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5116.
@dgt41
After "Apply first PR #4888 (!important)" like you wrote in the instruction test,
i got Error 500
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5116.
@yanivRozenman Please check your Joomla version you’re using to test this. If it is the current staging you don’t have to apply #4888 as this is already merged. If you are on 3.3.6 and still getting errors check patch tester. The error message indicates that something gone wrong and the file behavior.php is missing
| Status | Pending | ⇒ | Ready to Commit |
Moving to RTC as we have 2 successful tests.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5116.
brianteeman
- | Labels |
Added:
?
|
||
Merged into staging. Thanks!
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-12-02 20:13:19 |
| Labels |
Removed:
?
|
||
@test success. It is just a minor code change. Front-end site works as usual.