Correct the error message and keep the command line information.
authorGunnar Wrobel <wrobel@temple.(none)>
Wed, 10 Feb 2010 15:11:10 +0000 (16:11 +0100)
committerGunnar Wrobel <wrobel@temple.(none)>
Wed, 10 Feb 2010 15:11:10 +0000 (16:11 +0100)
The command line was originally reported as user information on a PEAR Error object. During the
import to git the PEAR Error was exchanged with an exception and the command line provided as
the second argument to the exception. As strings in this position get reset the information will
be lost. I assume that concatenating the command line to the error might actually be what was
desired.

framework/SpellChecker/lib/Horde/SpellChecker/Aspell.php

index 7788bd6..16051ad 100644 (file)
@@ -43,7 +43,7 @@ class Horde_SpellChecker_Aspell extends Horde_SpellChecker
 
         $process = proc_open($this->_cmd(), $descspec, $pipes);
         if (!is_resource($process)) {
-            throw new Horde_Exception('Spellcheck failed. Command line: ', $this->_cmd());
+            throw new Horde_Exception('Spellcheck failed. Command line: ' . $this->_cmd());
         }
 
         $charset = Horde_Nls::getCharset();
@@ -79,7 +79,7 @@ class Horde_SpellChecker_Aspell extends Horde_SpellChecker
             if ($this->_encoding) {
                 $err = Horde_String::convertCharset($err, $this->_encoding, $charset);
             }
-            throw new Horde_Exception('Spellcheck failed. Command line: ', $this->_cmd());
+            throw new Horde_Exception('Spellcheck failed. Command line: ' . $this->_cmd());
         }
 
         if ($this->_encoding) {