Mistralys / application-localization

src/Localization/Translator.php

Indentation +10 added lines, -10 removed lines

@@ -57,8 +57,8 @@ 
     protected $reverseStrings = array();
 
     /**
-    * @var Localization_Source[]
-    */
+     * @var Localization_Source[]
+     */
     private $sources = array();
 
     /**
@@ -191,14 +191,14 @@ 
         );
     }
     
-   /**
-    * Retrieves all available strings for the specified locale,
-    * as hash => text value pairs.
-    * 
-    * @param Localization_Locale $locale
-    * @throws Localization_Exception
-    * @return string[]
-    */
+    /**
+     * Retrieves all available strings for the specified locale,
+     * as hash => text value pairs.
+     * 
+     * @param Localization_Locale $locale
+     * @throws Localization_Exception
+     * @return string[]
+     */
     public function getStrings(Localization_Locale $locale) : array
     {
         $this->load($locale);

Spacing +14 added lines, -14 removed lines

@@ -79,7 +79,7 @@ 
      */
     public function addSources(array $sources) : void
     {
-        foreach($sources as $source) {
+        foreach ($sources as $source) {
             $this->addSource($source);
         }
     }
@@ -124,7 +124,7 @@ 
             $this->strings[$localeName] = array();
         }
 
-        foreach($this->sources as $source)
+        foreach ($this->sources as $source)
         {
             $file = $this->resolveStorageFile($locale, $source);
             if (!file_exists($file)) {
@@ -133,7 +133,7 @@ 
             
             $data = parse_ini_file($file, false);
             
-            if($data === false) 
+            if ($data === false) 
             {
                 throw new Localization_Exception(
                     'Malformatted localization file',
@@ -205,7 +205,7 @@ 
         
         $name = $locale->getName();
         
-        if(isset($this->strings[$name])) {
+        if (isset($this->strings[$name])) {
             return $this->strings[$name];
         }
         
@@ -240,11 +240,11 @@ 
      * @param Localization_Locale $locale
      * @param boolean $editable
      */
-    protected function renderStringsFile(string $type, Localization_Source $source, array $hashes, string $file, Localization_Locale $locale, bool $editable=true) : void
+    protected function renderStringsFile(string $type, Localization_Source $source, array $hashes, string $file, Localization_Locale $locale, bool $editable = true) : void
     {
         $writer = new Localization_Writer($locale, $type, $file);
 
-        if($editable)
+        if ($editable)
         {
             $writer->makeEditable();
         }
@@ -253,14 +253,14 @@ 
         
         foreach ($hashes as $hash) 
         {
-            if(!$hash->hasSourceID($sourceID)) {
+            if (!$hash->hasSourceID($sourceID)) {
                 continue;
             }
             
             $text = $this->getHashTranslation($hash->getHash(), $locale);
 
             // skip any empty strings
-            if($text === null || trim($text) == '') {
+            if ($text === null || trim($text) == '') {
                 continue;
             }
             
@@ -338,7 +338,7 @@ 
     {
         // to avoid re-creating the hash for the same texts over and over,
         // we keep track of the hashes we created, and re-use them.
-        if(isset($this->reverseStrings[$text])) {
+        if (isset($this->reverseStrings[$text])) {
             $hash = $this->reverseStrings[$text];
         } else {
             $hash = md5($text);
@@ -403,15 +403,15 @@ 
      * @param Localization_Locale|null $locale
      * @return string|NULL
      */
-    public function getHashTranslation(string $hash, ?Localization_Locale $locale=null) : ?string
+    public function getHashTranslation(string $hash, ?Localization_Locale $locale = null) : ?string
     {
-        if(!$locale) {
+        if (!$locale) {
             $locale = $this->targetLocale;
         }
         
         $localeName = $locale->getName();
         
-        if(isset($this->strings[$localeName]) && isset($this->strings[$localeName][$hash])) {
+        if (isset($this->strings[$localeName]) && isset($this->strings[$localeName][$hash])) {
             return $this->strings[$localeName][$hash];
         }
 
@@ -429,10 +429,10 @@ 
     {
         $result = array();
         
-        foreach($this->sources as $source) 
+        foreach ($this->sources as $source) 
         {
             $localeFile = self::getClientStorageFile($locale, $source);
-            if(!file_exists($localeFile)) {
+            if (!file_exists($localeFile)) {
                 continue;
             }
 

src/Localization/Parser/Token.php

Indentation +7 added lines, -7 removed lines

@@ -8,14 +8,14 @@ 
 
 abstract class Localization_Parser_Token
 {
-   /**
-    * @var array|string
-    */
+    /**
+     * @var array|string
+     */
     protected $definition;
     
-   /**
-    * @var Localization_Parser_Token
-    */
+    /**
+     * @var Localization_Parser_Token
+     */
     protected $parentToken;
 
     /**
@@ -98,6 +98,6 @@ 
             'value' => $this->getValue(),
             'line' => $this->getLine(),
             'isEncapsedString' => ConvertHelper::bool2string($this->isEncapsedString())
-         );
+            );
     }
 }

Spacing +2 added lines, -2 removed lines

@@ -42,13 +42,13 @@
      * @param array|string $definition
      * @param Localization_Parser_Token|null $parentToken
      */
-    public function __construct($definition, ?Localization_Parser_Token $parentToken=null)
+    public function __construct($definition, ?Localization_Parser_Token $parentToken = null)
     {
         $this->definition = $definition;
         $this->parentToken = $parentToken;
 
         $names = $this->getFunctionNames();
-        foreach($names as $name) {
+        foreach ($names as $name) {
             $this->nameLookup[$name] = true;
         }
 

src/Localization/Parser/Language.php

Indentation +79 added lines, -79 removed lines

@@ -23,49 +23,49 @@ 
      */
     protected $parser;
 
-   /**
-    * The function names that are included in the search.
-    * @var array
-    */
+    /**
+     * The function names that are included in the search.
+     * @var array
+     */
     protected $functionNames = array();
     
-   /**
-    * The tokens definitions.
-    * @var array
-    */
+    /**
+     * The tokens definitions.
+     * @var array
+     */
     protected $tokens = array();
     
-   /**
-    * The total amount of tokens found in the content.
-    * @var integer
-    */
+    /**
+     * The total amount of tokens found in the content.
+     * @var integer
+     */
     protected $totalTokens = 0;
     
-   /**
-    * All texts that have been collected.
-    * @var Text[]
-    */
+    /**
+     * All texts that have been collected.
+     * @var Text[]
+     */
     protected $texts = array();
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $content = '';
 
-   /**
-    * @var string|NULL
-    */
+    /**
+     * @var string|NULL
+     */
     protected $id;
     
-   /**
-    * @var Localization_Parser_Warning[]
-    */
+    /**
+     * @var Localization_Parser_Warning[]
+     */
     protected $warnings = array();
     
-   /**
-    * The source file that was parsed (if any)
-    * @var string
-    */
+    /**
+     * The source file that was parsed (if any)
+     * @var string
+     */
     protected $sourceFile = '';
     
     public function __construct(Localization_Parser $parser)
@@ -76,10 +76,10 @@ 
     
     abstract protected function getTokens() : array;
     
-   /**
-    * Retrieves the ID of the language.
-    * @return string E.g. "PHP", "Javascript"
-    */
+    /**
+     * Retrieves the ID of the language.
+     * @return string E.g. "PHP", "Javascript"
+     */
     public function getID() : string
     {
         if(!isset($this->id)) {
@@ -99,12 +99,12 @@ 
         return $this->sourceFile;
     }
     
-   /**
-    * Parses the code from a file.
-    * 
-    * @param string $path
-    * @throws Localization_Exception
-    */
+    /**
+     * Parses the code from a file.
+     * 
+     * @param string $path
+     * @throws Localization_Exception
+     */
     public function parseFile(string $path) : void
     {
         if(!file_exists($path)) 
@@ -141,10 +141,10 @@ 
         $this->parse();
     }
     
-   /**
-    * Parses a source code string.
-    * @param string $content
-    */
+    /**
+     * Parses a source code string.
+     * @param string $content
+     */
     public function parseString(string $content) : void
     {
         $this->content = $content;
@@ -185,11 +185,11 @@ 
         $this->texts[] = new Text($text, $line, $explanation);
     }
 
-   /**
-    * Retrieves a list of all the function names that are
-    * used as translation functions in the language.
-    * @return array
-    */
+    /**
+     * Retrieves a list of all the function names that are
+     * used as translation functions in the language.
+     * @return array
+     */
     public function getFunctionNames() : array
     {
         return $this->createToken('dummy')->getFunctionNames();
@@ -200,13 +200,13 @@ 
         Localization::log(sprintf('%1$s parser | %2$s', $this->getID(), $message));
     }
 
-   /**
-    * Adds a warning message when a text cannot be parsed correctly for some reason.
-    * 
-    * @param Localization_Parser_Token $token
-    * @param string $message
-    * @return Localization_Parser_Warning
-    */
+    /**
+     * Adds a warning message when a text cannot be parsed correctly for some reason.
+     * 
+     * @param Localization_Parser_Token $token
+     * @param string $message
+     * @return Localization_Parser_Warning
+     */
     protected function addWarning(Localization_Parser_Token $token, string $message) : Localization_Parser_Warning
     {
         $warning = new Localization_Parser_Warning($this, $token, $message);
@@ -216,34 +216,34 @@ 
         return $warning;
     }
     
-   /**
-    * Whether any warnings were generated during parsing.
-    * @return bool
-    */
+    /**
+     * Whether any warnings were generated during parsing.
+     * @return bool
+     */
     public function hasWarnings() : bool
     {
         return !empty($this->warnings);
     }
     
-   /**
-    * Retrieves all warnings that were generated during parsing,
-    * if any.
-    * 
-    * @return Localization_Parser_Warning[]
-    */
+    /**
+     * Retrieves all warnings that were generated during parsing,
+     * if any.
+     * 
+     * @return Localization_Parser_Warning[]
+     */
     public function getWarnings() : array
     {
         return $this->warnings;
     }
     
-   /**
-    * Creates a token instance: this retrieves information on
-    * the language token being parsed.
-    * 
-    * @param array|string $definition The token definition.
-    * @param Localization_Parser_Token|NULL $parentToken
-    * @return Localization_Parser_Token
-    */
+    /**
+     * Creates a token instance: this retrieves information on
+     * the language token being parsed.
+     * 
+     * @param array|string $definition The token definition.
+     * @param Localization_Parser_Token|NULL $parentToken
+     * @return Localization_Parser_Token
+     */
     protected function createToken($definition, Localization_Parser_Token $parentToken=null) : Localization_Parser_Token
     {
         $class = Localization_Parser_Token::class.'_'.$this->getID();
@@ -251,12 +251,12 @@ 
         return new $class($definition, $parentToken);
     }
 
-   /**
-    * Parses a translation function token.
-    * 
-    * @param int $number
-    * @param Localization_Parser_Token $token
-    */
+    /**
+     * Parses a translation function token.
+     * 
+     * @param int $number
+     * @param Localization_Parser_Token $token
+     */
     protected function parseToken(int $number, Localization_Parser_Token $token) : void
     {
         $textParts = array();

Braces +1 added lines, -2 removed lines

@@ -124,8 +124,7 @@
         try
         {
             $this->content = FileHelper::readContents($path);
-        }
-        catch (FileHelper_Exception $e)
+        } catch (FileHelper_Exception $e)
         {
             throw new Localization_Exception(
                 sprintf('Source code file [%s] could not be read', basename($path)),

Spacing +25 added lines, -25 removed lines

@@ -82,7 +82,7 @@ 
     */
     public function getID() : string
     {
-        if(!isset($this->id)) {
+        if (!isset($this->id)) {
             $this->id = str_replace(Localization_Parser_Language::class.'_', '', get_class($this));
         }
         
@@ -107,7 +107,7 @@ 
     */
     public function parseFile(string $path) : void
     {
-        if(!file_exists($path)) 
+        if (!file_exists($path)) 
         {
             throw new Localization_Exception(
                 sprintf('Source code file [%s] not found', basename($path)),
@@ -160,12 +160,12 @@ 
         $this->tokens = $this->getTokens();
         $this->totalTokens = count($this->tokens);
         
-        for($i = 0; $i < $this->totalTokens; $i++)
+        for ($i = 0; $i < $this->totalTokens; $i++)
         {
             $token = $this->createToken($this->tokens[$i]);
             
-            if($token->isTranslationFunction()) {
-                $this->parseToken($i+1, $token);
+            if ($token->isTranslationFunction()) {
+                $this->parseToken($i + 1, $token);
             }
         }
     }
@@ -178,7 +178,7 @@ 
         return $this->texts;
     }
     
-    protected function addResult(string $text, int $line=0, string $explanation='') : void
+    protected function addResult(string $text, int $line = 0, string $explanation = '') : void
     {
         $this->log(sprintf('Line [%1$s] | Found string [%2$s]', $line, $text));
 
@@ -244,7 +244,7 @@ 
     * @param Localization_Parser_Token|NULL $parentToken
     * @return Localization_Parser_Token
     */
-    protected function createToken($definition, Localization_Parser_Token $parentToken=null) : Localization_Parser_Token
+    protected function createToken($definition, Localization_Parser_Token $parentToken = null) : Localization_Parser_Token
     {
         $class = Localization_Parser_Token::class.'_'.$this->getID();
         
@@ -264,48 +264,48 @@ 
         $open = false;
         $explanation = '';
         
-        for($i = $number; $i < $max; $i++)
+        for ($i = $number; $i < $max; $i++)
         {
-            if(!isset($this->tokens[$i])) {
+            if (!isset($this->tokens[$i])) {
                 break;
             }
             
             $subToken = $this->createToken($this->tokens[$i], $token);
             
-            if(!$open && $subToken->isOpeningFuncParams())
+            if (!$open && $subToken->isOpeningFuncParams())
             {
                 $open = true;
                 continue;
             }
             
-            if($open && $subToken->isClosingFuncParams()) {
+            if ($open && $subToken->isClosingFuncParams()) {
                 break;
             }
             
             // additional parameters in the translation function, we don't want to capture these now.
-            if($open && $subToken->isArgumentSeparator())
+            if ($open && $subToken->isArgumentSeparator())
             {
-                if($token->isExplanationFunction()) {
-                    $leftover = array_slice($this->tokens, $i+1);
+                if ($token->isExplanationFunction()) {
+                    $leftover = array_slice($this->tokens, $i + 1);
                     $explanation = $this->parseExplanation($token, $leftover);
                 }
                 break;
             }
             
-            if($open && $subToken->isEncapsedString())
+            if ($open && $subToken->isEncapsedString())
             {
                 $textParts[] = $this->trimText(strval($subToken->getValue()));
                 continue;
             }
             
-            if($open && $subToken->isVariableOrFunction()) {
+            if ($open && $subToken->isVariableOrFunction()) {
                 $textParts = null;
                 $this->addWarning($subToken, t('Variables or functions are not supported in translation functions.'));
                 break;
             }
         }
         
-        if(empty($textParts)) {
+        if (empty($textParts)) {
             return;
         }
         
@@ -319,38 +319,38 @@ 
         $textParts = array();
         $max = 200;
 
-        for($i = 0; $i < $max; $i++)
+        for ($i = 0; $i < $max; $i++)
         {
-            if(!isset($tokens[$i])) {
+            if (!isset($tokens[$i])) {
                 break;
             }
 
             $subToken = $this->createToken($tokens[$i], $token);
 
-            if($subToken->isClosingFuncParams()) {
+            if ($subToken->isClosingFuncParams()) {
                 break;
             }
 
             // additional parameters in the translation function, we don't want to capture these now.
-            if($subToken->isArgumentSeparator())
+            if ($subToken->isArgumentSeparator())
             {
                 break;
             }
 
-            if($subToken->isEncapsedString())
+            if ($subToken->isEncapsedString())
             {
                 $textParts[] = $this->trimText(strval($subToken->getValue()));
                 continue;
             }
 
-            if($subToken->isVariableOrFunction()) {
+            if ($subToken->isVariableOrFunction()) {
                 $textParts = null;
                 $this->addWarning($subToken, t('Variables or functions are not supported in translation functions.'));
                 break;
             }
         }
 
-        if(empty($textParts)) {
+        if (empty($textParts)) {
             return '';
         }
 
@@ -359,7 +359,7 @@ 
 
     protected function debug(string $text) : void
     {
-        if($this->debug) {
+        if ($this->debug) {
             echo $text;
         }
     }

src/Localization/Event/LocaleChanged.php

Indentation +15 added lines, -15 removed lines

@@ -25,9 +25,9 @@ 
     const ERROR_NO_CURRENT_LOCALE_SPECIFIED = 91401;
 
     /**
-    * The locale that was used before the change, if any.
-    * @return Localization_Locale|NULL
-    */
+     * The locale that was used before the change, if any.
+     * @return Localization_Locale|NULL
+     */
     public function getPrevious() : ?Localization_Locale
     {
         $arg = $this->getArgument(1);
@@ -59,28 +59,28 @@ 
         );
     }
     
-   /**
-    * The namespace in which the locale change occurred.
-    * @return string
-    */
+    /**
+     * The namespace in which the locale change occurred.
+     * @return string
+     */
     public function getNamespace() : string
     {
         return strval($this->getArgument(0));
     }
 
-   /**
-    * Whether the change occurred for an application locale.
-    * @return bool
-    */
+    /**
+     * Whether the change occurred for an application locale.
+     * @return bool
+     */
     public function isAppLocale() : bool
     {
         return $this->getNamespace() === Localization::NAMESPACE_APPLICATION;
     }
     
-   /**
-    * Whether the change occurred for a content locale.
-    * @return bool
-    */
+    /**
+     * Whether the change occurred for a content locale.
+     * @return bool
+     */
     public function isContentLocale() : bool
     {
         return $this->getNamespace() === Localization::NAMESPACE_CONTENT;

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
     public function getPrevious() : ?Localization_Locale
     {
         $arg = $this->getArgument(1);
-        if($arg instanceof Localization_Locale) {
+        if ($arg instanceof Localization_Locale) {
             return $arg;
         }
         
@@ -48,7 +48,7 @@ 
     {
         $arg = $this->getArgument(2);
 
-        if($arg instanceof Localization_Locale) {
+        if ($arg instanceof Localization_Locale) {
             return $arg;
         }
 

src/Localization/Scanner.php

Indentation +18 added lines, -18 removed lines

@@ -24,24 +24,24 @@ 
  */
 class Localization_Scanner
 {
-   /**
-    * @var float
-    */
+    /**
+     * @var float
+     */
     protected $timeStart;
     
-   /**
-    * @var float
-    */
+    /**
+     * @var float
+     */
     protected $timeEnd;
     
-   /**
-    * @var array
-    */
+    /**
+     * @var array
+     */
     protected $stringInfos = array();
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $storageFile;
 
     /**
@@ -172,12 +172,12 @@ 
         return $this->getCollection()->countWarnings();
     }
     
-   /**
-    * Retrieves all warnings that have been registered
-    * during the last search for translatable texts.
-    * 
-    * @return Localization_Scanner_StringsCollection_Warning[]
-    */
+    /**
+     * Retrieves all warnings that have been registered
+     * during the last search for translatable texts.
+     * 
+     * @return Localization_Scanner_StringsCollection_Warning[]
+     */
     public function getWarnings() : array
     {
         return $this->getCollection()->getWarnings();

Spacing +7 added lines, -7 removed lines

@@ -72,14 +72,14 @@ 
     
     public function scan() : void
     {
-        if(isset($this->collection)) {
+        if (isset($this->collection)) {
             $this->collection = null;
         }
         
         $this->timeStart = microtime(true);
         
         $sources = Localization::getSources();
-        foreach($sources as $source) 
+        foreach ($sources as $source) 
         {
             $source->scan($this);
         }
@@ -91,11 +91,11 @@ 
 
     public function load() : void
     {
-        if(!$this->isScanAvailable()) {
+        if (!$this->isScanAvailable()) {
             return;
         }
         
-        if($this->loaded) {
+        if ($this->loaded) {
             return;
         }
         
@@ -103,7 +103,7 @@ 
         
         $data = FileHelper::parseJSONFile($this->storageFile);
         
-        if($this->getCollection()->fromArray($data) === true) {
+        if ($this->getCollection()->fromArray($data) === true) {
             return;
         }
         
@@ -121,7 +121,7 @@ 
     
     public function getParser() : Localization_Parser
     {
-        if(!isset($this->parser)) {
+        if (!isset($this->parser)) {
             $this->parser = new Localization_Parser($this);
         }
         
@@ -130,7 +130,7 @@ 
     
     public function getCollection() : Localization_Scanner_StringsCollection
     {
-        if(!isset($this->collection)) {
+        if (!isset($this->collection)) {
             $this->collection = new Localization_Scanner_StringsCollection($this);
         }
         

src/Localization/Currency.php

Spacing +7 added lines, -7 removed lines

@@ -73,11 +73,11 @@ 
             return self::$instances[$id];
         }
 
-        $className = Localization_Currency::class.'_' . $id;
+        $className = Localization_Currency::class.'_'.$id;
 
         $country = new $className($country);
 
-        if($country instanceof Localization_Currency)
+        if ($country instanceof Localization_Currency)
         {
             return $country;
         }
@@ -138,7 +138,7 @@ 
      */
     public static function isCurrencyKnown(string $currencyName) : bool
     {
-        return file_exists(__DIR__ . '/Currency/' . $currencyName . '.php');
+        return file_exists(__DIR__.'/Currency/'.$currencyName.'.php');
     }
 
     /**
@@ -350,7 +350,7 @@ 
      * @return string
      * @throws Localization_Exception
      */
-    public function makeReadable($number, int $decimalPositions = 2, bool $addSymbol=true) : string
+    public function makeReadable($number, int $decimalPositions = 2, bool $addSymbol = true) : string
     {
         if ($number === null || $number === '') {
             return '';
@@ -360,14 +360,14 @@ 
 
         $number = $this->formatNumber($parsed->getFloat(), $decimalPositions);
 
-        if(!$addSymbol) {
+        if (!$addSymbol) {
             return $number;
         }
         
         if ($this->isSymbolOnFront()) {
-            return $this->getSymbol() . ' ' . $number;
+            return $this->getSymbol().' '.$number;
         }
 
-        return $number . ' ' . $this->getSymbol();
+        return $number.' '.$this->getSymbol();
     }
 }

Braces +1 added lines, -2 removed lines

@@ -245,8 +245,7 @@
             if ($decimals == '-') {
                 $decimals = 0;
             }
-        } 
-        else 
+        } else 
         {
             $decimals = 0;
             $thousands = implode('', $parts);

src/Localization/Scanner/StringsCollection.php

Indentation +49 added lines, -49 removed lines

@@ -12,19 +12,19 @@ 
     
     const STORAGE_FORMAT_VERSION = 2;
     
-   /**
-    * @var Localization_Scanner
-    */
+    /**
+     * @var Localization_Scanner
+     */
     protected $scanner;
     
-   /**
-    * @var Localization_Scanner_StringHash[]
-    */
+    /**
+     * @var Localization_Scanner_StringHash[]
+     */
     protected $hashes = array();
     
-   /**
-    * @var array
-    */
+    /**
+     * @var array
+     */
     protected $warnings = array();
     
     public function __construct(Localization_Scanner $scanner)
@@ -52,12 +52,12 @@ 
         return new Localization_Scanner_StringInfo($this, $sourceID, $sourceType, $text);
     }
     
-   /**
-    * Adds a single translatable string.
-    * 
-    * @param Localization_Scanner_StringInfo $string
-    * @return Localization_Scanner_StringsCollection
-    */
+    /**
+     * Adds a single translatable string.
+     * 
+     * @param Localization_Scanner_StringInfo $string
+     * @return Localization_Scanner_StringsCollection
+     */
     protected function add(Localization_Scanner_StringInfo $string) : Localization_Scanner_StringsCollection
     {
         $hash = $string->getHash();
@@ -70,12 +70,12 @@ 
         return $this;
     }
     
-   /**
-    * Retrieves all available translatable strings,
-    * grouped by their hash to identify unique strings.
-    * 
-    * @return Localization_Scanner_StringHash[]
-    */
+    /**
+     * Retrieves all available translatable strings,
+     * grouped by their hash to identify unique strings.
+     * 
+     * @return Localization_Scanner_StringHash[]
+     */
     public function getHashes() : array
     {
         return array_values($this->hashes);
@@ -146,32 +146,32 @@ 
         return true;
     }
     
-   /**
-    * Whether the parser reported warnings during the
-    * search for translatable texts.
-    * 
-    * @return bool
-    */
+    /**
+     * Whether the parser reported warnings during the
+     * search for translatable texts.
+     * 
+     * @return bool
+     */
     public function hasWarnings() : bool
     {
         return !empty($this->warnings);
     }
     
-   /**
-    * Retrieves the amount of warnings.
-    * @return int
-    */
+    /**
+     * Retrieves the amount of warnings.
+     * @return int
+     */
     public function countWarnings() : int
     {
         return count($this->warnings);
     }
     
-   /**
-    * Retrieves all warning messages that were added
-    * during the search for translatable texts, if any.
-    * 
-    * @return Localization_Scanner_StringsCollection_Warning[]
-    */
+    /**
+     * Retrieves all warning messages that were added
+     * during the search for translatable texts, if any.
+     * 
+     * @return Localization_Scanner_StringsCollection_Warning[]
+     */
     public function getWarnings() : array
     {
         $result = array();
@@ -198,12 +198,12 @@ 
         return $amount;
     }
     
-   /**
-    * Retrieves all string hashed for the specified source.
-    * 
-    * @param string $id
-    * @return Localization_Scanner_StringHash[]
-    */
+    /**
+     * Retrieves all string hashed for the specified source.
+     * 
+     * @param string $id
+     * @return Localization_Scanner_StringHash[]
+     */
     public function getHashesBySourceID(string $id) : array
     {
         $hashes = array();
@@ -217,12 +217,12 @@ 
         return $hashes;
     }
     
-   /**
-    * Retrieves all hashes for the specified language ID.
-    * 
-    * @param string $languageID The language ID, e.g. "PHP"
-    * @return Localization_Scanner_StringHash[]
-    */
+    /**
+     * Retrieves all hashes for the specified language ID.
+     * 
+     * @param string $languageID The language ID, e.g. "PHP"
+     * @return Localization_Scanner_StringHash[]
+     */
     public function getHashesByLanguageID(string $languageID) : array
     {
         $hashes = array();

Spacing +11 added lines, -11 removed lines

@@ -62,7 +62,7 @@ 
     {
         $hash = $string->getHash();
         
-        if(!isset($this->hashes[$hash])) {
+        if (!isset($this->hashes[$hash])) {
             $this->hashes[$hash] = new Localization_Scanner_StringHash($this, $hash);
         }
         
@@ -93,7 +93,7 @@ 
      */
     public function getHash(string $hash) : Localization_Scanner_StringHash
     {
-        if(isset($this->hashes[$hash])) {
+        if (isset($this->hashes[$hash])) {
             return $this->hashes[$hash];
         }
         
@@ -115,7 +115,7 @@ 
             'warnings' => array()
         );
         
-        foreach($this->hashes as $hash)
+        foreach ($this->hashes as $hash)
         {
             $data['hashes'] = array_merge($data['hashes'], $hash->toArray());
         }
@@ -131,11 +131,11 @@ 
      */
     public function fromArray(array $array) : bool
     {
-        if(!isset($array['formatVersion']) || $array['formatVersion'] != self::STORAGE_FORMAT_VERSION) {
+        if (!isset($array['formatVersion']) || $array['formatVersion'] != self::STORAGE_FORMAT_VERSION) {
             return false;
         }
         
-        foreach($array['hashes'] as $entry) 
+        foreach ($array['hashes'] as $entry) 
         {
             $string = Localization_Scanner_StringInfo::fromArray($this, $entry);
             $this->add($string);
@@ -176,7 +176,7 @@ 
     {
         $result = array();
         
-        foreach($this->warnings as $def) {
+        foreach ($this->warnings as $def) {
             $result[] = new Localization_Scanner_StringsCollection_Warning($def);
         }
         
@@ -191,7 +191,7 @@ 
     public function countFiles() : int
     {
         $amount = 0;
-        foreach($this->hashes as $hash) {
+        foreach ($this->hashes as $hash) {
             $amount = $amount + $hash->countFiles();
         }
         
@@ -208,8 +208,8 @@ 
     {
         $hashes = array();
         
-        foreach($this->hashes as $hash) {
-            if($hash->hasSourceID($id)) {
+        foreach ($this->hashes as $hash) {
+            if ($hash->hasSourceID($id)) {
                 $hashes[] = $hash;
             }
         }
@@ -227,8 +227,8 @@ 
     {
         $hashes = array();
         
-        foreach($this->hashes as $hash) {
-            if($hash->hasLanguageType($languageID)) {
+        foreach ($this->hashes as $hash) {
+            if ($hash->hasLanguageType($languageID)) {
                 $hashes[] = $hash;
             }
         }

src/Localization/Editor/OutputBuffering.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     {
         $html = ob_get_clean();
 
-        if($html !== false)
+        if ($html !== false)
         {
             return $html;
         }

src/Localization/Editor.php

Indentation +180 added lines, -180 removed lines

@@ -37,54 +37,54 @@ 
     const ERROR_NO_SOURCES_AVAILABLE = 40001;
     const ERROR_LOCAL_PATH_NOT_FOUND = 40002;
 
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $installPath;
     
-   /**
-    * @var Localization_Source[]
-    */
+    /**
+     * @var Localization_Source[]
+     */
     protected $sources;
     
-   /**
-    * @var Request
-    */
+    /**
+     * @var Request
+     */
     protected $request;
     
-   /**
-    * @var Localization_Source
-    */
+    /**
+     * @var Localization_Source
+     */
     protected $activeSource;
     
-   /**
-    * @var Localization_Scanner
-    */
+    /**
+     * @var Localization_Scanner
+     */
     protected $scanner;
     
-   /**
-    * @var Localization_Locale[]
-    */
+    /**
+     * @var Localization_Locale[]
+     */
     protected $appLocales = array();
     
-   /**
-    * @var Localization_Locale
-    */
+    /**
+     * @var Localization_Locale
+     */
     protected $activeAppLocale;
     
-   /**
-    * @var Localization_Editor_Filters
-    */
+    /**
+     * @var Localization_Editor_Filters
+     */
     protected $filters;
 
-   /**
-    * @var array<string,string>
-    */
+    /**
+     * @var array<string,string>
+     */
     protected $requestParams = array();
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $varPrefix = 'applocalize_';
 
     /**
@@ -125,15 +125,15 @@ 
         return $this->request;
     }
     
-   /**
-    * Adds a request parameter that will be persisted in all URLs
-    * within the editor. This can be used when integrating the
-    * editor in an existing page that needs specific request params.
-    * 
-    * @param string $name
-    * @param string $value
-    * @return Localization_Editor
-    */
+    /**
+     * Adds a request parameter that will be persisted in all URLs
+     * within the editor. This can be used when integrating the
+     * editor in an existing page that needs specific request params.
+     * 
+     * @param string $name
+     * @param string $value
+     * @return Localization_Editor
+     */
     public function addRequestParam(string $name, string $value) : Localization_Editor
     {
         $this->requestParams[$name] = $value;
@@ -295,34 +295,34 @@ 
                                 	</a>
                                     <div class="dropdown-menu" aria-labelledby="dropdown01">
                                     	<?php 
-                                    	    foreach($this->sources as $source)
-                                    	    {
-                                    	       ?>
+                                            foreach($this->sources as $source)
+                                            {
+                                                ?>
                                         			<a class="dropdown-item" href="<?php echo $this->getSourceURL($source) ?>">
                                         				<?php 
-                                            				if($source->getID() === $this->activeSource->getID()) 
-                                            				{
-                                            				    ?>
+                                                            if($source->getID() === $this->activeSource->getID()) 
+                                                            {
+                                                                ?>
                                             				    	<b><?php echo $source->getLabel() ?></b>
                                         				    	<?php 
-                                            				}
-                                            				else
-                                            				{
-                                            				    echo $source->getLabel();
-                                            				}
-                                        				?>
+                                                            }
+                                                            else
+                                                            {
+                                                                echo $source->getLabel();
+                                                            }
+                                                        ?>
                                         				<?php
-                                        				    $untranslated = $source->countUntranslated($this->scanner);
-                                        				    if($untranslated > 0) {
-                                        				        ?>
+                                                            $untranslated = $source->countUntranslated($this->scanner);
+                                                            if($untranslated > 0) {
+                                                                ?>
                                         				        	(<span class="text-danger" title="<?php ptex('%1$s texts have not been translated in this text source.', 'Amount of texts', $untranslated) ?>"><?php echo $untranslated ?></span>)
                                 				            	<?php 
-                                        				    }
-                                    				    ?>
+                                                            }
+                                                        ?>
                                     				</a>
                                     			<?php 
-                                    	    }
-                                	    ?>
+                                            }
+                                        ?>
                                     </div>
                                 </li>
                                 <li class="nav-item dropdown">
@@ -331,15 +331,15 @@ 
                                 	</a>
                                     <div class="dropdown-menu" aria-labelledby="dropdown01">
                                     	<?php 
-                                    	    foreach($this->appLocales as $locale)
-                                    	    {
-                                    	       ?>
+                                            foreach($this->appLocales as $locale)
+                                            {
+                                                ?>
                                         			<a class="dropdown-item" href="<?php echo $this->getLocaleURL($locale) ?>">
                                         				<?php echo $locale->getLabel() ?>
                                     				</a>
                                     			<?php 
-                                    	    }
-                                	    ?>
+                                            }
+                                        ?>
                                     </div>
                                 </li>
                                 <li class="nav-item">
@@ -349,8 +349,8 @@ 
                                     </a>
                     			</li>
                     			<?php 
-                        			if($this->scanner->hasWarnings()) {
-                        			    ?>
+                                    if($this->scanner->hasWarnings()) {
+                                        ?>
                         			    	<li class="nav-item">
                         			    		<a href="<?php echo $this->getWarningsURL() ?>">
                             			    		<span class="badge badge-warning" title="<?php pts('The last scan for translatable texts reported warnings.'); pts('Click for details.'); ?>" data-toggle="tooltip">
@@ -360,8 +360,8 @@ 
                         			    		</a>
                         			    	</li>
                         			    <?php 
-                        			}
-                    			?>
+                                    }
+                                ?>
                             </ul>
                         <?php 
                     }
@@ -377,36 +377,36 @@ 
                         	</a>
                     	<?php 
                     }
-            	?>
+                ?>
     		</div>
 		</nav>
 		<main role="main" class="container">
 			<div>
     			<?php 
-    			    if(empty($this->appLocales))
-    			    {
-    			        ?>
+                    if(empty($this->appLocales))
+                    {
+                        ?>
     			        	<div class="alert alert-danger">
     			        		<i class="fa fa-exclamation-triangle"></i>
     			        		<b><?php pt('Nothing to translate:') ?></b>
     			        		<?php pt('No application locales were added to translate to.') ?>
     			        	</div>
     			        <?php 
-    			    }
-    			    else if($this->request->getBool($this->getVarName('warnings')))
-    			    {
-    			        echo $this->renderWarnings();
-    			    }
-    			    else
-    			    {
-    			        ?>
+                    }
+                    else if($this->request->getBool($this->getVarName('warnings')))
+                    {
+                        echo $this->renderWarnings();
+                    }
+                    else
+                    {
+                        ?>
             				<h1><?php echo $this->activeSource->getLabel() ?></h1>
             				<?php 
-                				if(!empty($_SESSION['localization_messages'])) 
-                				{
-                				    foreach($_SESSION['localization_messages'] as $def)
-                				    {
-                				        ?>
+                                if(!empty($_SESSION['localization_messages'])) 
+                                {
+                                    foreach($_SESSION['localization_messages'] as $def)
+                                    {
+                                        ?>
                 				        	<div class="alert alert-<?php echo $def['type'] ?>" role="alert">
                                         		<?php echo $def['text'] ?>
                                         		<button type="button" class="close" data-dismiss="alert" aria-label="<?php pt('Close') ?>" title="<?php pt('Dismiss this message.') ?>" data-toggle="tooltip">
@@ -414,28 +414,28 @@ 
             									</button>
                                         	</div>
             				        	<?php 
-                				    }
+                                    }
                 				    
-                				    // reset the messages after having displayed them
-                				    $_SESSION['localization_messages'] = array();
-                				}
-            				?>
+                                    // reset the messages after having displayed them
+                                    $_SESSION['localization_messages'] = array();
+                                }
+                            ?>
             				<p>
             					<?php 
-            				        pt(
-                					    'You are translating to %1$s', 
-                					    '<span class="badge badge-info">'.
-                					       $this->activeAppLocale->getLabel().
-                				        '</span>'
+                                    pt(
+                                        'You are translating to %1$s', 
+                                        '<span class="badge badge-info">'.
+                                            $this->activeAppLocale->getLabel().
+                                        '</span>'
                                     );
-            				    ?><br>
+                                ?><br>
             					<?php pt('Found %1$s texts to translate.', $this->activeSource->countUntranslated($this->scanner)) ?>
             				</p>
             				<br>
             				<?php
-                				if(!$this->scanner->isScanAvailable()) 
-                				{
-                				    ?>
+                                if(!$this->scanner->isScanAvailable()) 
+                                {
+                                    ?>
                 				    	<div class="alert alert-primary" role="alert">
                                         	<b><?php pt('No texts found:') ?></b> 
                                         	<?php pt('The source folders have not been scanned yet.') ?>
@@ -447,15 +447,15 @@ 
                                             </a>
                                         </p>
                 				    <?php 
-                				}
-                				else
-                				{
-                				    echo $this->filters->renderForm();
+                                }
+                                else
+                                {
+                                    echo $this->filters->renderForm();
 
-                				    $this->displayList();
-                				}
-        				}
-    				?>
+                                    $this->displayList();
+                                }
+                        }
+                    ?>
 			</div>
 		</main>
 	</body>
@@ -473,22 +473,22 @@ 
         	<h1><?php pt('Warnings') ?></h1>
         	<p class="abstract">
         		<?php 
-        		    pts('The following shows all texts where the system decided that they cannot be translated.');
-       		    ?>
+                    pts('The following shows all texts where the system decided that they cannot be translated.');
+                    ?>
         	</p>
         	<dl>
         		<?php 
-        		    $warnings = $this->scanner->getWarnings();
+                    $warnings = $this->scanner->getWarnings();
         		    
-        		    foreach($warnings as $warning)
-        		    {
-        		        ?>
+                    foreach($warnings as $warning)
+                    {
+                        ?>
         		        	<dt><?php echo FileHelper::relativizePathByDepth($warning->getFile(), 3) ?>:<?php echo $warning->getLine() ?></dt>
         		        	<dd><?php echo $warning->getMessage() ?></dd>
         		        <?php 
-        		    }
+                    }
         		        
-        		?>
+                ?>
         	</dl>
     	<?php 
 
@@ -558,13 +558,13 @@ 
 			<form method="post">
 				<div class="form-hiddens">
 					<?php 
-    					$params = $this->getRequestParams();
-    					foreach($params as $name => $value) {
-    					    ?>
+                        $params = $this->getRequestParams();
+                        foreach($params as $name => $value) {
+                            ?>
     					    	<input type="hidden" name="<?php echo $name ?>" value="<?php echo $value ?>">
     					    <?php 
-    					}
-					?>
+                        }
+                    ?>
 				</div>
             	<table class="table table-hover">
     				<thead>
@@ -577,20 +577,20 @@ 
     				</thead>
     				<tbody>
     					<?php 
-    					    foreach($keep as $string)
-    					    {
-    					        $this->renderListEntry($string);
-    					    }
-    					?>
+                            foreach($keep as $string)
+                            {
+                                $this->renderListEntry($string);
+                            }
+                        ?>
     				</tbody>
     			</table>
     			<?php 
-        			if($pager->hasPages()) 
-        			{
-        			    $prevUrl = $this->getPaginationURL($pager->getPreviousPage());
-        			    $nextUrl = $this->getPaginationURL($pager->getNextPage());
+                    if($pager->hasPages()) 
+                    {
+                        $prevUrl = $this->getPaginationURL($pager->getPreviousPage());
+                        $nextUrl = $this->getPaginationURL($pager->getNextPage());
         			    
-        			    ?>
+                        ?>
         			    	<nav aria-label="<?php pt('Navigate available pages of texts.') ?>">
                                 <ul class="pagination">
                                     <li class="page-item">
@@ -599,20 +599,20 @@ 
                                 		</a>
                             		</li>
                             		<?php 
-                            		    $numbers = $pager->getPageNumbers();
-                            		    foreach($numbers as $number) 
-                            		    {
-                            		        $url = $this->getPaginationURL($number);
+                                        $numbers = $pager->getPageNumbers();
+                                        foreach($numbers as $number) 
+                                        {
+                                            $url = $this->getPaginationURL($number);
                             		        
-                            		        ?>
+                                            ?>
                             		        	<li class="page-item <?php if($pager->isCurrentPage($number)) { echo 'active'; } ?>">
                             		        		<a class="page-link" href="<?php echo $url ?>">
                             		        			<?php echo $number ?>
                         		        			</a>
                         		        		</li>
                             		        <?php 
-                            		    }
-                            		?>
+                                        }
+                                    ?>
                                     <li class="page-item">
                                     	<a class="page-link" href="<?php echo $nextUrl ?>">
                                     		<i class="fa fa-arrow-right"></i>
@@ -621,8 +621,8 @@ 
                                 </ul>
                             </nav>
         			    <?php 
-        			}
-    			?>
+                    }
+                ?>
 				<br>
 				<p>
 					<button type="submit" name="<?php echo $this->getVarName('save') ?>" value="yes" class="btn btn-primary">
@@ -693,46 +693,46 @@ 
         			<div class="files-list">
             			<p>
             				<?php 
-            				    $totalFiles = count($files);
+                                $totalFiles = count($files);
             				    
-            				    if($totalFiles == 1)
-            				    {
-            				        pt('Found in a single file:');
-            				    }
-            				    else
-            				    {
-            				        pt('Found in %1$s files:', $totalFiles);
-            				    }
-    				        ?>
+                                if($totalFiles == 1)
+                                {
+                                    pt('Found in a single file:');
+                                }
+                                else
+                                {
+                                    pt('Found in %1$s files:', $totalFiles);
+                                }
+                            ?>
             			</p>
         				<div class="files-scroller">
                 			<ul>
                 				<?php 
-                				    $locations = $string->getStrings();
+                                    $locations = $string->getStrings();
                 				    
-                    				foreach($locations as $location) 
-                    				{
-                    				    $file = $location->getSourceFile();
-                    				    $line = $location->getLine();
+                                    foreach($locations as $location) 
+                                    {
+                                        $file = $location->getSourceFile();
+                                        $line = $location->getLine();
                     				    
-                    				    $ext = FileHelper::getExtension($file);
+                                        $ext = FileHelper::getExtension($file);
                     				    
-                    				    if($ext == 'php') {
-                    				        $icon = 'fab fa-php';
-                    				    } else if($ext == 'js') {
-                    				        $icon = 'fab fa-js-square';
-                    				    } else {
-                    				        $icon = 'fas fa-file-code';
-                    				    }
+                                        if($ext == 'php') {
+                                            $icon = 'fab fa-php';
+                                        } else if($ext == 'js') {
+                                            $icon = 'fab fa-js-square';
+                                        } else {
+                                            $icon = 'fas fa-file-code';
+                                        }
                     				    
-                    				    ?>
+                                        ?>
                     				    	<li>
                     				    		<i class="<?php echo $icon ?>"></i>
                     				    		<?php echo $file ?><span class="line-number">:<?php echo $line ?></span>
                     				    	</li>
                     				    <?php 
-                    				}
-                				?>
+                                    }
+                                ?>
                 			</ul>
             			</div>
         			</div>
@@ -949,13 +949,13 @@ 
         );
     }
     
-   /**
-    * Sets the application name shown in the main navigation
-    * in the user interface.
-    * 
-    * @param string $name
-    * @return Localization_Editor
-    */
+    /**
+     * Sets the application name shown in the main navigation
+     * in the user interface.
+     * 
+     * @param string $name
+     * @return Localization_Editor
+     */
     public function setAppName(string $name) : Localization_Editor
     {
         $this->setOption('appname', $name);
@@ -985,15 +985,15 @@ 
         return $this;
     }
     
-   /**
-    * Sets an URL that the translators can use to go back to
-    * the main application, for example if it is integrated into
-    * an existing application.
-    * 
-    * @param string $url The URL to use for the link
-    * @param string $label Label of the link
-    * @return Localization_Editor
-    */
+    /**
+     * Sets an URL that the translators can use to go back to
+     * the main application, for example if it is integrated into
+     * an existing application.
+     * 
+     * @param string $url The URL to use for the link
+     * @param string $label Label of the link
+     * @return Localization_Editor
+     */
     public function setBackURL(string $url, string $label) : Localization_Editor
     {
         $this->setOption('back-url', $url);

Spacing +61 added lines, -61 removed lines

@@ -11,7 +11,7 @@ 
 
 namespace AppLocalize;
 
-use AppLocalize\Editor\OutputBuffering;use AppUtils\ConvertHelper;
+use AppLocalize\Editor\OutputBuffering; use AppUtils\ConvertHelper;
 use AppUtils\PaginationHelper;
 use AppUtils\Traits_Optionable;
 use AppUtils\Interface_Optionable;
@@ -99,7 +99,7 @@ 
     public function __construct()
     {
         $path = realpath(__DIR__.'/../');
-        if($path === false)
+        if ($path === false)
         {
             throw new Localization_Exception(
                 'Local path not found',
@@ -152,11 +152,11 @@ 
     
     protected function initSession() : void
     {
-        if(session_status() != PHP_SESSION_ACTIVE) {
+        if (session_status() != PHP_SESSION_ACTIVE) {
             session_start();
         }
         
-        if(!isset($_SESSION['localization_messages'])) {
+        if (!isset($_SESSION['localization_messages'])) {
             $_SESSION['localization_messages'] = array();
         }
     }
@@ -173,7 +173,7 @@ 
     {
         $this->sources = Localization::getSources();
         
-        if(empty($this->sources)) 
+        if (empty($this->sources)) 
         {
             throw new Localization_Exception(
                 'Cannot start editor: no sources defined.',
@@ -183,7 +183,7 @@ 
         }
         
         $activeID = $this->request->registerParam($this->getVarName('source'))->setEnum(Localization::getSourceIDs())->get();
-        if(empty($activeID)) {
+        if (empty($activeID)) {
             $activeID = $this->getDefaultSourceID();
         }
         
@@ -193,7 +193,7 @@ 
     protected function getDefaultSourceID() : string
     {
         $default = $this->getOption('default-source');
-        if(!empty($default) && Localization::sourceAliasExists($default)) {
+        if (!empty($default) && Localization::sourceAliasExists($default)) {
             return Localization::getSourceByAlias($default)->getID();
         }
         
@@ -205,21 +205,21 @@ 
         $names = array();
         
         $locales = Localization::getAppLocales();
-        foreach($locales as $locale) {
-            if(!$locale->isNative()) {
+        foreach ($locales as $locale) {
+            if (!$locale->isNative()) {
                 $this->appLocales[] = $locale;
                 $names[] = $locale->getName();
             }
         }
         
         // use the default locale if no other is available.
-        if(empty($names)) {
+        if (empty($names)) {
             $this->activeAppLocale = Localization::getAppLocale();
             return;
         }
        
         $activeID = $this->request->registerParam($this->getVarName('locale'))->setEnum($names)->get();
-        if(empty($activeID)) {
+        if (empty($activeID)) {
             $activeID = $this->appLocales[0]->getName();
         }
         
@@ -234,11 +234,11 @@ 
         
         $this->filters = new Localization_Editor_Filters($this);
         
-        if($this->request->getBool($this->getVarName('scan'))) 
+        if ($this->request->getBool($this->getVarName('scan'))) 
         {
             $this->executeScan();
         } 
-        else if($this->request->getBool($this->getVarName('save'))) 
+        else if ($this->request->getBool($this->getVarName('save'))) 
         {
             $this->executeSave();
         }
@@ -285,7 +285,7 @@ 
             </button>
             <div class="collapse navbar-collapse" id="navbarsExampleDefault">
                 <?php 
-                    if(!empty($this->appLocales))
+                    if (!empty($this->appLocales))
                     {
                         ?>
                             <ul class="navbar-nav mr-auto">
@@ -295,12 +295,12 @@ 
                                 	</a>
                                     <div class="dropdown-menu" aria-labelledby="dropdown01">
                                     	<?php 
-                                    	    foreach($this->sources as $source)
+                                    	    foreach ($this->sources as $source)
                                     	    {
                                     	       ?>
                                         			<a class="dropdown-item" href="<?php echo $this->getSourceURL($source) ?>">
                                         				<?php 
-                                            				if($source->getID() === $this->activeSource->getID()) 
+                                            				if ($source->getID() === $this->activeSource->getID()) 
                                             				{
                                             				    ?>
                                             				    	<b><?php echo $source->getLabel() ?></b>
@@ -313,7 +313,7 @@ 
                                         				?>
                                         				<?php
                                         				    $untranslated = $source->countUntranslated($this->scanner);
-                                        				    if($untranslated > 0) {
+                                        				    if ($untranslated > 0) {
                                         				        ?>
                                         				        	(<span class="text-danger" title="<?php ptex('%1$s texts have not been translated in this text source.', 'Amount of texts', $untranslated) ?>"><?php echo $untranslated ?></span>)
                                 				            	<?php 
@@ -331,7 +331,7 @@ 
                                 	</a>
                                     <div class="dropdown-menu" aria-labelledby="dropdown01">
                                     	<?php 
-                                    	    foreach($this->appLocales as $locale)
+                                    	    foreach ($this->appLocales as $locale)
                                     	    {
                                     	       ?>
                                         			<a class="dropdown-item" href="<?php echo $this->getLocaleURL($locale) ?>">
@@ -349,7 +349,7 @@ 
                                     </a>
                     			</li>
                     			<?php 
-                        			if($this->scanner->hasWarnings()) {
+                        			if ($this->scanner->hasWarnings()) {
                         			    ?>
                         			    	<li class="nav-item">
                         			    		<a href="<?php echo $this->getWarningsURL() ?>">
@@ -368,7 +368,7 @@ 
                 ?>
                 <?php 
                     $backURL = $this->getOption('back-url');
-                    if(!empty($backURL)) 
+                    if (!empty($backURL)) 
                     {
                         ?>
                             <a href="<?php echo $backURL ?>" class="btn btn-light btn-sm">
@@ -383,7 +383,7 @@ 
 		<main role="main" class="container">
 			<div>
     			<?php 
-    			    if(empty($this->appLocales))
+    			    if (empty($this->appLocales))
     			    {
     			        ?>
     			        	<div class="alert alert-danger">
@@ -393,7 +393,7 @@ 
     			        	</div>
     			        <?php 
     			    }
-    			    else if($this->request->getBool($this->getVarName('warnings')))
+    			    else if ($this->request->getBool($this->getVarName('warnings')))
     			    {
     			        echo $this->renderWarnings();
     			    }
@@ -402,9 +402,9 @@ 
     			        ?>
             				<h1><?php echo $this->activeSource->getLabel() ?></h1>
             				<?php 
-                				if(!empty($_SESSION['localization_messages'])) 
+                				if (!empty($_SESSION['localization_messages'])) 
                 				{
-                				    foreach($_SESSION['localization_messages'] as $def)
+                				    foreach ($_SESSION['localization_messages'] as $def)
                 				    {
                 				        ?>
                 				        	<div class="alert alert-<?php echo $def['type'] ?>" role="alert">
@@ -433,7 +433,7 @@ 
             				</p>
             				<br>
             				<?php
-                				if(!$this->scanner->isScanAvailable()) 
+                				if (!$this->scanner->isScanAvailable()) 
                 				{
                 				    ?>
                 				    	<div class="alert alert-primary" role="alert">
@@ -480,7 +480,7 @@ 
         		<?php 
         		    $warnings = $this->scanner->getWarnings();
         		    
-        		    foreach($warnings as $warning)
+        		    foreach ($warnings as $warning)
         		    {
         		        ?>
         		        	<dt><?php echo FileHelper::relativizePathByDepth($warning->getFile(), 3) ?>:<?php echo $warning->getLine() ?></dt>
@@ -504,9 +504,9 @@ 
         
         $result = array();
         
-        foreach($strings as $string)
+        foreach ($strings as $string)
         {
-            if($this->filters->isStringMatch($string)) {
+            if ($this->filters->isStringMatch($string)) {
                 $result[] = $string;
             }
         }
@@ -537,7 +537,7 @@ 
     {
         $strings = $this->getFilteredStrings();
         
-        if(empty($strings))
+        if (empty($strings))
         {
             ?>
             	<div class="alert alert-info">
@@ -559,7 +559,7 @@ 
 				<div class="form-hiddens">
 					<?php 
     					$params = $this->getRequestParams();
-    					foreach($params as $name => $value) {
+    					foreach ($params as $name => $value) {
     					    ?>
     					    	<input type="hidden" name="<?php echo $name ?>" value="<?php echo $value ?>">
     					    <?php 
@@ -577,7 +577,7 @@ 
     				</thead>
     				<tbody>
     					<?php 
-    					    foreach($keep as $string)
+    					    foreach ($keep as $string)
     					    {
     					        $this->renderListEntry($string);
     					    }
@@ -585,7 +585,7 @@ 
     				</tbody>
     			</table>
     			<?php 
-        			if($pager->hasPages()) 
+        			if ($pager->hasPages()) 
         			{
         			    $prevUrl = $this->getPaginationURL($pager->getPreviousPage());
         			    $nextUrl = $this->getPaginationURL($pager->getNextPage());
@@ -600,12 +600,12 @@ 
                             		</li>
                             		<?php 
                             		    $numbers = $pager->getPageNumbers();
-                            		    foreach($numbers as $number) 
+                            		    foreach ($numbers as $number) 
                             		    {
                             		        $url = $this->getPaginationURL($number);
                             		        
                             		        ?>
-                            		        	<li class="page-item <?php if($pager->isCurrentPage($number)) { echo 'active'; } ?>">
+                            		        	<li class="page-item <?php if ($pager->isCurrentPage($number)) { echo 'active'; } ?>">
                             		        		<a class="page-link" href="<?php echo $url ?>">
                             		        			<?php echo $number ?>
                         		        			</a>
@@ -635,7 +635,7 @@ 
         <?php 
     }
     
-    protected function getPaginationURL(int $page, array $params=array()) : string
+    protected function getPaginationURL(int $page, array $params = array()) : string
     {
         $params[$this->getVarName('page')] = $page;
         
@@ -648,11 +648,11 @@ 
         $text = $string->getText();
         
         $previewText = $string->getTranslatedText();
-        if(empty($previewText)) {
+        if (empty($previewText)) {
             $previewText = $text->getText();
         }
         
-        $shortText =  $this->renderText($previewText, 50);
+        $shortText = $this->renderText($previewText, 50);
         
         $files = $string->getFiles();
         
@@ -672,7 +672,7 @@ 
         			</p>
                     <?php
                         $explanation = $text->getExplanation();
-                        if(!empty($explanation))
+                        if (!empty($explanation))
                         {
                             ?>
                             <p>
@@ -695,7 +695,7 @@ 
             				<?php 
             				    $totalFiles = count($files);
             				    
-            				    if($totalFiles == 1)
+            				    if ($totalFiles == 1)
             				    {
             				        pt('Found in a single file:');
             				    }
@@ -710,16 +710,16 @@ 
                 				<?php 
                 				    $locations = $string->getStrings();
                 				    
-                    				foreach($locations as $location) 
+                    				foreach ($locations as $location) 
                     				{
                     				    $file = $location->getSourceFile();
                     				    $line = $location->getLine();
                     				    
                     				    $ext = FileHelper::getExtension($file);
                     				    
-                    				    if($ext == 'php') {
+                    				    if ($ext == 'php') {
                     				        $icon = 'fab fa-php';
-                    				    } else if($ext == 'js') {
+                    				    } else if ($ext == 'js') {
                     				        $icon = 'fab fa-js-square';
                     				    } else {
                     				        $icon = 'fas fa-file-code';
@@ -741,13 +741,13 @@ 
         <?php 
     }
     
-    protected function renderText(string $text, int $cutAt=0) : string
+    protected function renderText(string $text, int $cutAt = 0) : string
     {
-        if(empty($text)) {
+        if (empty($text)) {
             return '';
         }
 
-        if($cutAt > 0) {
+        if ($cutAt > 0) {
             $text = ConvertHelper::text_cut($text, $cutAt);
         }
 
@@ -755,7 +755,7 @@ 
         
         $vars = $this->detectVariables($text);
         
-        foreach($vars as $var) {
+        foreach ($vars as $var) {
             $text = str_replace($var, '<span class="placeholder">'.$var.'</span>', $text);
         }
         
@@ -767,7 +767,7 @@ 
         $result = array();
         preg_match_all('/%[0-9]+d|%s|%[0-9]+\$s/i', $string, $result, PREG_PATTERN_ORDER);
 
-        if(isset($result[0]) && !empty($result[0])) {
+        if (isset($result[0]) && !empty($result[0])) {
             return $result[0];
         }
         
@@ -783,10 +783,10 @@ 
         
         // add a counter of the additional files if the total
         // is higher than the maximum to show
-        if($total > $max) 
+        if ($total > $max) 
         {
             $length = $max;
-            if($length > $keepTotal) {
+            if ($length > $keepTotal) {
                 $length = $keepTotal; 
             }
             
@@ -814,14 +814,14 @@ 
         return FileHelper::readContents($this->installPath.'/css/editor.css');
     }
     
-    public function getSourceURL(Localization_Source $source, array $params=array()) : string
+    public function getSourceURL(Localization_Source $source, array $params = array()) : string
     {
         $params[$this->getVarName('source')] = $source->getID();
         
         return $this->getURL($params);
     }
     
-    public function getLocaleURL(Localization_Locale $locale, array $params=array()) : string
+    public function getLocaleURL(Localization_Locale $locale, array $params = array()) : string
     {
         $params[$this->getVarName('locale')] = $locale->getName();
         
@@ -838,12 +838,12 @@ 
         return $this->getSourceURL($this->activeSource, array($this->getVarName('warnings') => 'yes'));
     }
     
-    public function getURL(array $params=array()) : string
+    public function getURL(array $params = array()) : string
     {
         $persist = $this->getRequestParams();
         
-        foreach($persist as $name => $value) {
-            if(!isset($params[$name])) {
+        foreach ($persist as $name => $value) {
+            if (!isset($params[$name])) {
                 $params[$name] = $value;
             }
         }
@@ -880,11 +880,11 @@ 
         $translator = Localization::getTranslator($this->activeAppLocale);
         
         $strings = $data[$this->getVarName('strings')];
-        foreach($strings as $hash => $text) 
+        foreach ($strings as $hash => $text) 
         {
             $text = trim($text);
             
-            if(empty($text)) {
+            if (empty($text)) {
                 continue;
             } 
             
@@ -906,7 +906,7 @@ 
     
     protected function renderStatus(Localization_Scanner_StringHash $hash) : string
     {
-        if($hash->isTranslated()) {
+        if ($hash->isTranslated()) {
             return '<i class="fa fa-check text-success"></i>';
         }        
         
@@ -917,18 +917,18 @@ 
     {
         $types = array();
         
-        if($hash->hasLanguageType('PHP')) {
+        if ($hash->hasLanguageType('PHP')) {
             $types[] = t('Server');
         }
         
-        if($hash->hasLanguageType('Javascript')) {
+        if ($hash->hasLanguageType('Javascript')) {
             $types[] = t('Client');
         }
         
         return implode(', ', $types);
     }
     
-    protected function addMessage(string $message, string $type=self::MESSAGE_INFO) : void
+    protected function addMessage(string $message, string $type = self::MESSAGE_INFO) : void
     {
         $_SESSION['localization_messages'][] = array(
             'text' => $message,
@@ -965,7 +965,7 @@ 
     public function getAppName() : string
     {
         $name = $this->getOption('appname');
-        if(!empty($name)) {
+        if (!empty($name)) {
             return $name;
         }
         

Braces +6 added lines, -12 removed lines

@@ -237,8 +237,7 @@ 
         if($this->request->getBool($this->getVarName('scan'))) 
         {
             $this->executeScan();
-        } 
-        else if($this->request->getBool($this->getVarName('save'))) 
+        } else if($this->request->getBool($this->getVarName('save'))) 
         {
             $this->executeSave();
         }
@@ -305,8 +304,7 @@ 
                                             				    ?>
                                             				    	<b><?php echo $source->getLabel() ?></b>
                                         				    	<?php 
-                                            				}
-                                            				else
+                                            				} else
                                             				{
                                             				    echo $source->getLabel();
                                             				}
@@ -392,12 +390,10 @@ 
     			        		<?php pt('No application locales were added to translate to.') ?>
     			        	</div>
     			        <?php 
-    			    }
-    			    else if($this->request->getBool($this->getVarName('warnings')))
+    			    } else if($this->request->getBool($this->getVarName('warnings')))
     			    {
     			        echo $this->renderWarnings();
-    			    }
-    			    else
+    			    } else
     			    {
     			        ?>
             				<h1><?php echo $this->activeSource->getLabel() ?></h1>
@@ -447,8 +443,7 @@ 
                                             </a>
                                         </p>
                 				    <?php 
-                				}
-                				else
+                				} else
                 				{
                 				    echo $this->filters->renderForm();
 
@@ -698,8 +693,7 @@ 
             				    if($totalFiles == 1)
             				    {
             				        pt('Found in a single file:');
-            				    }
-            				    else
+            				    } else
             				    {
             				        pt('Found in %1$s files:', $totalFiles);
             				    }