Purge empty campaigns + fix magical campaign_id

app/Console/Commands/CleanupCampaigns.php

@@ -0,0 +1,48 @@
+<?php
+
+namespace App\Console\Commands;
+
+use App\Services\Campaign\PurgeService;
+use Carbon\Carbon;
+use Illuminate\Console\Command;
+
+class CleanupCampaigns extends Command
+{
+    /**
+     * The name and signature of the console command.
+     *
+     * @var string
+     */
+    protected $signature = 'campaigns:cleanup {dry=1}';
+
+    /**
+     * The console command description.
+     *
+     * @var string
+     */
+    protected $description = 'Delete empty campaigns';
+
+    /**
+     * Execute the console command.
+     */
+    public function handle()
+    {
+        $this->info(Carbon::now());
+
+        /** @var PurgeService $service */
+        $service = app()->make(PurgeService::class);
+
+        $dry = $this->argument('dry');
+        if ($dry === '0') {
+            $service->real();
+        }
+
+        $count = $service->purgeEmpty();
+
+        if ($dry === '0') {
+            $this->info(Carbon::now() . ': Deleted ' . $count . ' empty campaigns.');
+        } else {
+            $this->info(Carbon::now() . ': There are ' . $count . ' empty campaigns that can be deleted.');
+        }
+    }
+}

app/Models/Ability.php

@@ -42,6 +42,7 @@ class Ability extends MiscModel
 
     /** @var string[]  */
     protected $fillable = [
+        'campaign_id',
         'name',
         'slug',
         'type',

app/Models/Family.php

@@ -27,8 +27,7 @@
  */
 class Family extends MiscModel
 {
-    use Acl
-    ;
+    use Acl;
     use CampaignTrait;
     use ExportableTrait;
     use Nested;
@@ -38,6 +37,7 @@ class Family extends MiscModel
 
     /** @var string[]  */
     protected $fillable = [
+        'campaign_id',
         'name',
         'slug',
         'entry',

app/Models/Map.php

@@ -64,6 +64,7 @@ class Map extends MiscModel
 
     /** @var string[]  */
     protected $fillable = [
+        'campaign_id',
         'name',
         'slug',
         'type',

app/Models/Note.php

@@ -21,8 +21,7 @@
  */
 class Note extends MiscModel
 {
-    use Acl
-    ;
+    use Acl;
     use CampaignTrait;
     use ExportableTrait;
     use Nested;
@@ -31,6 +30,7 @@ class Note extends MiscModel
 
     /** @var string[]  */
     protected $fillable = [
+        'campaign_id',
         'name',
         'slug',
         'entry',

app/Models/Organisation.php

@@ -28,8 +28,7 @@
  */
 class Organisation extends MiscModel
 {
-    use Acl
-    ;
+    use Acl;
     use CampaignTrait;
     use ExportableTrait;
     use Nested;
@@ -39,6 +38,7 @@ class Organisation extends MiscModel
 
     /** @var string[]  */
     protected $fillable = [
+        'campaign_id',
         'name',
         'slug',
         'entry',

app/Models/Timeline.php

@@ -24,12 +24,12 @@ class Timeline extends MiscModel
     use Acl;
     use CampaignTrait;
     use ExportableTrait;
-    use Nested
-    ;
+    use Nested;
     use SoftDeletes;
     use SortableTrait;
 
     public $fillable = [
+        'campaign_id',
         'name',
         'type',
         'calendar_id',

app/Services/Campaign/PurgeService.php

@@ -0,0 +1,50 @@
+<?php
+
+namespace App\Services\Campaign;
+
+use App\Models\Campaign;
+use App\Services\ImageService;
+use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Support\Facades\Log;
+
+class PurgeService
+{
+    protected int $count = 0;
+    protected bool $dry = true;
+
+    public function real(): self
+    {
+        $this->dry = false;
+        return $this;
+    }
+
+    public function count(): int
+    {
+        return $this->baseAll()
+            ->count();
+    }
+
+    public function purgeEmpty(): int
+    {
+        $this->baseAll()
+            ->chunk(500, function ($campaigns) {
+                /** @var Campaign $campaign */
+                foreach ($campaigns as $campaign) {
+                    if (!$this->dry) {
+                        ImageService::cleanup($campaign);
+                        $campaign->delete();
+                        Log::info('Services\Campaigns\PurgeService', ['campaign' => $campaign->id]);
+                    }
+                    $this->count++;
+                }
+            });
+        return $this->count;
+    }
+
+    protected function baseAll(): Builder
+    {
+        return Campaign::select('campaigns.id')
+            ->leftJoin('campaign_user as cu', 'cu.campaign_id', 'campaigns.id')
+            ->whereNull('cu.id');
+    }
+}