siduko / laravel-activitylog

src/ActivityLogger.php

Doc Comments +4 added lines

@@ -173,6 +173,10 @@
         }, $description);
     }
 
+    /**
+     * @param string $start
+     * @param string $end
+     */
     function getBetween($content,$start,$end){
         $r = explode($start, $content);
         if (isset($r[1])){

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Illuminate\Support\Traits\Macroable;
 use LaiVu\ActivityLog\Exceptions\InvalidConfiguration;
 use LaiVu\ActivityLog\Handlers\ActivityLogHandlerInterface;
-use LaiVu\ActivityLog\Models\Activity;
 
 class ActivityLogger
 {

Spacing +7 added lines, -7 removed lines

@@ -150,7 +150,7 @@ 
     {
         $maxAgeInDays = config('activitylog.delete_records_older_than_days');
 
-        if(is_null($maxAgeInDays) || $maxAgeInDays ==''){
+        if (is_null($maxAgeInDays) || $maxAgeInDays == '') {
             return false;
         }
 
@@ -162,20 +162,20 @@ 
 
     protected function replacePlaceholders($description)
     {
-        return preg_replace_callback('/:[a-z0-9._-]+/i', function ($match) {
+        return preg_replace_callback('/:[a-z0-9._-]+/i', function($match) {
             $match = $match[0];
-            $attribute =$this->getBetween($match,':', '.');
-            if (! in_array($attribute, ['performedOn', 'causedBy', 'properties'])) {
+            $attribute = $this->getBetween($match, ':', '.');
+            if (!in_array($attribute, ['performedOn', 'causedBy', 'properties'])) {
                 return $match;
             }
             $propertyName = substr($match, strpos($match, '.') + 1);
-            return isset($this->{$attribute})&&isset($this->{$attribute}->{$propertyName})?$this->{$attribute}->{$propertyName}:$match;
+            return isset($this->{$attribute}) && isset($this->{$attribute}->{$propertyName}) ? $this->{$attribute}->{$propertyName}:$match;
         }, $description);
     }
 
-    function getBetween($content,$start,$end){
+    function getBetween($content, $start, $end) {
         $r = explode($start, $content);
-        if (isset($r[1])){
+        if (isset($r[1])) {
             $r = explode($end, $r[1]);
             return $r[0];
         }

src/Handlers/LogHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
      * @param array $properties
      * @param $logName
      * @param string $description
-     * @return mixed
+     * @return boolean
      */
     function log($performOn, $causerBy, $properties = [], $logName, $description)
     {
@@ -33,7 +33,7 @@ 
 
     /**
      * @param $maxAgeInMonth
-     * @return boolean
+     * @return boolean|null
      */
     function cleanLog($maxAgeInMonth)
     {

src/Exceptions/InvalidConfiguration.php

Spacing +4 added lines, -4 removed lines

@@ -16,14 +16,14 @@
 {
     public static function modelIsNotValid($className)
     {
-        return new static("The given model class `$className` does not extend `".Activity::class.'`');
+        return new static("The given model class `$className` does not extend `" . Activity::class . '`');
     }
 
-    public static function handlerNotFound($handlerName){
+    public static function handlerNotFound($handlerName) {
         return new static("The given handler `$handlerName` does not exist");
     }
 
-    public static function handlerTypeNotValid($className){
-        return new static("The given handler class `$className` does not extend `".ActivityLogHandlerInterface::class.'`');
+    public static function handlerTypeNotValid($className) {
+        return new static("The given handler class `$className` does not extend `" . ActivityLogHandlerInterface::class . '`');
     }
 }
\ No newline at end of file

src/Models/Activity.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      */
     public function getChangesAttribute()
     {
-        return collect(array_filter($this->properties->toArray(), function ($key) {
+        return collect(array_filter($this->properties->toArray(), function($key) {
             return in_array($key, ['attributes', 'old']);
         }, ARRAY_FILTER_USE_KEY));
     }

src/Traits/DetectsChanges.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
     protected static function bootDetectsChanges()
     {
         if (static::eventsToBeRecorded()->contains('updated')) {
-            static::updating(function (Model $model) {
+            static::updating(function(Model $model) {
                 //temporary hold the original attributes on the model
                 //as we'll need these in the updating event
                 $oldValues = $model->replicate()->setRawAttributes($model->getOriginal());
@@ -24,14 +24,14 @@ 
     }
     public function attributesToBeLogged()
     {
-        if (! isset(static::$logAttributes)) {
+        if (!isset(static::$logAttributes)) {
             return [];
         }
         return static::$logAttributes;
     }
-    public function attributeValuesToBeLogged( $processingEvent)
+    public function attributeValuesToBeLogged($processingEvent)
     {
-        if (! count($this->attributesToBeLogged())) {
+        if (!count($this->attributesToBeLogged())) {
             return [];
         }
         $properties['attributes'] = static::logChanges($this);

src/Traits/LogsActivity.php

Spacing +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
 
     protected static function bootLogsActivity()
     {
-        static::eventsToBeRecorded()->each(function ($eventName) {
-            return static::$eventName(function (Model $model) use ($eventName) {
+        static::eventsToBeRecorded()->each(function($eventName) {
+            return static::$eventName(function(Model $model) use ($eventName) {
                 if (!$model->shouldLogEvent($eventName)) {
                     return;
                 }

src/ActivityLogServiceProvider.php

Spacing +4 added lines, -4 removed lines

@@ -15,15 +15,15 @@
     public function boot()
     {
         $this->publishes([
-            __DIR__.'/../config/activitylog.php' => config_path('activitylog.php'),
+            __DIR__ . '/../config/activitylog.php' => config_path('activitylog.php'),
         ], 'config');
 
-        $this->mergeConfigFrom(__DIR__.'/../config/activitylog.php', 'activitylog');
+        $this->mergeConfigFrom(__DIR__ . '/../config/activitylog.php', 'activitylog');
 
-        if (! class_exists('CreateActivityLogTable')) {
+        if (!class_exists('CreateActivityLogTable')) {
             $timestamp = date('Y_m_d_His', time());
             $this->publishes([
-                __DIR__.'/../migrations/create_activity_log_table.php.stub' => database_path("/migrations/{$timestamp}_create_activity_log_table.php"),
+                __DIR__ . '/../migrations/create_activity_log_table.php.stub' => database_path("/migrations/{$timestamp}_create_activity_log_table.php"),
             ], 'migrations');
         }
     }

prefill.php

Spacing +23 added lines, -23 removed lines

@@ -4,35 +4,35 @@ 
 define('COL_DEFAULT', 2);
 
 $fields = [
-    'author_name' =>            ['Your name',             '',                                                ''],
-    'author_github_username' => ['Your Github username',  '<username> in https://github.com/username',       ''],
-    'author_email' =>           ['Your email address',    '',                                                ''],
-    'author_twitter' =>         ['Your twitter username', '',                                                '@{author_github_username}'],
-    'author_website' =>         ['Your website',          '',                                                'https://github.com/{author_github_username}'],
+    'author_name' =>            ['Your name', '', ''],
+    'author_github_username' => ['Your Github username', '<username> in https://github.com/username', ''],
+    'author_email' =>           ['Your email address', '', ''],
+    'author_twitter' =>         ['Your twitter username', '', '@{author_github_username}'],
+    'author_website' =>         ['Your website', '', 'https://github.com/{author_github_username}'],
 
-    'package_vendor' =>         ['Package vendor',        '<vendor> in https://github.com/vendor/package',   '{author_github_username}'],
-    'package_name' =>           ['Package name',          '<package> in https://github.com/vendor/package',  ''],
-    'package_description' =>    ['Package very short description',   '',                                     ''],
+    'package_vendor' =>         ['Package vendor', '<vendor> in https://github.com/vendor/package', '{author_github_username}'],
+    'package_name' =>           ['Package name', '<package> in https://github.com/vendor/package', ''],
+    'package_description' =>    ['Package very short description', '', ''],
 
-    'psr4_namespace' =>         ['PSR-4 namespace',       'usually, Vendor\\Package',                        '{package_vendor}\\{package_name}'],
+    'psr4_namespace' =>         ['PSR-4 namespace', 'usually, Vendor\\Package', '{package_vendor}\\{package_name}'],
 ];
 
 $values = [];
 
 $replacements = [
-    ':vendor\\\\:package_name\\\\' => function () use(&$values) { return str_replace('\\', '\\\\', $values['psr4_namespace']) . '\\\\'; },
-    ':author_name'                 => function () use(&$values) { return $values['author_name']; },
-    ':author_username'             => function () use(&$values) { return $values['author_github_username']; },
-    ':author_website'              => function () use(&$values) { return $values['author_website'] ?: ('https://github.com/' . $values['author_github_username']); },
-    ':author_email'                => function () use(&$values) { return $values['author_email'] ?: ($values['author_github_username'] . '@example.com'); },
-    ':vendor'                      => function () use(&$values) { return $values['package_vendor']; },
-    ':package_name'                => function () use(&$values) { return $values['package_name']; },
-    ':package_description'         => function () use(&$values) { return $values['package_description']; },
-    'League\\Skeleton'             => function () use(&$values) { return $values['psr4_namespace']; },
+    ':vendor\\\\:package_name\\\\' => function() use(&$values) { return str_replace('\\', '\\\\', $values['psr4_namespace']) . '\\\\'; },
+    ':author_name'                 => function() use(&$values) { return $values['author_name']; },
+    ':author_username'             => function() use(&$values) { return $values['author_github_username']; },
+    ':author_website'              => function() use(&$values) { return $values['author_website'] ?: ('https://github.com/' . $values['author_github_username']); },
+    ':author_email'                => function() use(&$values) { return $values['author_email'] ?: ($values['author_github_username'] . '@example.com'); },
+    ':vendor'                      => function() use(&$values) { return $values['package_vendor']; },
+    ':package_name'                => function() use(&$values) { return $values['package_name']; },
+    ':package_description'         => function() use(&$values) { return $values['package_description']; },
+    'League\\Skeleton'             => function() use(&$values) { return $values['psr4_namespace']; },
 ];
 
-function read_from_console ($prompt) {
-    if ( function_exists('readline') ) {
+function read_from_console($prompt) {
+    if (function_exists('readline')) {
         $line = trim(readline($prompt));
         if (!empty($line)) {
             readline_add_history($line);
@@ -68,12 +68,12 @@ 
     echo "Please, provide the following information:\n";
     echo "----------------------------------------------------------------------\n";
     foreach ($fields as $f => $field) {
-        $default = isset($field[COL_DEFAULT]) ? interpolate($field[COL_DEFAULT], $values): '';
+        $default = isset($field[COL_DEFAULT]) ? interpolate($field[COL_DEFAULT], $values) : '';
         $prompt = sprintf(
             '%s%s%s: ',
             $field[COL_DESCRIPTION],
-            $field[COL_HELP] ? ' (' . $field[COL_HELP] . ')': '',
-            $field[COL_DEFAULT] !== '' ? ' [' . $default . ']': ''
+            $field[COL_HELP] ? ' (' . $field[COL_HELP] . ')' : '',
+            $field[COL_DEFAULT] !== '' ? ' [' . $default . ']' : ''
         );
         $values[$f] = read_from_console($prompt);
         if (empty($values[$f])) {