Daniel Kinzler has uploaded a new change for review.
https://gerrit.wikimedia.org/r/67956
Change subject: Fix string length check in StringtValidator.
......................................................................
Fix string length check in StringtValidator.
Unit tests for validators are still missing.
No point in adding them now with a redesign imminent,
but we should really have them soon.
Change-Id: I92e2b7cb98203119455ec91bfb5096a06dbe7fa1
---
M ValueValidators/includes/validators/StringValidator.php
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/DataValues
refs/changes/56/67956/1
diff --git a/ValueValidators/includes/validators/StringValidator.php
b/ValueValidators/includes/validators/StringValidator.php
index 0fcd394..fa876ee 100644
--- a/ValueValidators/includes/validators/StringValidator.php
+++ b/ValueValidators/includes/validators/StringValidator.php
@@ -64,7 +64,7 @@
if ( $lowerBound !== false || $upperBound !== false ) {
$rangeValidator = new RangeValidator();
$rangeValidator->setRange( $lowerBound, $upperBound );
- $this->runSubValidator( count( $value ),
$rangeValidator, 'length' );
+ $this->runSubValidator( strlen( $value ),
$rangeValidator, 'length' );
}
// TODO: this needs tests
--
To view, visit https://gerrit.wikimedia.org/r/67956
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I92e2b7cb98203119455ec91bfb5096a06dbe7fa1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DataValues
Gerrit-Branch: master
Gerrit-Owner: Daniel Kinzler <daniel.kinz...@wikimedia.de>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
