diff --git a/model/URLSegmentFilter.php b/model/URLSegmentFilter.php index 88436f15d..505709b6e 100644 --- a/model/URLSegmentFilter.php +++ b/model/URLSegmentFilter.php @@ -32,6 +32,7 @@ class URLSegmentFilter extends Object { '/\s|\+/u' => '-', // remove whitespace/plus '/[_.]+/u' => '-', // underscores and dots to dashes '/[^A-Za-z0-9\-]+/u' => '', // remove non-ASCII chars, only allow alphanumeric and dashes + '/\/+/u' => '-', // remove forward slashes in case multibyte is allowed (and ASCII chars aren't removed) '/[\-]{2,}/u' => '-', // remove duplicate dashes '/^[\-]+/u' => '', // Remove all leading dashes '/[\-]+$/u' => '' // Remove all trailing dashes diff --git a/tests/model/URLSegmentFilterTest.php b/tests/model/URLSegmentFilterTest.php index 177ad793f..433e73469 100644 --- a/tests/model/URLSegmentFilterTest.php +++ b/tests/model/URLSegmentFilterTest.php @@ -55,6 +55,15 @@ class URLSegmentFilterTest extends SapphireTest { ); } + public function testReplacesForwardSlashesWithAllowMultiByteOption() { + $f = new URLSegmentFilter(); + $f->setAllowMultibyte(true); + $this->assertEquals( + urlencode('this-that'), + $f->filter('this/that') + ); + } + public function testReplacements() { $f = new URLSegmentFilter(); $this->assertEquals(