danielgp / fk-scale-mysql

source/FKinterface.php

Spacing +16 added lines, -16 removed lines

@@ -47,7 +47,7 @@ 
         $sReturn[] = $this->buildInputs(['field' => 'dt', 'label' => 'Data type to change to'], $sGb);
         $sReturn[] = '<input type="submit" value="Generate SQL queries for scaling" />';
         $sReturn[] = $this->displayMySqlConfiguration($mysqlConfig);
-        return '<form method="get" action="' . filter_var($sGb->server->get('PHP_SELF'), FILTER_SANITIZE_URL) . '">'
+        return '<form method="get" action="'.filter_var($sGb->server->get('PHP_SELF'), FILTER_SANITIZE_URL).'">'
                 . implode('<br/>', $sReturn)
                 . '</form>';
     }
@@ -55,7 +55,7 @@ 
     protected function buildInputFormTab($mysqlConfig, $tParams, \Symfony\Component\HttpFoundation\Request $sGb)
     {
         return '<div class="tabber" id="tabberFKscaleMySQL">'
-                . '<div class="tabbertab' . ($tParams ? '' : ' tabbertabdefault')
+                . '<div class="tabbertab'.($tParams ? '' : ' tabbertabdefault')
                 . '" id="FKscaleMySQLparameters" title="Parameters for scaling">'
                 . $this->buildInputFormForFKscaling($mysqlConfig, $sGb)
                 . '</div><!-- end of Parameters tab -->';
@@ -63,9 +63,9 @@ 
 
     private function buildInputs($inArray, $sGb)
     {
-        return '<label for="' . $inArray['field'] . 'Name">' . $inArray['label'] . ':</label>'
-                . '<input type="text" id="' . $inArray['field'] . 'Name" name="' . $inArray['field']
-                . '" placeholder="' . explode(' ', $inArray['label'])[0] . ' name" '
+        return '<label for="'.$inArray['field'].'Name">'.$inArray['label'].':</label>'
+                . '<input type="text" id="'.$inArray['field'].'Name" name="'.$inArray['field']
+                . '" placeholder="'.explode(' ', $inArray['label'])[0].' name" '
                 . $this->returnInputsCleaned($inArray['field'], $sGb)
                 . 'size="30" maxlength="64" class="labell" />';
     }
@@ -76,15 +76,15 @@ 
         return '<p>For security reasons the MySQL connection details are not available '
                 . 'to be set/modified through the interface and must be set directly '
                 . 'into the "configurationMySQL.php" file. Currently these settings are:<ul>'
-                . '<li>Host name where MySQL server resides: <span style="' . $styleForMySQLparams . '">'
-                . $mysqlConfig['host'] . '</span></li>'
-                . '<li>MySQL port used: <span style="' . $styleForMySQLparams . '">'
-                . $mysqlConfig['port'] . '</span></li>'
-                . '<li>MySQL database to connect to: <span style="' . $styleForMySQLparams . '">'
-                . $mysqlConfig['database'] . '</span></li>'
-                . '<li>MySQL username used: <span style="' . $styleForMySQLparams . '">'
-                . $mysqlConfig['username'] . '</span></li>'
-                . '<li>MySQL password used: <span style="' . $styleForMySQLparams . '">'
+                . '<li>Host name where MySQL server resides: <span style="'.$styleForMySQLparams.'">'
+                . $mysqlConfig['host'].'</span></li>'
+                . '<li>MySQL port used: <span style="'.$styleForMySQLparams.'">'
+                . $mysqlConfig['port'].'</span></li>'
+                . '<li>MySQL database to connect to: <span style="'.$styleForMySQLparams.'">'
+                . $mysqlConfig['database'].'</span></li>'
+                . '<li>MySQL username used: <span style="'.$styleForMySQLparams.'">'
+                . $mysqlConfig['username'].'</span></li>'
+                . '<li>MySQL password used: <span style="'.$styleForMySQLparams.'">'
                 . 'cannot be disclosed due to security reasons</span></li>'
                 . '</ul></p>';
     }
@@ -93,7 +93,7 @@ 
     {
         $sReturn = '';
         if (!is_null($sGb->get($inputFieldName))) {
-            $sReturn = 'value="' . filter_var($sGb->get($inputFieldName), FILTER_SANITIZE_STRING) . '" ';
+            $sReturn = 'value="'.filter_var($sGb->get($inputFieldName), FILTER_SANITIZE_STRING).'" ';
         }
         return $sReturn;
     }
@@ -129,6 +129,6 @@ 
             'title'      => $pageTitle,
         ];
         return $this->setHeaderCommon($headerArray)
-                . '<h1>' . $pageTitle . '</h1>';
+                . '<h1>'.$pageTitle.'</h1>';
     }
 }

source/FKchange.php

Spacing +22 added lines, -22 removed lines

@@ -66,7 +66,7 @@ 
     {
         $sReturn             = [];
         $targetTableTextFlds = $this->getForeignKeys($elToModify);
-        $sReturn[]           = '<div class="tabbertab' . ($tParams ? ' tabbertabdefault' : '')
+        $sReturn[]           = '<div class="tabbertab'.($tParams ? ' tabbertabdefault' : '')
                 . '" id="FKscaleMySQLresults" title="Results">';
         if (is_array($targetTableTextFlds)) {
             $sReturn[]    = $this->createDropForeignKeysAndGetTargetColumnDefinition($targetTableTextFlds);
@@ -75,7 +75,7 @@ 
                 'style'                => 'color:blue;font-weight:bold;',
                 'includeOldColumnType' => true,
             ]);
-            $sReturn[]    = $this->recreateFKs($elToModify, $targetTableTextFlds);
+            $sReturn[] = $this->recreateFKs($elToModify, $targetTableTextFlds);
         } elseif (!is_array($targetTableTextFlds)) {
             $sReturn[] = $this->returnMessagesInCaseOfNoResults($mConnection);
         }
@@ -86,11 +86,11 @@ 
 
     private function createChangeColumn($params, $adtnlFeatures = null)
     {
-        return '<div style="' . (isset($adtnlFeatures['style']) ? $adtnlFeatures['style'] : 'color:blue;') . '">'
-                . 'ALTER TABLE `' . $params['Database'] . '`.`' . $params['Table'] . '` '
-                . 'CHANGE `' . $params['Column'] . '` `' . $params['Column'] . '` ' . $params['NewDataType'] . ' '
-                . $this->setColumnDefinition($params) . ';'
-                . (isset($adtnlFeatures['includeOldColumnType']) ? ' /* from ' . $params['OldDataType'] . ' */' : '')
+        return '<div style="'.(isset($adtnlFeatures['style']) ? $adtnlFeatures['style'] : 'color:blue;').'">'
+                . 'ALTER TABLE `'.$params['Database'].'`.`'.$params['Table'].'` '
+                . 'CHANGE `'.$params['Column'].'` `'.$params['Column'].'` '.$params['NewDataType'].' '
+                . $this->setColumnDefinition($params).';'
+                . (isset($adtnlFeatures['includeOldColumnType']) ? ' /* from '.$params['OldDataType'].' */' : '')
                 . '</div>';
     }
 
@@ -112,8 +112,8 @@ 
     private function createDropForeignKey($parameters)
     {
         return '<div style="color:red;">'
-                . 'ALTER TABLE `' . $parameters['Database'] . '`.`' . $parameters['Table']
-                . '` DROP FOREIGN KEY `' . $parameters['ForeignKeyName'] . '`;'
+                . 'ALTER TABLE `'.$parameters['Database'].'`.`'.$parameters['Table']
+                . '` DROP FOREIGN KEY `'.$parameters['ForeignKeyName'].'`;'
                 . '</div>';
     }
 
@@ -121,7 +121,7 @@ 
     {
         $sReturn = [];
         foreach ($targetTableTextFlds as $key => $value) {
-            $sReturn[]                                    = $this->createDropForeignKey([
+            $sReturn[] = $this->createDropForeignKey([
                 'Database'       => $value['TABLE_SCHEMA'],
                 'Table'          => $value['TABLE_NAME'],
                 'ForeignKeyName' => $value['CONSTRAINT_NAME'],
@@ -138,12 +138,12 @@ 
     private function createForeignKey($params)
     {
         return '<div style="color:green;">'
-                . 'ALTER TABLE `' . $params['Database'] . '`.`' . $params['Table'] . '` '
-                . 'ADD CONSTRAINT `' . $params['ForeignKeyName'] . '` '
-                . 'FOREIGN KEY (`' . $params['Column'] . '`) REFERENCES `' . $params['ReferencedDatabase'] . '`.`'
-                . $params['ReferencedTable'] . '` (`' . $params['ReferencedColumn'] . '`) '
-                . 'ON DELETE ' . ($params['RuleDelete'] == 'NULL' ? 'SET NULL' : $params['RuleDelete']) . ' '
-                . 'ON UPDATE ' . ($params['RuleUpdate'] == 'NULL' ? 'SET NULL' : $params['RuleUpdate']) . ';'
+                . 'ALTER TABLE `'.$params['Database'].'`.`'.$params['Table'].'` '
+                . 'ADD CONSTRAINT `'.$params['ForeignKeyName'].'` '
+                . 'FOREIGN KEY (`'.$params['Column'].'`) REFERENCES `'.$params['ReferencedDatabase'].'`.`'
+                . $params['ReferencedTable'].'` (`'.$params['ReferencedColumn'].'`) '
+                . 'ON DELETE '.($params['RuleDelete'] == 'NULL' ? 'SET NULL' : $params['RuleDelete']).' '
+                . 'ON UPDATE '.($params['RuleUpdate'] == 'NULL' ? 'SET NULL' : $params['RuleUpdate']).';'
                 . '</div>';
     }
 
@@ -171,7 +171,7 @@ 
             'REFERENCED_COLUMN_NAME'  => $elToModify['Column'],
             'REFERENCED_TABLE_NAME'   => 'NOT NULL',
         ];
-        $query              = $this->sQueryMySqlIndexes($additionalFeatures);
+        $query = $this->sQueryMySqlIndexes($additionalFeatures);
         return $this->setMySQLquery2Server($query, 'full_array_key_numbered')['result'];
     }
 
@@ -182,12 +182,12 @@ 
             'TABLE_NAME'   => $elToModify['Table'],
             'COLUMN_NAME'  => $elToModify['Column'],
         ];
-        $col           = $this->getMySQLlistColumns($colToIdentify);
+        $col = $this->getMySQLlistColumns($colToIdentify);
         return [
             'Database'       => $targetTableTextFlds[0]['REFERENCED_TABLE_SCHEMA'],
             'Table'          => $targetTableTextFlds[0]['REFERENCED_TABLE_NAME'],
             'Column'         => $targetTableTextFlds[0]['REFERENCED_COLUMN_NAME'],
-            'OldDataType'    => strtoupper($col[0]['COLUMN_TYPE']) . ' ' . $this->setColumnDefinition($col[0]),
+            'OldDataType'    => strtoupper($col[0]['COLUMN_TYPE']).' '.$this->setColumnDefinition($col[0]),
             'NewDataType'    => $elToModify['NewDataType'],
             'IS_NULLABLE'    => $col[0]['IS_NULLABLE'],
             'COLUMN_DEFAULT' => $col[0]['COLUMN_DEFAULT'],
@@ -213,19 +213,19 @@ 
             $colDefinition .= ' AUTO_INCREMENT';
         }
         if (strlen($inArray['COLUMN_COMMENT']) > 0) {
-            $colDefinition .= ' COMMENT "' . $inArray['COLUMN_COMMENT'] . '"';
+            $colDefinition .= ' COMMENT "'.$inArray['COLUMN_COMMENT'].'"';
         }
         return $colDefinition;
     }
 
     private function setColumnDefinitionPrefix($nullableYesNo, $defaultValue)
     {
-        $colDefinition = 'NOT NULL DEFAULT "' . $defaultValue . '"';
+        $colDefinition = 'NOT NULL DEFAULT "'.$defaultValue.'"';
         if (is_null($defaultValue)) {
             $colDefinition = 'NOT NULL';
         }
         if ($nullableYesNo == 'YES') {
-            $colDefinition = 'DEFAULT "' . $defaultValue . '"';
+            $colDefinition = 'DEFAULT "'.$defaultValue.'"';
             if ($defaultValue === null) {
                 $colDefinition = 'DEFAULT NULL';
             }