From 66707b8f8ce152ac0e2f6ca146cbcc30dd202c2a Mon Sep 17 00:00:00 2001 From: Welling Guzman Date: Fri, 21 Jul 2017 19:10:04 -0400 Subject: [PATCH] update files/users status column reference --- .../TableGateway/DirectusUsersTableGateway.php | 12 ------------ .../Database/TableGateway/RelationalTableGateway.php | 2 +- .../Directus/Util/Installation/InstallerUtils.php | 2 +- .../20150204015251_CreateDirectusFilesTable.php | 2 +- .../20150204042725_CreateDirectusUsersTable.php | 2 +- api/routes/A1/Auth.php | 2 +- api/routes/A1/Users.php | 2 +- 7 files changed, 6 insertions(+), 18 deletions(-) diff --git a/api/core/Directus/Database/TableGateway/DirectusUsersTableGateway.php b/api/core/Directus/Database/TableGateway/DirectusUsersTableGateway.php index 15aa6e9f34fed..ea5cb696d96e4 100644 --- a/api/core/Directus/Database/TableGateway/DirectusUsersTableGateway.php +++ b/api/core/Directus/Database/TableGateway/DirectusUsersTableGateway.php @@ -54,18 +54,6 @@ public function fetchAllWithGroupData() return ['rows' => $results]; } - public function getTableSchema($tableName = null) - { - $schema = parent::getTableSchema($tableName); - - // TODO: Add proper information to all system columns - $schema->setStatusColumn('active'); - $schema->setPrimaryColumn('id'); - - return $schema; - } - - public function findActiveUserIdsByGroupIds($ids = []) { $statusColumnName = $this->getTableSchema()->getStatusColumn(); diff --git a/api/core/Directus/Database/TableGateway/RelationalTableGateway.php b/api/core/Directus/Database/TableGateway/RelationalTableGateway.php index dc2ef015d34a7..398e0014e809c 100644 --- a/api/core/Directus/Database/TableGateway/RelationalTableGateway.php +++ b/api/core/Directus/Database/TableGateway/RelationalTableGateway.php @@ -299,7 +299,7 @@ public function deleteFiles($tableName, $recordData) return false; } - $statusColumnName = 'active'; + $statusColumnName = TableSchema::getTableSchema($tableName)->getStatusColumn(); // Get status delete value $tableSchema = $this->getTableSchema(); diff --git a/api/core/Directus/Util/Installation/InstallerUtils.php b/api/core/Directus/Util/Installation/InstallerUtils.php index 393ae167b35c7..6d8765e6c451b 100644 --- a/api/core/Directus/Util/Installation/InstallerUtils.php +++ b/api/core/Directus/Util/Installation/InstallerUtils.php @@ -183,7 +183,7 @@ public static function addDefaultUser($data) $data['avatar'] = get_gravatar($data['directus_email']); $tableGateway->insert([ - 'active' => 1, + 'status' => 1, 'first_name' => 'Admin', 'last_name' => 'User', 'email' => $data['directus_email'], diff --git a/api/migrations/schema/20150204015251_CreateDirectusFilesTable.php b/api/migrations/schema/20150204015251_CreateDirectusFilesTable.php index 4a1cac0030b94..32689cdb0a3dc 100644 --- a/api/migrations/schema/20150204015251_CreateDirectusFilesTable.php +++ b/api/migrations/schema/20150204015251_CreateDirectusFilesTable.php @@ -41,7 +41,7 @@ public function up() 'auto_increment' => true, 'primary_key' => true ]); - $t->column('active', 'tinyinteger', [ + $t->column('status', 'tinyinteger', [ 'limit' => 1, 'default' => 1 ]); diff --git a/api/migrations/schema/20150204042725_CreateDirectusUsersTable.php b/api/migrations/schema/20150204042725_CreateDirectusUsersTable.php index 8a60f9c547c4c..10616eff9f849 100644 --- a/api/migrations/schema/20150204042725_CreateDirectusUsersTable.php +++ b/api/migrations/schema/20150204042725_CreateDirectusUsersTable.php @@ -49,7 +49,7 @@ public function up() 'auto_increment' => true, 'primary_key' => true ]); - $t->column('active', 'tinyinteger', [ + $t->column('status', 'tinyinteger', [ 'limit' => 1, 'default' => 1 ]); diff --git a/api/routes/A1/Auth.php b/api/routes/A1/Auth.php index a1d811598421b..58992de99305f 100644 --- a/api/routes/A1/Auth.php +++ b/api/routes/A1/Auth.php @@ -101,7 +101,7 @@ public function login() $isUserActive = false; // TODO: Add a method in RowGateway to check whether the user is active or not // TODO: Add information about the user status - if ($user['active'] == STATUS_ACTIVE_NUM) { + if ($user['status'] == STATUS_ACTIVE_NUM) { $isUserActive = true; } diff --git a/api/routes/A1/Users.php b/api/routes/A1/Users.php index 7bc9f56b21b1b..ef2bc445c7c77 100644 --- a/api/routes/A1/Users.php +++ b/api/routes/A1/Users.php @@ -68,7 +68,7 @@ protected function sendInvitationTo($email) $token = StringUtils::randomString(128); $result = $tableGateway->insert([ - 'active' => STATUS_DRAFT_NUM, + 'status' => STATUS_DRAFT_NUM, 'email' => $email, 'token' => StringUtils::randomString(32), 'invite_token' => $token,