Merge branch 'hotfix/5183'

Close zendframework/zendframework#5183

src/Word/CamelCaseToSeparator.php

@@ -25,7 +25,7 @@ public function filter($value)
             $pattern     = array('#(?<=(?:\p{Lu}))(\p{Lu}\p{Ll})#', '#(?<=(?:\p{Ll}|\p{Nd}))(\p{Lu})#');
             $replacement = array($this->separator . '\1', $this->separator . '\1');
         } else {
-            $pattern     = array('#(?<=(?:[A-Z]))([A-Z]+)([A-Z][A-z])#', '#(?<=(?:[a-z0-9]))([A-Z])#');
+            $pattern     = array('#(?<=(?:[A-Z]))([A-Z]+)([A-Z][a-z])#', '#(?<=(?:[a-z0-9]))([A-Z])#');
             $replacement = array('\1' . $this->separator . '\2', $this->separator . '\1');
         }
 

test/Word/CamelCaseToSeparatorNoPcreUnicodeTest.php

@@ -0,0 +1,39 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @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
+ */
+
+namespace ZendTest\Filter\Word;
+
+use ReflectionProperty;
+use Zend\Filter\Word\CamelCaseToSeparator as CamelCaseToSeparatorFilter;
+use Zend\Stdlib\StringUtils;
+
+/**
+ * Test class for Zend\Filter\Word\CamelCaseToSeparator which simulates the
+ * PCRE Unicode features disabled
+ */
+class CamelCaseToSeparatorNoPcreUnicodeTest extends CamelCaseToSeparatorTest
+{
+    protected $reflection;
+
+    public function setUp()
+    {
+        if (!StringUtils::hasPcreUnicodeSupport()) {
+            return $this->markTestSkipped('PCRE is not compiled with Unicode support');
+        }
+
+        $this->reflection = new ReflectionProperty('Zend\Stdlib\StringUtils', 'hasPcreUnicodeSupport');
+        $this->reflection->setAccessible(true);
+        $this->reflection->setValue(false);
+    }
+
+    public function tearDown()
+    {
+        $this->reflection->setValue(true);
+    }
+}

test/Word/CamelCaseToSeparatorTest.php

@@ -37,4 +37,14 @@ public function testFilterSeparatesCamelCasedWordsWithProvidedSeparator()
         $this->assertNotEquals($string, $filtered);
         $this->assertEquals('Camel:-#Cased:-#Words', $filtered);
     }
+
+    public function testFilterSeperatesMultipleUppercasedLettersAndUnderscores()
+    {
+        $string   = 'TheseAre_SOME_CamelCASEDWords';
+        $filter   = new CamelCaseToSeparatorFilter('_');
+        $filtered = $filter($string);
+
+        $this->assertNotEquals($string, $filtered);
+        $this->assertEquals('These_Are_SOME_Camel_CASED_Words', $filtered);
+    }
 }