AC-9843:: i18n:collect-phrases breaks the translations integrity

Author: Pooja.Manral

setup/src/Magento/Setup/Module/I18n/Parser/Adapter/Js.php

@@ -15,18 +15,18 @@
 class Js extends AbstractAdapter
 {
     /**
-     * @var \Magento\Framework\Filesystem\Driver\File
+     * @var File
      */
-    protected $_filesystem;
+    private $filesystem;
 
     /**
      * Adapter construct
      *
-     * @param \Magento\Framework\Filesystem\Driver\File $fileSystem
+     * @param File $filesystem
      */
-    public function __construct()
+    public function __construct(File $filesystem)
     {
-        $this->_filesystem = new \Magento\Framework\Filesystem\Driver\File();
+        $this->filesystem = $filesystem;
     }
     /**
      * Covers
@@ -47,10 +47,11 @@ public function __construct()
 
     /**
      * @inheritdoc
+     * @throws FileSystemException
      */
     protected function _parse()
     {
-        $fileHandle = $this->_filesystem->fileOpen($this->_file, 'r');
+        $fileHandle = $this->filesystem->fileOpen($this->_file, 'r');
         $lineNumber = 0;
         try {
             while (($line = $this->fileReadLine($fileHandle, 0)) !== false) {
@@ -74,12 +75,12 @@ protected function _parse()
                 }
             }
         } catch (\Exception $e) {
-            $this->_filesystem->fileClose($fileHandle);
+            $this->filesystem->fileClose($fileHandle);
             throw new FileSystemException(
                 new \Magento\Framework\Phrase('Stream get line failed %1', [$e->getMessage()])
             );
         }
-        $this->_filesystem->fileClose($fileHandle);
+        $this->filesystem->fileClose($fileHandle);
     }
 
     /**

setup/src/Magento/Setup/Module/I18n/ServiceLocator.php

@@ -6,6 +6,7 @@
 namespace Magento\Setup\Module\I18n;
 
 use Magento\Framework\Component\ComponentRegistrar;
+use Magento\Framework\Filesystem\Driver\File;
 
 /**
  *  Service Locator (instead DI container)
@@ -52,10 +53,11 @@ public static function getDictionaryGenerator()
             $filesCollector = new FilesCollector();
 
             $phraseCollector = new Parser\Adapter\Php\Tokenizer\PhraseCollector(new Parser\Adapter\Php\Tokenizer());
+            $fileSystem = new File;
             $adapters = [
                 'php' => new Parser\Adapter\Php($phraseCollector),
                 'html' => new Parser\Adapter\Html(),
-                'js' => new Parser\Adapter\Js(),
+                'js' => new Parser\Adapter\Js($fileSystem),
                 'xml' => new Parser\Adapter\Xml(),
             ];