evolution-cms / evolution

core/vendor/illuminate/database/Console/Migrations/StatusCommand.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Illuminate\Database\Console\Migrations;
 
-use Illuminate\Support\Collection;
 use Illuminate\Database\Migrations\Migrator;
+use Illuminate\Support\Collection;
 use Symfony\Component\Console\Input\InputOption;
 
 class StatusCommand extends BaseCommand

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
     {
         $this->migrator->setConnection($this->option('database'));
 
-        if (! $this->migrator->repositoryExists()) {
+        if (!$this->migrator->repositoryExists()) {
             return $this->error('No migrations found.');
         }
 
@@ -76,7 +76,7 @@ 
     protected function getStatusFor(array $ran, array $batches)
     {
         return Collection::make($this->getAllMigrationFiles())
-                    ->map(function ($migration) use ($ran, $batches) {
+                    ->map(function($migration) use ($ran, $batches) {
                         $migrationName = $this->migrator->getMigrationName($migration);
 
                         return in_array($migrationName, $ran)

Braces +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     protected function getStatusFor(array $ran, array $batches)
     {
         return Collection::make($this->getAllMigrationFiles())
-                    ->map(function ($migration) use ($ran, $batches) {
+                    ->map(function ($migration) use ($ran, $batches){
                         $migrationName = $this->migrator->getMigrationName($migration);
 
                         return in_array($migrationName, $ran)

core/vendor/illuminate/database/Console/Seeds/SeedCommand.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -3,10 +3,10 @@
 namespace Illuminate\Database\Console\Seeds;
 
 use Illuminate\Console\Command;
-use Illuminate\Database\Eloquent\Model;
 use Illuminate\Console\ConfirmableTrait;
-use Symfony\Component\Console\Input\InputOption;
 use Illuminate\Database\ConnectionResolverInterface as Resolver;
+use Illuminate\Database\Eloquent\Model;
+use Symfony\Component\Console\Input\InputOption;
 
 class SeedCommand extends Command
 {

Spacing +2 added lines, -2 removed lines

@@ -53,13 +53,13 @@
      */
     public function handle()
     {
-        if (! $this->confirmToProceed()) {
+        if (!$this->confirmToProceed()) {
             return;
         }
 
         $this->resolver->setDefaultConnection($this->getDatabase());
 
-        Model::unguarded(function () {
+        Model::unguarded(function(){
             $this->getSeeder()->__invoke();
         });
     }

Braces +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
         $this->resolver->setDefaultConnection($this->getDatabase());
 
-        Model::unguarded(function () {
+        Model::unguarded(function (){
             $this->getSeeder()->__invoke();
         });
     }

core/vendor/illuminate/database/Console/Seeds/SeederMakeCommand.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,9 +2,9 @@
 
 namespace Illuminate\Database\Console\Seeds;
 
-use Illuminate\Support\Composer;
-use Illuminate\Filesystem\Filesystem;
 use Illuminate\Console\GeneratorCommand;
+use Illuminate\Filesystem\Filesystem;
+use Illuminate\Support\Composer;
 
 class SeederMakeCommand extends GeneratorCommand
 {

core/vendor/illuminate/database/DatabaseManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -273,7 +273,7 @@ 
     /**
      * Get all of the support drivers.
      *
-     * @return array
+     * @return string[]
      */
     public function supportedDrivers()
     {
@@ -283,7 +283,7 @@ 
     /**
      * Get all of the drivers that are actually available.
      *
-     * @return array
+     * @return string[]
      */
     public function availableDrivers()
     {

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,11 +2,11 @@
 
 namespace Illuminate\Database;
 
-use PDO;
+use Illuminate\Database\Connectors\ConnectionFactory;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Str;
 use InvalidArgumentException;
-use Illuminate\Database\Connectors\ConnectionFactory;
+use PDO;
 
 /**
  * @mixin \Illuminate\Database\Connection

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
         // If we haven't created this connection, we'll create it based on the config
         // provided in the application. Once we've created the connections we will
         // set the "fetch mode" for PDO which determines the query return types.
-        if (! isset($this->connections[$name])) {
+        if (!isset($this->connections[$name])) {
             $this->connections[$name] = $this->configure(
                 $this->makeConnection($database), $type
             );
@@ -164,7 +164,7 @@ 
         // Here we'll set a reconnector callback. This reconnector can be any callable
         // so we will set a Closure to reconnect from this manager with the name of
         // the connection, which will allow us to reconnect from the connections.
-        $connection->setReconnector(function ($connection) {
+        $connection->setReconnector(function($connection){
             $this->reconnect($connection->getName());
         });
 
@@ -227,7 +227,7 @@ 
     {
         $this->disconnect($name = $name ?: $this->getDefaultConnection());
 
-        if (! isset($this->connections[$name])) {
+        if (!isset($this->connections[$name])) {
             return $this->connection($name);
         }
 

Braces +1 added lines, -1 removed lines

@@ -164,7 +164,7 @@
         // Here we'll set a reconnector callback. This reconnector can be any callable
         // so we will set a Closure to reconnect from this manager with the name of
         // the connection, which will allow us to reconnect from the connections.
-        $connection->setReconnector(function ($connection) {
+        $connection->setReconnector(function ($connection){
             $this->reconnect($connection->getName());
         });
 

core/vendor/illuminate/database/DatabaseServiceProvider.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -4,12 +4,12 @@
 
 use Faker\Factory as FakerFactory;
 use Faker\Generator as FakerGenerator;
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Support\ServiceProvider;
 use Illuminate\Contracts\Queue\EntityResolver;
 use Illuminate\Database\Connectors\ConnectionFactory;
-use Illuminate\Database\Eloquent\QueueEntityResolver;
 use Illuminate\Database\Eloquent\Factory as EloquentFactory;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\QueueEntityResolver;
+use Illuminate\Support\ServiceProvider;
 
 class DatabaseServiceProvider extends ServiceProvider
 {

Spacing +6 added lines, -6 removed lines

@@ -51,18 +51,18 @@ 
         // The connection factory is used to create the actual connection instances on
         // the database. We will inject the factory into the manager so that it may
         // make the connections while they are actually needed and not of before.
-        $this->app->singleton('db.factory', function ($app) {
+        $this->app->singleton('db.factory', function($app){
             return new ConnectionFactory($app);
         });
 
         // The database manager is used to resolve various connections, since multiple
         // connections might be managed. It also implements the connection resolver
         // interface which may be used by other components requiring connections.
-        $this->app->singleton('db', function ($app) {
+        $this->app->singleton('db', function($app){
             return new DatabaseManager($app, $app['db.factory']);
         });
 
-        $this->app->bind('db.connection', function ($app) {
+        $this->app->bind('db.connection', function($app){
             return $app['db']->connection();
         });
     }
@@ -74,11 +74,11 @@ 
      */
     protected function registerEloquentFactory()
     {
-        $this->app->singleton(FakerGenerator::class, function ($app) {
+        $this->app->singleton(FakerGenerator::class, function($app){
             return FakerFactory::create($app['config']->get('app.faker_locale', 'en_US'));
         });
 
-        $this->app->singleton(EloquentFactory::class, function ($app) {
+        $this->app->singleton(EloquentFactory::class, function($app){
             return EloquentFactory::construct(
                 $app->make(FakerGenerator::class), $this->app->databasePath('factories')
             );
@@ -92,7 +92,7 @@ 
      */
     protected function registerQueueableEntityResolver()
     {
-        $this->app->singleton(EntityResolver::class, function () {
+        $this->app->singleton(EntityResolver::class, function(){
             return new QueueEntityResolver;
         });
     }

Braces +6 added lines, -6 removed lines

@@ -51,18 +51,18 @@ 
         // The connection factory is used to create the actual connection instances on
         // the database. We will inject the factory into the manager so that it may
         // make the connections while they are actually needed and not of before.
-        $this->app->singleton('db.factory', function ($app) {
+        $this->app->singleton('db.factory', function ($app){
             return new ConnectionFactory($app);
         });
 
         // The database manager is used to resolve various connections, since multiple
         // connections might be managed. It also implements the connection resolver
         // interface which may be used by other components requiring connections.
-        $this->app->singleton('db', function ($app) {
+        $this->app->singleton('db', function ($app){
             return new DatabaseManager($app, $app['db.factory']);
         });
 
-        $this->app->bind('db.connection', function ($app) {
+        $this->app->bind('db.connection', function ($app){
             return $app['db']->connection();
         });
     }
@@ -74,11 +74,11 @@ 
      */
     protected function registerEloquentFactory()
     {
-        $this->app->singleton(FakerGenerator::class, function ($app) {
+        $this->app->singleton(FakerGenerator::class, function ($app){
             return FakerFactory::create($app['config']->get('app.faker_locale', 'en_US'));
         });
 
-        $this->app->singleton(EloquentFactory::class, function ($app) {
+        $this->app->singleton(EloquentFactory::class, function ($app){
             return EloquentFactory::construct(
                 $app->make(FakerGenerator::class), $this->app->databasePath('factories')
             );
@@ -92,7 +92,7 @@ 
      */
     protected function registerQueueableEntityResolver()
     {
-        $this->app->singleton(EntityResolver::class, function () {
+        $this->app->singleton(EntityResolver::class, function (){
             return new QueueEntityResolver;
         });
     }

core/vendor/illuminate/database/DetectsLostConnections.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Illuminate\Database;
 
-use Throwable;
 use Illuminate\Support\Str;
+use Throwable;
 
 trait DetectsLostConnections
 {

core/vendor/illuminate/database/Eloquent/Builder.php

Doc Comments +6 added lines, -6 removed lines

@@ -233,8 +233,8 @@ 
      * Add an "or where" clause to the query.
      *
      * @param  \Closure|array|string  $column
-     * @param  mixed  $operator
-     * @param  mixed  $value
+     * @param  string  $operator
+     * @param  integer  $value
      * @return \Illuminate\Database\Eloquent\Builder|static
      */
     public function orWhere($column, $operator = null, $value = null)
@@ -453,7 +453,7 @@ 
      * Execute the query as a "select" statement.
      *
      * @param  array  $columns
-     * @return \Illuminate\Database\Eloquent\Collection|static[]
+     * @return \Illuminate\Support\Collection
      */
     public function get($columns = ['*'])
     {
@@ -784,7 +784,7 @@ 
      * Increment a column's value by a given amount.
      *
      * @param  string  $column
-     * @param  float|int  $amount
+     * @param  integer  $amount
      * @param  array  $extra
      * @return int
      */
@@ -799,7 +799,7 @@ 
      * Decrement a column's value by a given amount.
      *
      * @param  string  $column
-     * @param  float|int  $amount
+     * @param  integer  $amount
      * @param  array  $extra
      * @return int
      */
@@ -847,7 +847,7 @@ 
      *
      * Since we do not apply scopes here, the row will actually be deleted.
      *
-     * @return mixed
+     * @return integer
      */
     public function forceDelete()
     {

Unused Use Statements +4 added lines, -4 removed lines

@@ -2,15 +2,15 @@
 
 namespace Illuminate\Database\Eloquent;
 
-use Closure;
 use BadMethodCallException;
-use Illuminate\Support\Arr;
-use Illuminate\Support\Str;
-use Illuminate\Pagination\Paginator;
+use Closure;
 use Illuminate\Contracts\Support\Arrayable;
 use Illuminate\Database\Concerns\BuildsQueries;
 use Illuminate\Database\Eloquent\Relations\Relation;
 use Illuminate\Database\Query\Builder as QueryBuilder;
+use Illuminate\Pagination\Paginator;
+use Illuminate\Support\Arr;
+use Illuminate\Support\Str;
 
 /**
  * @mixin \Illuminate\Database\Query\Builder

Spacing +27 added lines, -27 removed lines

@@ -133,7 +133,7 @@ 
      */
     public function withoutGlobalScope($scope)
     {
-        if (! is_string($scope)) {
+        if (!is_string($scope)) {
             $scope = get_class($scope);
         }
 
@@ -152,7 +152,7 @@ 
      */
     public function withoutGlobalScopes(array $scopes = null)
     {
-        if (! is_array($scopes)) {
+        if (!is_array($scopes)) {
             $scopes = array_keys($this->scopes);
         }
 
@@ -256,7 +256,7 @@ 
     {
         $instance = $this->newModelInstance();
 
-        return $instance->newCollection(array_map(function ($item) use ($instance) {
+        return $instance->newCollection(array_map(function($item) use ($instance) {
             return $instance->newFromBuilder($item);
         }, $items));
     }
@@ -324,7 +324,7 @@ 
             if (count($result) === count(array_unique($id))) {
                 return $result;
             }
-        } elseif (! is_null($result)) {
+        } elseif (!is_null($result)) {
             return $result;
         }
 
@@ -342,7 +342,7 @@ 
      */
     public function findOrNew($id, $columns = ['*'])
     {
-        if (! is_null($model = $this->find($id, $columns))) {
+        if (!is_null($model = $this->find($id, $columns))) {
             return $model;
         }
 
@@ -358,7 +358,7 @@ 
      */
     public function firstOrNew(array $attributes, array $values = [])
     {
-        if (! is_null($instance = $this->where($attributes)->first())) {
+        if (!is_null($instance = $this->where($attributes)->first())) {
             return $instance;
         }
 
@@ -374,11 +374,11 @@ 
      */
     public function firstOrCreate(array $attributes, array $values = [])
     {
-        if (! is_null($instance = $this->where($attributes)->first())) {
+        if (!is_null($instance = $this->where($attributes)->first())) {
             return $instance;
         }
 
-        return tap($this->newModelInstance($attributes + $values), function ($instance) {
+        return tap($this->newModelInstance($attributes + $values), function($instance){
             $instance->save();
         });
     }
@@ -392,7 +392,7 @@ 
      */
     public function updateOrCreate(array $attributes, array $values = [])
     {
-        return tap($this->firstOrNew($attributes), function ($instance) use ($values) {
+        return tap($this->firstOrNew($attributes), function($instance) use ($values) {
             $instance->fill($values)->save();
         });
     }
@@ -407,7 +407,7 @@ 
      */
     public function firstOrFail($columns = ['*'])
     {
-        if (! is_null($model = $this->first($columns))) {
+        if (!is_null($model = $this->first($columns))) {
             return $model;
         }
 
@@ -429,7 +429,7 @@ 
             $columns = ['*'];
         }
 
-        if (! is_null($model = $this->first($columns))) {
+        if (!is_null($model = $this->first($columns))) {
             return $model;
         }
 
@@ -541,7 +541,7 @@ 
         // We want to run a relationship query without any constrains so that we will
         // not have to remove these where clauses manually which gets really hacky
         // and error prone. We don't want constraints because we add eager ones.
-        $relation = Relation::noConstraints(function () use ($name) {
+        $relation = Relation::noConstraints(function() use ($name) {
             try {
                 return $this->getModel()->newInstance()->$name();
             } catch (BadMethodCallException $e) {
@@ -679,13 +679,13 @@ 
         // If the model has a mutator for the requested column, we will spin through
         // the results and mutate the values so that the mutated version of these
         // columns are returned as you would expect from these Eloquent models.
-        if (! $this->model->hasGetMutator($column) &&
-            ! $this->model->hasCast($column) &&
-            ! in_array($column, $this->model->getDates())) {
+        if (!$this->model->hasGetMutator($column) &&
+            !$this->model->hasCast($column) &&
+            !in_array($column, $this->model->getDates())) {
             return $results;
         }
 
-        return $results->map(function ($value) use ($column) {
+        return $results->map(function($value) use ($column) {
             return $this->model->newFromBuilder([$column => $value])->{$column};
         });
     }
@@ -751,7 +751,7 @@ 
      */
     public function create(array $attributes = [])
     {
-        return tap($this->newModelInstance($attributes), function ($instance) {
+        return tap($this->newModelInstance($attributes), function($instance){
             $instance->save();
         });
     }
@@ -764,7 +764,7 @@ 
      */
     public function forceCreate(array $attributes)
     {
-        return $this->model->unguarded(function () use ($attributes) {
+        return $this->model->unguarded(function() use ($attributes) {
             return $this->newModelInstance()->create($attributes);
         });
     }
@@ -818,7 +818,7 @@ 
      */
     protected function addUpdatedAtColumn(array $values)
     {
-        if (! $this->model->usesTimestamps()) {
+        if (!$this->model->usesTimestamps()) {
             return $values;
         }
 
@@ -902,18 +902,18 @@ 
      */
     public function applyScopes()
     {
-        if (! $this->scopes) {
+        if (!$this->scopes) {
             return $this;
         }
 
         $builder = clone $this;
 
         foreach ($this->scopes as $identifier => $scope) {
-            if (! isset($builder->scopes[$identifier])) {
+            if (!isset($builder->scopes[$identifier])) {
                 continue;
             }
 
-            $builder->callScope(function (Builder $builder) use ($scope) {
+            $builder->callScope(function(Builder $builder) use ($scope) {
                 // If the scope is a Closure we will just go ahead and call the scope with the
                 // builder instance. The "callScope" method will properly group the clauses
                 // that are added to this query so "where" clauses maintain proper logic.
@@ -1087,7 +1087,7 @@ 
 
                 list($name, $constraints) = Str::contains($name, ':')
                             ? $this->createSelectWithConstraint($name)
-                            : [$name, function () {
+                            : [$name, function(){
                                 //
                             }];
             }
@@ -1111,7 +1111,7 @@ 
      */
     protected function createSelectWithConstraint($name)
     {
-        return [explode(':', $name)[0], function ($query) use ($name) {
+        return [explode(':', $name)[0], function($query) use ($name) {
             $query->select(explode(',', explode(':', $name)[1]));
         }];
     }
@@ -1133,8 +1133,8 @@ 
         foreach (explode('.', $name) as $segment) {
             $progress[] = $segment;
 
-            if (! isset($results[$last = implode('.', $progress)])) {
-                $results[$last] = function () {
+            if (!isset($results[$last = implode('.', $progress)])) {
+                $results[$last] = function(){
                     //
                 };
             }
@@ -1305,7 +1305,7 @@ 
             return;
         }
 
-        if (! isset(static::$macros[$method])) {
+        if (!isset(static::$macros[$method])) {
             throw new BadMethodCallException(sprintf(
                 'Method %s::%s does not exist.', static::class, $method
             ));

Braces +11 added lines, -11 removed lines

@@ -256,7 +256,7 @@ 
     {
         $instance = $this->newModelInstance();
 
-        return $instance->newCollection(array_map(function ($item) use ($instance) {
+        return $instance->newCollection(array_map(function ($item) use ($instance){
             return $instance->newFromBuilder($item);
         }, $items));
     }
@@ -378,7 +378,7 @@ 
             return $instance;
         }
 
-        return tap($this->newModelInstance($attributes + $values), function ($instance) {
+        return tap($this->newModelInstance($attributes + $values), function ($instance){
             $instance->save();
         });
     }
@@ -392,7 +392,7 @@ 
      */
     public function updateOrCreate(array $attributes, array $values = [])
     {
-        return tap($this->firstOrNew($attributes), function ($instance) use ($values) {
+        return tap($this->firstOrNew($attributes), function ($instance) use ($values){
             $instance->fill($values)->save();
         });
     }
@@ -541,7 +541,7 @@ 
         // We want to run a relationship query without any constrains so that we will
         // not have to remove these where clauses manually which gets really hacky
         // and error prone. We don't want constraints because we add eager ones.
-        $relation = Relation::noConstraints(function () use ($name) {
+        $relation = Relation::noConstraints(function () use ($name){
             try {
                 return $this->getModel()->newInstance()->$name();
             } catch (BadMethodCallException $e) {
@@ -685,7 +685,7 @@ 
             return $results;
         }
 
-        return $results->map(function ($value) use ($column) {
+        return $results->map(function ($value) use ($column){
             return $this->model->newFromBuilder([$column => $value])->{$column};
         });
     }
@@ -751,7 +751,7 @@ 
      */
     public function create(array $attributes = [])
     {
-        return tap($this->newModelInstance($attributes), function ($instance) {
+        return tap($this->newModelInstance($attributes), function ($instance){
             $instance->save();
         });
     }
@@ -764,7 +764,7 @@ 
      */
     public function forceCreate(array $attributes)
     {
-        return $this->model->unguarded(function () use ($attributes) {
+        return $this->model->unguarded(function () use ($attributes){
             return $this->newModelInstance()->create($attributes);
         });
     }
@@ -913,7 +913,7 @@ 
                 continue;
             }
 
-            $builder->callScope(function (Builder $builder) use ($scope) {
+            $builder->callScope(function (Builder $builder) use ($scope){
                 // If the scope is a Closure we will just go ahead and call the scope with the
                 // builder instance. The "callScope" method will properly group the clauses
                 // that are added to this query so "where" clauses maintain proper logic.
@@ -1087,7 +1087,7 @@ 
 
                 list($name, $constraints) = Str::contains($name, ':')
                             ? $this->createSelectWithConstraint($name)
-                            : [$name, function () {
+                            : [$name, function (){
                                 //
                             }];
             }
@@ -1111,7 +1111,7 @@ 
      */
     protected function createSelectWithConstraint($name)
     {
-        return [explode(':', $name)[0], function ($query) use ($name) {
+        return [explode(':', $name)[0], function ($query) use ($name){
             $query->select(explode(',', explode(':', $name)[1]));
         }];
     }
@@ -1134,7 +1134,7 @@ 
             $progress[] = $segment;
 
             if (! isset($results[$last = implode('.', $progress)])) {
-                $results[$last] = function () {
+                $results[$last] = function (){
                     //
                 };
             }

core/vendor/illuminate/database/Eloquent/Collection.php

Doc Comments +2 added lines, -2 removed lines

@@ -173,7 +173,7 @@ 
     /**
      * Determine if a key exists in the collection.
      *
-     * @param  mixed  $key
+     * @param  \Closure  $key
      * @param  mixed  $operator
      * @param  mixed  $value
      * @return bool
@@ -210,7 +210,7 @@ 
     /**
      * Merge the collection with the given items.
      *
-     * @param  \ArrayAccess|array  $items
+     * @param  Collection  $items
      * @return static
      */
     public function merge($items)

Unused Use Statements +4 added lines, -4 removed lines

@@ -2,13 +2,13 @@
 
 namespace Illuminate\Database\Eloquent;
 
-use LogicException;
-use Illuminate\Support\Arr;
-use Illuminate\Support\Str;
+use Illuminate\Contracts\Queue\QueueableCollection;
 use Illuminate\Contracts\Support\Arrayable;
 use Illuminate\Database\Eloquent\Relations\Pivot;
-use Illuminate\Contracts\Queue\QueueableCollection;
+use Illuminate\Support\Arr;
 use Illuminate\Support\Collection as BaseCollection;
+use Illuminate\Support\Str;
+use LogicException;
 
 class Collection extends BaseCollection implements QueueableCollection
 {

Spacing +16 added lines, -16 removed lines

@@ -37,7 +37,7 @@ 
             return $this->whereIn($this->first()->getKeyName(), $key);
         }
 
-        return Arr::first($this->items, function ($model) use ($key) {
+        return Arr::first($this->items, function($model) use ($key) {
             return $model->getKey() == $key;
         }, $default);
     }
@@ -115,8 +115,8 @@ 
             $relation = reset($relation);
         }
 
-        $models->filter(function ($model) use ($name) {
-            return ! is_null($model) && ! $model->relationLoaded($name);
+        $models->filter(function($model) use ($name) {
+            return !is_null($model) && !$model->relationLoaded($name);
         })->load($relation);
 
         if (empty($path)) {
@@ -143,13 +143,13 @@ 
     {
         $this->pluck($relation)
             ->filter()
-            ->groupBy(function ($model) {
+            ->groupBy(function($model){
                 return get_class($model);
             })
-            ->filter(function ($models, $className) use ($relations) {
+            ->filter(function($models, $className) use ($relations) {
                 return Arr::has($relations, $className);
             })
-            ->each(function ($models, $className) use ($relations) {
+            ->each(function($models, $className) use ($relations) {
                 $className::with($relations[$className])
                     ->eagerLoadRelations($models->all());
             });
@@ -185,12 +185,12 @@ 
         }
 
         if ($key instanceof Model) {
-            return parent::contains(function ($model) use ($key) {
+            return parent::contains(function($model) use ($key) {
                 return $model->is($key);
             });
         }
 
-        return parent::contains(function ($model) use ($key) {
+        return parent::contains(function($model) use ($key) {
             return $model->getKey() == $key;
         });
     }
@@ -202,7 +202,7 @@ 
      */
     public function modelKeys()
     {
-        return array_map(function ($model) {
+        return array_map(function($model){
             return $model->getKey();
         }, $this->items);
     }
@@ -234,8 +234,8 @@ 
     {
         $result = parent::map($callback);
 
-        return $result->contains(function ($item) {
-            return ! $item instanceof Model;
+        return $result->contains(function($item){
+            return !$item instanceof Model;
         }) ? $result->toBase() : $result;
     }
 
@@ -259,7 +259,7 @@ 
             ->get()
             ->getDictionary();
 
-        return $this->map(function ($model) use ($freshModels) {
+        return $this->map(function($model) use ($freshModels) {
             return $model->exists && isset($freshModels[$model->getKey()])
                     ? $freshModels[$model->getKey()] : null;
         });
@@ -278,7 +278,7 @@ 
         $dictionary = $this->getDictionary($items);
 
         foreach ($this->items as $item) {
-            if (! isset($dictionary[$item->getKey()])) {
+            if (!isset($dictionary[$item->getKey()])) {
                 $diff->add($item);
             }
         }
@@ -316,7 +316,7 @@ 
      */
     public function unique($key = null, $strict = false)
     {
-        if (! is_null($key)) {
+        if (!is_null($key)) {
             return parent::unique($key, $strict);
         }
 
@@ -488,7 +488,7 @@ 
 
         $class = get_class($this->first());
 
-        $this->each(function ($model) use ($class) {
+        $this->each(function($model) use ($class) {
             if (get_class($model) !== $class) {
                 throw new LogicException('Queueing collections with multiple model types is not supported.');
             }
@@ -537,7 +537,7 @@ 
 
         $connection = $this->first()->getConnectionName();
 
-        $this->each(function ($model) use ($connection) {
+        $this->each(function($model) use ($connection) {
             if ($model->getConnectionName() !== $connection) {
                 throw new LogicException('Queueing collections with multiple model connections is not supported.');
             }

Braces +12 added lines, -12 removed lines

@@ -37,7 +37,7 @@ 
             return $this->whereIn($this->first()->getKeyName(), $key);
         }
 
-        return Arr::first($this->items, function ($model) use ($key) {
+        return Arr::first($this->items, function ($model) use ($key){
             return $model->getKey() == $key;
         }, $default);
     }
@@ -115,7 +115,7 @@ 
             $relation = reset($relation);
         }
 
-        $models->filter(function ($model) use ($name) {
+        $models->filter(function ($model) use ($name){
             return ! is_null($model) && ! $model->relationLoaded($name);
         })->load($relation);
 
@@ -143,13 +143,13 @@ 
     {
         $this->pluck($relation)
             ->filter()
-            ->groupBy(function ($model) {
+            ->groupBy(function ($model){
                 return get_class($model);
             })
-            ->filter(function ($models, $className) use ($relations) {
+            ->filter(function ($models, $className) use ($relations){
                 return Arr::has($relations, $className);
             })
-            ->each(function ($models, $className) use ($relations) {
+            ->each(function ($models, $className) use ($relations){
                 $className::with($relations[$className])
                     ->eagerLoadRelations($models->all());
             });
@@ -185,12 +185,12 @@ 
         }
 
         if ($key instanceof Model) {
-            return parent::contains(function ($model) use ($key) {
+            return parent::contains(function ($model) use ($key){
                 return $model->is($key);
             });
         }
 
-        return parent::contains(function ($model) use ($key) {
+        return parent::contains(function ($model) use ($key){
             return $model->getKey() == $key;
         });
     }
@@ -202,7 +202,7 @@ 
      */
     public function modelKeys()
     {
-        return array_map(function ($model) {
+        return array_map(function ($model){
             return $model->getKey();
         }, $this->items);
     }
@@ -234,7 +234,7 @@ 
     {
         $result = parent::map($callback);
 
-        return $result->contains(function ($item) {
+        return $result->contains(function ($item){
             return ! $item instanceof Model;
         }) ? $result->toBase() : $result;
     }
@@ -259,7 +259,7 @@ 
             ->get()
             ->getDictionary();
 
-        return $this->map(function ($model) use ($freshModels) {
+        return $this->map(function ($model) use ($freshModels){
             return $model->exists && isset($freshModels[$model->getKey()])
                     ? $freshModels[$model->getKey()] : null;
         });
@@ -488,7 +488,7 @@ 
 
         $class = get_class($this->first());
 
-        $this->each(function ($model) use ($class) {
+        $this->each(function ($model) use ($class){
             if (get_class($model) !== $class) {
                 throw new LogicException('Queueing collections with multiple model types is not supported.');
             }
@@ -537,7 +537,7 @@ 
 
         $connection = $this->first()->getConnectionName();
 
-        $this->each(function ($model) use ($connection) {
+        $this->each(function ($model) use ($connection){
             if ($model->getConnectionName() !== $connection) {
                 throw new LogicException('Queueing collections with multiple model connections is not supported.');
             }

core/vendor/illuminate/database/Eloquent/Concerns/HasAttributes.php

Doc Comments +4 added lines, -4 removed lines

@@ -166,7 +166,7 @@ 
      *
      * @param  array  $attributes
      * @param  array  $mutatedAttributes
-     * @return array
+     * @return string
      */
     protected function addCastAttributesToArray(array $attributes, array $mutatedAttributes)
     {
@@ -753,7 +753,7 @@ 
      * Determine if the given value is a standard date format.
      *
      * @param  string  $value
-     * @return bool
+     * @return integer
      */
     protected function isStandardDateFormat($value)
     {
@@ -836,7 +836,7 @@ 
      * Determine whether an attribute should be cast to a native type.
      *
      * @param  string  $key
-     * @param  array|string|null  $types
+     * @param  string[]  $types
      * @return bool
      */
     public function hasCast($key, $types = null)
@@ -1162,7 +1162,7 @@ 
     /**
      * Get all of the attribute mutator methods.
      *
-     * @param  mixed  $class
+     * @param  string  $class
      * @return array
      */
     protected static function getMutatorMethods($class)

Unused Use Statements +5 added lines, -5 removed lines

@@ -2,15 +2,15 @@
 
 namespace Illuminate\Database\Eloquent\Concerns;
 
-use LogicException;
 use DateTimeInterface;
-use Illuminate\Support\Arr;
-use Illuminate\Support\Str;
-use Illuminate\Support\Carbon;
 use Illuminate\Contracts\Support\Arrayable;
+use Illuminate\Database\Eloquent\JsonEncodingException;
 use Illuminate\Database\Eloquent\Relations\Relation;
+use Illuminate\Support\Arr;
+use Illuminate\Support\Carbon;
 use Illuminate\Support\Collection as BaseCollection;
-use Illuminate\Database\Eloquent\JsonEncodingException;
+use Illuminate\Support\Str;
+use LogicException;
 
 trait HasAttributes
 {

Indentation +2 added lines, -2 removed lines

@@ -1092,10 +1092,10 @@
             return false;
         } elseif ($this->isDateAttribute($key)) {
             return $this->fromDateTime($current) ===
-                   $this->fromDateTime($original);
+                    $this->fromDateTime($original);
         } elseif ($this->hasCast($key)) {
             return $this->castAttribute($key, $current) ===
-                   $this->castAttribute($key, $original);
+                    $this->castAttribute($key, $original);
         }
 
         return is_numeric($current) && is_numeric($original)

Spacing +16 added lines, -16 removed lines

@@ -121,7 +121,7 @@ 
     protected function addDateAttributesToArray(array $attributes)
     {
         foreach ($this->getDates() as $key) {
-            if (! isset($attributes[$key])) {
+            if (!isset($attributes[$key])) {
                 continue;
             }
 
@@ -146,7 +146,7 @@ 
             // We want to spin through all the mutated attributes for this model and call
             // the mutator for the attribute. We cache off every mutated attributes so
             // we don't have to constantly check on attributes that actually change.
-            if (! array_key_exists($key, $attributes)) {
+            if (!array_key_exists($key, $attributes)) {
                 continue;
             }
 
@@ -171,7 +171,7 @@ 
     protected function addCastAttributesToArray(array $attributes, array $mutatedAttributes)
     {
         foreach ($this->getCasts() as $key => $value) {
-            if (! array_key_exists($key, $attributes) || in_array($key, $mutatedAttributes)) {
+            if (!array_key_exists($key, $attributes) || in_array($key, $mutatedAttributes)) {
                 continue;
             }
 
@@ -215,7 +215,7 @@ 
      */
     protected function getArrayableAppends()
     {
-        if (! count($this->appends)) {
+        if (!count($this->appends)) {
             return [];
         }
 
@@ -305,7 +305,7 @@ 
      */
     public function getAttribute($key)
     {
-        if (! $key) {
+        if (!$key) {
             return;
         }
 
@@ -355,7 +355,7 @@ 
         // instance on retrieval, which makes it quite convenient to work with
         // date fields without having to create a mutator for each property.
         if (in_array($key, $this->getDates()) &&
-            ! is_null($value)) {
+            !is_null($value)) {
             return $this->asDateTime($value);
         }
 
@@ -410,13 +410,13 @@ 
     {
         $relation = $this->$method();
 
-        if (! $relation instanceof Relation) {
+        if (!$relation instanceof Relation) {
             throw new LogicException(sprintf(
                 '%s::%s must return a relationship instance.', static::class, $method
             ));
         }
 
-        return tap($relation->getResults(), function ($results) use ($method) {
+        return tap($relation->getResults(), function($results) use ($method) {
             $this->setRelation($method, $results);
         });
     }
@@ -553,7 +553,7 @@ 
             $value = $this->fromDateTime($value);
         }
 
-        if ($this->isJsonCastable($key) && ! is_null($value)) {
+        if ($this->isJsonCastable($key) && !is_null($value)) {
             $value = $this->castAttributeAsJson($key, $value);
         }
 
@@ -632,7 +632,7 @@ 
      */
     protected function getArrayAttributeWithValue($path, $key, $value)
     {
-        return tap($this->getArrayAttributeByKey($key), function (&$array) use ($path, $value) {
+        return tap($this->getArrayAttributeByKey($key), function(&$array) use ($path, $value) {
             Arr::set($array, str_replace('->', '.', $path), $value);
         });
     }
@@ -689,7 +689,7 @@ 
      */
     public function fromJson($value, $asObject = false)
     {
-        return json_decode($value, ! $asObject);
+        return json_decode($value, !$asObject);
     }
 
     /**
@@ -999,7 +999,7 @@ 
      */
     public function isClean($attributes = null)
     {
-        return ! $this->isDirty(...func_get_args());
+        return !$this->isDirty(...func_get_args());
     }
 
     /**
@@ -1053,7 +1053,7 @@ 
         $dirty = [];
 
         foreach ($this->getAttributes() as $key => $value) {
-            if (! $this->originalIsEquivalent($key, $value)) {
+            if (!$this->originalIsEquivalent($key, $value)) {
                 $dirty[$key] = $value;
             }
         }
@@ -1080,7 +1080,7 @@ 
      */
     protected function originalIsEquivalent($key, $current)
     {
-        if (! array_key_exists($key, $this->original)) {
+        if (!array_key_exists($key, $this->original)) {
             return false;
         }
 
@@ -1139,7 +1139,7 @@ 
     {
         $class = static::class;
 
-        if (! isset(static::$mutatorCache[$class])) {
+        if (!isset(static::$mutatorCache[$class])) {
             static::cacheMutatedAttributes($class);
         }
 
@@ -1154,7 +1154,7 @@ 
      */
     public static function cacheMutatedAttributes($class)
     {
-        static::$mutatorCache[$class] = collect(static::getMutatorMethods($class))->map(function ($match) {
+        static::$mutatorCache[$class] = collect(static::getMutatorMethods($class))->map(function($match){
             return lcfirst(static::$snakeAttributes ? Str::snake($match) : $match);
         })->all();
     }

Braces +3 added lines, -3 removed lines

@@ -416,7 +416,7 @@ 
             ));
         }
 
-        return tap($relation->getResults(), function ($results) use ($method) {
+        return tap($relation->getResults(), function ($results) use ($method){
             $this->setRelation($method, $results);
         });
     }
@@ -632,7 +632,7 @@ 
      */
     protected function getArrayAttributeWithValue($path, $key, $value)
     {
-        return tap($this->getArrayAttributeByKey($key), function (&$array) use ($path, $value) {
+        return tap($this->getArrayAttributeByKey($key), function (&$array) use ($path, $value){
             Arr::set($array, str_replace('->', '.', $path), $value);
         });
     }
@@ -1154,7 +1154,7 @@ 
      */
     public static function cacheMutatedAttributes($class)
     {
-        static::$mutatorCache[$class] = collect(static::getMutatorMethods($class))->map(function ($match) {
+        static::$mutatorCache[$class] = collect(static::getMutatorMethods($class))->map(function ($match){
             return lcfirst(static::$snakeAttributes ? Str::snake($match) : $match);
         })->all();
     }