From 9bae9461780cb7883adb4eee950e08379b681fd3 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Fri, 15 Mar 2013 18:27:48 +0100 Subject: [PATCH] Fix: Return of function with no return should not be set into a variable. --- htdocs/core/class/CMailFile.class.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/htdocs/core/class/CMailFile.class.php b/htdocs/core/class/CMailFile.class.php index 5fbfa36371a..0f80563e293 100644 --- a/htdocs/core/class/CMailFile.class.php +++ b/htdocs/core/class/CMailFile.class.php @@ -62,7 +62,7 @@ class CMailFile var $css; //! Defined css style for body background var $styleCSS; - //! Defined bacckground directly in body tag + //! Defined background directly in body tag var $bodyCSS; // Image @@ -256,7 +256,7 @@ class CMailFile if (!empty($css)) { $this->css = $css; - $this->styleCSS = $this->buildCSS(); + $this->buildCSS(); } $msg = $this->html; $msg = $this->checkIfHTML($msg); @@ -308,7 +308,7 @@ class CMailFile if (!empty($css)) { $this->css = $css; - $this->styleCSS = $this->buildCSS(); + $this->buildCSS(); } $msg = $this->html; $msg = $this->checkIfHTML($msg); @@ -624,7 +624,7 @@ class CMailFile if ($this->css['bgcolor']) { $this->styleCSS.= ' background-color: '.$this->css['bgcolor'].';'; - $this->bodyCSS.= ' BGCOLOR="'.$this->css['bgcolor'].'"'; + $this->bodyCSS.= ' bgcolor="'.$this->css['bgcolor'].'"'; } if ($this->css['bgimage']) {