Author: Tobias Schlitt
Date: 2006-01-13 13:39:30 +0100 (Fri, 13 Jan 2006)
New Revision: 1823

Log:
- Fixed unreported bug in crop filter code, spotted by Bard (thanks!). 

Modified:
   packages/ConsoleTools/trunk/src/output.php
   packages/ConsoleTools/trunk/src/progressbar.php
   packages/ConsoleTools/trunk/src/statusbar.php
   packages/ConsoleTools/trunk/src/table.php
   packages/ImageConversion/trunk/src/filters/imagemagick.php
   packages/ImageConversion/trunk/tests/filtersshell_test.php
   packages/ImageConversion/trunk/tests/handlershell_test.php

Modified: packages/ConsoleTools/trunk/src/output.php
===================================================================
--- packages/ConsoleTools/trunk/src/output.php  2006-01-13 12:18:27 UTC (rev 
1822)
+++ packages/ConsoleTools/trunk/src/output.php  2006-01-13 12:39:30 UTC (rev 
1823)
@@ -210,7 +210,7 @@
      * @throws ezcBaseValueException 
      *         If a the value for the property options is not an instance of 
      *         ezcConsoleOutputOptions. 
-     * @throws ezcBasePropertyException 
+     * @throws ezcBaseValueException 
      *         If a the value for the property formats is not an instance of 
      *         ezcConsoleOutputFormats. 
      * @return void
@@ -222,14 +222,14 @@
             case 'options':
                 if ( !( $val instanceof ezcConsoleOutputOptions ) )
                 {
-                    throw new ezcBasePropertyException( $key, $val, 
'ezcConsoleOutputOptions' );
+                    throw new ezcBaseValueException( $key, $val, 
'ezcConsoleOutputOptions' );
                 }
                 $this->options = $val;
                 return;
             case 'formats':
                 if ( !( $val instanceof ezcConsoleOutputFormats ) )
                 {
-                    throw new ezcBasePropertyException( $key, $val, 
'ezcConsoleOutputFormats' );
+                    throw new ezcBaseValueException( $key, $val, 
'ezcConsoleOutputFormats' );
                 }
                 $this->formats = $val;
                 return;

Modified: packages/ConsoleTools/trunk/src/progressbar.php
===================================================================
--- packages/ConsoleTools/trunk/src/progressbar.php     2006-01-13 12:18:27 UTC 
(rev 1822)
+++ packages/ConsoleTools/trunk/src/progressbar.php     2006-01-13 12:39:30 UTC 
(rev 1823)
@@ -194,7 +194,7 @@
             case 'step':
                 if ( !is_int( $val ) || $val < 0 )
                 {
-                    throw new ezcBasePropertyException( $key, $val, 'int >= 0' 
);
+                    throw new ezcBaseValueException( $key, $val, 'int >= 0' );
                 }
                 break;
             default:

Modified: packages/ConsoleTools/trunk/src/statusbar.php
===================================================================
--- packages/ConsoleTools/trunk/src/statusbar.php       2006-01-13 12:18:27 UTC 
(rev 1822)
+++ packages/ConsoleTools/trunk/src/statusbar.php       2006-01-13 12:39:30 UTC 
(rev 1823)
@@ -128,7 +128,7 @@
             case 'failureChar':
                 if ( strlen( $val ) < 1 )
                 {
-                    throw new ezcBasePropertyException( $key, $val, 'string, 
not empty' );
+                    throw new ezcBaseValueException( $key, $val, 'string, not 
empty' );
                 }
                 break;
             default:

Modified: packages/ConsoleTools/trunk/src/table.php
===================================================================
--- packages/ConsoleTools/trunk/src/table.php   2006-01-13 12:18:27 UTC (rev 
1822)
+++ packages/ConsoleTools/trunk/src/table.php   2006-01-13 12:39:30 UTC (rev 
1823)
@@ -403,7 +403,7 @@
             case 'options':
                 if ( !( $val instanceof ezcConsoleTableOptions ) )
                 {
-                    throw new ezcBasePropertyException( $key, $val, 
'ezcConsoleTableOptions' );
+                    throw new ezcBaseValueException( $key, $val, 
'ezcConsoleTableOptions' );
                 }
                 $this->options = $val;
                 return;

Modified: packages/ImageConversion/trunk/src/filters/imagemagick.php
===================================================================
--- packages/ImageConversion/trunk/src/filters/imagemagick.php  2006-01-13 
12:18:27 UTC (rev 1822)
+++ packages/ImageConversion/trunk/src/filters/imagemagick.php  2006-01-13 
12:39:30 UTC (rev 1823)
@@ -195,7 +195,7 @@
         $this->handler->addFilterOption(
             $this->handler->getActiveReference(),
             '-crop ',
-            abs( $width ).'x'.abs( $height ).$xStart.'x'.$yStart
+            abs( $width ).'x'.abs( $height ).$xStart.$yStart
         );
     }
 

Modified: packages/ImageConversion/trunk/tests/filtersshell_test.php
===================================================================
--- packages/ImageConversion/trunk/tests/filtersshell_test.php  2006-01-13 
12:18:27 UTC (rev 1822)
+++ packages/ImageConversion/trunk/tests/filtersshell_test.php  2006-01-13 
12:39:30 UTC (rev 1823)
@@ -36,9 +36,9 @@
         'testColorspaceGrey' => '57d8e16a4a01131fbaecd519162aa095',
         'testColorspaceMonochrome' => '53bc6b9f9e012e875b27143bd3e29e21',
         'testColorspaceSepia' => '1db4c3a057a2546dd56f0d61f8480911',
-        'testCrop_1' => 'cbba8251105a848860ef111ed20dacf2',
-        'testCrop_2' => 'cbba8251105a848860ef111ed20dacf2',
-        'testCrop_3' => 'aa349d0114ac04540a16f3a77bcd4573',
+        'testCrop_1' => 'dc5a31eb43e3b5c9edee54d3c24fc7a3',
+        'testCrop_2' => 'dc5a31eb43e3b5c9edee54d3c24fc7a3',
+        'testCrop_3' => '524dc9211b054a967db9a444f4b59597',
         'testScale' => 'd0969b833c95c0cabb4e5dbf0100eaa7',
         'testScaleDown_do' => 'd896401cee2e7e9913fc3e0ac8946d6e',
         'testScaleDown_dont' => '902e24014b1d1490a94eaf89be5113a6',

Modified: packages/ImageConversion/trunk/tests/handlershell_test.php
===================================================================
--- packages/ImageConversion/trunk/tests/handlershell_test.php  2006-01-13 
12:18:27 UTC (rev 1822)
+++ packages/ImageConversion/trunk/tests/handlershell_test.php  2006-01-13 
12:39:30 UTC (rev 1823)
@@ -217,7 +217,7 @@
         $this->handler->save( $ref, $dstPath );
 
         $this->assertEquals(
-            '4c7c2e2f70f2ea616984afcf471b3ec7',
+            '34d2952b5ca0bdbd897a4ab3265f2a06',
             md5_file( $dstPath ),
             'Applying multiple filter through handler failed.'
         );

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

Reply via email to