Thiemo Mättig (WMDE) has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/338739 )
Change subject: Block all vertical whitespace in LexemeValidatorFactory ...................................................................... Block all vertical whitespace in LexemeValidatorFactory The \v shortcut is supported since PHP 5.2. See http://www.pcre.org/original/doc/html/pcrepattern.html for a list of all characters it contains. Yes, I'm sure we want to block all of them. Change-Id: Ieb9256b6bec1363f19a463b3adaa1eb6e845aba1 --- M src/Validators/LexemeValidatorFactory.php 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikibaseLexeme refs/changes/39/338739/1 diff --git a/src/Validators/LexemeValidatorFactory.php b/src/Validators/LexemeValidatorFactory.php index 367a452..910954d 100644 --- a/src/Validators/LexemeValidatorFactory.php +++ b/src/Validators/LexemeValidatorFactory.php @@ -47,7 +47,7 @@ [ new TypeValidator( 'string' ), new StringLengthValidator( 1, $this->maxTermLength, 'mb_strlen' ), - new RegexValidator( '/^\s|[\r\n\t]|\s$/', true ), + new RegexValidator( '/^\s|\v|\s$/', true ), ], true ); -- To view, visit https://gerrit.wikimedia.org/r/338739 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ieb9256b6bec1363f19a463b3adaa1eb6e845aba1 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/WikibaseLexeme Gerrit-Branch: master Gerrit-Owner: Thiemo Mättig (WMDE) <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
