Mistralys / application-localization

localization/index.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     $autoload = realpath($root.'/../vendor/autoload.php');
     
     // we need the autoloader to be present
-    if(!file_exists($autoload)) {
+    if (!file_exists($autoload)) {
         die('<b>ERROR:</b> Autoloader not present. Run composer update first.');
     }
     

example/index.php

Indentation +3 added lines, -3 removed lines

@@ -16,9 +16,9 @@
         die('<b>ERROR:</b> Autoloader not present. Run composer update first.');
     }
 
-   /**
-    * The composer autoloader
-    */
+    /**
+     * The composer autoloader
+     */
     require_once $autoload;
 
     // the folder in which the localization .ini files are stored

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     $autoload = realpath($root.'/../vendor/autoload.php');
     
     // we need the autoloader to be present
-    if($autoload === false) {
+    if ($autoload === false) {
         die('<b>ERROR:</b> Autoloader not present. Run composer update first.');
     }
 

example/sources/php/sources3.php

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@
     $text = t('This text is in the global scope.');
     
     $text2 = t(
-        'It is possible to write long texts ' .
-        'using text concatenation in the source ' .
+        'It is possible to write long texts '.
+        'using text concatenation in the source '.
         'code to keep it readable.'
     );
\ No newline at end of file

src/Localization/Source/Folder.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
     
     public function excludeFolder(string $folder) : Localization_Source_Folder
     {
-        if(!in_array($folder, $this->excludes['folders'])) {
+        if (!in_array($folder, $this->excludes['folders'])) {
             $this->excludes['folders'][] = $folder;
         }
         
@@ -58,7 +58,7 @@ 
     
     public function excludeFolders(array $folders) : Localization_Source_Folder
     {
-        foreach($folders as $folder) {
+        foreach ($folders as $folder) {
             $this->excludeFolder($folder);
         }
         

Indentation +14 added lines, -14 removed lines

@@ -23,24 +23,24 @@
  */
 class Localization_Source_Folder extends Localization_Source
 {
-   /**
-    * The folder under which all translatable files are kept.
-    * @var string
-    */
+    /**
+     * The folder under which all translatable files are kept.
+     * @var string
+     */
     protected $sourcesFolder;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $id;
 
-   /**
-    * @param string $alias An alias for this source, to recognize it by.
-    * @param string $label The human-readable label, used in the editor.
-    * @param string $group A human-readable group label to group several sources by. Used in the editor.
-    * @param string $storageFolder The folder in which to store the localization files.
-    * @param string $sourcesFolder The folder in which to analyze files to find translatable strings.
-    */
+    /**
+     * @param string $alias An alias for this source, to recognize it by.
+     * @param string $label The human-readable label, used in the editor.
+     * @param string $group A human-readable group label to group several sources by. Used in the editor.
+     * @param string $storageFolder The folder in which to store the localization files.
+     * @param string $sourcesFolder The folder in which to analyze files to find translatable strings.
+     */
     public function __construct(string $alias, string $label, string $group, string $storageFolder, string $sourcesFolder)
     {
         parent::__construct($alias, $label, $group, $storageFolder);

src/Localization/Event.php

Indentation +10 added lines, -10 removed lines

@@ -20,9 +20,9 @@ 
  */
 abstract class Localization_Event
 {
-   /**
-    * @var array
-    */
+    /**
+     * @var array
+     */
     protected $args;
     
     public function __construct(array $args)
@@ -30,13 +30,13 @@ 
         $this->args = $args;
     }
 
-   /**
-    * Fetches the argument at the specified index in the 
-    * event's arguments list, if it exists. 
-    * 
-    * @param int $index Zero-based index number.
-    * @return mixed|NULL
-    */ 
+    /**
+     * Fetches the argument at the specified index in the 
+     * event's arguments list, if it exists. 
+     * 
+     * @param int $index Zero-based index number.
+     * @return mixed|NULL
+     */ 
     public function getArgument(int $index)
     {
         if(isset($this->args[$index])) {

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     */ 
     public function getArgument(int $index)
     {
-        if(isset($this->args[$index])) {
+        if (isset($this->args[$index])) {
             return $this->args[$index];
         }
         

src/Localization/Generator.php

Braces +1 added lines, -2 removed lines

@@ -258,8 +258,7 @@
         if(empty($tokens))
         {
             $content = '/* No strings found. */';
-        }
-        else
+        } else
         {
             $content =
             '/**'.PHP_EOL.

Indentation +25 added lines, -25 removed lines

@@ -17,24 +17,24 @@ 
     const FILES_LOCALES = 'locales';
     const FILES_CACHE_KEY = 'cachekey';
     
-   /**
-    * @var bool
-    */
+    /**
+     * @var bool
+     */
     protected $force = false;
     
-   /**
-    * @var Localization_Translator
-    */
+    /**
+     * @var Localization_Translator
+     */
     protected $translator;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $targetFolder;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $cacheKeyFile;
 
     /**
@@ -119,14 +119,14 @@ 
         $this->writeCacheKey();
     }
     
-   /**
-    * Retrieves a list of all localization client 
-    * files that are written to disk. This includes
-    * the locale files and the libraries required
-    * to make it work clientside.
-    * 
-    * @return string[]
-    */
+    /**
+     * Retrieves a list of all localization client 
+     * files that are written to disk. This includes
+     * the locale files and the libraries required
+     * to make it work clientside.
+     * 
+     * @return string[]
+     */
     public function getFilesList() : array
     {
         $files = array();
@@ -279,11 +279,11 @@ 
         $this->written[self::FILES_LOCALES] = true;
     }
     
-   /**
-    * Generates the cache key file, which is used to determine
-    * automatically whether the client libraries need to be 
-    * refreshed.
-    */
+    /**
+     * Generates the cache key file, which is used to determine
+     * automatically whether the client libraries need to be 
+     * refreshed.
+     */
     protected function writeCacheKey() : void
     {
         if(file_exists($this->cacheKeyFile) && !$this->force) {

Spacing +20 added lines, -20 removed lines

@@ -66,7 +66,7 @@ 
     protected function initCache() : void
     {
         // ignore it if it does not exist.
-        if(!file_exists($this->cacheKeyFile)) {
+        if (!file_exists($this->cacheKeyFile)) {
             return;
         }
         
@@ -78,22 +78,22 @@ 
      * @throws Localization_Exception
      * @throws FileHelper_Exception
      */
-    public function writeFiles(bool $force=false) : void
+    public function writeFiles(bool $force = false) : void
     {
         // reset the write states for all files
         $fileIDs = array_keys($this->written);
-        foreach($fileIDs as $fileID) {
+        foreach ($fileIDs as $fileID) {
             $this->written[$fileID] = false;
         }
         
         // no client libraries folder set: ignore.
-        if(empty($this->targetFolder)) {
+        if (empty($this->targetFolder)) {
             return;
         }
         
         FileHelper::createFolder($this->targetFolder);
         
-        if(!is_writable($this->targetFolder)) 
+        if (!is_writable($this->targetFolder)) 
         {
             throw new Localization_Exception(
                 sprintf(
@@ -108,7 +108,7 @@ 
             );
         }
         
-        if($this->cacheKey !== Localization::getClientLibrariesCacheKey()) {
+        if ($this->cacheKey !== Localization::getClientLibrariesCacheKey()) {
             $force = true;
         }
         
@@ -131,16 +131,16 @@ 
     {
         $files = array();
         
-        foreach($this->libraries as $fileName)
+        foreach ($this->libraries as $fileName)
         {
             $files[] = $this->getLibraryFilePath($fileName);
         }
         
         $locales = Localization::getAppLocales();
         
-        foreach($locales as $locale)
+        foreach ($locales as $locale)
         {
-            if($locale->isNative()) {
+            if ($locale->isNative()) {
                 continue;
             }
             
@@ -154,9 +154,9 @@ 
     {
         $locales = Localization::getAppLocales();
         
-        foreach($locales as $locale)
+        foreach ($locales as $locale)
         {
-            if($locale->isNative()) {
+            if ($locale->isNative()) {
                 continue;
             }
             
@@ -180,7 +180,7 @@ 
     {
         $sourceFolder = realpath(__DIR__.'/../js');
         
-        if($sourceFolder === false) 
+        if ($sourceFolder === false) 
         {
             throw new Localization_Exception(
                 'Unexpected folder structure encountered.',
@@ -192,11 +192,11 @@ 
             );
         }
         
-        foreach($this->libraries as $fileName)
+        foreach ($this->libraries as $fileName)
         {
             $targetFile = $this->getLibraryFilePath($fileName);
             
-            if(file_exists($targetFile) && !$this->force) {
+            if (file_exists($targetFile) && !$this->force) {
                 continue;
             }
             
@@ -241,16 +241,16 @@ 
     {
         $path = $this->getLocaleFilePath($locale);
         
-        if(file_exists($path) && !$this->force) {
+        if (file_exists($path) && !$this->force) {
             return;
         }
         
         $strings = $this->translator->getClientStrings($locale);
         
         $tokens = array();
-        foreach($strings as $hash => $text)
+        foreach ($strings as $hash => $text)
         {
-            if(empty($text)) {
+            if (empty($text)) {
                 continue;
             }
             
@@ -261,7 +261,7 @@ 
             );
         }
         
-        if(empty($tokens))
+        if (empty($tokens))
         {
             $content = '/* No strings found. */';
         }
@@ -286,7 +286,7 @@ 
     */
     protected function writeCacheKey() : void
     {
-        if(file_exists($this->cacheKeyFile) && !$this->force) {
+        if (file_exists($this->cacheKeyFile) && !$this->force) {
             return;
         }
         
@@ -311,7 +311,7 @@ 
      */
     public function areFilesWritten(string $filesID) : bool
     {
-        if(isset($this->written[$filesID])) {
+        if (isset($this->written[$filesID])) {
             return $this->written[$filesID];
         }
         

src/Localization/Parser/Token/Javascript.php

Spacing +2 added lines, -2 removed lines

@@ -9,12 +9,12 @@
     protected function parseDefinition() : void
     {
         // some entries are strings, like parenthesises, semicolons and the like.
-        if(is_string($this->definition))
+        if (is_string($this->definition))
         {
             $this->token = $this->definition;
             $this->value = null;
             
-            if(isset($this->parentToken)) {
+            if (isset($this->parentToken)) {
                 $this->line = $this->parentToken->getLine();
             }
         }

Braces +1 added lines, -2 removed lines

@@ -19,8 +19,7 @@
             if(isset($this->parentToken)) {
                 $this->line = $this->parentToken->getLine();
             }
-        }
-        else
+        } else
         {
             $this->token = JTokenizer::getTokenName($this->definition[0]);
             $this->value = $this->definition[1];

src/Localization/Parser/Token/PHP.php

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
             if(isset($this->parentToken)) {
                 $this->line = $this->parentToken->getLine();
             }
-        }
-        else
+        } else
         {
             $this->token = token_name($this->definition[0]);
             $this->value = $this->definition[1];

Spacing +2 added lines, -2 removed lines

@@ -15,12 +15,12 @@
     protected function parseDefinition() : void
     {
         // some entries are strings, like parentheses, semicolons and the like.
-        if(is_string($this->definition))
+        if (is_string($this->definition))
         {
             $this->token = $this->definition;
             $this->value = null;
             
-            if(isset($this->parentToken)) {
+            if (isset($this->parentToken)) {
                 $this->line = $this->parentToken->getLine();
             }
         }

src/Localization/Editor/Filters.php

Braces +2 added lines, -4 removed lines

@@ -50,8 +50,7 @@ 
             foreach($defaults as $name => $val) {
                 $this->setValue($name, $val);
             }
-        }
-        else if($this->request->getBool($this->vars['filter'])) 
+        } else if($this->request->getBool($this->vars['filter'])) 
         {
             $this->parseSearchTerms($this->request->getParam($this->vars['search']));
             
@@ -72,8 +71,7 @@ 
                 ->setEnum('', 'client', 'server')
                 ->get('')
             );
-        }
-        else
+        } else
         {
             $this->parseSearchTerms($this->getValue($this->vars['search']));
         }

Indentation +55 added lines, -55 removed lines

@@ -23,25 +23,25 @@ 
  */
 class Localization_Editor_Filters
 {
-   /**
-    * @var Localization_Editor
-    */
+    /**
+     * @var Localization_Editor
+     */
     protected $editor;
     
-   /**
-    * @var Request
-    */
+    /**
+     * @var Request
+     */
     
     protected $request; 
      
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $sessionName = 'localize_filters';
     
-   /**
-    * @var string[]
-    */
+    /**
+     * @var string[]
+     */
     protected $vars = array(
         'resetfilter' => '',
         'filter' => '',
@@ -204,13 +204,13 @@ 
             <form class="form-inline">
             	<div class="form-hiddens">
             		<?php 
-    					$params = $this->editor->getRequestParams();
-    					foreach($params as $name => $value) {
-    					    ?>
+                        $params = $this->editor->getRequestParams();
+                        foreach($params as $name => $value) {
+                            ?>
     					    	<input type="hidden" name="<?php echo $name ?>" value="<?php echo $value ?>">
     					    <?php 
-    					}
-					?>
+                        }
+                    ?>
             	</div>
             	<div class="form-row">
             		<div class="col-auto">
@@ -218,42 +218,42 @@ 
     		        </div>
                     <div class="col-auto">
                     	<?php
-                    	    echo $this->renderSelect(
-                    	       $this->vars['status'],
-                    	       array(
-                        	       array(
-                        	            'value' => '',
-                        	            'label' => t('Status...')
-                        	       ),
-                        	       array(
-                        	           'value' => 'untranslated',
-                        	           'label' => t('Not translated')
-                        	       ),
-                        	       array(
-                        	           'value' => 'translated',
-                        	           'label' => t('Translated')
-                        	       )
-                    	       )
-                    	   );
+                            echo $this->renderSelect(
+                                $this->vars['status'],
+                                array(
+                                    array(
+                                        'value' => '',
+                                        'label' => t('Status...')
+                                    ),
+                                    array(
+                                        'value' => 'untranslated',
+                                        'label' => t('Not translated')
+                                    ),
+                                    array(
+                                        'value' => 'translated',
+                                        'label' => t('Translated')
+                                    )
+                                )
+                            );
                     	
-                    	   echo $this->renderSelect(
-                    	       $this->vars['location'],
-                    	       array(
-                    	           array(
-                    	               'value' => '',
-                    	               'label' => t('Location...')
-                    	           ),
-                    	           array(
-                    	               'value' => 'client',
-                    	               'label' => t('Clientside')
-                    	           ),
-                    	           array(
-                    	               'value' => 'server',
-                    	               'label' => t('Serverside')
-                    	           )
-                    	       )
-                	       );
-                    	?>
+                            echo $this->renderSelect(
+                                $this->vars['location'],
+                                array(
+                                    array(
+                                        'value' => '',
+                                        'label' => t('Location...')
+                                    ),
+                                    array(
+                                        'value' => 'client',
+                                        'label' => t('Clientside')
+                                    ),
+                                    array(
+                                        'value' => 'server',
+                                        'label' => t('Serverside')
+                                    )
+                                )
+                            );
+                        ?>
                     </div>
                     <div class="col-auto">
 	    				<button type="submit" name="<?php echo $this->vars['filter'] ?>" value="yes" class="btn btn-primary mb-2" title="<?php pt('Filter the list with the selected criteria.') ?>" data-toggle="tooltip">
@@ -268,9 +268,9 @@ 
             </form>
             <p>
             	<small class="text-muted"><?php 
-            	   pts('Hint:'); 
-            	   pt('Search works in translated and untranslated text, as well as the file name.') 
-        	   ?></small>
+                    pts('Hint:'); 
+                    pt('Search works in translated and untranslated text, as well as the file name.') 
+                ?></small>
             </p>
 			<br>
         <?php

Spacing +20 added lines, -20 removed lines

@@ -55,23 +55,23 @@ 
         $this->editor = $editor;
         $this->request = $editor->getRequest();
         
-        foreach($this->vars as $var => $name) {
+        foreach ($this->vars as $var => $name) {
             $this->vars[$var] = $this->editor->getVarName($var);
         }
 
-        if(!isset($_SESSION[$this->sessionName])) {
+        if (!isset($_SESSION[$this->sessionName])) {
             $_SESSION[$this->sessionName] = array();
         }
 
-        if($this->request->getBool($this->vars['resetfilter']))
+        if ($this->request->getBool($this->vars['resetfilter']))
         {
             $defaults = $this->getDefaultValues();
             
-            foreach($defaults as $name => $val) {
+            foreach ($defaults as $name => $val) {
                 $this->setValue($name, $val);
             }
         }
-        else if($this->request->getBool($this->vars['filter'])) 
+        else if ($this->request->getBool($this->vars['filter'])) 
         {
             $this->parseSearchTerms($this->request->getParam($this->vars['search']));
             
@@ -106,12 +106,12 @@ 
     
     protected function getValue(string $filterName) : string
     {
-        if(isset($_SESSION[$this->sessionName][$filterName])) {
+        if (isset($_SESSION[$this->sessionName][$filterName])) {
             return $_SESSION[$this->sessionName][$filterName];
         }
         
         $defaults = $this->getDefaultValues();
-        if(isset($defaults[$filterName])) {
+        if (isset($defaults[$filterName])) {
             return $defaults[$filterName];
         }
         
@@ -130,7 +130,7 @@ 
     
     protected function parseSearchTerms(string $searchString) : void
     {
-        if(empty($searchString)) 
+        if (empty($searchString)) 
         {
             $this->searchTerms = array();
             $this->searchString = '';
@@ -144,8 +144,8 @@ 
         $terms = array_map('trim', $terms);
         
         $keep = array();
-        foreach($terms as $term) {
-            if(!empty($term)) {
+        foreach ($terms as $term) {
+            if (!empty($term)) {
                 $keep[] = $term;
             }
         }
@@ -156,28 +156,28 @@ 
     
     public function isStringMatch(Localization_Scanner_StringHash $string) : bool
     {
-        if(!empty($this->searchTerms)) 
+        if (!empty($this->searchTerms)) 
         {
             $haystack = $string->getSearchString();
             
-            foreach($this->searchTerms as $term) {
-                if(!mb_stristr($haystack, $term)) {
+            foreach ($this->searchTerms as $term) {
+                if (!mb_stristr($haystack, $term)) {
                     return false;
                 }
             }
         }
         
         $status = $this->getValue($this->vars['status']);
-        if($status === 'untranslated' && $string->isTranslated()) {
+        if ($status === 'untranslated' && $string->isTranslated()) {
             return false;
-        } else if($status === 'translated' && !$string->isTranslated()) {
+        } else if ($status === 'translated' && !$string->isTranslated()) {
             return false;
         }
         
         $location = $this->getValue($this->vars['location']);
-        if($location === 'client' && !$string->hasLanguageType('Javascript')) {
+        if ($location === 'client' && !$string->hasLanguageType('Javascript')) {
             return false;
-        } else if($location === 'server' && !$string->hasLanguageType('PHP')) {
+        } else if ($location === 'server' && !$string->hasLanguageType('PHP')) {
             return false;
         }
         
@@ -205,7 +205,7 @@ 
             	<div class="form-hiddens">
             		<?php 
     					$params = $this->editor->getRequestParams();
-    					foreach($params as $name => $value) {
+    					foreach ($params as $name => $value) {
     					    ?>
     					    	<input type="hidden" name="<?php echo $name ?>" value="<?php echo $value ?>">
     					    <?php 
@@ -293,10 +293,10 @@ 
         ?>
         	<select class="form-control" name="<?php echo $filterName ?>">
         		<?php 
-                    foreach($entries as $entry) 
+                    foreach ($entries as $entry) 
                     {
                         $selected = '';
-                        if($entry['value'] === $value) {
+                        if ($entry['value'] === $value) {
                             $selected = ' selected';
                         }