matecat / xliff-parser

src/Constants/XliffTags.php

Spacing +1 added lines, -1 removed lines

@@ -7,5 +7,5 @@
     /**
      * List of allowed Xliff tags
      */
-    public static $tags = [ 'g', 'x', 'bx', 'ex', 'bpt', 'ept', 'ph', 'pc', 'ec', 'sc', 'it', 'mrk' ];
+    public static $tags = ['g', 'x', 'bx', 'ex', 'bpt', 'ept', 'ph', 'pc', 'ec', 'sc', 'it', 'mrk'];
 }
\ No newline at end of file

src/Constants/TranslationStatus.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@
             self::STATUS_REBUTTED
     ];
 
-    public static function isReviewedStatus( $status ) {
-        return in_array( $status, TranslationStatus::$REVISION_STATUSES );
+    public static function isReviewedStatus($status) {
+        return in_array($status, TranslationStatus::$REVISION_STATUSES);
     }
 }

src/XliffUtils/XmlParser.php

Spacing +34 added lines, -34 removed lines

@@ -27,65 +27,65 @@ 
      * @throws InvalidXmlException When parsing of XML with schema or callable produces any errors unrelated to the XML parsing itself
      * @throws RuntimeException   When DOM extension is missing
      */
-    public static function parse( $content, $schemaOrCallable = null ) {
-        if ( !extension_loaded( 'dom' ) ) {
-            throw new RuntimeException( 'Extension DOM is required.' );
+    public static function parse($content, $schemaOrCallable = null) {
+        if (!extension_loaded('dom')) {
+            throw new RuntimeException('Extension DOM is required.');
         }
 
-        $internalErrors  = libxml_use_internal_errors( true );
-        $disableEntities = libxml_disable_entity_loader( true );
+        $internalErrors  = libxml_use_internal_errors(true);
+        $disableEntities = libxml_disable_entity_loader(true);
         libxml_clear_errors();
 
-        $dom                  = new DOMDocument( '1.0', 'UTF-8' );
+        $dom                  = new DOMDocument('1.0', 'UTF-8');
         $dom->validateOnParse = true;
-        if ( !$dom->loadXML( $content, LIBXML_NONET | ( defined( 'LIBXML_COMPACT' ) ? LIBXML_COMPACT : 0 ) ) ) {
-            libxml_disable_entity_loader( $disableEntities );
+        if (!$dom->loadXML($content, LIBXML_NONET | (defined('LIBXML_COMPACT') ? LIBXML_COMPACT : 0))) {
+            libxml_disable_entity_loader($disableEntities);
 
-            throw new XmlParsingException( implode( "\n", static::getXmlErrors( $internalErrors ) ) );
+            throw new XmlParsingException(implode("\n", static::getXmlErrors($internalErrors)));
         }
 
         $dom->normalizeDocument();
 
-        libxml_use_internal_errors( $internalErrors );
-        libxml_disable_entity_loader( $disableEntities );
+        libxml_use_internal_errors($internalErrors);
+        libxml_disable_entity_loader($disableEntities);
 
-        foreach ( $dom->childNodes as $child ) {
-            if ( XML_DOCUMENT_TYPE_NODE === $child->nodeType ) {
-                throw new XmlParsingException( 'Document types are not allowed.' );
+        foreach ($dom->childNodes as $child) {
+            if (XML_DOCUMENT_TYPE_NODE === $child->nodeType) {
+                throw new XmlParsingException('Document types are not allowed.');
             }
         }
 
-        if ( null !== $schemaOrCallable ) {
-            $internalErrors = libxml_use_internal_errors( true );
+        if (null !== $schemaOrCallable) {
+            $internalErrors = libxml_use_internal_errors(true);
             libxml_clear_errors();
 
             $e = null;
-            if ( is_callable( $schemaOrCallable ) ) {
+            if (is_callable($schemaOrCallable)) {
                 try {
-                    $valid = call_user_func( $schemaOrCallable, $dom, $internalErrors );
-                } catch ( Exception $e ) {
+                    $valid = call_user_func($schemaOrCallable, $dom, $internalErrors);
+                } catch (Exception $e) {
                     $valid = false;
                 }
-            } elseif ( !is_array( $schemaOrCallable ) && is_file( (string)$schemaOrCallable ) ) {
-                $schemaSource = file_get_contents( (string)$schemaOrCallable );
-                $valid        = @$dom->schemaValidateSource( $schemaSource );
+            } elseif (!is_array($schemaOrCallable) && is_file((string)$schemaOrCallable)) {
+                $schemaSource = file_get_contents((string)$schemaOrCallable);
+                $valid        = @$dom->schemaValidateSource($schemaSource);
             } else {
-                libxml_use_internal_errors( $internalErrors );
+                libxml_use_internal_errors($internalErrors);
 
-                throw new XmlParsingException( 'The schemaOrCallable argument has to be a valid path to XSD file or callable.' );
+                throw new XmlParsingException('The schemaOrCallable argument has to be a valid path to XSD file or callable.');
             }
 
-            if ( !$valid ) {
-                $messages = static::getXmlErrors( $internalErrors );
-                if ( empty( $messages ) ) {
-                    throw new InvalidXmlException( 'The XML is not valid.', 0, $e );
+            if (!$valid) {
+                $messages = static::getXmlErrors($internalErrors);
+                if (empty($messages)) {
+                    throw new InvalidXmlException('The XML is not valid.', 0, $e);
                 }
-                throw new XmlParsingException( implode( "\n", $messages ), 0, $e );
+                throw new XmlParsingException(implode("\n", $messages), 0, $e);
             }
         }
 
         libxml_clear_errors();
-        libxml_use_internal_errors( $internalErrors );
+        libxml_use_internal_errors($internalErrors);
 
         return $dom;
     }
@@ -95,14 +95,14 @@ 
      *
      * @return array
      */
-    private static function getXmlErrors( $internalErrors ) {
+    private static function getXmlErrors($internalErrors) {
         $errors = [];
-        foreach ( libxml_get_errors() as $error ) {
+        foreach (libxml_get_errors() as $error) {
             $errors[] = sprintf(
                     '[%s %s] %s (in %s - line %d, column %d)',
                     LIBXML_ERR_WARNING == $error->level ? 'WARNING' : 'ERROR',
                     $error->code,
-                    trim( $error->message ),
+                    trim($error->message),
                     $error->file ?: 'n/a',
                     $error->line,
                     $error->column
@@ -110,7 +110,7 @@ 
         }
 
         libxml_clear_errors();
-        libxml_use_internal_errors( $internalErrors );
+        libxml_use_internal_errors($internalErrors);
 
         return $errors;
     }

src/XliffReplacer/AbstractXliffReplacer.php

Spacing +52 added lines, -52 removed lines

@@ -8,28 +8,28 @@ 
 abstract class AbstractXliffReplacer {
     protected $originalFP;
 
-    protected $tuTagName;                 // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
-    protected $inTU                = false;  // flag to check whether we are in a <trans-unit>
-    protected $inTarget            = false;  // flag to check whether we are in a <target>, to ignore everything
-    protected $isEmpty             = false;  // flag to check whether we are in an empty tag (<tag/>)
-    protected $targetWasWritten    = false;  // flag to check is <target> was written in the current unit
-    protected $segmentPositionInTu = -1;  // the current position of segment in the current <unit> (forXliff v 2.*)
-
-    protected $CDATABuffer    = "";       // buffer for special tag
-    protected $bufferIsActive = false;    // buffer for special tag
-
-    protected $offset              = 0;         // offset for SAX pointer
-    protected $outputFP;                  // output stream pointer
-    protected $currentBuffer;             // the current piece of text it's been parsed
-    protected $len;                       // length of the currentBuffer
-    protected $segments;                  // array of translations
+    protected $tuTagName; // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
+    protected $inTU                = false; // flag to check whether we are in a <trans-unit>
+    protected $inTarget            = false; // flag to check whether we are in a <target>, to ignore everything
+    protected $isEmpty             = false; // flag to check whether we are in an empty tag (<tag/>)
+    protected $targetWasWritten    = false; // flag to check is <target> was written in the current unit
+    protected $segmentPositionInTu = -1; // the current position of segment in the current <unit> (forXliff v 2.*)
+
+    protected $CDATABuffer    = ""; // buffer for special tag
+    protected $bufferIsActive = false; // buffer for special tag
+
+    protected $offset = 0; // offset for SAX pointer
+    protected $outputFP; // output stream pointer
+    protected $currentBuffer; // the current piece of text it's been parsed
+    protected $len; // length of the currentBuffer
+    protected $segments; // array of translations
     protected $lastTransUnit       = [];
-    protected $currentTransUnitId;        // id of current <trans-unit>
+    protected $currentTransUnitId; // id of current <trans-unit>
     protected $currentTransUnitTranslate; // 'translate' attribute of current <trans-unit>
-    protected $currentSegmentArray = [];  // id of current <segment> (forXliff v 2.*)
-    protected $unitContainsMda     = false;   // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
-    protected $hasWrittenCounts    = false;  // check if <unit> already wrote segment counts (forXliff v 2.*)
-    protected $sourceAttributes    = [];     // current <source> attributes (needed when handling xliff files without <target>)
+    protected $currentSegmentArray = []; // id of current <segment> (forXliff v 2.*)
+    protected $unitContainsMda     = false; // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
+    protected $hasWrittenCounts    = false; // check if <unit> already wrote segment counts (forXliff v 2.*)
+    protected $sourceAttributes    = []; // current <source> attributes (needed when handling xliff files without <target>)
 
     protected $targetLang;
 
@@ -76,8 +76,8 @@ 
             XliffReplacerCallbackInterface $callback = null
     ) {
         self::$INTERNAL_TAG_PLACEHOLDER = $this->getInternalTagPlaceholder();
-        $this->createOutputFileIfDoesNotExist( $outputFilePath );
-        $this->setFileDescriptors( $originalXliffPath, $outputFilePath );
+        $this->createOutputFileIfDoesNotExist($outputFilePath);
+        $this->setFileDescriptors($originalXliffPath, $outputFilePath);
         $this->xliffVersion = $xliffVersion;
         $this->setTuTagName();
         $this->segments       = $segments;
@@ -95,19 +95,19 @@ 
         return "§" .
                 substr(
                         str_replace(
-                                [ '+', '/' ],
+                                ['+', '/'],
                                 '',
-                                base64_encode( openssl_random_pseudo_bytes( 10, $_crypto_strong ) )
+                                base64_encode(openssl_random_pseudo_bytes(10, $_crypto_strong))
                         ),
                         0,
                         4
                 );
     }
 
-    private function createOutputFileIfDoesNotExist( $outputFilePath ) {
+    private function createOutputFileIfDoesNotExist($outputFilePath) {
         // create output file
-        if ( !file_exists( $outputFilePath ) ) {
-            touch( $outputFilePath );
+        if (!file_exists($outputFilePath)) {
+            touch($outputFilePath);
         }
     }
 
@@ -115,13 +115,13 @@ 
      * @param $originalXliffPath
      * @param $outputFilePath
      */
-    private function setFileDescriptors( $originalXliffPath, $outputFilePath ) {
-        $this->outputFP = fopen( $outputFilePath, 'w+' );
+    private function setFileDescriptors($originalXliffPath, $outputFilePath) {
+        $this->outputFP = fopen($outputFilePath, 'w+');
 
         $streamArgs = null;
 
-        if ( !( $this->originalFP = fopen( $originalXliffPath, "r", false, stream_context_create( $streamArgs ) ) ) ) {
-            throw new RuntimeException( "could not open XML input" );
+        if (!($this->originalFP = fopen($originalXliffPath, "r", false, stream_context_create($streamArgs)))) {
+            throw new RuntimeException("could not open XML input");
         }
     }
 
@@ -130,7 +130,7 @@ 
      * <trans-unit> (xliff v1.*) || <unit> (xliff v2.*)
      */
     private function setTuTagName() {
-        $this->tuTagName = ( $this->xliffVersion === 2 ) ? 'unit' : 'trans-unit';
+        $this->tuTagName = ($this->xliffVersion === 2) ? 'unit' : 'trans-unit';
     }
 
     /**
@@ -139,8 +139,8 @@ 
     public function __destruct() {
         //this stream can be closed outside the class
         //to permit multiple concurrent downloads, so suppress warnings
-        @fclose( $this->originalFP );
-        fclose( $this->outputFP );
+        @fclose($this->originalFP);
+        fclose($this->outputFP);
     }
 
     abstract public function replaceTranslation();
@@ -151,11 +151,11 @@ 
      * @return resource
      */
     protected function initSaxParser() {
-        $xmlSaxParser = xml_parser_create( 'UTF-8' );
-        xml_set_object( $xmlSaxParser, $this );
-        xml_parser_set_option( $xmlSaxParser, XML_OPTION_CASE_FOLDING, false );
-        xml_set_element_handler( $xmlSaxParser, 'tagOpen', 'tagClose' );
-        xml_set_character_data_handler( $xmlSaxParser, 'characterData' );
+        $xmlSaxParser = xml_parser_create('UTF-8');
+        xml_set_object($xmlSaxParser, $this);
+        xml_parser_set_option($xmlSaxParser, XML_OPTION_CASE_FOLDING, false);
+        xml_set_element_handler($xmlSaxParser, 'tagOpen', 'tagClose');
+        xml_set_character_data_handler($xmlSaxParser, 'characterData');
 
         return $xmlSaxParser;
     }
@@ -163,8 +163,8 @@ 
     /**
      * @param resource $xmlSaxParser
      */
-    protected function closeSaxParser( $xmlSaxParser ) {
-        xml_parser_free( $xmlSaxParser );
+    protected function closeSaxParser($xmlSaxParser) {
+        xml_parser_free($xmlSaxParser);
     }
 
     /**
@@ -174,7 +174,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagOpen( $parser, $name, $attr );
+    abstract protected function tagOpen($parser, $name, $attr);
 
     /**
      * @param $parser
@@ -182,7 +182,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagClose( $parser, $name );
+    abstract protected function tagClose($parser, $name);
 
     /**
      * @param $parser
@@ -190,7 +190,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function characterData( $parser, $data );
+    abstract protected function characterData($parser, $data);
 
     /**
      * postprocess escaped data and write to disk
@@ -199,18 +199,18 @@ 
      * @param string   $data
      * @param bool     $treatAsCDATA
      */
-    protected function postProcAndFlush( $fp, $data, $treatAsCDATA = false ) {
+    protected function postProcAndFlush($fp, $data, $treatAsCDATA = false) {
         //postprocess string
-        $data = preg_replace( "/" . self::$INTERNAL_TAG_PLACEHOLDER . '(.*?)' . self::$INTERNAL_TAG_PLACEHOLDER . "/", '&$1;', $data );
-        $data = str_replace( '&nbsp;', ' ', $data );
-        if ( !$treatAsCDATA ) {
+        $data = preg_replace("/" . self::$INTERNAL_TAG_PLACEHOLDER . '(.*?)' . self::$INTERNAL_TAG_PLACEHOLDER . "/", '&$1;', $data);
+        $data = str_replace('&nbsp;', ' ', $data);
+        if (!$treatAsCDATA) {
             //unix2dos
-            $data = str_replace( "\r\n", "\r", $data );
-            $data = str_replace( "\n", "\r", $data );
-            $data = str_replace( "\r", "\r\n", $data );
+            $data = str_replace("\r\n", "\r", $data);
+            $data = str_replace("\n", "\r", $data);
+            $data = str_replace("\r", "\r\n", $data);
         }
 
         //flush to disk
-        fwrite( $fp, $data );
+        fwrite($fp, $data);
     }
 }

src/Utils/Files.php

Spacing +28 added lines, -28 removed lines

@@ -18,15 +18,15 @@ 
      *
      * @return array|mixed
      */
-    public static function pathInfo( $path, $options = 15 ) {
-        $rawPath = explode( DIRECTORY_SEPARATOR, $path );
+    public static function pathInfo($path, $options = 15) {
+        $rawPath = explode(DIRECTORY_SEPARATOR, $path);
 
-        $basename = array_pop( $rawPath );
-        $dirname  = implode( DIRECTORY_SEPARATOR, $rawPath );
+        $basename = array_pop($rawPath);
+        $dirname  = implode(DIRECTORY_SEPARATOR, $rawPath);
 
-        $explodedFileName = explode( ".", $basename );
-        $extension        = strtolower( array_pop( $explodedFileName ) );
-        $filename         = implode( ".", $explodedFileName );
+        $explodedFileName = explode(".", $basename);
+        $extension        = strtolower(array_pop($explodedFileName));
+        $filename         = implode(".", $explodedFileName);
 
         $returnArray = [];
 
@@ -39,17 +39,17 @@ 
 
         // foreach flag, add in $return_array the corresponding field,
         // obtained by variable name correspondence
-        foreach ( $flagMap as $field => $i ) {
+        foreach ($flagMap as $field => $i) {
             //binary AND
-            if ( ( $options & $i ) > 0 ) {
+            if (($options & $i) > 0) {
                 //variable substitution: $field can be one between 'dirname', 'basename', 'extension', 'filename'
                 // $$field gets the value of the variable named $field
-                $returnArray[ $field ] = $$field;
+                $returnArray[$field] = $$field;
             }
         }
 
-        if ( count( $returnArray ) == 1 ) {
-            $returnArray = array_pop( $returnArray );
+        if (count($returnArray) == 1) {
+            $returnArray = array_pop($returnArray);
         }
 
         return $returnArray;
@@ -60,14 +60,14 @@ 
      *
      * @return false|string
      */
-    public static function getExtension( $path ) {
-        $pathInfo = self::pathInfo( $path );
+    public static function getExtension($path) {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return false;
         }
 
-        return strtolower( $pathInfo[ 'extension' ] );
+        return strtolower($pathInfo['extension']);
     }
 
     /**
@@ -75,14 +75,14 @@ 
      *
      * @return bool
      */
-    public static function isXliff( $path ) {
-        $extension = self::getExtension( $path );
+    public static function isXliff($path) {
+        $extension = self::getExtension($path);
 
-        if ( !$extension ) {
+        if (!$extension) {
             return false;
         }
 
-        switch ( $extension ) {
+        switch ($extension) {
             case 'xliff':
             case 'sdlxliff':
             case 'tmx':
@@ -98,14 +98,14 @@ 
      *
      * @return bool|string
      */
-    public static function getMemoryFileType( $path ) {
-        $pathInfo = self::pathInfo( $path );
+    public static function getMemoryFileType($path) {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return false;
         }
 
-        switch ( strtolower( $pathInfo[ 'extension' ] ) ) {
+        switch (strtolower($pathInfo['extension'])) {
             case 'tmx':
                 return 'tmx';
             case 'g':
@@ -120,8 +120,8 @@ 
      *
      * @return bool
      */
-    public static function isTMXFile( $path ) {
-        return self::getMemoryFileType( $path ) === 'tmx';
+    public static function isTMXFile($path) {
+        return self::getMemoryFileType($path) === 'tmx';
     }
 
     /**
@@ -129,7 +129,7 @@ 
      *
      * @return bool
      */
-    public static function isGlossaryFile( $path ) {
-        return self::getMemoryFileType( $path ) === 'glossary';
+    public static function isGlossaryFile($path) {
+        return self::getMemoryFileType($path) === 'glossary';
     }
 }

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;
                 }