diff --git a/src/Illuminate/Database/Eloquent/Factories/Factory.php b/src/Illuminate/Database/Eloquent/Factories/Factory.php index 43d37719dfda..e1e4ef4cf90c 100644 --- a/src/Illuminate/Database/Eloquent/Factories/Factory.php +++ b/src/Illuminate/Database/Eloquent/Factories/Factory.php @@ -76,7 +76,7 @@ abstract class Factory /** * The name of the database connection that will be used to create the models. * - * @var string + * @var string|null */ protected $connection; @@ -129,11 +129,11 @@ public function __construct($count = null, $connection = null) { $this->count = $count; - $this->states = $states ?: new Collection; - $this->has = $has ?: new Collection; - $this->for = $for ?: new Collection; - $this->afterMaking = $afterMaking ?: new Collection; - $this->afterCreating = $afterCreating ?: new Collection; + $this->states = $states ?? new Collection; + $this->has = $has ?? new Collection; + $this->for = $for ?? new Collection; + $this->afterMaking = $afterMaking ?? new Collection; + $this->afterCreating = $afterCreating ?? new Collection; $this->connection = $connection; $this->faker = $this->withFaker(); } @@ -516,7 +516,7 @@ public function has(self $factory, $relationship = null) { return $this->newInstance([ 'has' => $this->has->concat([new Relationship( - $factory, $relationship ?: $this->guessRelationship($factory->modelName()) + $factory, $relationship ?? $this->guessRelationship($factory->modelName()) )]), ]); } @@ -548,7 +548,7 @@ public function hasAttached($factory, $pivot = [], $relationship = null) 'has' => $this->has->concat([new BelongsToManyRelationship( $factory, $pivot, - $relationship ?: Str::camel(Str::plural(class_basename( + $relationship ?? Str::camel(Str::plural(class_basename( $factory instanceof Factory ? $factory->modelName() : Collection::wrap($factory)->first() @@ -568,7 +568,7 @@ public function for($factory, $relationship = null) { return $this->newInstance(['for' => $this->for->concat([new BelongsToRelationship( $factory, - $relationship ?: Str::camel(class_basename( + $relationship ?? Str::camel(class_basename( $factory instanceof Factory ? $factory->modelName() : $factory )) )])]); @@ -688,7 +688,7 @@ public function newModel(array $attributes = []) */ public function modelName() { - $resolver = static::$modelNameResolver ?: function (self $factory) { + $resolver = static::$modelNameResolver ?? function (self $factory) { $namespacedFactoryBasename = Str::replaceLast( 'Factory', '', Str::replaceFirst(static::$namespace, '', get_class($factory)) ); @@ -702,7 +702,7 @@ public function modelName() : $appNamespace.$factoryBasename; }; - return $this->model ?: $resolver($this); + return $this->model ?? $resolver($this); } /** @@ -769,7 +769,7 @@ protected function withFaker() */ public static function resolveFactoryName(string $modelName) { - $resolver = static::$factoryNameResolver ?: function (string $modelName) { + $resolver = static::$factoryNameResolver ?? function (string $modelName) { $appNamespace = static::appNamespace(); $modelName = Str::startsWith($modelName, $appNamespace.'Models\\') @@ -820,7 +820,7 @@ public function __call($method, $parameters) $relatedModel = get_class($this->newModel()->{$relationship}()->getRelated()); if (method_exists($relatedModel, 'newFactory')) { - $factory = $relatedModel::newFactory() ?: static::factoryForModel($relatedModel); + $factory = $relatedModel::newFactory() ?? static::factoryForModel($relatedModel); } else { $factory = static::factoryForModel($relatedModel); }