SetBased / php-audit

src/Columns.php

Doc Comments +1 added lines, -1 removed lines

@@ -144,7 +144,7 @@
    *
    * @param string $columnName The column name.
    *
-   * @return null|string
+   * @return integer|null
    */
   public function getPreviousColumn($columnName)
   {

Indentation +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
         if (!isset($columns2->columns[$column1['column_name']]))
         {
           $diff[] = ['column_name' => $column1['column_name'],
-                     'column_type' => $column1['column_type']];
+                      'column_type' => $column1['column_type']];
         }
       }
     }

Spacing +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
     $columns = array_keys($this->columns);
     $key     = array_search($columnName, $columns);
 
-    if ($key>=1)
+    if ($key >= 1)
     {
       return $columns[$key - 1];
     }

src/Table.php

Doc Comments +1 added lines, -1 removed lines

@@ -268,7 +268,7 @@
   /**
    * Selects and returns the metadata of the columns of this table from information_schema.
    *
-   * @return array[]
+   * @return \array[]
    */
   private function getColumnsFromInformationSchema()
   {

Indentation +4 added lines, -4 removed lines

@@ -259,7 +259,7 @@ 
   private function getAlteredColumns()
   {
     $alteredColumnsTypes = Columns::differentColumnTypes($this->dataTableColumnsDatabase,
-                                                         $this->dataTableColumnsConfig);
+                                                          $this->dataTableColumnsConfig);
 
     return $alteredColumnsTypes;
   }
@@ -382,9 +382,9 @@ 
     foreach ($alteredColumns as $column)
     {
       $this->io->logInfo('Type of <dbo>%s.%s</dbo> has been altered to <dbo>%s</dbo>',
-                         $this->tableName,
-                         $column['column_name'],
-                         $column['column_type']);
+                          $this->tableName,
+                          $column['column_name'],
+                          $column['column_type']);
     }
   }
 

src/MySql/DataLayer.php

Indentation +14 added lines, -14 removed lines

@@ -109,14 +109,14 @@ 
                                             $additionSql)
   {
     $helper = new CreateAuditTrigger($dataSchemaName,
-                                     $auditSchemaName,
-                                     $tableName,
-                                     $triggerAction,
-                                     $triggerName,
-                                     $tableColumns,
-                                     $auditColumns,
-                                     $skipVariable,
-                                     $additionSql);
+                                      $auditSchemaName,
+                                      $tableName,
+                                      $triggerAction,
+                                      $triggerName,
+                                      $tableColumns,
+                                      $auditColumns,
+                                      $skipVariable,
+                                      $additionSql);
     $sql    = $helper->buildStatement();
 
     self::executeNone($sql);
@@ -259,8 +259,8 @@ 
 where  TABLE_SCHEMA = %s
 and    TABLE_NAME   = %s
 order by ORDINAL_POSITION',
-                   self::$dl->quoteString($schemaName),
-                   self::$dl->quoteString($tableName));
+                    self::$dl->quoteString($schemaName),
+                    self::$dl->quoteString($tableName));
 
     return self::$dl->executeRows($sql);
   }
@@ -284,8 +284,8 @@ 
 inner join information_schema.COLLATION_CHARACTER_SET_APPLICABILITY t2  on  t2.COLLATION_NAME = t1.TABLE_COLLATION
 WHERE t1.TABLE_SCHEMA = %s
 AND   t1.TABLE_NAME   = %s',
-                   self::$dl->quoteString($schemaName),
-                   self::$dl->quoteString($tableName));
+                    self::$dl->quoteString($schemaName),
+                    self::$dl->quoteString($tableName));
 
     return self::$dl->executeRow1($sql);
   }
@@ -307,8 +307,8 @@ 
 where  TRIGGER_SCHEMA     = %s
 and    EVENT_OBJECT_TABLE = %s
 order by Trigger_Name',
-                   self::$dl->quoteString($schemaName),
-                   self::$dl->quoteString($tableName));
+                    self::$dl->quoteString($schemaName),
+                    self::$dl->quoteString($tableName));
 
     return self::$dl->executeRows($sql);
   }

Spacing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
                                      $auditColumns,
                                      $skipVariable,
                                      $additionSql);
-    $sql    = $helper->buildStatement();
+    $sql = $helper->buildStatement();
 
     self::executeNone($sql);
   }

src/MySql/Helper/CompoundSyntaxStore.php

Spacing +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
     if ($trim) $line = trim($line);
 
     $words = explode(' ', $line);
-    if (count($words)>0)
+    if (count($words) > 0)
     {
       switch ($words[0])
       {

Braces +4 added lines, -1 removed lines

@@ -111,7 +111,10 @@
    */
   private function appendLine($line, $trim)
   {
-    if ($trim) $line = trim($line);
+    if ($trim)
+    {
+      $line = trim($line);
+    }
 
     $words = explode(' ', $line);
     if (count($words)>0)

src/MySql/Command/AuditCommand.php

Indentation +4 added lines, -4 removed lines

@@ -101,8 +101,8 @@ 
       {
         $this->io->writeln(sprintf('<info>Found new table %s</info>', $table['table_name']));
         $this->config['tables'][$table['table_name']] = ['audit' => false,
-                                                         'alias' => null,
-                                                         'skip'  => null];
+                                                          'alias' => null,
+                                                          'skip'  => null];
       }
     }
   }
@@ -114,8 +114,8 @@ 
   protected function configure()
   {
     $this->setName('audit')
-         ->setDescription('Create (missing) audit table and (re)creates audit triggers')
-         ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file', 'etc/audit.json');
+          ->setDescription('Create (missing) audit table and (re)creates audit triggers')
+          ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file', 'etc/audit.json');
   }
 
   //--------------------------------------------------------------------------------------------------------------------

Spacing +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
                                   $this->config['audit_columns'],
                                   $this->config['tables'][$table['table_name']]['alias'],
                                   $this->config['tables'][$table['table_name']]['skip']);
-        $res          = StaticDataLayer::searchInRowSet('table_name', $currentTable->getTableName(), $this->auditSchemaTables);
+        $res = StaticDataLayer::searchInRowSet('table_name', $currentTable->getTableName(), $this->auditSchemaTables);
         if (!isset($res))
         {
           $currentTable->createMissingAuditTable();

src/MySql/Sql/CreateAuditTrigger.php

Braces +24 added lines, -6 removed lines

@@ -155,7 +155,10 @@ 
     $this->code->append('for each row');
     $this->code->append('begin');
 
-    if ($this->skipVariable!==null) $this->code->append(sprintf('if (%s is null) then', $this->skipVariable));
+    if ($this->skipVariable!==null)
+    {
+      $this->code->append(sprintf('if (%s is null) then', $this->skipVariable));
+    }
 
     $this->code->append($this->additionalSql);
 
@@ -165,7 +168,10 @@ 
       $this->createInsertStatement($rowState[1]);
     }
 
-    if ($this->skipVariable!==null) $this->code->append('end if;');
+    if ($this->skipVariable!==null)
+    {
+      $this->code->append('end if;');
+    }
     $this->code->append('end');
 
     return $this->code->getCode();
@@ -194,14 +200,20 @@ 
     // First the audit columns.
     foreach ($this->auditColumns->getColumns() as $column)
     {
-      if ($columnNames) $columnNames .= ',';
+      if ($columnNames)
+      {
+        $columnNames .= ',';
+      }
       $columnNames .= sprintf('`%s`', $column['column_name']);
     }
 
     // Second the audit columns.
     foreach ($this->tableColumns->getColumns() as $column)
     {
-      if ($columnNames) $columnNames .= ',';
+      if ($columnNames)
+      {
+        $columnNames .= ',';
+      }
       $columnNames .= sprintf('`%s`', $column['column_name']);
     }
 
@@ -221,7 +233,10 @@ 
     // First the values for the audit columns.
     foreach ($this->auditColumns->getColumns() as $column)
     {
-      if ($values) $values .= ',';
+      if ($values)
+      {
+        $values .= ',';
+      }
       switch (true)
       {
         case (isset($column['audit_value_type'])):
@@ -252,7 +267,10 @@ 
     // Second the values for the audit columns.
     foreach ($this->tableColumns->getColumns() as $column)
     {
-      if ($values) $values .= ',';
+      if ($values)
+      {
+        $values .= ',';
+      }
       $values .= sprintf('%s.`%s`', $rowState, $column['column_name']);
     }
 

src/Command/AboutCommand.php

Indentation +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
   protected function configure()
   {
     $this->setName('about')
-         ->setDescription('Short information about AuditApplication')
-         ->setHelp('<info>audit about</info>');
+          ->setDescription('Short information about AuditApplication')
+          ->setHelp('<info>audit about</info>');
   }
 
   //--------------------------------------------------------------------------------------------------------------------

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     $this->io->write(<<<EOT
 <info>AuditApplication - Database Auditing</info>
 <comment>Creates audit tables and triggers to track data changes in databases.</comment>
-EOT
+eot
     );
   }
 

src/Command/DiffCommand.php

Indentation +3 added lines, -3 removed lines

@@ -73,9 +73,9 @@
   protected function configure()
   {
     $this->setName('diff')
-         ->setDescription('Compares data tables and audit tables')
-         ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file', 'etc/audit.json')
-         ->addOption('full', 'f', InputOption::VALUE_NONE, 'Show all columns');
+          ->setDescription('Compares data tables and audit tables')
+          ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file', 'etc/audit.json')
+          ->addOption('full', 'f', InputOption::VALUE_NONE, 'Show all columns');
   }
 
   //--------------------------------------------------------------------------------------------------------------------

Spacing +1 added lines, -1 removed lines

@@ -201,7 +201,7 @@
           // Removing audit columns.
           $auditColumns = array_udiff($auditColumns,
                                       $this->config['audit_columns'],
-            function ($a, $b)
+            function($a, $b)
             {
               return strcmp($a['column_name'], $b['column_name']);
             });

src/Command/BaseCommand.php

Indentation +2 added lines, -2 removed lines

@@ -70,8 +70,8 @@
       if ($mandatory)
       {
         throw new RuntimeException("Setting '%s' not found in section '%s' configuration file.",
-                                   $settingName,
-                                   $sectionName);
+                                    $settingName,
+                                    $sectionName);
       }
       else
       {

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
   {
     $content = file_get_contents($this->configFileName);
 
-    $this->config = (array)json_decode($content, true);
+    $this->config = (array) json_decode($content, true);
     if (json_last_error()!=JSON_ERROR_NONE)
     {
       throw new RuntimeException("Error decoding JSON: '%s'.", json_last_error_msg());

Braces +4 added lines, -1 removed lines

@@ -137,7 +137,10 @@
     if (file_exists($filename))
     {
       $old_data = file_get_contents($filename);
-      if ($data==$old_data) $write_flag = false;
+      if ($data==$old_data)
+      {
+        $write_flag = false;
+      }
     }
 
     if ($write_flag)