jlourenco\support\Traits\Creation::bootCreation()

Complexity

Conditions	9
Paths	1

Size

Total Lines	32
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

<?php namespace jlourenco\support\Traits;
    public static function bootCreation()
    {

        // create a event to happen on deleting
        static::deleting(function($table)  {
            if (class_exists('Cartalyst\Sentinel\Laravel\Facades\Sentinel'))
                $table->deleted_by = Sentinel::getUser()->id;
            else
                $table->deleted_by = Auth::user()->id;
        });

        // create a event to happen on saving
        static::saving(function($table)  {

            if (class_exists('Cartalyst\Sentinel\Laravel\Facades\Sentinel'))
            {
                $table->modified_by = Sentinel::getUser()->id;

                if ($user = Sentinel::check() && ($table->created_by == null || !($table->created_by > 0)))

$user is not used, you could remove the assignment.

Consider adding parentheses for clarity. Current Interpretation: $user = (\Cartalyst\Sentinel\Laravel\Facades\Sentinel::check() && ($table->created_by == null || !($table->created_by > 0))), Probably Intended Meaning: ($user = \Cartalyst\Sentinel\Laravel\Facades\Sentinel::check()) && $table->created_by == null || !($table->created_by > 0)

                    $table->created_by = Sentinel::getUser()->id;
            }
            else
            {
                $table->modified_by = Auth::user()->id;

                if (!Auth::guest() && ($table->created_by == null || !($table->created_by > 0)))
                    $table->created_by = Auth::user()->id;
            }

        });

    }

}