Tickets :: Comment on [#6625] imp/lib/SpellChecker/aspell.php test for ecode broken

* Your Email Address
* Spam protection	Enter the letters below: .__ .__ . .. , . [__)[__)\|\/\| \./ \| \| \\| \\| \| \| \__\|
Comment	> I don't actually know any of the PHP developers, so I've tacked a > comment onto one of the bugs I perceived to be applicable (and that > was still allowing comments). If one of you has an in with the folks > at php.net, could you pursue this? > > > > In the meantime, what about replacing... > > > > if (($ecode = proc_close($process)) != 0) { > > > > with something like.. > > > > if (strlen($out) == 0) { > > > > or > > > > if (empty($out)) { > > > > if there's a problem invoking aspell, $out stays empty, but $err gets > filled.. so this seemed like it might work ok?
Attachment
Watch this ticket