From 798608cebd778e7c984a99686e57bb088e289923 Mon Sep 17 00:00:00 2001
From: Deon George <deon@leenooks.net>
Date: Fri, 19 Aug 2022 13:52:16 +1000
Subject: [PATCH] Fix site_id table references

---
 app/Jobs/ImportCosts.php                      | 10 ++--
 .../2022_08_19_133535_fix_siteid.php          | 52 +++++++++++++++++++
 2 files changed, 57 insertions(+), 5 deletions(-)
 create mode 100644 database/migrations/2022_08_19_133535_fix_siteid.php

diff --git a/app/Jobs/ImportCosts.php b/app/Jobs/ImportCosts.php
index 0136b37..de12645 100644
--- a/app/Jobs/ImportCosts.php
+++ b/app/Jobs/ImportCosts.php
@@ -35,19 +35,19 @@ class ImportCosts implements ShouldQueue
 		$this->file = $file;
 		Config::set('site',$site);
 
-		$this->co = Cost::where('site_id',$site->id)
+		$this->co = Cost::where('site_id',$site->site_id)
 			->where('supplier_id',$so->id)
 			->where('billed_at',$invoice_date)
 			->firstOrNew();
 		$this->co->active = TRUE;
-		$this->co->site_id = $site->id;
+		$this->co->site_id = $site->site_id;
 		$this->co->billed_at = $invoice_date;
 		$this->co->supplier_id = $so->id;
 		$this->co->save();
 
-		Cost\Broadband::where('cost_id',$this->co->id)->where('site_id',$site->id)->delete();
-		Cost\Phone::where('cost_id',$this->co->id)->where('site_id',$site->id)->delete();
-		Cost\Generic::where('cost_id',$this->co->id)->where('site_id',$site->id)->delete();
+		Cost\Broadband::where('cost_id',$this->co->id)->where('site_id',$site->site_id)->delete();
+		Cost\Phone::where('cost_id',$this->co->id)->where('site_id',$site->site_id)->delete();
+		Cost\Generic::where('cost_id',$this->co->id)->where('site_id',$site->site_id)->delete();
 
 		// @todo to be stored in supplier config
 		$headers = [
diff --git a/database/migrations/2022_08_19_133535_fix_siteid.php b/database/migrations/2022_08_19_133535_fix_siteid.php
new file mode 100644
index 0000000..ca4fe57
--- /dev/null
+++ b/database/migrations/2022_08_19_133535_fix_siteid.php
@@ -0,0 +1,52 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+		foreach ([
+			'checkouts',
+			'costs',
+			'domain_registrars',
+			'product_broadband',
+			'product_email',
+			'product_generic',
+			'product_host',
+			'product_ssl',
+			'products',
+			'provider_oauth',
+			'supplier_broadband',
+			'supplier_domain',
+			'supplier_email',
+			'supplier_generic',
+			'supplier_host',
+			'supplier_host_servers',
+			'supplier_ssl',
+		] as $table)
+		{
+			Schema::table($table, function (Blueprint $table) {
+				$table->dropForeign(['site_id']);
+				$table->foreign(['site_id'])->references(['site_id'])->on('sites');
+			});
+		}
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        abort(500,'cant go back');
+    }
+};