Get rid of CurrencyRepository

app/Http/Controllers/TransactionController.php

@@ -2,26 +2,23 @@
 
 namespace App\Http\Controllers;
 
+use App\Models\Currency;
 use App\Models\Space;
 use App\Models\Tag;
-use App\Repositories\CurrencyRepository;
 use App\Repositories\RecurringRepository;
 use App\Repositories\TransactionRepository;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Auth;
 
 class TransactionController extends Controller
 {
-    private $currencyRepository;
     private $recurringRepository;
 
     public function __construct(
         TransactionRepository $transactionRepository,
-        CurrencyRepository $currencyRepository,
         RecurringRepository $recurringRepository
     ) {
         $this->repository = $transactionRepository;
-        $this->currencyRepository = $currencyRepository;
         $this->recurringRepository = $recurringRepository;
     }
 
@@ -50,9 +47,18 @@ public function create()
             ];
         }
 
+        $spaceCurrencyId = Space::find(session('space_id'))->currency_id;
+
+        $currencies = Currency::select('currencies.*')
+            ->leftJoin('conversion_rates AS cr', 'cr.base_currency_id', '=', 'currencies.id')
+            ->where('cr.target_currency_id', $spaceCurrencyId)
+            ->orWhere('currencies.id', $spaceCurrencyId)
+            ->groupBy('currencies.id')
+            ->get();
+
         return view('transactions.create', [
             'tags' => $tags,
-            'currencies' => $this->currencyRepository->getIfConversionRatePresent(),
+            'currencies' => $currencies,
             'defaultTransactionType' => Auth::user()->default_transaction_type,
             'firstDayOfWeek' => Auth::user()->first_day_of_week,
             'defaultCurrencyId' => Space::find(session('space_id'))->currency_id,

app/Jobs/FetchConversionRate.php

@@ -2,8 +2,8 @@
 
 namespace App\Jobs;
 
+use App\Models\Currency;
 use App\Repositories\ConversionRateRepository;
-use App\Repositories\CurrencyRepository;
 use Exception;
 use GuzzleHttp\Client;
 use Illuminate\Bus\Queueable;
@@ -19,32 +19,27 @@ class FetchConversionRate implements ShouldQueue
     use Queueable;
     use SerializesModels;
 
-    protected $baseCurrencyId;
-
-    private $currencyRepository;
     private $conversionRateRepository;
 
-    public function __construct(int $baseCurrencyId)
+    public function __construct(protected int $baseCurrencyId)
     {
-        $this->baseCurrencyId = $baseCurrencyId;
+        //
     }
 
-    public function handle(
-        CurrencyRepository $currencyRepository,
-        ConversionRateRepository $conversionRateRepository
-    ): void {
-        $this->currencyRepository = $currencyRepository;
+    public function handle(ConversionRateRepository $conversionRateRepository): void
+    {
         $this->conversionRateRepository = $conversionRateRepository;
 
-        $baseCurrency = $this->currencyRepository->getById($this->baseCurrencyId);
+        /** @var Currency $baseCurrency */
+        $baseCurrency = Currency::query()->find($this->baseCurrencyId);
 
         if (!$baseCurrency || !$baseCurrency->iso) {
             return;
         }
 
         $client = new Client();
 
-        foreach ($this->currencyRepository->getAll() as $targetCurrency) {
+        foreach (Currency::query()->get() as $targetCurrency) {
             if (!$targetCurrency->iso || $baseCurrency->iso === $targetCurrency->iso) {
                 continue;
             }

app/Jobs/FetchConversionRates.php

@@ -2,7 +2,7 @@
 
 namespace App\Jobs;
 
-use App\Repositories\CurrencyRepository;
+use App\Models\Currency;
 use Illuminate\Bus\Queueable;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
@@ -16,13 +16,9 @@ class FetchConversionRates implements ShouldQueue
     use Queueable;
     use SerializesModels;
 
-    private $currencyRepository;
-
-    public function handle(CurrencyRepository $currencyRepository): void
+    public function handle(): void
     {
-        $this->currencyRepository = $currencyRepository;
-
-        foreach ($this->currencyRepository->getAll() as $currency) {
+        foreach (Currency::query()->get() as $currency) {
             FetchConversionRate::dispatch($currency->id);
         }
     }