commandURLCleanup now strips all unwanted chars at the end of the string

lib/ezi18n/classes/ezchartransform.php

@@ -395,12 +395,10 @@ static function commandUrlCleanup( $text, $charsetName )
         $sep  = eZCharTransform::wordSeparator();
         $sepQ = preg_quote( $sep );
         $text = preg_replace( array( "#[^a-zA-Z0-9_!\.-]+#",
-                                     "#^[\.]+|[!\.]+$#", # Remove dots at beginning/end
                                      "#\.\.+#", # Remove double dots
                                      "#[{$sepQ}]+#", # Turn multiple separators into one
-                                     "#^[{$sepQ}]+|[{$sepQ}]+$#" ), # Strip separator from beginning/end
+                                     "#^[{$sepQ}\.]+|[{$sepQ}!\.]+$#" ), # Strip separator and dot from beginning, strip exclemation mark, dot and separator from end
                               array( $sep,
-                                     $sep,
                                      $sep,
                                      $sep,
                                      "" ),

tests/tests/lib/ezi18n/ezchartransform_test.php

@@ -0,0 +1,41 @@
+<?php
+/**
+ * File containing the eZCharTransformTests class
+ *
+ * @package tests
+ */
+
+class eZCharTransformTests extends ezpTestCase
+{
+
+    public function __construct()
+    {
+        parent::__construct();
+        $this->setName( "eZCharTransformTests" );
+    }
+
+    public function setUp()
+    {
+        parent::setUp();
+    }
+
+    public function tearDown()
+    {
+        parent::tearDown();
+    }
+
+    public function testCommandUrlCleanupMultipleSpecialCharsAtEnd()
+    {
+        $objectName = 'test."';
+        $transformed = eZCharTransform::commandUrlCleanup( $objectName );
+        $this->assertEquals( $transformed, 'test' );
+
+        $objectName = 'te.st."';
+        $transformed = eZCharTransform::commandUrlCleanup( $objectName );
+        $this->assertEquals( $transformed, 'te.st' );
+
+        $objectName = '.test!"';
+        $transformed = eZCharTransform::commandUrlCleanup( $objectName );
+        $this->assertEquals( $transformed, 'test' );
+    }
+}

tests/tests/lib/ezi18n/suite.php

@@ -0,0 +1,22 @@
+<?php
+/**
+ * File containing the eZFileTestSuite class
+ *
+ * @package tests
+ */
+
+class eZFileTestSuite extends ezpTestSuite
+{
+    public function __construct()
+    {
+        parent::__construct();
+        $this->setName( "eZFile Test Suite" );
+        $this->addTestSuite( 'eZCharTransformTests' );
+    }
+
+    public static function suite()
+    {
+        return new self();
+    }
+
+}