Skip to content

Commit

Permalink
MAGETWO-52000: [Github][PR]impossible to see what is wrong with cron …
Browse files Browse the repository at this point in the history
…- unhelpful error message #3189

 - fix unknown variable merge error and path in docblock
  • Loading branch information
monkeysee committed May 15, 2016
1 parent 2a9cebf commit bdaf001
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions setup/src/Magento/Setup/Model/Cron/ReadinessCheck.php
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ class ReadinessCheck
/**#@-*/

/**
* @var DbValidator
* @var \Magento\Setup\Validator\DbValidator
*/
private $dbValidator;

Expand Down Expand Up @@ -138,6 +138,7 @@ public function runReadinessCheck()

// Prepare list of magento specific files and directory paths for updater application to check write
// permissions
$errorMessage = '';
try {
$filePaths = $this->basePackageInfo->getPaths();
$resultJsonRawData[self::KEY_FILE_PATHS][self::KEY_LIST] = $filePaths;
Expand All @@ -146,7 +147,7 @@ public function runReadinessCheck()
$resultJsonRawData[self::KEY_FILE_PATHS][self::KEY_LIST] = [];
$errorLogMessages[] = $errorMessage;
}
$resultJsonRawData[self::KEY_FILE_PATHS][self::KEY_ERROR] = $errorMsg;
$resultJsonRawData[self::KEY_FILE_PATHS][self::KEY_ERROR] = $errorMessage;

// updates timestamp
$write = $this->filesystem->getDirectoryWrite(\Magento\Framework\App\Filesystem\DirectoryList::VAR_DIR);
Expand Down

0 comments on commit bdaf001

Please sign in to comment.