Merge pull request #3840 from murrant/fix-validate-empty-modified

Validate: Also check the output of git before saying there is modified files
This commit is contained in:
Neil Lathwood
2016-07-14 23:37:03 +01:00
committed by GitHub

View File

@ -93,9 +93,9 @@ if($username === 'root') {
$modifiedcmd = 'su '.$config['user'].' -c "'.$modifiedcmd.'"'; $modifiedcmd = 'su '.$config['user'].' -c "'.$modifiedcmd.'"';
} }
exec($modifiedcmd, $cmdoutput, $code); exec($modifiedcmd, $cmdoutput, $code);
if($code !== 0) { if($code !== 0 && !empty($cmdoutput)) {
print_warn("Your local git contains modified files, this could prevent automatic updates.\nModified files:"); print_warn("Your local git contains modified files, this could prevent automatic updates.\nModified files:");
echo(implode("\n", $cmdoutput) . "\n"); echo(' ' . implode("\n ", $cmdoutput) . "\n");
} }
echo "DB Schema: ".$versions['db_schema']."\n"; echo "DB Schema: ".$versions['db_schema']."\n";