bb-drummer / my-application

module/Application/src/Application/Model/DbStructUpdater.php

Indentation +90 added lines, -90 removed lines

@@ -60,20 +60,20 @@ 
     var $config = array();//updater configuration
 
     /**
-    * Constructor
+     * Constructor
      *
-    * @access public
-    */
+     * @access public
+     */
     public function __construct()
     {
         $this->init();
     }
 
     /**
-    * Constructor
+     * Constructor
      *
-    * @access public
-    */
+     * @access public
+     */
     public function dbStructUpdater()
     {
         $this->init();
@@ -96,10 +96,10 @@ 
     }
 
     /**
-    * merges current updater config with the given one
+     * merges current updater config with the given one
      *
-    * @param assoc_array $config new configuration values
-    */
+     * @param assoc_array $config new configuration values
+     */
     public function setConfig($config=array())
     {
         if (is_array($config)) {
@@ -108,14 +108,14 @@ 
     }
 
     /**
-    * Returns array of update SQL with default options, $source, $dest - database structures
+     * Returns array of update SQL with default options, $source, $dest - database structures
      *
-    * @access public
-    * @param  string $source   structure dump of database to update
-    * @param  string $dest     structure dump of the reference database
-    * @param  bool   $asString if true - result will be a string, otherwise - array
-    * @return array|string update sql statements - in array or string (separated with ';')
-    */
+     * @access public
+     * @param  string $source   structure dump of database to update
+     * @param  string $dest     structure dump of the reference database
+     * @param  bool   $asString if true - result will be a string, otherwise - array
+     * @return array|string update sql statements - in array or string (separated with ';')
+     */
     public function getUpdates($source, $dest, $asString=false)
     {
         $result = $asString?'':array();
@@ -135,8 +135,8 @@ 
     }
 
     /**
-    * Filters comparison result and lefts only sync actions allowed by 'updateTypes' option
-    */
+     * Filters comparison result and lefts only sync actions allowed by 'updateTypes' option
+     */
     public function filterDiffs($compRes)
     {
         $result = array();
@@ -185,9 +185,9 @@ 
     }
     

     /**
-    * Gets structured general info about the databases diff :
-    * array(sourceOrphans=>array(...), destOrphans=>array(...), different=>array(...))
-    */
+     * Gets structured general info about the databases diff :
+     * array(sourceOrphans=>array(...), destOrphans=>array(...), different=>array(...))
+     */
     public function getDiffInfo($compRes)
     {        

         if (!is_array($compRes)) {
@@ -211,20 +211,20 @@ 
     }
 
     /**
-    * Makes comparison of the given database structures, support some options
+     * Makes comparison of the given database structures, support some options
      *
-    * @access private
-    * @param  string $source and $dest are strings - database tables structures
-    * @return array
-    * - table (array)
-    *        - destOrphan (boolean)
-    *        - sourceOrphan (boolean)
-    *        - differs (array) OR (boolean) false if no diffs
-    *            - [0](array)
-    *                - source (string) structure definition line in the out-of-date table
-    *                - dest (string) structure definition line in the reference table
-    *            - [1](array) ...
-    */
+     * @access private
+     * @param  string $source and $dest are strings - database tables structures
+     * @return array
+     * - table (array)
+     *        - destOrphan (boolean)
+     *        - sourceOrphan (boolean)
+     *        - differs (array) OR (boolean) false if no diffs
+     *            - [0](array)
+     *                - source (string) structure definition line in the out-of-date table
+     *                - dest (string) structure definition line in the reference table
+     *            - [1](array) ...
+     */
     private function compare($source, $dest)
     {
         $this->sourceStruct = $source;
@@ -270,11 +270,11 @@ 
     }
 
     /**
-    * Retrieves list of table names from the database structure dump
+     * Retrieves list of table names from the database structure dump
      *
-    * @access private
-    * @param  string $struct database structure listing
-    */
+     * @access private
+     * @param  string $struct database structure listing
+     */
     private function getTableList($struct)
     {
         $result = array();
@@ -288,14 +288,14 @@ 
     }
 
     /**
-    * Retrieves table structure definition from the database structure dump
+     * Retrieves table structure definition from the database structure dump
      *
-    * @access private
-    * @param  string $struct         database structure listing
-    * @param  string $tab            table name
-    * @param  bool   $removeDatabase - either to remove database name in "CREATE TABLE database.tab"-like declarations
-    * @return string table structure definition
-    */
+     * @access private
+     * @param  string $struct         database structure listing
+     * @param  string $tab            table name
+     * @param  bool   $removeDatabase - either to remove database name in "CREATE TABLE database.tab"-like declarations
+     * @return string table structure definition
+     */
     private function getTabSql($struct, $tab, $removeDatabase=true)
     {
         $result = '';
@@ -323,8 +323,8 @@ 
     }
     

     /**
-    * Splits table sql into indexed array
-    */
+     * Splits table sql into indexed array
+     */
     public function splitTabSql($sql)
     {
         $result = array();
@@ -365,18 +365,18 @@ 
     }
 
     /**
-    * returns array of fields or keys definitions that differs in the given tables structure
+     * returns array of fields or keys definitions that differs in the given tables structure
      *
-    * @access private
-    * @param  sring $sourceSql table structure
-    * @param  sring $destSql   right table structure supports some $options
-    * supports some $options
-    * @return array
-    *     - [0]
-    *         - source (string) out-of-date table field definition
-    *         - dest (string) reference table field definition
-    *     - [1]...
-    */
+     * @access private
+     * @param  sring $sourceSql table structure
+     * @param  sring $destSql   right table structure supports some $options
+     * supports some $options
+     * @return array
+     *     - [0]
+     *         - source (string) out-of-date table field definition
+     *         - dest (string) reference table field definition
+     *     - [1]...
+     */
     private function compareSql($sourceSql, $destSql)//$sourceSql, $destSql
     {
         $result = array();        

@@ -439,14 +439,14 @@ 
     }
 
     /**
-    * Transforms table structure defnition line into key=>value pair where the key is a string that uniquely
-    * defines field or key desribed
+     * Transforms table structure defnition line into key=>value pair where the key is a string that uniquely
+     * defines field or key desribed
      *
-    * @access private
-    * @param  string $line field definition string
-    * @return array array with single key=>value pair as described in the description
-    * implements some options
-    */
+     * @access private
+     * @param  string $line field definition string
+     * @return array array with single key=>value pair as described in the description
+     * implements some options
+     */
     private function processLine($line)
     {
         $options = $this->config;
@@ -484,14 +484,14 @@ 
     }
 
     /**
-    * Takes an output of compare() method to generate the set of sql needed to update source table to make it
-    * look as a destination one
+     * Takes an output of compare() method to generate the set of sql needed to update source table to make it
+     * look as a destination one
      *
-    * @access private
-    * @param  array $diff compare() method output
-    * @return array list of sql statements
-    * supports query generation options
-    */
+     * @access private
+     * @param  array $diff compare() method output
+     * @return array list of sql statements
+     * supports query generation options
+     */
     private function getDiffSql($diff)//maybe add option to ommit or force 'IF NOT EXISTS', skip autoincrement
     {
         $options = $this->config;
@@ -548,14 +548,14 @@ 
     }
 
     /**
-    * Compiles update sql
+     * Compiles update sql
      *
-    * @access private
-    * @param  string $action - 'drop', 'add' or 'modify'
-    * @param  string $tab    table name
-    * @param  string $sql    definition of the element to change
-    * @return string update sql
-    */
+     * @access private
+     * @param  string $action - 'drop', 'add' or 'modify'
+     * @param  string $tab    table name
+     * @param  string $sql    definition of the element to change
+     * @return string update sql
+     */
     private function getActionSql($action, $tab, $sql)
     {
         $result = 'ALTER TABLE `'.$tab.'` ';
@@ -621,14 +621,14 @@ 
     }
 
     /**
-    * Searches for the position of the next delimiter which is not inside string literal like 'this ; ' or
-    * like "this ; ".
-    *
-    * Handles escaped \" and \'. Also handles sql comments.
-    * Actualy it is regex-based Finit State Machine (FSN)
-    *
-    * $skipInBrackets - if true, delimiter will be skipped if located inside (brackets) which are not string literals or comment parts
-    */
+     * Searches for the position of the next delimiter which is not inside string literal like 'this ; ' or
+     * like "this ; ".
+     *
+     * Handles escaped \" and \'. Also handles sql comments.
+     * Actualy it is regex-based Finit State Machine (FSN)
+     *
+     * $skipInBrackets - if true, delimiter will be skipped if located inside (brackets) which are not string literals or comment parts
+     */
     public function getDelimPos($string, $offset=0, $delim=';', $skipInBrackets=false)
     {
         $stack = array();
@@ -701,9 +701,9 @@ 
     }
     

     /**
-    * works the same as getDelimPos except returns position of the first occurence of the delimiter starting from
-    * the end of the string
-    */
+     * works the same as getDelimPos except returns position of the first occurence of the delimiter starting from
+     * the end of the string
+     */
     public function getDelimRpos($string, $offset=0, $delim=';', $skipInBrackets=false)
     {
         $pos = $this->getDelimPos($string, $offset, $delim, $skipInBrackets);

Switch Indentation +33 added lines, -33 removed lines

@@ -568,39 +568,39 @@
             $fields = trim($m[3]);
             switch($action)
             {
-            case 'drop':
-                if ($type=='primary') {
-                    $result.= 'DROP PRIMARY KEY';
-                }
-                else
-                {
-                    $result.= 'DROP INDEX `'.$name.'`';
-                }
-                break;
-            case 'add':
-                if ($type=='primary') {
-                    $result.= 'ADD PRIMARY KEY '.$fields;
-                }
-                elseif ($type=='') {
-                    $result.= 'ADD INDEX `'.$name.'` '.$fields;
-                }
-                else
-                {
-                    $result .='ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
-                }
-                break;
-            case 'modify':
-                if ($type=='primary') {
-                    $result.='DROP PRIMARY KEY, ADD PRIMARY KEY '.$fields;
-                }
-                elseif ($type=='') {
-                    $result.='DROP INDEX `'.$name.'`, ADD INDEX `'.$name.'` '.$fields;
-                }
-                else
-                {
-                    $result.='DROP INDEX `'.$name.'`, ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
-                }
-                break;
+                case 'drop':
+                    if ($type=='primary') {
+                        $result.= 'DROP PRIMARY KEY';
+                    }
+                    else
+                    {
+                        $result.= 'DROP INDEX `'.$name.'`';
+                    }
+                    break;
+                case 'add':
+                    if ($type=='primary') {
+                        $result.= 'ADD PRIMARY KEY '.$fields;
+                    }
+                    elseif ($type=='') {
+                        $result.= 'ADD INDEX `'.$name.'` '.$fields;
+                    }
+                    else
+                    {
+                        $result .='ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
+                    }
+                    break;
+                case 'modify':
+                    if ($type=='primary') {
+                        $result.='DROP PRIMARY KEY, ADD PRIMARY KEY '.$fields;
+                    }
+                    elseif ($type=='') {
+                        $result.='DROP INDEX `'.$name.'`, ADD INDEX `'.$name.'` '.$fields;
+                    }
+                    else
+                    {
+                        $result.='DROP INDEX `'.$name.'`, ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
+                    }
+                    break;
 
             }
         }

Spacing +74 added lines, -74 removed lines

@@ -55,9 +55,9 @@ 
 */
 class DbStructUpdater
 {
-    var $sourceStruct = '';//structure dump of the reference database
-    var $destStruct = '';//structure dump of database to update
-    var $config = array();//updater configuration
+    var $sourceStruct = ''; //structure dump of the reference database
+    var $destStruct = ''; //structure dump of database to update
+    var $config = array(); //updater configuration
 
     /**
     * Constructor
@@ -100,7 +100,7 @@ 
      *
     * @param assoc_array $config new configuration values
     */
-    public function setConfig($config=array())
+    public function setConfig($config = array())
     {
         if (is_array($config)) {
             $this->config = array_merge($this->config, $config);
@@ -116,9 +116,9 @@ 
     * @param  bool   $asString if true - result will be a string, otherwise - array
     * @return array|string update sql statements - in array or string (separated with ';')
     */
-    public function getUpdates($source, $dest, $asString=false)
+    public function getUpdates($source, $dest, $asString = false)
     {
-        $result = $asString?'':array();
+        $result = $asString ? '' : array();
         $compRes = $this->compare($source, $dest);
         if (empty($compRes)) {
             return $result;
@@ -149,7 +149,7 @@ 
         }
         $allowedActions = array('create', 'drop', 'add', 'remove', 'modify');
         $updateActions = array_intersect($updateActions, $allowedActions);
-        foreach($compRes as $table=>$info)
+        foreach ($compRes as $table=>$info)
         {
             if ($info['sourceOrphan']) {
                 if (in_array('create', $updateActions)) {
@@ -161,7 +161,7 @@ 
                     $result[$table] = $info;
                 }
             }
-            elseif($info['differs']) {
+            elseif ($info['differs']) {
                 $resultInfo = $info;
                 unset($resultInfo['differs']);
                 foreach ($info['differs'] as $diff)
@@ -172,7 +172,7 @@ 
                     elseif (empty($diff['source']) && in_array('remove', $updateActions)) {
                         $resultInfo['differs'][] = $diff;
                     }
-                    elseif(in_array('modify', $updateActions)) {
+                    elseif (in_array('modify', $updateActions)) {
                         $resultInfo['differs'][] = $diff;
                     }
                 }
@@ -194,7 +194,7 @@ 
             return false;
         }
         $result = array('sourceOrphans'=>array(), 'destOrphans'=>array(), 'different'=>array());
-        foreach($compRes as $table=>$info)
+        foreach ($compRes as $table=>$info)
         {
             if ($info['sourceOrphan']) {
                 $result['sourceOrphans'][] = $table;
@@ -242,7 +242,7 @@ 
         foreach ($all as $tab)
         {
             $info = array('destOrphan'=>false, 'sourceOrphan'=>false, 'differs'=>false);
-            if(in_array($tab, $destOrphans)) {
+            if (in_array($tab, $destOrphans)) {
                 $info['destOrphan'] = true;
             }
             elseif (in_array($tab, $sourceOrphans)) {
@@ -253,7 +253,7 @@ 
                 $destSql = $this->getTabSql($this->destStruct, $tab, true);
                 $sourceSql = $this->getTabSql($this->sourceStruct, $tab, true);
                 $diffs = $this->compareSql($sourceSql, $destSql);                

-                if ($diffs===false) {
+                if ($diffs === false) {
                     trigger_error('[WARNING] error parsing definition of table "'.$tab.'" - skipped');
                     continue;
                 }
@@ -261,7 +261,7 @@ 
                 {
                     $info['differs'] = $diffs;                    

                 }                

-                else { continue;//empty array
+                else { continue; //empty array
                 }         
             }
             $result[$tab] = $info;
@@ -279,7 +279,7 @@ 
     {
         $result = array();
         if (preg_match_all('/CREATE(?:\s*TEMPORARY)?\s*TABLE\s*(?:IF NOT EXISTS\s*)?(?:`?(\w+)`?\.)?`?(\w+)`?/i', $struct, $m)) {
-            foreach($m[2] as $match)//m[1] is a database name if any
+            foreach ($m[2] as $match)//m[1] is a database name if any
             {
                 $result[] = $match;
             }
@@ -296,7 +296,7 @@ 
     * @param  bool   $removeDatabase - either to remove database name in "CREATE TABLE database.tab"-like declarations
     * @return string table structure definition
     */
-    private function getTabSql($struct, $tab, $removeDatabase=true)
+    private function getTabSql($struct, $tab, $removeDatabase = true)
     {
         $result = '';
         /* create table should be single line in this case*/
@@ -305,14 +305,14 @@ 
             $tableDef = $m[0][0];
             $start = $m[0][1];
             $database = $m[2][0];
-            $offset = $start+strlen($m[0][0]);
+            $offset = $start + strlen($m[0][0]);
             $end = $this->getDelimPos($struct, $offset);
             if ($end === false) {
                 $result = substr($struct, $start);
             }
             else
             {
-                $result = substr($struct, $start, $end-$start);//already without ';'
+                $result = substr($struct, $start, $end - $start); //already without ';'
             }
         }
         $result = trim($result);
@@ -330,25 +330,25 @@ 
         $result = array();
         //find opening bracket, get the prefix along with it
         $openBracketPos = $this->getDelimPos($sql, 0, '(');
-        if ($openBracketPos===false) {
+        if ($openBracketPos === false) {
             trigger_error('[WARNING] can not find opening bracket in table definition');
             return false;
         }
-        $prefix = substr($sql, 0, $openBracketPos+1);//prefix can not be empty, so do not check it, just trim
+        $prefix = substr($sql, 0, $openBracketPos + 1); //prefix can not be empty, so do not check it, just trim
         $result[] = trim($prefix);
-        $body = substr($sql, strlen($prefix));//fields, indexes and part after closing bracket
+        $body = substr($sql, strlen($prefix)); //fields, indexes and part after closing bracket
         //split by commas, get part by part
-        while(($commaPos = $this->getDelimPos($body, 0, ',', true))!==false)
+        while (($commaPos = $this->getDelimPos($body, 0, ',', true)) !== false)
         {
-            $part = trim(substr($body, 0, $commaPos+1));//read another part and shorten $body
+            $part = trim(substr($body, 0, $commaPos + 1)); //read another part and shorten $body
             if ($part) {
                 $result[] = $part;
             }
-            $body = substr($body, $commaPos+1);
+            $body = substr($body, $commaPos + 1);
         }
         //here we have last field (or index) definition + part after closing bracket (ENGINE, ect)
         $closeBracketPos = $this->getDelimRpos($body, 0, ')');
-        if ($closeBracketPos===false) {
+        if ($closeBracketPos === false) {
             trigger_error('[WARNING] can not find closing bracket in table definition');
             return false;
         }
@@ -381,27 +381,27 @@ 
     {
         $result = array();        

         //split with comma delimiter, not line breaks
-        $sourceParts =  $this->splitTabSql($sourceSql);
-        if ($sourceParts===false)//error parsing sql
+        $sourceParts = $this->splitTabSql($sourceSql);
+        if ($sourceParts === false)//error parsing sql
         {
             trigger_error('[WARNING] error parsing source sql');
             return false;
         }
         $destParts = $this->splitTabSql($destSql);
-        if ($destParts===false) {
+        if ($destParts === false) {
             trigger_error('[WARNING] error parsing destination sql');
             return false;
         }
         $sourcePartsIndexed = array();
         $destPartsIndexed = array();
-        foreach($sourceParts as $line)
+        foreach ($sourceParts as $line)
         {            

             $lineInfo = $this->processLine($line);
             if (!$lineInfo) { continue; 
             }
             $sourcePartsIndexed[$lineInfo['key']] = $lineInfo['line'];
         }
-        foreach($destParts as $line)
+        foreach ($destParts as $line)
         {            

             $lineInfo = $this->processLine($line);
             if (!$lineInfo) { continue; 
@@ -411,16 +411,16 @@ 
         $sourceKeys = array_keys($sourcePartsIndexed);
         $destKeys = array_keys($destPartsIndexed);
         $all = array_unique(array_merge($sourceKeys, $destKeys));
-        sort($all);//fields first, then indexes - because fields are prefixed with '!'
+        sort($all); //fields first, then indexes - because fields are prefixed with '!'
 
         foreach ($all as $key)
         {
             $info = array('source'=>'', 'dest'=>'');
-            $inSource= in_array($key, $sourceKeys);
-            $inDest= in_array($key, $destKeys);
+            $inSource = in_array($key, $sourceKeys);
+            $inDest = in_array($key, $destKeys);
             $sourceOrphan = $inSource && !$inDest;
             $destOrphan = $inDest && !$inSource;
-            $different =  $inSource && $inDest && $destPartsIndexed[$key]!=$sourcePartsIndexed[$key];
+            $different = $inSource && $inDest && $destPartsIndexed[$key] != $sourcePartsIndexed[$key];
             if ($sourceOrphan) {
                 $info['source'] = $sourcePartsIndexed[$key];
             }
@@ -462,11 +462,11 @@ 
         }
         elseif (preg_match('/^`?\w+`?/i', $line, $m))//field definition
         {
-            $key = '!'.$m[0];//to make sure fields will be synchronised before the keys
+            $key = '!'.$m[0]; //to make sure fields will be synchronised before the keys
         }
         else
         {
-            return false;//line has no valuable info (empty or comment)
+            return false; //line has no valuable info (empty or comment)
         }
         //$key = str_replace('`', '', $key);
         if (!empty($options['varcharDefaultIgnore'])) {
@@ -479,7 +479,7 @@ 
             $line = preg_replace("/ AUTO_INCREMENT=[0-9]+/i", '', $line);
         }
         $result['key'] = $key;
-        $result['line']= $line;
+        $result['line'] = $line;
         return $result;
     }
 
@@ -499,7 +499,7 @@ 
         if (!is_array($diff) || empty($diff)) {
             return $sqls;
         }
-        foreach($diff as $tab=>$info)
+        foreach ($diff as $tab=>$info)
         {
             if ($info['sourceOrphan'])//delete it
             {
@@ -522,7 +522,7 @@ 
             }
             else
             {
-                foreach($info['differs'] as $finfo)
+                foreach ($info['differs'] as $finfo)
                 {
                     $inDest = !empty($finfo['dest']);
                     $inSource = !empty($finfo['source']);
@@ -560,45 +560,45 @@ 
     {
         $result = 'ALTER TABLE `'.$tab.'` ';
         $action = strtolower($action);
-        $keyField = '`?\w`?(?:\(\d+\))?';//matches `name`(10)
-        $keyFieldList = '(?:'.$keyField.'(?:,\s?)?)+';//matches `name`(10),`desc`(255)
+        $keyField = '`?\w`?(?:\(\d+\))?'; //matches `name`(10)
+        $keyFieldList = '(?:'.$keyField.'(?:,\s?)?)+'; //matches `name`(10),`desc`(255)
         if (preg_match('/((?:PRIMARY )|(?:UNIQUE )|(?:FULLTEXT ))?KEY `?(\w+)?`?\s(\('.$keyFieldList.'\))/i', $sql, $m)) {   //key and index operations
             $type = strtolower(trim($m[1]));
             $name = trim($m[2]);
             $fields = trim($m[3]);
-            switch($action)
+            switch ($action)
             {
             case 'drop':
-                if ($type=='primary') {
-                    $result.= 'DROP PRIMARY KEY';
+                if ($type == 'primary') {
+                    $result .= 'DROP PRIMARY KEY';
                 }
                 else
                 {
-                    $result.= 'DROP INDEX `'.$name.'`';
+                    $result .= 'DROP INDEX `'.$name.'`';
                 }
                 break;
             case 'add':
-                if ($type=='primary') {
-                    $result.= 'ADD PRIMARY KEY '.$fields;
+                if ($type == 'primary') {
+                    $result .= 'ADD PRIMARY KEY '.$fields;
                 }
-                elseif ($type=='') {
-                    $result.= 'ADD INDEX `'.$name.'` '.$fields;
+                elseif ($type == '') {
+                    $result .= 'ADD INDEX `'.$name.'` '.$fields;
                 }
                 else
                 {
-                    $result .='ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
+                    $result .= 'ADD '.strtoupper($type).' `'.$name.'` '.$fields; //fulltext or unique
                 }
                 break;
             case 'modify':
-                if ($type=='primary') {
-                    $result.='DROP PRIMARY KEY, ADD PRIMARY KEY '.$fields;
+                if ($type == 'primary') {
+                    $result .= 'DROP PRIMARY KEY, ADD PRIMARY KEY '.$fields;
                 }
-                elseif ($type=='') {
-                    $result.='DROP INDEX `'.$name.'`, ADD INDEX `'.$name.'` '.$fields;
+                elseif ($type == '') {
+                    $result .= 'DROP INDEX `'.$name.'`, ADD INDEX `'.$name.'` '.$fields;
                 }
                 else
                 {
-                    $result.='DROP INDEX `'.$name.'`, ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
+                    $result .= 'DROP INDEX `'.$name.'`, ADD '.strtoupper($type).' `'.$name.'` '.$fields; //fulltext or unique
                 }
                 break;
 
@@ -607,14 +607,14 @@ 
         else //fields operations
         {
             $sql = rtrim(trim($sql), ',');
-            $result.= strtoupper($action);
-            if ($action=='drop') {
+            $result .= strtoupper($action);
+            if ($action == 'drop') {
                 $spacePos = strpos($sql, ' ');
-                $result.= ' '.substr($sql, 0, $spacePos);
+                $result .= ' '.substr($sql, 0, $spacePos);
             }
             else
             {
-                $result.= ' '.$sql;
+                $result .= ' '.$sql;
             }
         }
         return $result;
@@ -629,34 +629,34 @@ 
     *
     * $skipInBrackets - if true, delimiter will be skipped if located inside (brackets) which are not string literals or comment parts
     */
-    public function getDelimPos($string, $offset=0, $delim=';', $skipInBrackets=false)
+    public function getDelimPos($string, $offset = 0, $delim = ';', $skipInBrackets = false)
     {
         $stack = array();
-        $rbs = '\\\\';    //reg - escaped backslash
+        $rbs = '\\\\'; //reg - escaped backslash
         $regPrefix = "(?<!$rbs)(?:$rbs{2})*";
         $reg = $regPrefix.'("|\')|(/\\*)|(\\*/)|(-- )|(\r\n|\r|\n)|';
         if ($skipInBrackets) {
-            $reg.='(\(|\))|';
+            $reg .= '(\(|\))|';
         }
         else 
         {
-            $reg.='()';
+            $reg .= '()';
         }
         $reg .= '('.preg_quote($delim).')';
         while (preg_match('%'.$reg.'%', $string, $m, PREG_OFFSET_CAPTURE, $offset))
         {
-            $offset = $m[0][1]+strlen($m[0][0]);
-            if (end($stack)=='/*') {
+            $offset = $m[0][1] + strlen($m[0][0]);
+            if (end($stack) == '/*') {
                 if (!empty($m[3][0])) {
                     array_pop($stack);
                 }
-                continue;//here we could also simplify regexp
+                continue; //here we could also simplify regexp
             }
-            if (end($stack)=='-- ') {
+            if (end($stack) == '-- ') {
                 if (!empty($m[5][0])) {
                     array_pop($stack);
                 }
-                continue;//here we could also simplify regexp
+                continue; //here we could also simplify regexp
             }
 
             if (!empty($m[7][0]))// ';' found
@@ -671,16 +671,16 @@ 
             }
             if (!empty($m[6][0]))// '(' or ')' found
             {
-                if (empty($stack) && $m[6][0]=='(') {
+                if (empty($stack) && $m[6][0] == '(') {
                     array_push($stack, $m[6][0]);
                 }
-                elseif($m[6][0]==')' && end($stack)=='(') {
+                elseif ($m[6][0] == ')' && end($stack) == '(') {
                     array_pop($stack);
                 }
             }
             elseif (!empty($m[1][0]))// ' or " found
             {
-                if (end($stack)==$m[1][0]) {
+                if (end($stack) == $m[1][0]) {
                     array_pop($stack);
                 }
                 else
@@ -704,20 +704,20 @@ 
     * works the same as getDelimPos except returns position of the first occurence of the delimiter starting from
     * the end of the string
     */
-    public function getDelimRpos($string, $offset=0, $delim=';', $skipInBrackets=false)
+    public function getDelimRpos($string, $offset = 0, $delim = ';', $skipInBrackets = false)
     {
         $pos = $this->getDelimPos($string, $offset, $delim, $skipInBrackets);
-        if ($pos===false) {
+        if ($pos === false) {
             return false;
         }
         do
         {
-            $newPos=$this->getDelimPos($string, $pos+1, $delim, $skipInBrackets);
+            $newPos = $this->getDelimPos($string, $pos + 1, $delim, $skipInBrackets);
             if ($newPos !== false) {
                 $pos = $newPos;
             }
         }
-        while($newPos!==false);
+        while ($newPos !== false);
         return $pos;
     }
 }

Braces +67 added lines, -75 removed lines

@@ -142,8 +142,7 @@ 
         $result = array();
         if (is_array($this->config['updateTypes'])) {
             $updateActions = $this->config['updateTypes'];
-        }
-        else
+        } else
         {
             $updateActions = array_map('trim', explode(',', $this->config['updateTypes']));
         }
@@ -155,24 +154,20 @@ 
                 if (in_array('create', $updateActions)) {
                     $result[$table] = $info;
                 }
-            }
-            elseif ($info['destOrphan']) {
+            } elseif ($info['destOrphan']) {
                 if (in_array('drop', $updateActions)) {
                     $result[$table] = $info;
                 }
-            }
-            elseif($info['differs']) {
+            } elseif($info['differs']) {
                 $resultInfo = $info;
                 unset($resultInfo['differs']);
                 foreach ($info['differs'] as $diff)
                 {
                     if (empty($diff['dest']) && in_array('add', $updateActions)) {
                         $resultInfo['differs'][] = $diff;
-                    }
-                    elseif (empty($diff['source']) && in_array('remove', $updateActions)) {
+                    } elseif (empty($diff['source']) && in_array('remove', $updateActions)) {
                         $resultInfo['differs'][] = $diff;
-                    }
-                    elseif(in_array('modify', $updateActions)) {
+                    } elseif(in_array('modify', $updateActions)) {
                         $resultInfo['differs'][] = $diff;
                     }
                 }
@@ -198,11 +193,9 @@ 
         {
             if ($info['sourceOrphan']) {
                 $result['sourceOrphans'][] = $table;
-            }
-            elseif ($info['destOrphan']) {
+            } elseif ($info['destOrphan']) {
                 $result['destOrphans'][] = $table;
-            }
-            else
+            } else
             {
                 $result['different'][] = $table;
             }
@@ -244,11 +237,9 @@ 
             $info = array('destOrphan'=>false, 'sourceOrphan'=>false, 'differs'=>false);
             if(in_array($tab, $destOrphans)) {
                 $info['destOrphan'] = true;
-            }
-            elseif (in_array($tab, $sourceOrphans)) {
+            } elseif (in_array($tab, $sourceOrphans)) {
                 $info['sourceOrphan'] = true;
-            }
-            else
+            } else
             {                

                 $destSql = $this->getTabSql($this->destStruct, $tab, true);
                 $sourceSql = $this->getTabSql($this->sourceStruct, $tab, true);
@@ -256,12 +247,12 @@ 
                 if ($diffs===false) {
                     trigger_error('[WARNING] error parsing definition of table "'.$tab.'" - skipped');
                     continue;
-                }
-                elseif (!empty($diffs))//not empty array
+                } elseif (!empty($diffs)) {
+                    //not empty array
                 {
-                    $info['differs'] = $diffs;                    

-                }                

-                else { continue;//empty array
+                    $info['differs'] = $diffs;
+                }
+                } else { continue;//empty array
                 }         
             }
             $result[$tab] = $info;
@@ -279,10 +270,12 @@ 
     {
         $result = array();
         if (preg_match_all('/CREATE(?:\s*TEMPORARY)?\s*TABLE\s*(?:IF NOT EXISTS\s*)?(?:`?(\w+)`?\.)?`?(\w+)`?/i', $struct, $m)) {
-            foreach($m[2] as $match)//m[1] is a database name if any
+            foreach($m[2] as $match) {
+                //m[1] is a database name if any
             {
                 $result[] = $match;
             }
+            }
         }
         return $result;
     }
@@ -309,8 +302,7 @@ 
             $end = $this->getDelimPos($struct, $offset);
             if ($end === false) {
                 $result = substr($struct, $start);
-            }
-            else
+            } else
             {
                 $result = substr($struct, $start, $end-$start);//already without ';'
             }
@@ -382,9 +374,11 @@ 
         $result = array();        

         //split with comma delimiter, not line breaks
         $sourceParts =  $this->splitTabSql($sourceSql);
-        if ($sourceParts===false)//error parsing sql
+        if ($sourceParts===false) {
+            //error parsing sql
         {
             trigger_error('[WARNING] error parsing source sql');
+        }
             return false;
         }
         $destParts = $this->splitTabSql($destSql);
@@ -423,15 +417,12 @@ 
             $different =  $inSource && $inDest && $destPartsIndexed[$key]!=$sourcePartsIndexed[$key];
             if ($sourceOrphan) {
                 $info['source'] = $sourcePartsIndexed[$key];
-            }
-            elseif ($destOrphan) {
+            } elseif ($destOrphan) {
                 $info['dest'] = $destPartsIndexed[$key];
-            }
-            elseif ($different) {
+            } elseif ($different) {
                 $info['source'] = $sourcePartsIndexed[$key];
                 $info['dest'] = $destPartsIndexed[$key];
-            }
-            else { continue; 
+            } else { continue; 
             }
             $result[] = $info;
         }
@@ -452,19 +443,24 @@ 
         $options = $this->config;
         $result = array('key'=>'', 'line'=>'');
         $line = rtrim(trim($line), ',');
-        if (preg_match('/^(CREATE TABLE)|(\) ENGINE=)/i', $line))//first or last table definition line
+        if (preg_match('/^(CREATE TABLE)|(\) ENGINE=)/i', $line)) {
+            //first or last table definition line
         {
             return false;
         }
-        if (preg_match('/^(PRIMARY KEY)|(((UNIQUE )|(FULLTEXT ))?KEY `?\w+`?)/i', $line, $m))//key definition
+        }
+        if (preg_match('/^(PRIMARY KEY)|(((UNIQUE )|(FULLTEXT ))?KEY `?\w+`?)/i', $line, $m)) {
+            //key definition
         {
             $key = $m[0];
         }
-        elseif (preg_match('/^`?\w+`?/i', $line, $m))//field definition
+        } elseif (preg_match('/^`?\w+`?/i', $line, $m)) {
+            //field definition
         {
-            $key = '!'.$m[0];//to make sure fields will be synchronised before the keys
+            $key = '!'.$m[0];
         }
-        else
+        //to make sure fields will be synchronised before the keys
+        } else
         {
             return false;//line has no valuable info (empty or comment)
         }
@@ -501,13 +497,16 @@ 
         }
         foreach($diff as $tab=>$info)
         {
-            if ($info['sourceOrphan'])//delete it
+            if ($info['sourceOrphan']) {
+                //delete it
             {
                 $sqls[] = 'DROP TABLE `'.$tab.'`';
             }
-            elseif ($info['destOrphan'])//create destination table in source
+            } elseif ($info['destOrphan']) {
+                //create destination table in source
             {
                 $database = '';
+            }
                 $destSql = $this->getTabSql($this->destStruct, $tab, $database);
                 if (!empty($options['ignoreIncrement'])) {
                     $destSql = preg_replace("/\s*AUTO_INCREMENT=[0-9]+/i", '', $destSql);
@@ -519,8 +518,7 @@ 
                     $destSql = preg_replace('/(CREATE(?:\s*TEMPORARY)?\s*TABLE\s*)(?:IF NOT EXISTS\s*)?(`?\w+`?)/i', '$1IF NOT EXISTS $2', $destSql);
                 }
                 $sqls[] = $destSql;
-            }
-            else
+            } else
             {
                 foreach($info['differs'] as $finfo)
                 {
@@ -529,12 +527,10 @@ 
                     if ($inSource && !$inDest) {
                         $sql = $finfo['source'];
                         $action = 'drop';
-                    }
-                    elseif ($inDest && !$inSource) {
+                    } elseif ($inDest && !$inSource) {
                         $sql = $finfo['dest'];
                         $action = 'add';
-                    }
-                    else
+                    } else
                     {
                         $sql = $finfo['dest'];
                         $action = 'modify';
@@ -571,8 +567,7 @@ 
             case 'drop':
                 if ($type=='primary') {
                     $result.= 'DROP PRIMARY KEY';
-                }
-                else
+                } else
                 {
                     $result.= 'DROP INDEX `'.$name.'`';
                 }
@@ -580,11 +575,9 @@ 
             case 'add':
                 if ($type=='primary') {
                     $result.= 'ADD PRIMARY KEY '.$fields;
-                }
-                elseif ($type=='') {
+                } elseif ($type=='') {
                     $result.= 'ADD INDEX `'.$name.'` '.$fields;
-                }
-                else
+                } else
                 {
                     $result .='ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
                 }
@@ -592,27 +585,23 @@ 
             case 'modify':
                 if ($type=='primary') {
                     $result.='DROP PRIMARY KEY, ADD PRIMARY KEY '.$fields;
-                }
-                elseif ($type=='') {
+                } elseif ($type=='') {
                     $result.='DROP INDEX `'.$name.'`, ADD INDEX `'.$name.'` '.$fields;
-                }
-                else
+                } else
                 {
                     $result.='DROP INDEX `'.$name.'`, ADD '.strtoupper($type).' `'.$name.'` '.$fields;//fulltext or unique
                 }
                 break;
 
             }
-        }
-        else //fields operations
+        } else //fields operations
         {
             $sql = rtrim(trim($sql), ',');
             $result.= strtoupper($action);
             if ($action=='drop') {
                 $spacePos = strpos($sql, ' ');
                 $result.= ' '.substr($sql, 0, $spacePos);
-            }
-            else
+            } else
             {
                 $result.= ' '.$sql;
             }
@@ -637,8 +626,7 @@ 
         $reg = $regPrefix.'("|\')|(/\\*)|(\\*/)|(-- )|(\r\n|\r|\n)|';
         if ($skipInBrackets) {
             $reg.='(\(|\))|';
-        }
-        else 
+        } else 
         {
             $reg.='()';
         }
@@ -659,43 +647,47 @@ 
                 continue;//here we could also simplify regexp
             }
 
-            if (!empty($m[7][0]))// ';' found
+            if (!empty($m[7][0])) {
+                // ';' found
             {
                 if (empty($stack)) {
                     return $m[7][1];
-                }
-                else
+            }
+                } else
                 {
                     //var_dump($stack, substr($string, $offset-strlen($m[0][0])));
                 }
             }
-            if (!empty($m[6][0]))// '(' or ')' found
+            if (!empty($m[6][0])) {
+                // '(' or ')' found
             {
                 if (empty($stack) && $m[6][0]=='(') {
                     array_push($stack, $m[6][0]);
-                }
-                elseif($m[6][0]==')' && end($stack)=='(') {
+            }
+                } elseif($m[6][0]==')' && end($stack)=='(') {
                     array_pop($stack);
                 }
-            }
-            elseif (!empty($m[1][0]))// ' or " found
+            } elseif (!empty($m[1][0])) {
+                // ' or " found
             {
                 if (end($stack)==$m[1][0]) {
                     array_pop($stack);
-                }
-                else
+            }
+                } else
                 {
                     array_push($stack, $m[1][0]);
                 }
-            }
-            elseif (!empty($m[2][0])) // opening comment / *
+            } elseif (!empty($m[2][0])) {
+                // opening comment / *
             {
                 array_push($stack, $m[2][0]);
             }
-            elseif (!empty($m[4][0])) // opening comment --
+            } elseif (!empty($m[4][0])) {
+                // opening comment --
             {
                 array_push($stack, $m[4][0]);
             }
+            }
         }
         return false;
     }

module/UIComponents/src/UIComponents/View/Helper/Components/Element.php

Spacing +3 added lines, -3 removed lines

@@ -33,10 +33,10 @@
      */
     public function __invoke($options = array())
     {
-        if ( is_object($options) && method_exists($options, 'toArray') ) {
+        if (is_object($options) && method_exists($options, 'toArray')) {
             $options = $options->toArray();
-        } else if ( is_object($options) ) {
-            $options = (array)$options;
+        } else if (is_object($options)) {
+            $options = (array) $options;
         }
         
         if (isset($options['container']) && (null !== $options['container'])) {

module/UIComponents/src/UIComponents/View/Helper/Components/PluginManager.php

Indentation +13 added lines, -13 removed lines

@@ -33,8 +33,8 @@ 
      */
     protected $invokableClasses = [
 
-    	// panels
-    	//
+        // panels
+        //
         'element'              => 'UIComponents\View\Helper\Components\Element',
         'block'             => 'UIComponents\View\Helper\Components\Block',
         //'well'              => 'UIComponents\View\Helper\Components\Well',
@@ -43,33 +43,33 @@ 
         'widget'            => 'UIComponents\View\Helper\Components\Widget',
         'dashboard'         => 'UIComponents\View\Helper\Components\Dashboard',
 
-    	// page components
-    	//
+        // page components
+        //
         'nav'               => 'UIComponents\View\Helper\Components\Navbar', // set 'alias to' or combine with 'Navbars' !
         'topbar'            => 'UIComponents\View\Helper\Components\Navbar', // set 'alias to' or combine with 'Navbars' !
         'breadcrumbs'       => 'UIComponents\View\Helper\Components\Breadcrumbs',
         'languagemenu'      => 'UIComponents\View\Helper\Components\Languagemenu',
-    	'toolbar'           => 'UIComponents\View\Helper\Components\Toolbar',
+        'toolbar'           => 'UIComponents\View\Helper\Components\Toolbar',
 
         // controls
         //
         'button'            => 'UIComponents\View\Helper\Components\Button', // default, drop-down/up, splitted
         'buttongroup'       => 'UIComponents\View\Helper\Components\Buttongroup',
-    	//'inputgroup'        => 'UIComponents\View\Helper\Components\Inputgroup',
+        //'inputgroup'        => 'UIComponents\View\Helper\Components\Inputgroup',
         //'progressbar'       => 'UIComponents\View\Helper\Components\Progressbar',
 
-    	// forms
-    	//
+        // forms
+        //
         'formgroup'         => 'UIComponents\View\Helper\Components\Formgroup',
 
-    	// lists/tables
-    	//
+        // lists/tables
+        //
         //'listgroup'         => 'UIComponents\View\Helper\Components\Listgroup',
         'table'         	=> 'UIComponents\View\Helper\Components\Listgroup',
-    	//'pagination'        => 'UIComponents\View\Helper\Components\Pagination',
+        //'pagination'        => 'UIComponents\View\Helper\Components\Pagination',
 
-    	// widgets
-    	//
+        // widgets
+        //
         //'label'             => 'UIComponents\View\Helper\Components\Label',
         //'badge'             => 'UIComponents\View\Helper\Components\Badge',
         //'pageheader'        => 'UIComponents\View\Helper\Components\Pageheader',

module/UIComponents/src/UIComponents/View/Helper/Utilities.php

Spacing +3 added lines, -3 removed lines

@@ -71,7 +71,7 @@ 
     
             $controller = $params['controller'];
             if (isset($params['__NAMESPACE__'])) {
-                $controller = $params['__NAMESPACE__'] . '\\' . $params['controller'];
+                $controller = $params['__NAMESPACE__'].'\\'.$params['controller'];
             }
             $action = $params['action'];
     
@@ -102,8 +102,8 @@ 
     {
         $info = $this->requestInfo();
         if ($info) {
-            $slug = strtolower($info['module']) . '-' . 
-                strtolower($info['controller']) . '-' . 
+            $slug = strtolower($info['module']).'-'. 
+                strtolower($info['controller']).'-'. 
                 strtolower($info['action']);
     
             return $slug;

module/UIComponents/src/UIComponents/View/Helper/Navigation/Menu.php

Indentation +6 added lines, -6 removed lines

@@ -29,8 +29,8 @@ 
  */
 class Menu extends \Zend\View\Helper\Navigation\Menu
 {
-	use ComponentClassnamesTrait;
-	use ComponentAttributesTrait;
+    use ComponentClassnamesTrait;
+    use ComponentAttributesTrait;
 
     /**
      * default CSS class to use for li elements
@@ -353,10 +353,10 @@ 
      */
     public function findActive($container = null, $minDepth = null, $maxDepth = -1)
     {
-    	if ( null == $container ) {
-    		$container = $this->getContainer();
-    	}
-    	return parent::findActive($container, $minDepth, $maxDepth);
+        if ( null == $container ) {
+            $container = $this->getContainer();
+        }
+        return parent::findActive($container, $minDepth, $maxDepth);
     }
     
     /**

Spacing +30 added lines, -31 removed lines

@@ -212,7 +212,7 @@ 
         $escaper = $this->view->plugin('escapeHtmlAttr');
 
         if ($found) {
-            $foundPage    = $found['page'];
+            $foundPage = $found['page'];
             $foundDepth = $found['depth'];
         } else {
             $foundPage = null;
@@ -261,34 +261,33 @@ 
 
             // make sure indentation is correct
             $depth -= $minDepth;
-            $myIndent = $indent . str_repeat('    ', $depth);
+            $myIndent = $indent.str_repeat('    ', $depth);
             $attributes = $this->getAttributes();
             if ($depth > $prevDepth) {
                 // start new ul tag
-                $ulClass = '' . 
-                    ($depth == 0 ? $this->getUlClass() : 
-                            ($depth == 1 ? $this->getSubUlClassLevel1() : $this->getSubUlClass())
-                    ) . 
-                    ' level_' . $depth . 
+                $ulClass = ''. 
+                    ($depth == 0 ? $this->getUlClass() : ($depth == 1 ? $this->getSubUlClassLevel1() : $this->getSubUlClass())
+                    ). 
+                    ' level_'.$depth. 
                 '';
-                if ($ulClass && $depth ==    0) {
-                    $ulClass = ' class="' . $escaper($ulClass) . '"';
+                if ($ulClass && $depth == 0) {
+                    $ulClass = ' class="'.$escaper($ulClass).'"';
                 } else {
-                    $ulClass = ' class="' . $escaper($ulClass) . '"';
+                    $ulClass = ' class="'.$escaper($ulClass).'"';
                 }
-                $html .= $myIndent . '<ul' . $ulClass . ' '.($depth == 0 ? $this->htmlAttribs($attributes) : '').'>' . PHP_EOL;
+                $html .= $myIndent.'<ul'.$ulClass.' '.($depth == 0 ? $this->htmlAttribs($attributes) : '').'>'.PHP_EOL;
             } elseif ($prevDepth > $depth) {
                 // close li/ul tags until we're at current depth
                 for ($i = $prevDepth; $i > $depth; $i--) {
-                    $ind = $indent . str_repeat('        ', $i);
-                    $html .= $ind . '    </li>' . PHP_EOL;
-                    $html .= $ind . '</ul>' . PHP_EOL;
+                    $ind = $indent.str_repeat('        ', $i);
+                    $html .= $ind.'    </li>'.PHP_EOL;
+                    $html .= $ind.'</ul>'.PHP_EOL;
                 }
                 // close previous li tag
-                $html .= $myIndent . '    </li>' . PHP_EOL;
+                $html .= $myIndent.'    </li>'.PHP_EOL;
             } else {
                 // close previous li tag
-                $html .= $myIndent . '    </li>' . PHP_EOL;
+                $html .= $myIndent.'    </li>'.PHP_EOL;
             }
 
             // render li tag and page
@@ -308,10 +307,10 @@ 
             if ($addClassToListItem && $page->getClass()) {
                 $liClasses[] = $page->getClass();
             }
-            $liClass = empty($liClasses) ? '' : ' class="' . $escaper(implode(' ', $liClasses)) . '"';
+            $liClass = empty($liClasses) ? '' : ' class="'.$escaper(implode(' ', $liClasses)).'"';
 
-            $html .= $myIndent . '    <li' . $liClass . '>' . PHP_EOL
-                . $myIndent . '        ' . $this->htmlify($page, $escapeLabels, $addClassToListItem) . PHP_EOL;
+            $html .= $myIndent.'    <li'.$liClass.'>'.PHP_EOL
+                . $myIndent.'        '.$this->htmlify($page, $escapeLabels, $addClassToListItem).PHP_EOL;
 
             // store as previous depth for next iteration
             $prevDepth = $depth;
@@ -319,10 +318,10 @@ 
 
         if ($html) {
             // done iterating container; close open ul/li tags
-            for ($i = $prevDepth+1; $i > 0; $i--) {
-                $myIndent = $indent . str_repeat('        ', $i-1);
-                $html .= $myIndent . '    </li>' . PHP_EOL
-                    . $myIndent . '</ul>' . PHP_EOL;
+            for ($i = $prevDepth + 1; $i > 0; $i--) {
+                $myIndent = $indent.str_repeat('        ', $i - 1);
+                $html .= $myIndent.'    </li>'.PHP_EOL
+                    . $myIndent.'</ul>'.PHP_EOL;
             }
             $html = rtrim($html, PHP_EOL);
         }
@@ -353,7 +352,7 @@ 
      */
     public function findActive($container = null, $minDepth = null, $maxDepth = -1)
     {
-    	if ( null == $container ) {
+    	if (null == $container) {
     		$container = $this->getContainer();
     	}
     	return parent::findActive($container, $minDepth, $maxDepth);
@@ -370,7 +369,7 @@ 
      * @param    bool         $addClassToListItem Whether or not to add the page class to the list item
      * @return string
      */
-    public function htmlify(AbstractPage $page, $escapeLabel = true, $addClassToListItem = false, $extraAttributes = [] )
+    public function htmlify(AbstractPage $page, $escapeLabel = true, $addClassToListItem = false, $extraAttributes = [])
     {
         $partial = $this->getHtmlifyPartial();
         if ($partial) {
@@ -380,11 +379,11 @@ 
         $attribs = array_merge([
                 'id'     => $page->getId(),
                 'title'    => $this->translate($page->getTitle(), $page->getTextDomain()),
-                'data-test'    => 'cta-nav-' . $this->slugify($page->getLabel()),
+                'data-test'    => 'cta-nav-'.$this->slugify($page->getLabel()),
         ], $extraAttributes);
         
         $classnames = array();
-        if ( $addClassToListItem === false ) {
+        if ($addClassToListItem === false) {
             $class = $page->getClass();
             if (!empty($class)) {
                 $classnames[] = $page->getClass();
@@ -393,7 +392,7 @@ 
         $maxDepth = $this->getMaxDepth();
         $depth = $page->get('level');
         $isBelowMaxLevel = ($maxDepth > $depth) || ($maxDepth === null) || ($maxDepth === false);
-        if ( !empty($page->pages) && $isBelowMaxLevel ) {
+        if (!empty($page->pages) && $isBelowMaxLevel) {
             $classnames[] = 'dropdown-toggle';
             $attribs['data-toggle'] = (($depth == 0) ? $this->getSubLiClassLevel0() : $this->getSubLiClass());
         }
@@ -412,8 +411,8 @@ 
             $attribs['href'] = '#';
         }
         
-        $html    = '<' . $element . $this->htmlAttribs($attribs) . '>';
-        $html .= ($page->get('icon') ? '<span class="' . $this->getIconPrefixClass() . '' . $page->get('icon') . '"></span> ' : '' );
+        $html = '<'.$element.$this->htmlAttribs($attribs).'>';
+        $html .= ($page->get('icon') ? '<span class="'.$this->getIconPrefixClass().''.$page->get('icon').'"></span> ' : '');
         $label = $this->translate($page->getLabel(), $page->getTextDomain());
         if ($escapeLabel === true) {
             /** @var \Zend\View\Helper\EscapeHtml $escaper */
@@ -422,7 +421,7 @@ 
         } else {
             $html .= $label;
         }
-        $html .= '</' . $element . '>';
+        $html .= '</'.$element.'>';
     
         return $html;
     }

module/UIComponents/src/UIComponents/View/Helper/Components/Toolbar.php

Spacing +20 added lines, -21 removed lines

@@ -97,7 +97,7 @@ 
         $escaper = $this->view->plugin('escapeHtmlAttr');
 
         if ($found) {
-            $foundPage    = $found['page'];
+            $foundPage = $found['page'];
             $foundDepth = $found['depth'];
         } else {
             $foundPage = null;
@@ -146,28 +146,27 @@ 
 
             // make sure indentation is correct
             $depth -= $minDepth;
-            $myIndent = $indent . str_repeat('    ', $depth);
+            $myIndent = $indent.str_repeat('    ', $depth);
 
             if ($depth > $prevDepth) {
                 // start new ul tag
-                $ulClass = '' . 
-                    ($depth == 0 ? $this->getUlClass() : 
-                            ($depth == 1 ? $this->getSubUlClassLevel1() : $this->getSubUlClass())
-                    ) . 
-                    ' level_' . $depth . 
+                $ulClass = ''. 
+                    ($depth == 0 ? $this->getUlClass() : ($depth == 1 ? $this->getSubUlClassLevel1() : $this->getSubUlClass())
+                    ). 
+                    ' level_'.$depth. 
                 '';
-                if ($ulClass && $depth ==    0) {
-                    $ulClass = ' class="' . $escaper($ulClass) . '"';
+                if ($ulClass && $depth == 0) {
+                    $ulClass = ' class="'.$escaper($ulClass).'"';
                 } else {
-                    $ulClass = ' class="' . $escaper($ulClass) . '"';
+                    $ulClass = ' class="'.$escaper($ulClass).'"';
                 }
-                $html .= $myIndent . '<' . $this->getTagname() . $ulClass . '>' . PHP_EOL;
+                $html .= $myIndent.'<'.$this->getTagname().$ulClass.'>'.PHP_EOL;
             } elseif ($prevDepth > $depth) {
                 // close li/ul tags until we're at current depth
                 for ($i = $prevDepth; $i > $depth; $i--) {
-                    $ind = $indent . str_repeat('        ', $i);
+                    $ind = $indent.str_repeat('        ', $i);
                     //$html .= $ind . '    </li>' . PHP_EOL;
-                    $html .= $ind . '</' . $this->getTagname() . '>' . PHP_EOL;
+                    $html .= $ind.'</'.$this->getTagname().'>'.PHP_EOL;
                 }
                 // close previous li tag
                 //$html .= $myIndent . '    </li>' . PHP_EOL;
@@ -193,12 +192,12 @@ 
             if ($addClassToListItem && $page->getClass()) {
                 $liClasses[] = $page->getClass();
             }
-            $liClass = empty($liClasses) ? '' : ' class="' . $escaper(implode(' ', $liClasses)) . '"';
+            $liClass = empty($liClasses) ? '' : ' class="'.$escaper(implode(' ', $liClasses)).'"';
 
             $html .= /* $myIndent . '    <li' . $liClass . '>' . PHP_EOL
-                . */ $myIndent . '        ' . $this->htmlify($page, $escapeLabels, $addClassToListItem, [
-                    'data-test' => 'cta-toolbar-' . $this->slugify($page->getLabel()),
-                ]) . PHP_EOL;
+                . */ $myIndent.'        '.$this->htmlify($page, $escapeLabels, $addClassToListItem, [
+                    'data-test' => 'cta-toolbar-'.$this->slugify($page->getLabel()),
+                ]).PHP_EOL;
 
             // store as previous depth for next iteration
             $prevDepth = $depth;
@@ -206,10 +205,10 @@ 
 
         if ($html) {
             // done iterating container; close open ul/li tags
-            for ($i = $prevDepth+1; $i > 0; $i--) {
-                $myIndent = $indent . str_repeat('        ', $i-1);
+            for ($i = $prevDepth + 1; $i > 0; $i--) {
+                $myIndent = $indent.str_repeat('        ', $i - 1);
                 $html .= /*$myIndent . '    </li>' . PHP_EOL
-                    . */ $myIndent . '</' . $this->getTagname() . '>' . PHP_EOL;
+                    . */ $myIndent.'</'.$this->getTagname().'>'.PHP_EOL;
             }
             $html = rtrim($html, PHP_EOL);
         }
@@ -232,7 +231,7 @@ 
      * @param string $tagname
      */
     public function setTagname($tagname) {
-        if ( null !== $tagname ) {
+        if (null !== $tagname) {
             $this->tagname = $tagname;
         }
         return $this;

module/UIComponents/src/UIComponents/View/Helper/Components/Breadcrumbs.php

Spacing +11 added lines, -11 removed lines

@@ -112,24 +112,24 @@ 
         /** @var \Zend\View\Helper\EscapeHtml $escaper */
         $escaper = $this->view->plugin('escapeHtml');
         
-        $listHtmlOpen = '<ol class="'.$this->getOlClass().'" data-test="layout-breadcrumbs">' . PHP_EOL;
-        if ( !empty($this->getHeader()) ) {
-            $listHtmlOpen .= '<li class="' . $escaper($this->getHeaderClass()) . '" data-test="layout-breadcrumbs-header">' . $escaper($this->getHeader()) . '</li>' . PHP_EOL;
+        $listHtmlOpen = '<ol class="'.$this->getOlClass().'" data-test="layout-breadcrumbs">'.PHP_EOL;
+        if (!empty($this->getHeader())) {
+            $listHtmlOpen .= '<li class="'.$escaper($this->getHeaderClass()).'" data-test="layout-breadcrumbs-header">'.$escaper($this->getHeader()).'</li>'.PHP_EOL;
         }
         // put the deepest active page last in breadcrumbs
         if ($this->getLinkLast()) {
-            $html = '<li class="active" data-test="layout-breadcrumbs-current">' . $this->htmlify($active) . '</li>' . PHP_EOL;
+            $html = '<li class="active" data-test="layout-breadcrumbs-current">'.$this->htmlify($active).'</li>'.PHP_EOL;
         } else {
-            $html = '<li class="active" data-test="layout-breadcrumbs-current">' . $escaper(
+            $html = '<li class="active" data-test="layout-breadcrumbs-current">'.$escaper(
                 $this->translate($active->getLabel()) //, $active->getTextDomain())
-            ) . '</li>' . PHP_EOL;
+            ).'</li>'.PHP_EOL;
         }
 
         // walk back to root
         while ($parent = $active->getParent()) {
             if ($parent instanceof AbstractPage) {
                 // prepend crumb to html
-                $html = '<li data-test="layout-breadcrumbs-parent'.$parent->getLabel().'">' . $this->htmlify($parent) . '</li>' . PHP_EOL
+                $html = '<li data-test="layout-breadcrumbs-parent'.$parent->getLabel().'">'.$this->htmlify($parent).'</li>'.PHP_EOL
                     //. $this->getSeparator()
                     . $html;
             }
@@ -142,9 +142,9 @@ 
             $active = $parent;
         }
 
-        $listHtmlClose = '</ol>' . PHP_EOL;
+        $listHtmlClose = '</ol>'.PHP_EOL;
         
-        return strlen($html) ? $listHtmlOpen . $this->getIndent() . $html . $listHtmlClose : '';
+        return strlen($html) ? $listHtmlOpen.$this->getIndent().$html.$listHtmlClose : '';
     }
 
     /**
@@ -165,14 +165,14 @@ 
             'class'  => $page->getClass(),
             'href'   => $page->getHref(),
             'target' => $page->getTarget(),
-            'data-test' => 'cta-breadcrumbs-' . $this->slugify($page->getLabel())
+            'data-test' => 'cta-breadcrumbs-'.$this->slugify($page->getLabel())
         ];
 
         /** @var \Zend\View\Helper\EscapeHtml $escaper */
         $escaper = $this->view->plugin('escapeHtml');
         $label   = $escaper($label);
 
-        return '<a' . $this->htmlAttribs($attribs) . '>' . $label . '</a>';
+        return '<a'.$this->htmlAttribs($attribs).'>'.$label.'</a>';
     }
     
     /**

module/UIComponents/src/UIComponents/View/Helper/Traits/ComponentAttributesTrait.php

Spacing +9 added lines, -9 removed lines

@@ -36,7 +36,7 @@ 
      * @return the $attribute 
      */
     public function getAttribute($name) {
-        return ( isset($this->attributes[$name]) ? $this->attributes[$name] : null );
+        return (isset($this->attributes[$name]) ? $this->attributes[$name] : null);
     }
 
     /**
@@ -46,7 +46,7 @@ 
      * @param mixed $value
      */
     public function setAttribute($attribute, $value = "") {
-        if ( null !== $attribute ) {
+        if (null !== $attribute) {
             $this->attributes[$attribute] = $value;
         }
         return $this;
@@ -67,7 +67,7 @@ 
      * @param array $attributes
      */
     public function setAttributes($attributes) {
-        if ( is_array($attributes) ) {
+        if (is_array($attributes)) {
             $this->attributes = array_merge_recursive($this->attributes, $attributes);
         }
         return $this;
@@ -88,7 +88,7 @@ 
         // except for "data-" and "aria-" attributes
         foreach ($attribs as $key => $value) {
             if ($value === null || (is_string($value) && !strlen($value))) {
-                if ( (strpos($key, "data") === false) && (strpos($key, "aria") === false) ) {
+                if ((strpos($key, "data") === false) && (strpos($key, "aria") === false)) {
                     unset($attribs[$key]);
                 }
             }
@@ -96,9 +96,9 @@ 
 
         $xhtml = '';
         $oView = $this->getView();
-        $escaper        = function ($val) { return $val; };
-        $escapeHtmlAttr = function ($val) { return $val; };
-        if ( ($oView instanceof \Zend\View\Renderer\RendererInterface) ) {
+        $escaper        = function($val) { return $val; };
+        $escapeHtmlAttr = function($val) { return $val; };
+        if (($oView instanceof \Zend\View\Renderer\RendererInterface)) {
             $escaper        = $oView->plugin('escapehtml');
             $escapeHtmlAttr = $oView->plugin('escapehtmlattr');
         }
@@ -133,7 +133,7 @@ 
         return $xhtml;
     }
 
-    public function slugify ($text) {
+    public function slugify($text) {
         $rules = <<<'RULES'
             :: Any-Latin;
             :: NFD;
@@ -147,7 +147,7 @@ 
         RULES;
 
         return \Transliterator::createFromRules($rules)
-            ->transliterate( $text );
+            ->transliterate($text);
     }
 
 }