Author: Derick Rethans
Date: 2006-01-14 19:38:42 +0100 (Sat, 14 Jan 2006)
New Revision: 1852

Log:
- Fixed bug where the translation filename format that was passed as an option
  in the options array to the contructor was overwritten by the default.

Modified:
   packages/Translation/trunk/src/backends/ts_backend.php
   packages/Translation/trunk/tests/translation_backend_ts_test.php

Modified: packages/Translation/trunk/src/backends/ts_backend.php
===================================================================
--- packages/Translation/trunk/src/backends/ts_backend.php      2006-01-14 
15:24:19 UTC (rev 1851)
+++ packages/Translation/trunk/src/backends/ts_backend.php      2006-01-14 
18:38:42 UTC (rev 1852)
@@ -85,7 +85,7 @@
      *
      * @var string
      */
-    private $tsFilenameFormat;
+    private $tsFilenameFormat = '[LOCALE].xml';
 
     /**
      * The last read context, as read by next() method.
@@ -134,7 +134,6 @@
         }
         $this->setOptions( array( 'location' => $location ) );
         $this->setOptions( $options );
-        $this->tsFilenameFormat = '[LOCALE].xml';
     }
 
     /**

Modified: packages/Translation/trunk/tests/translation_backend_ts_test.php
===================================================================
--- packages/Translation/trunk/tests/translation_backend_ts_test.php    
2006-01-14 15:24:19 UTC (rev 1851)
+++ packages/Translation/trunk/tests/translation_backend_ts_test.php    
2006-01-14 18:38:42 UTC (rev 1852)
@@ -16,13 +16,15 @@
 {
     public function testConfigSetting()
     {
-        $backend = new ezcTranslationTsBackend( 'tests/translations', array ( 
'format' => '[LOCALE].xml' ) );
+        $backend = new ezcTranslationTsBackend( 'tests/translations', array ( 
'format' => 'test-[LOCALE].xml' ) );
+        self::assertPrivatePropertySame( $backend, 'tsFilenameFormat', 
'test-[LOCALE].xml' );
     }
 
     public function testConfigSettingAlternative()
     {
         $backend = new ezcTranslationTsBackend( 'tests/translations' );
-        $backend->setOptions( array ( 'format' => '[LOCALE].xml' ) );
+        $backend->setOptions( array ( 'format' => 'test-[LOCALE].xml' ) );
+        self::assertPrivatePropertySame( $backend, 'tsFilenameFormat', 
'test-[LOCALE].xml' );
     }
 
     public function testConfigSettingBroken()
@@ -41,15 +43,15 @@
     public function testBuildTranslationFileName1()
     {
         $backend = new ezcTranslationTsBackend( 'tests/translations' );
-        $backend->setOptions( array ( 'format' => '[LOCALE].xml' ) );
-        self::assertEquals( 'tests/translations/nl-nl.xml', 
$backend->buildTranslationFileName( 'nl-nl' ) );
+        $backend->setOptions( array ( 'format' => 'test-[LOCALE].xml' ) );
+        self::assertEquals( 'tests/translations/test-nl-nl.xml', 
$backend->buildTranslationFileName( 'nl-nl' ) );
     }
 
     public function testBuildTranslationFileName2()
     {
         $backend = new ezcTranslationTsBackend( 'tests/translations/' );
-        $backend->setOptions( array ( 'format' => '[LOCALE].xml' ) );
-        self::assertEquals( 'tests/translations/nl-nl.xml', 
$backend->buildTranslationFileName( 'nl-nl' ) );
+        $backend->setOptions( array ( 'format' => 'test-[LOCALE].xml' ) );
+        self::assertEquals( 'tests/translations/test-nl-nl.xml', 
$backend->buildTranslationFileName( 'nl-nl' ) );
     }
 
     public function testBuildTranslationFileName3()

-- 
svn-components mailing list
[email protected]
http://lists.ez.no/mailman/listinfo/svn-components

Reply via email to