From c21a79e0299585f4562f3b8568cfe922b40bef3a Mon Sep 17 00:00:00 2001
From: James Cole <james@firefly-iii.org>
Date: Sun, 29 Dec 2024 08:16:27 +0100
Subject: [PATCH] Better, but not perfect, currency switch.

---
 .../Events/PreferencesEventHandler.php        | 26 +++++++++++--------
 public/v1/js/ff/currencies/index.js           |  9 +++++++
 2 files changed, 24 insertions(+), 11 deletions(-)

diff --git a/app/Handlers/Events/PreferencesEventHandler.php b/app/Handlers/Events/PreferencesEventHandler.php
index e18fd2de3b..28d4e5c5e8 100644
--- a/app/Handlers/Events/PreferencesEventHandler.php
+++ b/app/Handlers/Events/PreferencesEventHandler.php
@@ -30,7 +30,9 @@ use FireflyIII\Models\PiggyBank;
 use FireflyIII\Models\UserGroup;
 use FireflyIII\Repositories\UserGroups\Budget\BudgetRepositoryInterface;
 use FireflyIII\Repositories\UserGroups\PiggyBank\PiggyBankRepositoryInterface;
+use FireflyIII\Support\Facades\Amount;
 use Illuminate\Database\Query\Builder;
+use Illuminate\Support\Facades\Artisan;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Log;
 
@@ -57,6 +59,10 @@ class PreferencesEventHandler
         $this->resetPiggyBanks($event->userGroup);
         $this->resetBudgets($event->userGroup);
         $this->resetTransactions($event->userGroup);
+        // fire laravel command to recalculate them all.
+        if (Amount::convertToNative()) {
+            Artisan::call('correction:recalculate-native-amounts');
+        }
     }
 
     private function resetPiggyBanks(UserGroup $userGroup): void
@@ -93,7 +99,7 @@ class PreferencesEventHandler
     {
         $repository = app(BudgetRepositoryInterface::class);
         $repository->setUserGroup($userGroup);
-        $set        = $repository->getBudgets();
+        $set = $repository->getBudgets();
 
         /** @var Budget $budget */
         foreach ($set as $budget) {
@@ -121,15 +127,13 @@ class PreferencesEventHandler
     {
         // custom query because of the potential size of this update.
         $success = DB::table('transactions')
-            ->join('transaction_journals', 'transaction_journals.id', '=', 'transactions.transaction_journal_id')
-            ->where('transaction_journals.user_group_id', $userGroup->id)
-            ->where(static function (Builder $q): void {
-                $q->whereNotNull('native_amount')
-                    ->orWhereNotNull('native_foreign_amount')
-                ;
-            })
-            ->update(['native_amount' => null, 'native_foreign_amount' => null])
-        ;
-        Log::debug(sprintf('Updated %d transactions.', $success));
+                     ->join('transaction_journals', 'transaction_journals.id', '=', 'transactions.transaction_journal_id')
+                     ->where('transaction_journals.user_group_id', $userGroup->id)
+                     ->where(static function (Builder $q): void {
+                         $q->whereNotNull('native_amount')
+                           ->orWhereNotNull('native_foreign_amount');
+                     })
+                     ->update(['native_amount' => null, 'native_foreign_amount' => null]);
+        Log::debug(sprintf('Reset %d transactions.', $success));
     }
 }
diff --git a/public/v1/js/ff/currencies/index.js b/public/v1/js/ff/currencies/index.js
index 99ecaf7e24..d8b670c18b 100644
--- a/public/v1/js/ff/currencies/index.js
+++ b/public/v1/js/ff/currencies/index.js
@@ -26,10 +26,19 @@ $(function () {
     $('.make_default').on('click', setDefaultCurrency);
     $('.enable-currency').on('click', enableCurrency);
     $('.disable-currency').on('click', disableCurrency);
+    console.log('Loaded3');
 });
 
 function setDefaultCurrency(e) {
+    console.log('Setting default currency');
     var button = $(e.currentTarget);
+    // disable everything.
+    button.prop('disabled', true);
+    $('a').css('pointer-events', 'none');
+
+    // change cursor to hourglass
+    $('body').css('cursor', 'wait');
+
     var currencyCode = button.data('code');
 
     var params = {