diff --git a/app/Http/Controllers/References/BankAccountsController.php b/app/Http/Controllers/References/BankAccountsController.php index 9462bf9..699809d 100644 --- a/app/Http/Controllers/References/BankAccountsController.php +++ b/app/Http/Controllers/References/BankAccountsController.php @@ -107,11 +107,13 @@ class BankAccountsController extends Controller { $bankAccounts = Option::where('key', 'bank_accounts')->first(); if ($bankAccounts && $bankAccounts->value) { - foreach (json_decode($bankAccounts->value, true) as $bankAccountData) { + $bankAccountList = json_decode($bankAccounts->value, true); + foreach ($bankAccountList as $bankAccountData) { $bankAccount = new BankAccount($bankAccountData); $bankAccount->save(); } $bankAccounts->delete(); + flash(__('bank_account.imported', ['count' => count($bankAccountList)]), 'success'); } return back(); diff --git a/resources/lang/de/bank_account.php b/resources/lang/de/bank_account.php index db3b9f7..1611f5e 100644 --- a/resources/lang/de/bank_account.php +++ b/resources/lang/de/bank_account.php @@ -8,6 +8,8 @@ return [ 'back_to_index' => 'Zurück zur Bankverbindunsliste', // Actions + 'import' => 'Import Bank Account', + 'imported' => ':count bank account(s) has imported.', 'create' => 'Neue Bankverbindung erstellen', 'created' => 'Neue Bankverbindung wurde erstellt.', 'show' => 'Zeige Bankverbindung Details', diff --git a/resources/lang/en/bank_account.php b/resources/lang/en/bank_account.php index 524cee8..b153168 100644 --- a/resources/lang/en/bank_account.php +++ b/resources/lang/en/bank_account.php @@ -8,6 +8,8 @@ return [ 'back_to_index' => 'Back to Bank Account list', // Actions + 'import' => 'Import Bank Account', + 'imported' => ':count bank account(s) has imported.', 'create' => 'Create new Bank Account', 'created' => 'New Bank Account has been created.', 'show' => 'Show Bank Account detail', diff --git a/resources/lang/id/bank_account.php b/resources/lang/id/bank_account.php index fc21def..b178ec2 100644 --- a/resources/lang/id/bank_account.php +++ b/resources/lang/id/bank_account.php @@ -8,6 +8,8 @@ return [ 'back_to_index' => 'Kembali ke daftar Rekening Bank', // Actions + 'import' => 'Import Rekening Bank', + 'imported' => ':count rekening bank telah diimport.', 'create' => 'Input Rekening Bank Baru', 'created' => 'Input Rekening Bank baru telah berhasil.', 'show' => 'Lihat Detail Rekening Bank', diff --git a/resources/views/bank-accounts/index.blade.php b/resources/views/bank-accounts/index.blade.php index 3c883ae..923f315 100644 --- a/resources/views/bank-accounts/index.blade.php +++ b/resources/views/bank-accounts/index.blade.php @@ -3,17 +3,18 @@ @section('title', trans('bank_account.list')) @section('content-dashboard') -{!! FormField::formButton(['route' => 'bank-accounts.import'], __('bank_account.import'), ['id' => 'import-bank-accounts']) !!}
@foreach ($bankAccounts as $bankAccount)
-

{{ $bankAccount->name }}

+
+

{{ $bankAccount->name }}

+
+ {{ $bankAccount->status }}

{{ trans('bank_account.number') }}:
{{ $bankAccount->number }}

{{ trans('bank_account.account_name') }}:
{{ $bankAccount->account_name }}

-

{{ __('app.status') }}:
{{ $bankAccount->status }}

@if ($bankAccount->description)

{{ trans('app.description') }}:
{{ $bankAccount->description }}

@endif @@ -44,6 +45,12 @@ ]) !!} @endif @includeWhen(Request::has('action'), 'bank-accounts.forms') + @if (Option::get('bank_accounts')) + {!! FormField::formButton(['route' => 'bank-accounts.import'], __('bank_account.import'), [ + 'id' => 'import-bank-accounts', + 'class' => 'btn btn-primary', + ]) !!} + @endif
@endsection diff --git a/tests/Feature/References/ManageBankAccountsTest.php b/tests/Feature/References/ManageBankAccountsTest.php index 501362c..8ac5d46 100644 --- a/tests/Feature/References/ManageBankAccountsTest.php +++ b/tests/Feature/References/ManageBankAccountsTest.php @@ -3,7 +3,7 @@ namespace Tests\Feature\References; use Tests\TestCase; -use Facades\App\Services\Option; +use App\Entities\Options\Option; use App\Entities\Invoices\BankAccount; use Illuminate\Foundation\Testing\DatabaseMigrations; @@ -119,14 +119,17 @@ class ManageBankAccountsTest extends TestCase 'description' => 'BankAccount 1 description', ]; - Option::set('bank_accounts', json_encode($bankAccounts)); + Option::create([ + 'key' => 'bank_accounts', + 'value' => json_encode($bankAccounts), + ]); $this->visit(route('bank-accounts.index')); $this->seeElement('button', ['id' => 'import-bank-accounts']); $this->press('import-bank-accounts'); $this->seePageIs(route('bank-accounts.index')); - + $this->seeText(__('bank_account.imported', ['count' => 1])); $this->dontSeeInDatabase('site_options', [ 'value' => json_encode($bankAccounts), ]);