Remove old 'active' column and replace some references with 'suspended' in place

Author: DaneEveritt

app/Console/Commands/RunTasks.php

@@ -67,7 +67,7 @@ public function __construct()
      */
     public function handle()
     {
-        $tasks = Models\Task::where('queued', 0)->where('active', 1)->where('next_run', '<=', (Carbon::now())->toAtomString())->get();
+        $tasks = Models\Task::where('queued', 0)->where('suspended', 0)->where('next_run', '<=', (Carbon::now())->toAtomString())->get();
 
         $this->info(sprintf('Preparing to queue %d tasks.', count($tasks)));
         $bar = $this->output->createProgressBar(count($tasks));

app/Http/Controllers/Admin/UserController.php

@@ -69,7 +69,6 @@ public function getView(Request $request, $id)
                 ->join('nodes', 'servers.node', '=', 'nodes.id')
                 ->join('locations', 'nodes.location', '=', 'locations.id')
                 ->where('owner', $id)
-                ->where('active', 1)
                 ->get(),
         ]);
     }

app/Models/Server.php

@@ -59,7 +59,7 @@ class Server extends Model
      */
      protected $casts = [
          'node' => 'integer',
-         'active' => 'integer',
+         'suspended' => 'integer',
          'owner' => 'integer',
          'memory' => 'integer',
          'swap' => 'integer',
@@ -117,7 +117,7 @@ protected static function getUserDaemonSecret(Server $server)
 
     /**
      * Returns array of all servers owned by the logged in user.
-     * Returns all active servers if user is a root admin.
+     * Returns all users servers if user is a root admin.
      *
      * @return \Illuminate\Database\Eloquent\Collection
      */
@@ -132,8 +132,7 @@ public static function getUserServers($paginate = null)
             'allocations.port'
         )->join('nodes', 'servers.node', '=', 'nodes.id')
         ->join('locations', 'nodes.location', '=', 'locations.id')
-        ->join('allocations', 'servers.allocation', '=', 'allocations.id')
-        ->where('active', 1);
+        ->join('allocations', 'servers.allocation', '=', 'allocations.id');
 
         if (self::$user->root_admin !== 1) {
             $query->whereIn('servers.id', Subuser::accessServers());
@@ -164,7 +163,7 @@ public static function getByUUID($uuid)
 
         $query = self::select('servers.*', 'services.file as a_serviceFile')
             ->join('services', 'services.id', '=', 'servers.service')
-            ->where('uuidShort', $uuid)->where('active', 1);
+            ->where('uuidShort', $uuid);
 
         if (self::$user->root_admin !== 1) {
             $query->whereIn('servers.id', Subuser::accessServers());

app/Repositories/ServerRepository.php

@@ -205,7 +205,7 @@ public function create(array $data)
                 'uuidShort' => $uuid->generateShort('servers', 'uuidShort', $generatedUuid),
                 'node' => $data['node'],
                 'name' => $data['name'],
-                'active' => 1,
+                'suspended' => 0,
                 'owner' => $user->id,
                 'memory' => $data['memory'],
                 'swap' => $data['swap'],

database/migrations/2016_09_01_211924_remove_active_column.php

@@ -0,0 +1,31 @@
+<?php
+
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Database\Migrations\Migration;
+
+class RemoveActiveColumn extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::table('servers', function (Blueprint $table) {
+            $table->dropColumn('active');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::table('servers', function (Blueprint $table) {
+            $table->tinyInteger('active')->after('name')->unsigned()->default(0);
+        });
+    }
+}