The \Magento\Framework\Filesystem\Directory\Read::readFile() method should firsly check whether the $path argument is empty string and raise an exception with a proper and clear message in this case instead of the current cryptic message

dmitry_fedyuk · November 20, 2015, 10:17am

magento/magento2/blob/2.0.0/lib/internal/Magento/Framework/Filesystem/Directory/Read.php#L195-L209


      
          /**
           * Retrieve file contents from given path
           *
           * @param string $path
           * @param string|null $flag
           * @param resource|null $context
           * @return string
           * @throws FileSystemException
           */
          public function readFile($path, $flag = null, $context = null)
          {
              $absolutePath = $this->driver->getAbsolutePath($this->path, $path);
              return $this->driver->fileGetContents($absolutePath, $flag, $context);
          
          }

For now, the message looks like:

Warning!file_get_contents(C:/work/mage2.pro/store/): failed to open stream: No such file or directory

I propose to make the message more clear, for example:

\Magento\Framework\Filesystem\Directory\Read::readFile: Invalid parameter: the path argument should not be empty.

dmitry_fedyuk · November 23, 2015, 10:08pm

See also: Magento 2 loses the problem Less file name in a «Compilation from source / Cannot read contents from file» error report