SetBased / php-audit

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/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
       {

src/Command/ConfigCommand.php

Indentation +2 added lines, -2 removed lines

@@ -28,8 +28,8 @@
   protected function configure()
   {
     $this->setName('config')
-         ->setDescription('Create or edit config file')
-         ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file', 'etc/audit.json');
+          ->setDescription('Create or edit config file')
+          ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file', 'etc/audit.json');
   }
 
   //--------------------------------------------------------------------------------------------------------------------

src/MySql/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');
   }
 
   //--------------------------------------------------------------------------------------------------------------------

src/MySql/Command/AuditCommand.php

Indentation +2 added lines, -2 removed lines

@@ -22,8 +22,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');
+          ->setDescription('Create (missing) audit table and (re)creates audit triggers')
+          ->addArgument('config file', InputArgument::OPTIONAL, 'The audit configuration file');
   }
 
   //--------------------------------------------------------------------------------------------------------------------

src/MySql/AuditDataLayer.php

Spacing +1 added lines, -1 removed lines

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

Indentation +15 added lines, -15 removed lines

@@ -110,14 +110,14 @@ 
                                             $additionSql)
   {
     $helper = new CreateAuditTrigger($dataSchemaName,
-                                     $auditSchemaName,
-                                     $tableName,
-                                     $triggerName,
-                                     $triggerAction,
-                                     $auditColumns,
-                                     $tableColumns,
-                                     $skipVariable,
-                                     $additionSql);
+                                      $auditSchemaName,
+                                      $tableName,
+                                      $triggerName,
+                                      $triggerAction,
+                                      $auditColumns,
+                                      $tableColumns,
+                                      $skipVariable,
+                                      $additionSql);
     $sql    = $helper->buildStatement();
 
     self::executeNone($sql);
@@ -302,8 +302,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::executeRows($sql);
   }
@@ -327,8 +327,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::executeRow1($sql);
   }
@@ -350,8 +350,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::executeRows($sql);
   }
@@ -393,7 +393,7 @@ 
 where  TRIGGER_SCHEMA     = %s
 order by EVENT_OBJECT_TABLE
 ,        TRIGGER_NAME',
-                   self::$dl->quoteString($schemaName));
+                    self::$dl->quoteString($schemaName));
 
     return self::executeRows($sql);
   }

src/MySql/Command/DropTriggersCommand.php

Indentation +4 added lines, -4 removed lines

@@ -21,8 +21,8 @@ 
   protected function configure()
   {
     $this->setName('drop-triggers')
-         ->setDescription('Drops all triggers')
-         ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file');
+          ->setDescription('Drops all triggers')
+          ->addArgument('config file', InputArgument::REQUIRED, 'The audit configuration file');
   }
 
   //--------------------------------------------------------------------------------------------------------------------
@@ -58,8 +58,8 @@ 
     foreach ($triggers as $trigger)
     {
       $this->io->logInfo('Dropping trigger <dbo>%s</dbo> from table <dbo>%s</dbo>',
-                         $trigger['trigger_name'],
-                         $trigger['table_name']);
+                          $trigger['trigger_name'],
+                          $trigger['table_name']);
 
       AuditDataLayer::dropTrigger($data_schema, $trigger['trigger_name']);
     }

src/MySql/AuditTable.php

Indentation +20 added lines, -20 removed lines

@@ -108,8 +108,8 @@ 
   public function createAuditTable()
   {
     $this->io->logInfo('Creating audit table <dbo>%s.%s<dbo>',
-                       $this->auditSchemaName,
-                       $this->configTable->getTableName());
+                        $this->auditSchemaName,
+                        $this->configTable->getTableName());
 
     // In the audit table all columns from the data table must be nullable.
     $dataTableColumnsDatabase = clone($this->dataTableColumnsDatabase);
@@ -117,9 +117,9 @@ 
 
     $columns = TableColumnsMetadata::combine($this->auditColumns, $dataTableColumnsDatabase);
     AuditDataLayer::createAuditTable($this->configTable->getSchemaName(),
-                                     $this->auditSchemaName,
-                                     $this->configTable->getTableName(),
-                                     $columns);
+                                      $this->auditSchemaName,
+                                      $this->configTable->getTableName(),
+                                      $columns);
   }
 
   //--------------------------------------------------------------------------------------------------------------------
@@ -242,14 +242,14 @@ 
                           $this->configTable->getTableName());
 
     AuditDataLayer::createAuditTrigger($this->configTable->getSchemaName(),
-                                       $this->auditSchemaName,
-                                       $this->configTable->getTableName(),
-                                       $triggerName,
-                                       $action,
-                                       $this->auditColumns,
-                                       $this->dataTableColumnsDatabase,
-                                       $skipVariable,
-                                       $additionSql);
+                                        $this->auditSchemaName,
+                                        $this->configTable->getTableName(),
+                                        $triggerName,
+                                        $action,
+                                        $this->auditColumns,
+                                        $this->dataTableColumnsDatabase,
+                                        $skipVariable,
+                                        $additionSql);
   }
 
   //--------------------------------------------------------------------------------------------------------------------
@@ -366,23 +366,23 @@ 
     foreach ($newColumns->getColumns() as $column)
     {
       $this->io->logInfo('New column <dbo>%s.%s</dbo>',
-                         $this->configTable->getTableName(),
-                         $column->getProperty('column_name'));
+                          $this->configTable->getTableName(),
+                          $column->getProperty('column_name'));
     }
 
     foreach ($obsoleteColumns->getColumns() as $column)
     {
       $this->io->logInfo('Obsolete column <dbo>%s.%s</dbo>',
-                         $this->configTable->getTableName(),
-                         $column->getProperty('column_name'));
+                          $this->configTable->getTableName(),
+                          $column->getProperty('column_name'));
     }
 
     foreach ($alteredColumns->getColumns() as $column)
     {
       $this->io->logInfo('Type of <dbo>%s.%s</dbo> has been altered to <dbo>%s</dbo>',
-                         $this->configTable->getTableName(),
-                         $column->getProperty('column_name'),
-                         $column->getProperty('column_type'));
+                          $this->configTable->getTableName(),
+                          $column->getProperty('column_name'),
+                          $column->getProperty('column_type'));
     }
   }
 

src/MySql/AuditDiffTable.php

Doc Comments +3 added lines, -3 removed lines

@@ -89,7 +89,7 @@ 
   /**
    * Check full and return array without new or obsolete columns if full not set.
    *
-   * @return DiffTableColumns
+   * @return TableColumnsMetadata
    */
   public function removeMatchingColumns()
   {
@@ -139,9 +139,9 @@ 
   /**
    * Add not null to audit columns if it not nullable.
    *
-   * @param array $theColumns Audit columns.
+   * @param \array[] $theColumns Audit columns.
    *
-   * @return array
+   * @return \array[]
    */
   private function addNotNull($theColumns)
   {

Braces +1 added lines, -2 removed lines

@@ -125,8 +125,7 @@
           && $audit_character_set_name==$config_character_set_name
           && $audit_collation_name==$config_collation_name
           && $audit_collation_name==$data_collation_name
-        )
-        {
+        ) {
           $metadata->removeColumn($columnName);
         }
       }