-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New Zend\Validator\Bitwise #4940
New Zend\Validator\Bitwise #4940
Conversation
* @link http://github.com/zendframework/zf2 for the canonical source repository | ||
* @copyright Copyright (c) 2005-2013 Zend Technologies USA Inc. (http://www.zend.com) | ||
* @license http://framework.zend.com/license/new-bsd New BSD License | ||
* @package Zend_Validator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove @Package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it ( @Package ) need to be removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, missed this spot the first time !
public function __construct($options = null) | ||
{ | ||
if ($options instanceof Traversable) { | ||
$options = ArrayUtils::iteratorToArray($options); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you do not have nested options (i.e., if the options are only one level deep), just use iterator_to_array
here.
@corentin-larose Looks good - incorporate or answer the various feedback, and I'll review to merge. |
Ok, I will take your changes and commit again. |
@weierophinney should be ok/compliant if you want to merge. |
@corentin-larose Your test are failing in Travis-CI
|
@Maks3w, my bad, fixed the constructor to work the same way than other validators, forgot to push the U.T |
@corentin-larose Tests run still failing. |
Tests are ok, seems that Travis didn't run the most recent code (possible ?) I'm running tests on my last commit: 9339412. corentin@QAPA-I:/media/localhost/zf2-fork/tests$ phpunit ZendTest/Validator/BitwiseTest.php Configuration read from /media/localhost/zf2-fork/tests/phpunit.xml ......... Time: 0 seconds, Memory: 3.00Mb OK (9 tests, 48 assertions) |
@corentin-larose Please read the log and make the necessary to changes for to have a clean Travis-CI run. If you see the log Travis use even a most recent version than you. Try using full namespace class name instead the just |
@Maks3w doc block fixed, travis still fails, but on the usual think which is not related with Bitwise validator. |
New Zend\Validator\Bitwise
…lidator-bitwise New Zend\Validator\Bitwise
Usage: