matecat / xliff-parser

src/XliffUtils/CheckXliffProprietaryPipeline.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
      *
      * @param $tmp
      */
-    public function __construct( $tmp ) {
+    public function __construct($tmp) {
         $this->tmp   = $tmp;
         $this->steps = [];
     }
@@ -28,7 +28,7 @@ 
     /**
      * @param CheckInterface $step
      */
-    public function addCheck( CheckInterface $step ) {
+    public function addCheck(CheckInterface $step) {
         $this->steps[] = $step;
     }
 
@@ -39,13 +39,13 @@ 
         $fileType = [];
 
         /** @var CheckInterface $step */
-        foreach ( $this->steps as $step ) {
-            if ( null !== $step->check( $this->tmp ) ) {
-                $fileType = $step->check( $this->tmp );
+        foreach ($this->steps as $step) {
+            if (null !== $step->check($this->tmp)) {
+                $fileType = $step->check($this->tmp);
             }
         }
 
-        if ( !empty( $fileType ) && $this->isValid( $fileType ) ) {
+        if (!empty($fileType) && $this->isValid($fileType)) {
             return $fileType;
         }
 
@@ -62,7 +62,7 @@ 
      *
      * @return bool
      */
-    private function isValid( $fileType ) {
+    private function isValid($fileType) {
         $mandatoryKeys = [
                 'proprietary',
                 'proprietary_name',
@@ -70,6 +70,6 @@ 
                 'converter_version',
         ];
 
-        return array_keys( $fileType ) === $mandatoryKeys;
+        return array_keys($fileType) === $mandatoryKeys;
     }
 }

src/XliffUtils/XliffProprietaryDetect.php

Spacing +58 added lines, -58 removed lines

@@ -22,11 +22,11 @@ 
      *
      * @return array
      */
-    public static function getInfoFromXliffContent( $xliffContent ) {
+    public static function getInfoFromXliffContent($xliffContent) {
         self::reset();
-        $tmp = self::getFirst1024CharsFromXliff( $xliffContent, null );
+        $tmp = self::getFirst1024CharsFromXliff($xliffContent, null);
 
-        return self::getInfoFromTmp( $tmp );
+        return self::getInfoFromTmp($tmp);
     }
 
     /**
@@ -34,12 +34,12 @@ 
      *
      * @return array
      */
-    public static function getInfo( $fullPathToFile ) {
+    public static function getInfo($fullPathToFile) {
         self::reset();
-        $tmp                      = self::getFirst1024CharsFromXliff( null, $fullPathToFile );
-        self::$fileType[ 'info' ] = Files::pathInfo( $fullPathToFile );
+        $tmp                      = self::getFirst1024CharsFromXliff(null, $fullPathToFile);
+        self::$fileType['info'] = Files::pathInfo($fullPathToFile);
 
-        return self::getInfoFromTmp( $tmp );
+        return self::getInfoFromTmp($tmp);
     }
 
     /**
@@ -47,21 +47,21 @@ 
      *
      * @return array
      */
-    private static function getInfoFromTmp( $tmp ) {
+    private static function getInfoFromTmp($tmp) {
         try {
-            self::checkVersion( $tmp );
-        } catch ( Exception $ignore ) {
+            self::checkVersion($tmp);
+        } catch (Exception $ignore) {
             // do nothing
             // self::$fileType[ 'version' ] is left empty
         }
 
         // run CheckXliffProprietaryPipeline
-        $pipeline = self::runPipeline( $tmp );
+        $pipeline = self::runPipeline($tmp);
 
-        self::$fileType[ 'proprietary' ]            = $pipeline[ 'proprietary' ];
-        self::$fileType[ 'proprietary_name' ]       = $pipeline[ 'proprietary_name' ];
-        self::$fileType[ 'proprietary_short_name' ] = $pipeline[ 'proprietary_short_name' ];
-        self::$fileType[ 'converter_version' ]      = $pipeline[ 'converter_version' ];
+        self::$fileType['proprietary']            = $pipeline['proprietary'];
+        self::$fileType['proprietary_name']       = $pipeline['proprietary_name'];
+        self::$fileType['proprietary_short_name'] = $pipeline['proprietary_short_name'];
+        self::$fileType['converter_version']      = $pipeline['converter_version'];
 
         return self::$fileType;
     }
@@ -71,12 +71,12 @@ 
      *
      * @return array
      */
-    private static function runPipeline( $tmp ) {
-        $pipeline = new CheckXliffProprietaryPipeline( $tmp );
-        $pipeline->addCheck( new CheckSDL() );
-        $pipeline->addCheck( new CheckGlobalSight() );
-        $pipeline->addCheck( new CheckMateCATConverter() );
-        $pipeline->addCheck( new CheckXliffVersion2() );
+    private static function runPipeline($tmp) {
+        $pipeline = new CheckXliffProprietaryPipeline($tmp);
+        $pipeline->addCheck(new CheckSDL());
+        $pipeline->addCheck(new CheckGlobalSight());
+        $pipeline->addCheck(new CheckMateCATConverter());
+        $pipeline->addCheck(new CheckXliffVersion2());
 
         return $pipeline->run();
     }
@@ -99,29 +99,29 @@ 
      *
      * @return array|false
      */
-    private static function getFirst1024CharsFromXliff( $stringData = null, $fullPathToFile = null ) {
-        if ( !empty( $stringData ) && empty( $fullPathToFile ) ) {
+    private static function getFirst1024CharsFromXliff($stringData = null, $fullPathToFile = null) {
+        if (!empty($stringData) && empty($fullPathToFile)) {
             $pathInfo   = [];
-            $stringData = substr( $stringData, 0, 1024 );
-        } elseif ( empty( $stringData ) && !empty( $fullPathToFile ) ) {
-            $pathInfo = Files::pathInfo( $fullPathToFile );
+            $stringData = substr($stringData, 0, 1024);
+        } elseif (empty($stringData) && !empty($fullPathToFile)) {
+            $pathInfo = Files::pathInfo($fullPathToFile);
 
-            if ( is_file( $fullPathToFile ) ) {
-                $file_pointer = fopen( "$fullPathToFile", 'r' );
+            if (is_file($fullPathToFile)) {
+                $file_pointer = fopen("$fullPathToFile", 'r');
                 // Checking Requirements (By specs, I know that xliff version is in the first 1KB)
-                $stringData = fread( $file_pointer, 1024 );
-                fclose( $file_pointer );
+                $stringData = fread($file_pointer, 1024);
+                fclose($file_pointer);
             }
-        } elseif ( !empty( $stringData ) && !empty( $fullPathToFile ) ) {
-            $pathInfo = Files::pathInfo( $fullPathToFile );
+        } elseif (!empty($stringData) && !empty($fullPathToFile)) {
+            $pathInfo = Files::pathInfo($fullPathToFile);
         }
 
-        if ( !empty( $pathInfo ) && !Files::isXliff( $fullPathToFile ) ) {
+        if (!empty($pathInfo) && !Files::isXliff($fullPathToFile)) {
             return false;
         }
 
-        if ( !empty( $stringData ) ) {
-            return [ $stringData ];
+        if (!empty($stringData)) {
+            return [$stringData];
         }
 
         return false;
@@ -133,9 +133,9 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    protected static function checkVersion( $tmp ) {
-        if ( isset( $tmp[ 0 ] ) ) {
-            self::$fileType[ 'version' ] = XliffVersionDetector::detect( $tmp[ 0 ] );
+    protected static function checkVersion($tmp) {
+        if (isset($tmp[0])) {
+            self::$fileType['version'] = XliffVersionDetector::detect($tmp[0]);
         }
     }
 
@@ -146,20 +146,20 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function getInfoByStringData( $stringData ) {
+    public static function getInfoByStringData($stringData) {
         self::reset();
 
-        $tmp                      = self::getFirst1024CharsFromXliff( $stringData );
-        self::$fileType[ 'info' ] = [];
-        self::checkVersion( $tmp );
+        $tmp                      = self::getFirst1024CharsFromXliff($stringData);
+        self::$fileType['info'] = [];
+        self::checkVersion($tmp);
 
         // run CheckXliffProprietaryPipeline
-        $pipeline = self::runPipeline( $tmp );
+        $pipeline = self::runPipeline($tmp);
 
-        self::$fileType[ 'proprietary' ]            = $pipeline[ 'proprietary' ];
-        self::$fileType[ 'proprietary_name' ]       = $pipeline[ 'proprietary_name' ];
-        self::$fileType[ 'proprietary_short_name' ] = $pipeline[ 'proprietary_short_name' ];
-        self::$fileType[ 'converter_version' ]      = $pipeline[ 'converter_version' ];
+        self::$fileType['proprietary']            = $pipeline['proprietary'];
+        self::$fileType['proprietary_name']       = $pipeline['proprietary_name'];
+        self::$fileType['proprietary_short_name'] = $pipeline['proprietary_short_name'];
+        self::$fileType['converter_version']      = $pipeline['converter_version'];
 
         return self::$fileType;
     }
@@ -171,21 +171,21 @@ 
      *
      * @return bool|int
      */
-    public static function fileMustBeConverted( $fullPath, $enforceOnXliff = false, $filterAddress = null ) {
+    public static function fileMustBeConverted($fullPath, $enforceOnXliff = false, $filterAddress = null) {
         $convert = true;
 
-        $fileType       = self::getInfo( $fullPath );
-        $memoryFileType = Files::getMemoryFileType( $fullPath );
+        $fileType       = self::getInfo($fullPath);
+        $memoryFileType = Files::getMemoryFileType($fullPath);
 
-        if ( Files::isXliff( $fullPath ) || $memoryFileType ) {
-            if ( !empty( $filterAddress ) ) {
+        if (Files::isXliff($fullPath) || $memoryFileType) {
+            if (!empty($filterAddress)) {
 
                 //conversion enforce
-                if ( !$enforceOnXliff ) {
+                if (!$enforceOnXliff) {
 
                     //if file is not proprietary AND Enforce is disabled
                     //we take it as is
-                    if ( !$fileType[ 'proprietary' ] || $memoryFileType ) {
+                    if (!$fileType['proprietary'] || $memoryFileType) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
@@ -193,16 +193,16 @@ 
                     //if conversion enforce is active
                     //we force all xliff files but not files produced by SDL Studio because we can handle them
                     if (
-                            $fileType[ 'proprietary_short_name' ] == 'matecat_converter'
-                            || $fileType[ 'proprietary_short_name' ] == 'trados'
-                            || $fileType[ 'proprietary_short_name' ] == 'xliff_v2'
+                            $fileType['proprietary_short_name'] == 'matecat_converter'
+                            || $fileType['proprietary_short_name'] == 'trados'
+                            || $fileType['proprietary_short_name'] == 'xliff_v2'
                             || $memoryFileType
                     ) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
                 }
-            } elseif ( $fileType[ 'proprietary' ] ) {
+            } elseif ($fileType['proprietary']) {
 
                 /**
                  * Application misconfiguration.

src/XliffUtils/CheckPipeline/CheckGlobalSight.php

Spacing +7 added lines, -7 removed lines

@@ -8,15 +8,15 @@
      *
      * @return array|null
      */
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            if ( stripos( $tmp[ 0 ], 'globalsight' ) !== false ) {
-                $fileType[ 'proprietary' ]            = true;
-                $fileType[ 'proprietary_name' ]       = 'GlobalSight Download File';
-                $fileType[ 'proprietary_short_name' ] = 'globalsight';
-                $fileType[ 'converter_version' ]      = 'legacy';
+        if (isset($tmp[0])) {
+            if (stripos($tmp[0], 'globalsight') !== false) {
+                $fileType['proprietary']            = true;
+                $fileType['proprietary_name']       = 'GlobalSight Download File';
+                $fileType['proprietary_short_name'] = 'globalsight';
+                $fileType['converter_version']      = 'legacy';
 
                 return $fileType;
             }

src/XliffUtils/CheckPipeline/CheckXliffVersion2.php

Spacing +12 added lines, -12 removed lines

@@ -8,22 +8,22 @@
      *
      * @return array|void|null
      */
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            preg_match( '|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr( $tmp[ 0 ], 0, 1000 ), $versionMatches );
-            preg_match( '|<xliff.*?\sxmlns\s?=\s?["\']urn:oasis:names:tc:xliff:document:(.*?)["\']|si', substr( $tmp[ 0 ], 0, 1000 ), $xmlnsMatches );
+        if (isset($tmp[0])) {
+            preg_match('|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr($tmp[0], 0, 1000), $versionMatches);
+            preg_match('|<xliff.*?\sxmlns\s?=\s?["\']urn:oasis:names:tc:xliff:document:(.*?)["\']|si', substr($tmp[0], 0, 1000), $xmlnsMatches);
 
-            if ( !empty( $versionMatches ) && !empty( $xmlnsMatches ) ) {
-                $version = $versionMatches[ 1 ];
-                $xmlns   = $xmlnsMatches[ 1 ];
+            if (!empty($versionMatches) && !empty($xmlnsMatches)) {
+                $version = $versionMatches[1];
+                $xmlns   = $xmlnsMatches[1];
 
-                if ( $version === $xmlns && $version >= 2 ) {
-                    $fileType[ 'proprietary' ]            = false;
-                    $fileType[ 'proprietary_name' ]       = 'Xliff v' . $version . ' File';
-                    $fileType[ 'proprietary_short_name' ] = 'xliff_v2';
-                    $fileType[ 'converter_version' ]      = '2.0';
+                if ($version === $xmlns && $version >= 2) {
+                    $fileType['proprietary']            = false;
+                    $fileType['proprietary_name']       = 'Xliff v' . $version . ' File';
+                    $fileType['proprietary_short_name'] = 'xliff_v2';
+                    $fileType['converter_version']      = '2.0';
 
                     return $fileType;
                 }

src/XliffUtils/CheckPipeline/CheckSDL.php

Spacing +7 added lines, -7 removed lines

@@ -3,16 +3,16 @@
 namespace Matecat\XliffParser\XliffUtils\CheckPipeline;
 
 class CheckSDL implements CheckInterface {
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            if ( stripos( $tmp[ 0 ], 'sdl:version' ) !== false ) {
+        if (isset($tmp[0])) {
+            if (stripos($tmp[0], 'sdl:version') !== false) {
                 //little trick, we consider not proprietary Sdlxliff files because we can handle them
-                $fileType[ 'proprietary' ]            = false;
-                $fileType[ 'proprietary_name' ]       = 'SDL Studio ';
-                $fileType[ 'proprietary_short_name' ] = 'trados';
-                $fileType[ 'converter_version' ]      = 'legacy';
+                $fileType['proprietary']            = false;
+                $fileType['proprietary_name']       = 'SDL Studio ';
+                $fileType['proprietary_short_name'] = 'trados';
+                $fileType['converter_version']      = 'legacy';
 
                 return $fileType;
             }

src/XliffUtils/CheckPipeline/CheckMateCATConverter.php

Spacing +10 added lines, -10 removed lines

@@ -8,20 +8,20 @@
      *
      * @return array|null
      */
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            preg_match( '#tool-id\s*=\s*"matecat-converter(\s+([^"]+))?"#i', $tmp[ 0 ], $matches );
-            if ( !empty( $matches ) ) {
-                $fileType[ 'proprietary' ]            = false;
-                $fileType[ 'proprietary_name' ]       = 'MateCAT Converter';
-                $fileType[ 'proprietary_short_name' ] = 'matecat_converter';
-                if ( isset( $matches[ 2 ] ) ) {
-                    $fileType[ 'converter_version' ] = $matches[ 2 ];
+        if (isset($tmp[0])) {
+            preg_match('#tool-id\s*=\s*"matecat-converter(\s+([^"]+))?"#i', $tmp[0], $matches);
+            if (!empty($matches)) {
+                $fileType['proprietary']            = false;
+                $fileType['proprietary_name']       = 'MateCAT Converter';
+                $fileType['proprietary_short_name'] = 'matecat_converter';
+                if (isset($matches[2])) {
+                    $fileType['converter_version'] = $matches[2];
                 } else {
                     // First converter release didn't specify version
-                    $fileType[ 'converter_version' ] = '1.0';
+                    $fileType['converter_version'] = '1.0';
                 }
 
                 return $fileType;

src/XliffUtils/CheckPipeline/CheckInterface.php

Spacing +1 added lines, -1 removed lines

@@ -8,5 +8,5 @@
      *
      * @return array|null
      */
-    public function check( $tmp );
+    public function check($tmp);
 }

src/XliffUtils/XliffVersionDetector.php

Spacing +12 added lines, -12 removed lines

@@ -9,12 +9,12 @@ 
     /**
      * @var array
      */
-    private static $versions_1 = [ '1.0', '1.1', '1.2' ];
+    private static $versions_1 = ['1.0', '1.1', '1.2'];
 
     /**
      * @var array
      */
-    private static $versions_2 = [ '2.0', '2.1' ];
+    private static $versions_2 = ['2.0', '2.1'];
 
     /**
      * @param string $xliffContent
@@ -23,16 +23,16 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function detect( $xliffContent ) {
-        preg_match( '|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr( $xliffContent, 0, 1000 ), $versionMatches );
+    public static function detect($xliffContent) {
+        preg_match('|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr($xliffContent, 0, 1000), $versionMatches);
 
-        if ( empty( $versionMatches ) ) {
-            throw new NotValidFileException( 'This is not a valid xliff file' );
+        if (empty($versionMatches)) {
+            throw new NotValidFileException('This is not a valid xliff file');
         }
 
-        $version = $versionMatches[ 1 ];
+        $version = $versionMatches[1];
 
-        return self::resolveVersion( $version );
+        return self::resolveVersion($version);
     }
 
     /**
@@ -41,15 +41,15 @@ 
      * @return int
      * @throws NotSupportedVersionException
      */
-    private static function resolveVersion( $version ) {
-        if ( in_array( $version, self::$versions_1 ) ) {
+    private static function resolveVersion($version) {
+        if (in_array($version, self::$versions_1)) {
             return 1;
         }
 
-        if ( in_array( $version, self::$versions_2 ) ) {
+        if (in_array($version, self::$versions_2)) {
             return 2;
         }
 
-        throw new NotSupportedVersionException( 'Not supported version' );
+        throw new NotSupportedVersionException('Not supported version');
     }
 }

src/XliffReplacer/XliffReplacerCallbackInterface.php

Spacing +1 added lines, -1 removed lines

@@ -11,5 +11,5 @@
      *
      * @return bool
      */
-    public function thereAreErrors( $segmentId, $segment, $translation, array $dataRefMap = [] );
+    public function thereAreErrors($segmentId, $segment, $translation, array $dataRefMap = []);
 }