Differential D2494 Diff 6991 src/database/migrations/2016_06_01_000001_create_oauth_auth_codes_table.php
Changeset View
Changeset View
Standalone View
Standalone View
src/database/migrations/2016_06_01_000001_create_oauth_auth_codes_table.php
- This file was added.
<?php | |||||
use Illuminate\Database\Migrations\Migration; | |||||
use Illuminate\Database\Schema\Blueprint; | |||||
use Illuminate\Support\Facades\Schema; | |||||
class CreateOauthAuthCodesTable extends Migration | |||||
{ | |||||
/** | |||||
* The database schema. | |||||
* | |||||
* @var \Illuminate\Database\Schema\Builder | |||||
*/ | |||||
protected $schema; | |||||
/** | |||||
* Create a new migration instance. | |||||
* | |||||
* @return void | |||||
*/ | |||||
public function __construct() | |||||
{ | |||||
$this->schema = Schema::connection($this->getConnection()); | |||||
} | |||||
/** | |||||
* Run the migrations. | |||||
* | |||||
* @return void | |||||
*/ | |||||
public function up() | |||||
{ | |||||
$this->schema->create('oauth_auth_codes', function (Blueprint $table) { | |||||
$table->string('id', 100)->primary(); | |||||
$table->unsignedBigInteger('user_id')->index(); | |||||
machniak: Shouldn't the user_id column be a foreign key with on delete cascade on update cascade? Here… | |||||
mollekopfAuthorUnsubmitted Done Inline ActionsI don't think it's strictly necessary, but seems reasonable. mollekopf: I don't think it's strictly necessary, but seems reasonable. | |||||
$table->unsignedBigInteger('client_id'); | |||||
$table->text('scopes')->nullable(); | |||||
$table->boolean('revoked'); | |||||
$table->dateTime('expires_at')->nullable(); | |||||
}); | |||||
} | |||||
/** | |||||
* Reverse the migrations. | |||||
* | |||||
* @return void | |||||
*/ | |||||
public function down() | |||||
{ | |||||
$this->schema->dropIfExists('oauth_auth_codes'); | |||||
} | |||||
/** | |||||
* Get the migration connection name. | |||||
* | |||||
* @return string|null | |||||
*/ | |||||
public function getConnection() | |||||
{ | |||||
return config('passport.storage.database.connection'); | |||||
} | |||||
Done Inline ActionsThis suggests existence of config/passport.php file. It is not needed or you forgot to commit it? machniak: This suggests existence of config/passport.php file. It is not needed or you forgot to commit… | |||||
Done Inline ActionsI don't have one either, so doesn't seem to be required. mollekopf: I don't have one either, so doesn't seem to be required. | |||||
} |
Shouldn't the user_id column be a foreign key with on delete cascade on update cascade? Here and in other tables.