[BUGFIX] Unnecessary warning in css_styled_content (division by zero)
authorThomas Layh <thomas@layh.com>
Wed, 2 May 2012 14:10:06 +0000 (16:10 +0200)
committerErnesto Baschny <ernst@cron-it.de>
Wed, 18 Jul 2012 09:20:09 +0000 (11:20 +0200)
If tt_content.image.20.maxW is not set, PHP throws a warning about
division by zero. This is not necessary. After that the variable
$scale is not set. This patch checks if $netW is greater zero and
otherwise the else condition will set $scale to 1.

Change-Id: Icd876b33d543080e486e0184c8af34c9a3831738
Fixes: #36777
Releases: 6.0, 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/12865
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
typo3/sysext/css_styled_content/pi1/class.tx_cssstyledcontent_pi1.php

index f7cc0a7..55b6bec 100755 (executable)
@@ -655,7 +655,7 @@ class tx_cssstyledcontent_pi1 extends tslib_pibase {
                                        $accumWidth = 0; // reset accumulated net width
                                        $accumDesiredWidth = 0; // reset accumulated desired width
                                        $rowTotalMaxW = $relations_cols[$rowIdx];
-                                       if ($rowTotalMaxW > $netW)      {
+                                       if ($rowTotalMaxW > $netW && $netW > 0) {
                                                $scale = $rowTotalMaxW / $netW;
                                        } else {
                                                $scale = 1;