? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
8 Nov 2014

Executive summary

This PR converts the form validation on com_newsfeed to use plain jquery (no mootools call on every form).
Also NO MORE INLINE SCRIPTS!

Testing

  1. Apply first PR #4888 (!important)
  2. Apply this PR
  3. In the admin area go to com_newsfeed and try to submit any form.

If no javascript errors are logged in your browser and the functionality remains the same your test is a pass 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
http://localhost/administrator/index.php?option=com_admin&view=sysinfo should demonstrate highlighter.js without mt
Logout and log in to demonstrate the use of noframes without mt.

Votes

# of Users Experiencing Issue
0/1
Average Importance Score
5.00

avatar dgt41 dgt41 - open - 8 Nov 2014
avatar jissues-bot jissues-bot - change - 8 Nov 2014
The description was changed
Labels Added: ?
avatar brianteeman brianteeman - change - 8 Nov 2014
Category JavaScript
avatar dgt41 dgt41 - change - 10 Nov 2014
The description was changed
avatar smanzi
smanzi - comment - 11 Nov 2014

@Test success

avatar anibalsanchez
anibalsanchez - comment - 13 Nov 2014

@Test success

avatar anibalsanchez anibalsanchez - test_item - 13 Nov 2014 - Tested successfully
avatar roland-d roland-d - alter_testresult - 18 Nov 2014 - smanzi: Tested successfully
avatar roland-d roland-d - change - 18 Nov 2014
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 18 Nov 2014

Moving to RTC as we have 2 successful tests.

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5050.

avatar dgt41
dgt41 - comment - 18 Nov 2014

@roland-d All these PRs depend on #4888 so this one has to be RTC as well!

avatar roland-d
roland-d - comment - 18 Nov 2014

@dgt41 Thanks for the heads-up, I read that and I am going through your PRs at this moment. So #4888 should be hit at some point as well :)

avatar brianteeman brianteeman - change - 29 Nov 2014
Labels Added: ?
avatar smanzi
smanzi - comment - 30 Nov 2014

@test success for this PR
Tested the set of: #5046 #5048 #5049 #5050 #5051 #5052 #5053 #5054 #5056 #5058 #5113 #5255

avatar phproberto phproberto - close - 30 Nov 2014
avatar zero-24 zero-24 - close - 30 Nov 2014
avatar phproberto phproberto - change - 30 Nov 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-11-30 02:05:55
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment