[Validator] Fixed Regex::getHtmlPattern() to work with complex and negated patterns
This commit is contained in:
parent
7a13556443
commit
bf006f5202
@ -45,7 +45,14 @@ class Regex extends Constraint
|
|||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Returns htmlPattern if exists or pattern is convertible.
|
* Converts the htmlPattern to a suitable format for HTML5 pattern.
|
||||||
|
* Example: /^[a-z]+$/ would be converted to [a-z]+
|
||||||
|
* However, if options are specified, it cannot be converted
|
||||||
|
*
|
||||||
|
* Pattern is also ignored if match=false since the pattern should
|
||||||
|
* then be reversed before application.
|
||||||
|
*
|
||||||
|
* @link http://dev.w3.org/html5/spec/single-page.html#the-pattern-attribute
|
||||||
*
|
*
|
||||||
* @return string|null
|
* @return string|null
|
||||||
*/
|
*/
|
||||||
@ -58,40 +65,34 @@ class Regex extends Constraint
|
|||||||
: $this->htmlPattern;
|
: $this->htmlPattern;
|
||||||
}
|
}
|
||||||
|
|
||||||
return $this->getNonDelimitedPattern();
|
// Quit if delimiters not at very beginning/end (e.g. when options are passed)
|
||||||
}
|
if ($this->pattern[0] !== $this->pattern[strlen($this->pattern) - 1]) {
|
||||||
|
|
||||||
/**
|
|
||||||
* Converts the htmlPattern to a suitable format for HTML5 pattern.
|
|
||||||
* Example: /^[a-z]+$/ would be converted to [a-z]+
|
|
||||||
* However, if options are specified, it cannot be converted
|
|
||||||
*
|
|
||||||
* Pattern is also ignored if match=false since the pattern should
|
|
||||||
* then be reversed before application.
|
|
||||||
*
|
|
||||||
* @todo reverse pattern in case match=false as per issue #5307
|
|
||||||
*
|
|
||||||
* @link http://dev.w3.org/html5/spec/single-page.html#the-pattern-attribute
|
|
||||||
*
|
|
||||||
* @return string|null
|
|
||||||
*/
|
|
||||||
private function getNonDelimitedPattern()
|
|
||||||
{
|
|
||||||
// If match = false, pattern should not be added to HTML5 validation
|
|
||||||
if (!$this->match) {
|
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (preg_match('/^(.)(\^?)(.*?)(\$?)\1$/', $this->pattern, $matches)) {
|
$delimiter = $this->pattern[0];
|
||||||
$delimiter = $matches[1];
|
|
||||||
$start = empty($matches[2]) ? '.*' : '';
|
|
||||||
$pattern = $matches[3];
|
|
||||||
$end = empty($matches[4]) ? '.*' : '';
|
|
||||||
|
|
||||||
// Unescape the delimiter in pattern
|
// Unescape the delimiter
|
||||||
$pattern = str_replace('\\'.$delimiter, $delimiter, $pattern);
|
$pattern = str_replace('\\'.$delimiter, $delimiter, substr($this->pattern, 1, -1));
|
||||||
|
|
||||||
return $start.$pattern.$end;
|
// If the pattern is inverted, we can simply wrap it in
|
||||||
|
// ((?!pattern).)*
|
||||||
|
if (!$this->match) {
|
||||||
|
return '((?!'.$pattern.').)*';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// If the pattern contains an or statement, wrap the pattern in
|
||||||
|
// .*(pattern).* and quit. Otherwise we'd need to parse the pattern
|
||||||
|
if (false !== strpos($pattern, '|')) {
|
||||||
|
return '.*('.$pattern.').*';
|
||||||
|
}
|
||||||
|
|
||||||
|
// Trim leading ^, otherwise prepend .*
|
||||||
|
$pattern = '^' === $pattern[0] ? substr($pattern, 1) : '.*'.$pattern;
|
||||||
|
|
||||||
|
// Trim trailing $, otherwise append .*
|
||||||
|
$pattern = '$' === $pattern[strlen($pattern) - 1] ? substr($pattern, 0, -1) : $pattern.'.*';
|
||||||
|
|
||||||
|
return $pattern;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -0,0 +1,87 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
/*
|
||||||
|
* This file is part of the Symfony package.
|
||||||
|
*
|
||||||
|
* (c) Fabien Potencier <fabien@symfony.com>
|
||||||
|
*
|
||||||
|
* For the full copyright and license information, please view the LICENSE
|
||||||
|
* file that was distributed with this source code.
|
||||||
|
*/
|
||||||
|
|
||||||
|
namespace Constraints;
|
||||||
|
|
||||||
|
use Symfony\Component\Validator\Constraints\Regex;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @author Bernhard Schussek <bschussek@gmail.com>
|
||||||
|
*/
|
||||||
|
class RegexTest extends \PHPUnit_Framework_TestCase
|
||||||
|
{
|
||||||
|
public function testConstraintGetDefaultOption()
|
||||||
|
{
|
||||||
|
$constraint = new Regex('/^[0-9]+$/');
|
||||||
|
|
||||||
|
$this->assertSame('/^[0-9]+$/', $constraint->pattern);
|
||||||
|
}
|
||||||
|
|
||||||
|
public function provideHtmlPatterns()
|
||||||
|
{
|
||||||
|
return array(
|
||||||
|
// HTML5 wraps the pattern in ^(?:pattern)$
|
||||||
|
array('/^[0-9]+$/', '[0-9]+'),
|
||||||
|
array('/[0-9]+$/', '.*[0-9]+'),
|
||||||
|
array('/^[0-9]+/', '[0-9]+.*'),
|
||||||
|
array('/[0-9]+/', '.*[0-9]+.*'),
|
||||||
|
// We need a smart way to allow matching of patterns that contain
|
||||||
|
// ^ and $ at various sub-clauses of an or-clause
|
||||||
|
// .*(pattern).* seems to work correctly
|
||||||
|
array('/[0-9]$|[a-z]+/', '.*([0-9]$|[a-z]+).*'),
|
||||||
|
array('/[0-9]$|^[a-z]+/', '.*([0-9]$|^[a-z]+).*'),
|
||||||
|
array('/^[0-9]|[a-z]+$/', '.*(^[0-9]|[a-z]+$).*'),
|
||||||
|
// Unescape escaped delimiters
|
||||||
|
array('/^[0-9]+\/$/', '[0-9]+/'),
|
||||||
|
array('#^[0-9]+\#$#', '[0-9]+#'),
|
||||||
|
// Cannot be converted
|
||||||
|
array('/^[0-9]+$/i', null),
|
||||||
|
|
||||||
|
// Inverse matches are simple, just wrap in
|
||||||
|
// ((?!pattern).)*
|
||||||
|
array('/^[0-9]+$/', '((?!^[0-9]+$).)*', false),
|
||||||
|
array('/[0-9]+$/', '((?![0-9]+$).)*', false),
|
||||||
|
array('/^[0-9]+/', '((?!^[0-9]+).)*', false),
|
||||||
|
array('/[0-9]+/', '((?![0-9]+).)*', false),
|
||||||
|
array('/[0-9]$|[a-z]+/', '((?![0-9]$|[a-z]+).)*', false),
|
||||||
|
array('/[0-9]$|^[a-z]+/', '((?![0-9]$|^[a-z]+).)*', false),
|
||||||
|
array('/^[0-9]|[a-z]+$/', '((?!^[0-9]|[a-z]+$).)*', false),
|
||||||
|
array('/^[0-9]+\/$/', '((?!^[0-9]+/$).)*', false),
|
||||||
|
array('#^[0-9]+\#$#', '((?!^[0-9]+#$).)*', false),
|
||||||
|
array('/^[0-9]+$/i', null, false),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @dataProvider provideHtmlPatterns
|
||||||
|
*/
|
||||||
|
public function testGetHtmlPattern($pattern, $htmlPattern, $match = true)
|
||||||
|
{
|
||||||
|
$constraint = new Regex(array(
|
||||||
|
'pattern' => $pattern,
|
||||||
|
'match' => $match,
|
||||||
|
));
|
||||||
|
|
||||||
|
$this->assertSame($pattern, $constraint->pattern);
|
||||||
|
$this->assertSame($htmlPattern, $constraint->getHtmlPattern());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function testGetCustomHtmlPattern()
|
||||||
|
{
|
||||||
|
$constraint = new Regex(array(
|
||||||
|
'pattern' => '((?![0-9]$|[a-z]+).)*',
|
||||||
|
'htmlPattern' => 'foobar',
|
||||||
|
));
|
||||||
|
|
||||||
|
$this->assertSame('((?![0-9]$|[a-z]+).)*', $constraint->pattern);
|
||||||
|
$this->assertSame('foobar', $constraint->getHtmlPattern());
|
||||||
|
}
|
||||||
|
}
|
@ -88,70 +88,4 @@ class RegexValidatorTest extends AbstractConstraintValidatorTest
|
|||||||
array('090foo'),
|
array('090foo'),
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testConstraintGetDefaultOption()
|
|
||||||
{
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/^[0-9]+$/',
|
|
||||||
));
|
|
||||||
|
|
||||||
$this->assertEquals('pattern', $constraint->getDefaultOption());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function testHtmlPatternEscaping()
|
|
||||||
{
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/^[0-9]+\/$/',
|
|
||||||
));
|
|
||||||
|
|
||||||
$this->assertEquals('[0-9]+/', $constraint->getHtmlPattern());
|
|
||||||
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '#^[0-9]+\#$#',
|
|
||||||
));
|
|
||||||
|
|
||||||
$this->assertEquals('[0-9]+#', $constraint->getHtmlPattern());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function testHtmlPattern()
|
|
||||||
{
|
|
||||||
// Specified htmlPattern
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/^[a-z]+$/i',
|
|
||||||
'htmlPattern' => '[a-zA-Z]+',
|
|
||||||
));
|
|
||||||
$this->assertEquals('[a-zA-Z]+', $constraint->getHtmlPattern());
|
|
||||||
|
|
||||||
// Disabled htmlPattern
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/^[a-z]+$/i',
|
|
||||||
'htmlPattern' => false,
|
|
||||||
));
|
|
||||||
$this->assertNull($constraint->getHtmlPattern());
|
|
||||||
|
|
||||||
// Cannot be converted
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/^[a-z]+$/i',
|
|
||||||
));
|
|
||||||
$this->assertNull($constraint->getHtmlPattern());
|
|
||||||
|
|
||||||
// Automatically converted
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/^[a-z]+$/',
|
|
||||||
));
|
|
||||||
$this->assertEquals('[a-z]+', $constraint->getHtmlPattern());
|
|
||||||
|
|
||||||
// Automatically converted, adds .*
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/[a-z]+/',
|
|
||||||
));
|
|
||||||
$this->assertEquals('.*[a-z]+.*', $constraint->getHtmlPattern());
|
|
||||||
|
|
||||||
// Dropped because of match=false
|
|
||||||
$constraint = new Regex(array(
|
|
||||||
'pattern' => '/[a-z]+/',
|
|
||||||
'match' => false,
|
|
||||||
));
|
|
||||||
$this->assertNull($constraint->getHtmlPattern());
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user