Passed
Branch master (406faa)
by P.R.
05:09
created
src/Audit/Audit.php 1 patch
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 
63 63
     $this->additionalAuditColumns =
64 64
       AuditDataLayer::resolveCanonicalAdditionalAuditColumns($this->config['database']['audit_schema'],
65
-                                                             $this->config['audit_columns']);
65
+                                                              $this->config['audit_columns']);
66 66
   }
67 67
 
68 68
   //--------------------------------------------------------------------------------------------------------------------
@@ -135,8 +135,8 @@  discard block
 block discarded – undo
135 135
       {
136 136
         $this->io->writeln(sprintf('<info>Found new table %s</info>', $table['table_name']));
137 137
         $this->config['tables'][$table['table_name']] = ['audit' => null,
138
-                                                         'alias' => null,
139
-                                                         'skip'  => null];
138
+                                                          'alias' => null,
139
+                                                          'skip'  => null];
140 140
       }
141 141
     }
142 142
   }
@@ -152,12 +152,12 @@  discard block
 block discarded – undo
152 152
       if ($this->config['tables'][$table['table_name']]['audit'])
153 153
       {
154 154
         $currentTable = new AuditTable($this->io,
155
-                                       $this->config['database']['data_schema'],
156
-                                       $this->config['database']['audit_schema'],
157
-                                       $table['table_name'],
158
-                                       $this->additionalAuditColumns,
159
-                                       $this->config['tables'][$table['table_name']]['alias'],
160
-                                       $this->config['tables'][$table['table_name']]['skip']);
155
+                                        $this->config['database']['data_schema'],
156
+                                        $this->config['database']['audit_schema'],
157
+                                        $table['table_name'],
158
+                                        $this->additionalAuditColumns,
159
+                                        $this->config['tables'][$table['table_name']]['alias'],
160
+                                        $this->config['tables'][$table['table_name']]['skip']);
161 161
 
162 162
         // Ensure the audit table exists.
163 163
         if (AuditDataLayer::searchInRowSet('table_name', $table['table_name'], $this->auditSchemaTables)===null)
Please login to merge, or discard this patch.
src/Audit/Diff.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
 
71 71
     $this->additionalAuditColumns =
72 72
       AuditDataLayer::resolveCanonicalAdditionalAuditColumns($this->config['database']['audit_schema'],
73
-                                                             $this->config['audit_columns']);
73
+                                                              $this->config['audit_columns']);
74 74
   }
75 75
 
76 76
   //--------------------------------------------------------------------------------------------------------------------
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@
 block discarded – undo
123 123
     $dataTableColumns = TableColumnsMetadata::combine($this->additionalAuditColumns, $dataTableColumns);
124 124
 
125 125
     // In the audit table columns coming from the data table don't have defaults.
126
-    foreach($auditTableColumns->getColumns() as $column)
126
+    foreach ($auditTableColumns->getColumns() as $column)
127 127
     {
128 128
       if (!in_array($column->getName(), $this->additionalAuditColumns->getColumnNames()))
129 129
       {
Please login to merge, or discard this patch.
src/Audit/AlterAuditTable.php 1 patch
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 
49 49
     $this->additionalAuditColumns =
50 50
       AuditDataLayer::resolveCanonicalAdditionalAuditColumns($this->config['database']['audit_schema'],
51
-                                                             $this->config['audit_columns']);
51
+                                                              $this->config['audit_columns']);
52 52
   }
53 53
 
54 54
   //--------------------------------------------------------------------------------------------------------------------
@@ -103,8 +103,8 @@  discard block
 block discarded – undo
103 103
       $maxLength = $diff->getLongestColumnNameLength();
104 104
 
105 105
       $this->codeStore->append(sprintf('alter table `%s`.`%s`',
106
-                                       $this->config['database']['audit_schema'],
107
-                                       $auditTable->getTableName()));
106
+                                        $this->config['database']['audit_schema'],
107
+                                        $auditTable->getTableName()));
108 108
 
109 109
       $first = true;
110 110
       foreach ($diff->getColumns() as $column)
@@ -115,11 +115,11 @@  discard block
 block discarded – undo
115 115
         if (!$first) $this->codeStore->appendToLastLine(',');
116 116
 
117 117
         $this->codeStore->append(sprintf('change column `%s`%s`%s`%s%s',
118
-                                         $name,
119
-                                         $filler,
120
-                                         $name,
121
-                                         $filler,
122
-                                         $column->getColumnAuditDefinition()));
118
+                                          $name,
119
+                                          $filler,
120
+                                          $name,
121
+                                          $filler,
122
+                                          $column->getColumnAuditDefinition()));
123 123
 
124 124
         $first = false;
125 125
       }
@@ -165,9 +165,9 @@  discard block
 block discarded – undo
165 165
       }
166 166
 
167 167
       $this->codeStore->append(sprintf('alter table `%s`.`%s` %s;',
168
-                                       $this->config['database']['audit_schema'],
169
-                                       $auditTable->getTableName(),
170
-                                       implode(' ', $parts)));
168
+                                        $this->config['database']['audit_schema'],
169
+                                        $auditTable->getTableName(),
170
+                                        implode(' ', $parts)));
171 171
       $this->codeStore->append('');
172 172
     }
173 173
   }
Please login to merge, or discard this patch.
src/DiffTable.php 1 patch
Indentation   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -229,20 +229,20 @@  discard block
 block discarded – undo
229 229
       if (in_array($column, $auditColumns))
230 230
       {
231 231
         $this->rows[] = ['name'     => $column,
232
-                         'audit'    => $this->auditTable->getColumns()->getColumn($column),
233
-                         'data'     => $this->dataTable->getColumns()->getColumn($column),
234
-                         'type'     => 'column',
235
-                         'new'      => false,
236
-                         'obsolete' => false];
232
+                          'audit'    => $this->auditTable->getColumns()->getColumn($column),
233
+                          'data'     => $this->dataTable->getColumns()->getColumn($column),
234
+                          'type'     => 'column',
235
+                          'new'      => false,
236
+                          'obsolete' => false];
237 237
       }
238 238
       else
239 239
       {
240 240
         $this->rows[] = ['name'     => $column,
241
-                         'audit'    => null,
242
-                         'data'     => $this->dataTable->getColumns()->getColumn($column),
243
-                         'type'     => 'column',
244
-                         'new'      => true,
245
-                         'obsolete' => false];
241
+                          'audit'    => null,
242
+                          'data'     => $this->dataTable->getColumns()->getColumn($column),
243
+                          'type'     => 'column',
244
+                          'new'      => true,
245
+                          'obsolete' => false];
246 246
       }
247 247
     }
248 248
 
@@ -251,11 +251,11 @@  discard block
 block discarded – undo
251 251
       if (!in_array($column, $dataColumns))
252 252
       {
253 253
         $this->rows[] = ['name'     => $column,
254
-                         'audit'    => $this->auditTable->getColumns()->getColumn($column),
255
-                         'data'     => null,
256
-                         'type'     => 'column',
257
-                         'new'      => false,
258
-                         'obsolete' => true];
254
+                          'audit'    => $this->auditTable->getColumns()->getColumn($column),
255
+                          'data'     => null,
256
+                          'type'     => 'column',
257
+                          'new'      => false,
258
+                          'obsolete' => true];
259 259
       }
260 260
     }
261 261
   }
@@ -272,10 +272,10 @@  discard block
 block discarded – undo
272 272
     foreach ($dataOptions as $option)
273 273
     {
274 274
       $this->rows[] = ['name'    => $option,
275
-                       'audit1'  => $this->auditTable->getProperty($option),
276
-                       'data1'   => $this->dataTable->getProperty($option),
277
-                       'type'    => 'option',
278
-                       'rowspan' => 1];
275
+                        'audit1'  => $this->auditTable->getProperty($option),
276
+                        'data1'   => $this->dataTable->getProperty($option),
277
+                        'type'    => 'option',
278
+                        'rowspan' => 1];
279 279
     }
280 280
 
281 281
     foreach ($auditOptions as $option)
@@ -283,10 +283,10 @@  discard block
 block discarded – undo
283 283
       if (!in_array($option, $dataOptions))
284 284
       {
285 285
         $this->rows[] = ['name'    => $option,
286
-                         'audit1'  => $this->auditTable->getProperty($option),
287
-                         'data2'   => null,
288
-                         'type'    => 'option',
289
-                         'rowspan' => 1];
286
+                          'audit1'  => $this->auditTable->getProperty($option),
287
+                          'data2'   => null,
288
+                          'type'    => 'option',
289
+                          'rowspan' => 1];
290 290
       }
291 291
     }
292 292
   }
Please login to merge, or discard this patch.
src/Command/DiffCommand.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -32,9 +32,9 @@
 block discarded – undo
32 32
   protected function configure()
33 33
   {
34 34
     $this->setName('diff')
35
-         ->setDescription('Compares data tables and audit tables')
36
-         ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file')
37
-         ->addOption('full', 'f', InputOption::VALUE_NONE, 'Show all columns');
35
+          ->setDescription('Compares data tables and audit tables')
36
+          ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file')
37
+          ->addOption('full', 'f', InputOption::VALUE_NONE, 'Show all columns');
38 38
   }
39 39
 
40 40
   //--------------------------------------------------------------------------------------------------------------------
Please login to merge, or discard this patch.
src/Command/DropTriggersCommand.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -21,10 +21,10 @@  discard block
 block discarded – undo
21 21
   protected function configure()
22 22
   {
23 23
     $this->setName('drop-triggers')
24
-         ->setDescription('Drops all triggers')
25
-         ->setHelp('Drops all triggers (including triggers not created by audit) from all tables (including tables '.
26
-                   'excluded for auditing) in the data schema.')
27
-         ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file');
24
+          ->setDescription('Drops all triggers')
25
+          ->setHelp('Drops all triggers (including triggers not created by audit) from all tables (including tables '.
26
+                    'excluded for auditing) in the data schema.')
27
+          ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file');
28 28
   }
29 29
 
30 30
   //--------------------------------------------------------------------------------------------------------------------
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
60 60
     foreach ($triggers as $trigger)
61 61
     {
62 62
       $this->io->logInfo('Dropping trigger <dbo>%s</dbo> from table <dbo>%s</dbo>',
63
-                         $trigger['trigger_name'],
64
-                         $trigger['table_name']);
63
+                          $trigger['trigger_name'],
64
+                          $trigger['table_name']);
65 65
 
66 66
       AuditDataLayer::dropTrigger($data_schema, $trigger['trigger_name']);
67 67
     }
Please login to merge, or discard this patch.
src/Command/AlterAuditTableCommand.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -31,9 +31,9 @@
 block discarded – undo
31 31
   protected function configure()
32 32
   {
33 33
     $this->setName('alter-audit-table')
34
-         ->setDescription('Creates alter SQL statements for audit tables')
35
-         ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file')
36
-         ->addArgument('sql file', InputArgument::OPTIONAL, 'The destination file for the SQL statements');
34
+          ->setDescription('Creates alter SQL statements for audit tables')
35
+          ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file')
36
+          ->addArgument('sql file', InputArgument::OPTIONAL, 'The destination file for the SQL statements');
37 37
 
38 38
     $this->setHelp(<<<EOL
39 39
 Generates alter table SQL statements for aligning the audit tables with the 
Please login to merge, or discard this patch.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 No SQL statements will be generated for missing or obsolete columns in the 
50 50
 audit tables. Use the command 'audit' for creating missing columns in audit
51 51
 tables.
52
-EOL
52
+eol
53 53
     );
54 54
   }
55 55
 
Please login to merge, or discard this patch.
src/Command/AuditCommand.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -22,12 +22,12 @@
 block discarded – undo
22 22
   protected function configure()
23 23
   {
24 24
     $this->setName('audit')
25
-         ->setDescription('Maintains audit tables and audit triggers')
26
-         ->setHelp("Maintains audit tables and audit triggers:\n".
27
-                   "- creates new audit tables\n".
28
-                   "- adds new columns to exiting audit tables\n".
29
-                   "- creates new and recreates existing audit triggers\n")
30
-         ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file');
25
+          ->setDescription('Maintains audit tables and audit triggers')
26
+          ->setHelp("Maintains audit tables and audit triggers:\n".
27
+                    "- creates new audit tables\n".
28
+                    "- adds new columns to exiting audit tables\n".
29
+                    "- creates new and recreates existing audit triggers\n")
30
+          ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file');
31 31
   }
32 32
 
33 33
   //--------------------------------------------------------------------------------------------------------------------
Please login to merge, or discard this patch.
src/MySql/AuditDataLayer.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
                                      $tableColumns,
91 91
                                      $skipVariable,
92 92
                                      $additionSql);
93
-    $sql    = $helper->buildStatement();
93
+    $sql = $helper->buildStatement();
94 94
 
95 95
     static::executeNone($sql);
96 96
   }
Please login to merge, or discard this patch.
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -86,14 +86,14 @@  discard block
 block discarded – undo
86 86
                                             array $additionSql): void
87 87
   {
88 88
     $helper = new CreateAuditTrigger($dataSchemaName,
89
-                                     $auditSchemaName,
90
-                                     $tableName,
91
-                                     $triggerName,
92
-                                     $triggerAction,
93
-                                     $additionalAuditColumns,
94
-                                     $tableColumns,
95
-                                     $skipVariable,
96
-                                     $additionSql);
89
+                                      $auditSchemaName,
90
+                                      $tableName,
91
+                                      $triggerName,
92
+                                      $triggerAction,
93
+                                      $additionalAuditColumns,
94
+                                      $tableColumns,
95
+                                      $skipVariable,
96
+                                      $additionSql);
97 97
     $sql    = $helper->buildStatement();
98 98
 
99 99
     static::executeNone($sql);
@@ -275,8 +275,8 @@  discard block
 block discarded – undo
275 275
 where  TABLE_SCHEMA = %s
276 276
 and    TABLE_NAME   = %s
277 277
 order by ORDINAL_POSITION",
278
-                   static::quoteString($schemaName),
279
-                   static::quoteString($tableName));
278
+                    static::quoteString($schemaName),
279
+                    static::quoteString($tableName));
280 280
 
281 281
     return static::executeRows($sql);
282 282
   }
@@ -302,8 +302,8 @@  discard block
 block discarded – undo
302 302
 inner join information_schema.COLLATION_CHARACTER_SET_APPLICABILITY t2  on  t2.COLLATION_NAME = t1.TABLE_COLLATION
303 303
 WHERE t1.TABLE_SCHEMA = %s
304 304
 AND   t1.TABLE_NAME   = %s',
305
-                   static::quoteString($schemaName),
306
-                   static::quoteString($tableName));
305
+                    static::quoteString($schemaName),
306
+                    static::quoteString($tableName));
307 307
 
308 308
     return static::executeRow1($sql);
309 309
   }
@@ -325,8 +325,8 @@  discard block
 block discarded – undo
325 325
 where  TRIGGER_SCHEMA     = %s
326 326
 and    EVENT_OBJECT_TABLE = %s
327 327
 order by Trigger_Name',
328
-                   static::quoteString($schemaName),
329
-                   static::quoteString($tableName));
328
+                    static::quoteString($schemaName),
329
+                    static::quoteString($tableName));
330 330
 
331 331
     return static::executeRows($sql);
332 332
   }
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
 where  TRIGGER_SCHEMA     = %s
369 369
 order by EVENT_OBJECT_TABLE
370 370
 ,        TRIGGER_NAME',
371
-                   static::quoteString($schemaName));
371
+                    static::quoteString($schemaName));
372 372
 
373 373
     return static::executeRows($sql);
374 374
   }
Please login to merge, or discard this patch.