#21 Shell_exec error not printed in reindexing plugin

[Samuele Carpene]

I want to suggest to add more controls to Magmi_ReindexingPlugin class
In the function updateIndexes the row of code
$out = $this->_mdh->exec_cmd($cl,"--reindex $idx", $this->_mdh->getMagentoDir());
can return false meaning that the execution of shell_exec goes wrong.
In this case it would be useful to show a message.. so the user of this plugin can understand that this command has some problems...

So this code can control the shell_exec problems

if($out ===false){//added
$this->log("Reindexing $idx error","error");//added
}//added
else{//added
$this->log($out,"info"); //original code
$tend=microtime(true); //original code
$this->log("done in ".round($tend-$tstart,2). " secs","info"); //original code
} //original code

In the attacments there is the file.

Thank you.

[Samuele Carpene]

I want to inform you that i'm using MAGMI with magento 1.8.1 and there are no problems.
I'm using it with these plugins:

• Magmi Magento Reindexer v1.0.2
• Image attributes processor v1.0.25
• Configurable Item processor v1.3.7

Another thing that i want to know is this... how can contribute to the development of this plugin?