matecat / xliff-parser

src/XliffReplacer/AbstractXliffReplacer.php

Spacing +110 added lines, -110 removed lines

@@ -7,26 +7,26 @@ 
 
 abstract class AbstractXliffReplacer {
     protected $originalFP;
-    protected $outputFP;                  // output stream pointer
+    protected $outputFP; // output stream pointer
 
-    protected string $tuTagName;                 // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
-    protected bool   $inTU             = false;  // flag to check whether we are in a <trans-unit>
-    protected bool   $inTarget         = false;  // flag to check whether we are in a <target>, to ignore everything
-    protected bool   $isEmpty          = false;  // flag to check whether we are in an empty tag (<tag/>)
-    protected bool   $targetWasWritten = false;  // flag to check is <target> was written in the current unit
-    protected string $CDATABuffer      = "";       // buffer for special tag
-    protected bool   $bufferIsActive   = false;    // buffer for special tag
+    protected string $tuTagName; // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
+    protected bool   $inTU             = false; // flag to check whether we are in a <trans-unit>
+    protected bool   $inTarget         = false; // flag to check whether we are in a <target>, to ignore everything
+    protected bool   $isEmpty          = false; // flag to check whether we are in an empty tag (<tag/>)
+    protected bool   $targetWasWritten = false; // flag to check is <target> was written in the current unit
+    protected string $CDATABuffer      = ""; // buffer for special tag
+    protected bool   $bufferIsActive   = false; // buffer for special tag
 
-    protected int $offset = 0;         // offset for SAX pointer
+    protected int $offset = 0; // offset for SAX pointer
 
-    protected string  $currentBuffer;             // the current piece of text it's been parsed
-    protected int     $len;                       // length of the currentBuffer
-    protected array   $segments;                  // array of translations
+    protected string  $currentBuffer; // the current piece of text it's been parsed
+    protected int     $len; // length of the currentBuffer
+    protected array   $segments; // array of translations
     protected array   $lastTransUnit                  = [];
     protected int     $segmentInUnitPosition          = 0;
-    protected ?string $currentTransUnitId             = null;        // id of current <trans-unit>
+    protected ?string $currentTransUnitId             = null; // id of current <trans-unit>
     protected ?string $currentTransUnitIsTranslatable = null; // 'translate' attribute of current <trans-unit>
-    protected bool    $hasWrittenCounts               = false;  // check if <unit> already wrote segment counts (forXliff v 2.*)
+    protected bool    $hasWrittenCounts               = false; // check if <unit> already wrote segment counts (forXliff v 2.*)
     protected string  $targetLang;
     protected bool    $sourceInTarget                 = false;
 
@@ -76,10 +76,10 @@ 
             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->tuTagName      = ( $this->xliffVersion === 2 ) ? 'unit' : 'trans-unit';
+        $this->tuTagName      = ($this->xliffVersion === 2) ? 'unit' : 'trans-unit';
         $this->segments       = $segments;
         $this->targetLang     = $trgLang;
         $this->sourceInTarget = $setSourceInTarget;
@@ -89,17 +89,17 @@ 
     }
 
     public function replaceTranslation() {
-        fwrite( $this->outputFP, '<?xml version="1.0" encoding="UTF-8"?>' );
+        fwrite($this->outputFP, '<?xml version="1.0" encoding="UTF-8"?>');
 
         //create Sax parser
         $xmlParser = $this->initSaxParser();
 
-        while ( $this->currentBuffer = fread( $this->originalFP, 4096 ) ) {
+        while ($this->currentBuffer = fread($this->originalFP, 4096)) {
             /*
                preprocess file
              */
             // obfuscate entities because sax automatically does html_entity_decode
-            $temporary_check_buffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+            $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
 
             //avoid cutting entities in half:
             //the last fread could have truncated an entity (say, '&lt;' in '&l'), thus invalidating the escaping
@@ -107,36 +107,36 @@ 
             // 9 is the max length of an entity. So, suppose that the & is at the end of buffer,
             // add 9 Bytes and substitute the entities, if the & is present, and it is not at the end
             //it can't be an entity, exit the loop
-            while ( true ) {
-                $_ampPos = strpos( $temporary_check_buffer, '&' );
+            while (true) {
+                $_ampPos = strpos($temporary_check_buffer, '&');
 
                 //check for real entity or escape it to safely exit from the loop!!!
-                if ( $_ampPos === false || strlen( substr( $temporary_check_buffer, $_ampPos ) ) > 9 ) {
+                if ($_ampPos === false || strlen(substr($temporary_check_buffer, $_ampPos)) > 9) {
                     break;
                 }
 
                 //if an entity is still present, fetch some more and repeat the escaping
-                $this->currentBuffer    .= fread( $this->originalFP, 9 );
-                $temporary_check_buffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+                $this->currentBuffer .= fread($this->originalFP, 9);
+                $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
             }
 
             //free stuff outside the loop
-            unset( $temporary_check_buffer );
+            unset($temporary_check_buffer);
 
-            $this->currentBuffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
-            $this->currentBuffer = str_replace( "&", self::$INTERNAL_TAG_PLACEHOLDER . 'amp' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+            $this->currentBuffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
+            $this->currentBuffer = str_replace("&", self::$INTERNAL_TAG_PLACEHOLDER . 'amp' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
 
             //get length of chunk
-            $this->len = strlen( $this->currentBuffer );
+            $this->len = strlen($this->currentBuffer);
 
             //parse chunk of text
-            if ( !xml_parse( $xmlParser, $this->currentBuffer, feof( $this->originalFP ) ) ) {
+            if (!xml_parse($xmlParser, $this->currentBuffer, feof($this->originalFP))) {
                 //if unable, raise an exception
-                throw new RuntimeException( sprintf(
+                throw new RuntimeException(sprintf(
                         "XML error: %s at line %d",
-                        xml_error_string( xml_get_error_code( $xmlParser ) ),
-                        xml_get_current_line_number( $xmlParser )
-                ) );
+                        xml_error_string(xml_get_error_code($xmlParser)),
+                        xml_get_current_line_number($xmlParser)
+                ));
             }
             /*
             * Get the accumulated this->offset in the document:
@@ -147,7 +147,7 @@ 
         }
 
         // close Sax parser
-        $this->closeSaxParser( $xmlParser );
+        $this->closeSaxParser($xmlParser);
 
     }
 
@@ -156,24 +156,24 @@ 
      *
      * @return string
      */
-    protected function getLastCharacter( $parser ): string {
+    protected function getLastCharacter($parser): string {
 
         //this logic helps detecting empty tags
         //get current position of SAX pointer in all the stream of data is has read so far:
         //it points at the end of current tag
-        $idx = xml_get_current_byte_index( $parser );
+        $idx = xml_get_current_byte_index($parser);
 
         //check whether the bounds of current tag are entirely in current buffer or the end of the current tag
         //is outside current buffer (in the latter case, it's in next buffer to be read by the while loop);
         //this check is necessary because we may have truncated a tag in half with current read,
         //and the other half may be encountered in the next buffer it will be passed
-        if ( isset( $this->currentBuffer[ $idx - $this->offset ] ) ) {
+        if (isset($this->currentBuffer[$idx - $this->offset])) {
             //if this tag entire lenght fitted in the buffer, the last char must be the last
             //symbol before the '>'; if it's an empty tag, it is assumed that it's a '/'
-            $lastChar = $this->currentBuffer[ $idx - $this->offset ];
+            $lastChar = $this->currentBuffer[$idx - $this->offset];
         } else {
             //if it's out, simple use the last character of the chunk
-            $lastChar = $this->currentBuffer[ $this->len - 1 ];
+            $lastChar = $this->currentBuffer[$this->len - 1];
         }
 
         return $lastChar;
@@ -187,19 +187,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( string $outputFilePath ) {
+    private function createOutputFileIfDoesNotExist(string $outputFilePath) {
         // create output file
-        if ( !file_exists( $outputFilePath ) ) {
-            touch( $outputFilePath );
+        if (!file_exists($outputFilePath)) {
+            touch($outputFilePath);
         }
     }
 
@@ -207,13 +207,13 @@ 
      * @param string $originalXliffPath
      * @param string $outputFilePath
      */
-    private function setFileDescriptors( string $originalXliffPath, string $outputFilePath ) {
-        $this->outputFP = fopen( $outputFilePath, 'w+' );
+    private function setFileDescriptors(string $originalXliffPath, string $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");
         }
     }
 
@@ -223,8 +223,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);
     }
 
     /**
@@ -233,11 +233,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;
     }
@@ -245,8 +245,8 @@ 
     /**
      * @param resource $xmlSaxParser
      */
-    protected function closeSaxParser( $xmlSaxParser ) {
-        xml_parser_free( $xmlSaxParser );
+    protected function closeSaxParser($xmlSaxParser) {
+        xml_parser_free($xmlSaxParser);
     }
 
     /**
@@ -256,7 +256,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagOpen( $parser, string $name, array $attr );
+    abstract protected function tagOpen($parser, string $name, array $attr);
 
     /**
      * @param resource $parser
@@ -264,7 +264,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagClose( $parser, string $name );
+    abstract protected function tagClose($parser, string $name);
 
     /**
      * @param resource $parser
@@ -272,11 +272,11 @@ 
      *
      * @return mixed
      */
-    protected function characterData( $parser, string $data ): void {
+    protected function characterData($parser, string $data): void {
         // don't write <target> data
-        if ( !$this->inTarget && !$this->bufferIsActive ) {
-            $this->postProcAndFlush( $this->outputFP, $data );
-        } elseif ( $this->bufferIsActive ) {
+        if (!$this->inTarget && !$this->bufferIsActive) {
+            $this->postProcAndFlush($this->outputFP, $data);
+        } elseif ($this->bufferIsActive) {
             $this->CDATABuffer .= $data;
         }
     }
@@ -288,19 +288,19 @@ 
      * @param string   $data
      * @param bool     $treatAsCDATA
      */
-    protected function postProcAndFlush( $fp, string $data, bool $treatAsCDATA = false ) {
+    protected function postProcAndFlush($fp, string $data, bool $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);
     }
 
     /**
@@ -309,19 +309,19 @@ 
      *
      * @return void
      */
-    protected function handleOpenUnit( string $name, array $attr ) {
+    protected function handleOpenUnit(string $name, array $attr) {
 
         // check if we are entering into a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->inTU = true;
 
             // get id
             // trim to first 100 characters because this is the limit on Matecat's DB
-            $this->currentTransUnitId = substr( $attr[ 'id' ], 0, 100 );
+            $this->currentTransUnitId = substr($attr['id'], 0, 100);
 
             // `translate` attribute can be only yes or no
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitIsTranslatable = empty( $attr[ 'translate' ] ) ? 'yes' : $attr[ 'translate' ];
+            $this->currentTransUnitIsTranslatable = empty($attr['translate']) ? 'yes' : $attr['translate'];
 
             $this->setLastTransUnitSegments();
 
@@ -335,15 +335,15 @@ 
      *
      * @return string
      */
-    protected function handleOpenXliffTag( string $name, array $attr, string $tag ): string {
+    protected function handleOpenXliffTag(string $name, array $attr, string $tag): string {
 
         // Add MateCat specific namespace.
         // Add trgLang
-        if ( $name === 'xliff' ) {
-            if ( !array_key_exists( 'xmlns:mtc', $attr ) ) {
+        if ($name === 'xliff') {
+            if (!array_key_exists('xmlns:mtc', $attr)) {
                 $tag .= ' xmlns:mtc="https://www.matecat.com" ';
             }
-            $tag = preg_replace( '/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag );
+            $tag = preg_replace('/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag);
         }
 
         return $tag;
@@ -355,10 +355,10 @@ 
      *
      * @return void
      */
-    protected function checkSetInTarget( string $name ) {
+    protected function checkSetInTarget(string $name) {
         // check if we are entering into a <target>
-        if ( 'target' === $name ) {
-            if ( $this->currentTransUnitIsTranslatable === 'no' ) {
+        if ('target' === $name) {
+            if ($this->currentTransUnitIsTranslatable === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -371,8 +371,8 @@ 
      *
      * @return void
      */
-    protected function setInBuffer( string $name ) {
-        if ( in_array( $name, $this->nodesToBuffer ) ) {
+    protected function setInBuffer(string $name) {
+        if (in_array($name, $this->nodesToBuffer)) {
             $this->bufferIsActive = true;
         }
     }
@@ -380,24 +380,24 @@ 
     /**
      * @param array $seg
      */
-    protected function updateSegmentCounts( array $seg = [] ) {
+    protected function updateSegmentCounts(array $seg = []) {
 
-        $raw_word_count = $seg[ 'raw_word_count' ];
-        $eq_word_count  = ( floor( $seg[ 'eq_word_count' ] * 100 ) / 100 );
+        $raw_word_count = $seg['raw_word_count'];
+        $eq_word_count  = (floor($seg['eq_word_count'] * 100) / 100);
 
-        $this->counts[ 'segments_count_array' ][ $seg[ 'sid' ] ] = [
+        $this->counts['segments_count_array'][$seg['sid']] = [
                 'raw_word_count' => $raw_word_count,
                 'eq_word_count'  => $eq_word_count,
         ];
 
-        $this->counts[ 'raw_word_count' ] += $raw_word_count;
-        $this->counts[ 'eq_word_count' ]  += $eq_word_count;
+        $this->counts['raw_word_count'] += $raw_word_count;
+        $this->counts['eq_word_count']  += $eq_word_count;
     }
 
     protected function resetCounts() {
-        $this->counts[ 'segments_count_array' ] = [];
-        $this->counts[ 'raw_word_count' ]       = 0;
-        $this->counts[ 'eq_word_count' ]        = 0;
+        $this->counts['segments_count_array'] = [];
+        $this->counts['raw_word_count']       = 0;
+        $this->counts['eq_word_count']        = 0;
     }
 
     /**
@@ -406,16 +406,16 @@ 
      *
      * @return void
      */
-    protected function checkForSelfClosedTagAndFlush( $parser, string $tag ) {
+    protected function checkForSelfClosedTagAndFlush($parser, string $tag) {
 
-        $lastChar = $this->getLastCharacter( $parser );
+        $lastChar = $this->getLastCharacter($parser);
 
         //trim last space
-        $tag = rtrim( $tag );
+        $tag = rtrim($tag);
 
         //detect empty tag
         $this->isEmpty = $lastChar == '/';
-        if ( $this->isEmpty ) {
+        if ($this->isEmpty) {
             $tag .= $lastChar;
         }
 
@@ -423,11 +423,11 @@ 
         $tag .= ">";
 
         //set a Buffer for the segSource Source tag
-        if ( $this->bufferIsActive ) { // we are opening a critical CDATA section
+        if ($this->bufferIsActive) { // we are opening a critical CDATA section
             //these are NOT source/seg-source/value empty tags, THERE IS A CONTENT, write it in buffer
             $this->CDATABuffer .= $tag;
         } else {
-            $this->postProcAndFlush( $this->outputFP, $tag );
+            $this->postProcAndFlush($this->outputFP, $tag);
         }
 
     }
@@ -452,18 +452,18 @@ 
          */
         $this->lastTransUnit = [];
 
-        if ( !isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
+        if (!isset($this->transUnits[$this->currentTransUnitId])) {
             return;
         }
 
-        $listOfSegmentsIds = $this->transUnits[ $this->currentTransUnitId ];
+        $listOfSegmentsIds = $this->transUnits[$this->currentTransUnitId];
         $last_value        = null;
-        $segmentsCount     = count( $listOfSegmentsIds );
-        for ( $i = 0; $i < $segmentsCount; $i++ ) {
-            $id = $listOfSegmentsIds[ $i ];
-            if ( isset( $this->segments[ $id ] ) && ( $i == 0 || $last_value + 1 == $listOfSegmentsIds[ $i ] ) ) {
-                $last_value            = $listOfSegmentsIds[ $i ];
-                $this->lastTransUnit[] = $this->segments[ $id ];
+        $segmentsCount     = count($listOfSegmentsIds);
+        for ($i = 0; $i < $segmentsCount; $i++) {
+            $id = $listOfSegmentsIds[$i];
+            if (isset($this->segments[$id]) && ($i == 0 || $last_value + 1 == $listOfSegmentsIds[$i])) {
+                $last_value            = $listOfSegmentsIds[$i];
+                $this->lastTransUnit[] = $this->segments[$id];
             }
         }
 
@@ -473,8 +473,8 @@ 
      * @return array
      */
     protected function getCurrentSegment(): array {
-        if ( $this->currentTransUnitIsTranslatable !== 'no' && isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
-            return $this->segments[ $this->segmentInUnitPosition ];
+        if ($this->currentTransUnitIsTranslatable !== 'no' && isset($this->transUnits[$this->currentTransUnitId])) {
+            return $this->segments[$this->segmentInUnitPosition];
         }
 
         return [];

src/Utils/Files.php

Spacing +28 added lines, -28 removed lines

@@ -18,15 +18,15 @@ 
      *
      * @return array|mixed
      */
-    public static function pathInfo( string $path, ?int $options = 15 ) {
-        $rawPath = explode( DIRECTORY_SEPARATOR, $path );
+    public static function pathInfo(string $path, ?int $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 ?string
      */
-    public static function getExtension( $path ): ?string {
-        $pathInfo = self::pathInfo( $path );
+    public static function getExtension($path): ?string {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return null;
         }
 
-        return strtolower( $pathInfo[ 'extension' ] );
+        return strtolower($pathInfo['extension']);
     }
 
     /**
@@ -75,14 +75,14 @@ 
      *
      * @return bool
      */
-    public static function isXliff( ?string $path ): bool {
-        $extension = self::getExtension( $path );
+    public static function isXliff(?string $path): bool {
+        $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( string $path ) {
-        $pathInfo = self::pathInfo( $path );
+    public static function getMemoryFileType(string $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 ): bool {
-        return self::getMemoryFileType( $path ) === 'tmx';
+    public static function isTMXFile($path): bool {
+        return self::getMemoryFileType($path) === 'tmx';
     }
 
     /**
@@ -129,7 +129,7 @@ 
      *
      * @return bool
      */
-    public static function isGlossaryFile( $path ): bool {
-        return self::getMemoryFileType( $path ) === 'glossary';
+    public static function isGlossaryFile($path): bool {
+        return self::getMemoryFileType($path) === 'glossary';
     }
 }

src/XliffUtils/XliffProprietaryDetect.php

Spacing +57 added lines, -57 removed lines

@@ -22,11 +22,11 @@ 
      *
      * @return array
      */
-    public static function getInfoFromXliffContent( string $xliffContent ): array {
+    public static function getInfoFromXliffContent(string $xliffContent): array {
         self::reset();
-        $tmp = self::getFirst1024CharsFromXliff( $xliffContent );
+        $tmp = self::getFirst1024CharsFromXliff($xliffContent);
 
-        return self::getInfoFromTmp( $tmp );
+        return self::getInfoFromTmp($tmp);
     }
 
     /**
@@ -34,12 +34,12 @@ 
      *
      * @return array
      */
-    public static function getInfo( string $fullPathToFile ): array {
+    public static function getInfo(string $fullPathToFile): array {
         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( array $tmp ): array {
+    private static function getInfoFromTmp(array $tmp): array {
         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( ?array $tmp = [] ): array {
-        $pipeline = new CheckXliffProprietaryPipeline( $tmp );
-        $pipeline->addCheck( new CheckSDL() );
-        $pipeline->addCheck( new CheckGlobalSight() );
-        $pipeline->addCheck( new CheckMateCATConverter() );
-        $pipeline->addCheck( new CheckXliffVersion2() );
+    private static function runPipeline(?array $tmp = []): array {
+        $pipeline = new CheckXliffProprietaryPipeline($tmp);
+        $pipeline->addCheck(new CheckSDL());
+        $pipeline->addCheck(new CheckGlobalSight());
+        $pipeline->addCheck(new CheckMateCATConverter());
+        $pipeline->addCheck(new CheckXliffVersion2());
 
         return $pipeline->run();
     }
@@ -93,21 +93,21 @@ 
         ];
     }
 
-    private static function getFirst1024CharsFromString( ?string $stringData ): string {
-        if ( !empty( $stringData ) ) {
-            return substr( $stringData, 0, 1024 );
+    private static function getFirst1024CharsFromString(?string $stringData): string {
+        if (!empty($stringData)) {
+            return substr($stringData, 0, 1024);
         }
 
         return '';
     }
 
-    private static function getFirst1024CharsFromFile( string $fullPathToFile ): string {
+    private static function getFirst1024CharsFromFile(string $fullPathToFile): string {
         $stringData = '';
-        if ( !empty( $fullPathToFile ) && is_file( $fullPathToFile ) ) {
-            $file_pointer = fopen( "$fullPathToFile", 'r' );
+        if (!empty($fullPathToFile) && 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);
         }
 
         return $stringData;
@@ -120,13 +120,13 @@ 
      *
      * @return string[]
      */
-    private static function getFirst1024CharsFromXliff( ?string $stringData = null, string $fullPathToFile = null ): ?array {
-        $stringData = static::getFirst1024CharsFromString( $stringData );
-        if ( empty( $stringData ) ) {
-            $stringData = static::getFirst1024CharsFromFile( $fullPathToFile );
+    private static function getFirst1024CharsFromXliff(?string $stringData = null, string $fullPathToFile = null): ?array {
+        $stringData = static::getFirst1024CharsFromString($stringData);
+        if (empty($stringData)) {
+            $stringData = static::getFirst1024CharsFromFile($fullPathToFile);
         }
 
-        return !empty( $stringData ) ? [ $stringData ] : [];
+        return !empty($stringData) ? [$stringData] : [];
     }
 
     /**
@@ -135,9 +135,9 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    protected static function checkVersion( array $tmp ) {
-        if ( isset( $tmp[ 0 ] ) ) {
-            self::$fileType[ 'version' ] = XliffVersionDetector::detect( $tmp[ 0 ] );
+    protected static function checkVersion(array $tmp) {
+        if (isset($tmp[0])) {
+            self::$fileType['version'] = XliffVersionDetector::detect($tmp[0]);
         }
     }
 
@@ -148,20 +148,20 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function getInfoByStringData( string $stringData ): array {
+    public static function getInfoByStringData(string $stringData): array {
         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;
     }
@@ -173,21 +173,21 @@ 
      *
      * @return bool|int
      */
-    public static function fileMustBeConverted( string $fullPath, ?bool $enforceOnXliff = false, ?string $filterAddress = null ) {
+    public static function fileMustBeConverted(string $fullPath, ?bool $enforceOnXliff = false, ?string $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
                     }
@@ -195,16 +195,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.