Differential D2779 Diff 8038 src/app/Traits/BinaryUuidKeyTrait.php

Changeset View

Standalone View

				<?php

				namespace App\Traits;

				trait BinaryUuidKeyTrait
				{
				/**
				* Boot function from Laravel.
				*/
				protected static function boot()
				{
				parent::boot();
				static::creating(function ($model) {
				if (empty($model->{$model->getKeyName()})) {
				$allegedly_unique = \App\Utils::uuidInt();
				machniakUnsubmitted Not Done Inline Actions We didn't check that before. machniak: We didn't check that before.
				mollekopfAuthorUnsubmitted Done Inline Actions But we need to if we want to make the trait compatible with softdelete models, right? mollekopf: But we need to if we want to make the trait compatible with softdelete models, right?
				machniakUnsubmitted Not Done Inline Actions I was referring to `if (empty($model->{$model->getKeyName()})) {`. There was no such check before. machniak: I was referring to ` if (empty($model->{$model->getKeyName()})) {`. There was no such check…

				// Verify if unique
				if (in_array('Illuminate\Database\Eloquent\SoftDeletes', class_uses($model))) {
				while ($model->withTrashed()->find($allegedly_unique)) {
				$allegedly_unique = \App\Utils::uuidInt();
				}
				} else {
				while ($model->find($allegedly_unique)) {
				$allegedly_unique = \App\Utils::uuidInt();
				}
				}

				$model->{$model->getKeyName()} = $allegedly_unique;
				}
				});
				}

				/**
				* Get if the key is incrementing.
				*
				* @return bool
				*/
				public function getIncrementing()
				{
				return false;
				}

				/**
				* Get the key type.
				*
				* @return string
				*/
				public function getKeyType()
				{
				return 'bigint';
				}
				}