? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
7 Jun 2016

Pull Request for Issue #10351

Summary of Changes

This succeeds @rdeutz 's PR at #10351 - it adds a generic on before validation event to allow manipulation of data to validate including the com_users PR that he mentioned earlier

Testing Instructions

Ensure there are no PHP errors in any core components

avatar wilsonge wilsonge - open - 7 Jun 2016
avatar wilsonge wilsonge - change - 7 Jun 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jun 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 8 Jun 2016
Category Plugins
avatar wilsonge
wilsonge - comment - 8 Jun 2016

@zero-24 fixed

avatar wilsonge wilsonge - change - 8 Jun 2016
Milestone Added:
avatar zero-24
zero-24 - comment - 8 Jun 2016

Thanks i have just noticed that there is a issue with the tests?!

There were 6 errors:
1) JModelFormTest::testValidateReturnsFalseOnJformException
JDatabaseExceptionExecuting: No database selected SQL=SELECT id, rules
FROM `#__viewlevels`
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver/mysqli.php:660
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver.php:1494
/home/travis/build/joomla/joomla-cms/libraries/joomla/access/access.php:935
/home/travis/build/joomla/joomla-cms/libraries/joomla/user/user.php:453
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:296
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:163
/home/travis/build/joomla/joomla-cms/libraries/legacy/model/form.php:333
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/legacy/model/JModelFormTest.php:57
2) JModelFormTest::testValidateReturnsFalseOnJformFalse
JDatabaseExceptionExecuting: No database selected SQL=SELECT id, rules
FROM `#__viewlevels`
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver/mysqli.php:660
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver.php:1494
/home/travis/build/joomla/joomla-cms/libraries/joomla/access/access.php:935
/home/travis/build/joomla/joomla-cms/libraries/joomla/user/user.php:453
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:296
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:163
/home/travis/build/joomla/joomla-cms/libraries/legacy/model/form.php:333
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/legacy/model/JModelFormTest.php:101
3) JModelFormTest::testValidateModifiesTagsArray with data set #0 (array('1'), array('1'))
JDatabaseExceptionExecuting: No database selected SQL=SELECT id, rules
FROM `#__viewlevels`
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver/mysqli.php:660
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver.php:1494
/home/travis/build/joomla/joomla-cms/libraries/joomla/access/access.php:935
/home/travis/build/joomla/joomla-cms/libraries/joomla/user/user.php:453
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:296
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:163
/home/travis/build/joomla/joomla-cms/libraries/legacy/model/form.php:333
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/legacy/model/JModelFormTest.php:150
4) JModelFormTest::testValidateModifiesTagsArray with data set #1 (array(array('1')), array(array('1'), '1'))
JDatabaseExceptionExecuting: No database selected SQL=SELECT id, rules
FROM `#__viewlevels`
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver/mysqli.php:660
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver.php:1494
/home/travis/build/joomla/joomla-cms/libraries/joomla/access/access.php:935
/home/travis/build/joomla/joomla-cms/libraries/joomla/user/user.php:453
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:296
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:163
/home/travis/build/joomla/joomla-cms/libraries/legacy/model/form.php:333
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/legacy/model/JModelFormTest.php:150
5) JModelFormTest::testValidateModifiesTagsArray with data set #2 (array('1', array('2')), array('1', array('2')))
JDatabaseExceptionExecuting: No database selected SQL=SELECT id, rules
FROM `#__viewlevels`
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver/mysqli.php:660
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver.php:1494
/home/travis/build/joomla/joomla-cms/libraries/joomla/access/access.php:935
/home/travis/build/joomla/joomla-cms/libraries/joomla/user/user.php:453
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:296
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:163
/home/travis/build/joomla/joomla-cms/libraries/legacy/model/form.php:333
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/legacy/model/JModelFormTest.php:150
6) JModelFormTest::testValidatePassesDataAndGroupToJform
JDatabaseExceptionExecuting: No database selected SQL=SELECT id, rules
FROM `#__viewlevels`
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver/mysqli.php:660
/home/travis/build/joomla/joomla-cms/libraries/joomla/database/driver.php:1494
/home/travis/build/joomla/joomla-cms/libraries/joomla/access/access.php:935
/home/travis/build/joomla/joomla-cms/libraries/joomla/user/user.php:453
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:296
/home/travis/build/joomla/joomla-cms/libraries/cms/plugin/helper.php:163
/home/travis/build/joomla/joomla-cms/libraries/legacy/model/form.php:333
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/legacy/model/JModelFormTest.php:186

avatar rdeutz rdeutz - test_item - 8 Jun 2016 - Tested successfully
avatar rdeutz
rdeutz - comment - 8 Jun 2016

I have tested this item successfully on 162e40f

works as expected (with the simplify user plugin) when also applying #10350


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

avatar joomla-cms-bot
joomla-cms-bot - comment - 8 Jun 2016

This PR has received new commits.

CC: @rdeutz


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

avatar joomla-cms-bot joomla-cms-bot - change - 8 Jun 2016
Labels Added: ?
avatar joomla-cms-bot
joomla-cms-bot - comment - 8 Jun 2016

This PR has received new commits.

CC: @rdeutz


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

avatar wilsonge wilsonge - change - 8 Jun 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-06-08 21:51:06
Closed_By wilsonge

Add a Comment

Login with GitHub to post a comment