| @@ -115,7 +115,7 @@ discard block | ||
| 115 | 115 | } | 
| 116 | 116 | |
| 117 | 117 | /** | 
| 118 | - * @param $action | |
| 118 | + * @param string $action | |
| 119 | 119 | * @throws \yii\db\Exception | 
| 120 | 120 | */ | 
| 121 | 121 | public function audit($action) | 
| @@ -261,14 +261,14 @@ discard block | ||
| 261 | 261 | /** | 
| 262 | 262 | * Save the audit trails for a create or update action | 
| 263 | 263 | * | 
| 264 | - * @param $action | |
| 264 | + * @param string $action | |
| 265 | 265 | * @param $newAttributes | 
| 266 | 266 | * @param $oldAttributes | 
| 267 | 267 | * @param $entry_id | 
| 268 | 268 | * @param $user_id | 
| 269 | - * @param $model | |
| 269 | + * @param string $model | |
| 270 | 270 | * @param $model_id | 
| 271 | - * @param $created | |
| 271 | + * @param string $created | |
| 272 | 272 | * @throws \yii\db\Exception | 
| 273 | 273 | */ | 
| 274 | 274 | protected function saveAuditTrail($action, $newAttributes, $oldAttributes, $entry_id, $user_id, $model, $model_id, $created) | 
| @@ -2,9 +2,9 @@ | ||
| 2 | 2 | namespace bedezign\yii2\audit; | 
| 3 | 3 | |
| 4 | 4 | use Yii; | 
| 5 | +use bedezign\yii2\audit\models\AuditTrail; | |
| 5 | 6 | use yii\base\Behavior; | 
| 6 | 7 | use yii\db\ActiveRecord; | 
| 7 | -use bedezign\yii2\audit\models\AuditTrail; | |
| 8 | 8 | use yii\db\Query; | 
| 9 | 9 | |
| 10 | 10 | /** | 
| @@ -177,7 +177,7 @@ discard block | ||
| 177 | 177 | */ | 
| 178 | 178 | protected function cleanAttributesIgnored($attributes) | 
| 179 | 179 |      { | 
| 180 | - if($this->ignore_timestamps) | |
| 180 | + if ($this->ignore_timestamps) | |
| 181 | 181 | $this->ignored = array_merge($this->ignored, [ | 
| 182 | 182 | 'created', 'updated', 'created_at', 'updated_at', 'timestamp']); | 
| 183 | 183 | |
| @@ -199,12 +199,12 @@ discard block | ||
| 199 | 199 | */ | 
| 200 | 200 | protected function cleanAttributesOverride($attributes) | 
| 201 | 201 |      { | 
| 202 | -        if (sizeof($this->override) > 0 && sizeof($attributes) >0) { | |
| 202 | +        if (sizeof($this->override) > 0 && sizeof($attributes) > 0) { | |
| 203 | 203 |              foreach ($this->override as $field => $queryParams) { | 
| 204 | 204 | $newOverrideValues = $this->getNewOverrideValues($attributes[$field], $queryParams); | 
| 205 | 205 | $saveField = \yii\helpers\ArrayHelper::getValue($queryParams, 'saveField', $field); | 
| 206 | 206 | |
| 207 | -                if (count($newOverrideValues) >1) { | |
| 207 | +                if (count($newOverrideValues) > 1) { | |
| 208 | 208 |                      $attributes[$saveField] = implode(', ', | 
| 209 | 209 | \yii\helpers\ArrayHelper::map($newOverrideValues, $queryParams['returnField'], $queryParams['returnField']) | 
| 210 | 210 | ); | 
| @@ -177,9 +177,10 @@ | ||
| 177 | 177 | */ | 
| 178 | 178 | protected function cleanAttributesIgnored($attributes) | 
| 179 | 179 |      { | 
| 180 | - if($this->ignore_timestamps) | |
| 181 | - $this->ignored = array_merge($this->ignored, [ | |
| 180 | +        if($this->ignore_timestamps) { | |
| 181 | + $this->ignored = array_merge($this->ignored, [ | |
| 182 | 182 | 'created', 'updated', 'created_at', 'updated_at', 'timestamp']); | 
| 183 | + } | |
| 183 | 184 | |
| 184 | 185 |          if (sizeof($this->ignored) > 0) { | 
| 185 | 186 |              foreach ($attributes as $f => $v) { |