refactor(client): extract base account generation to command
Move the logic for generating base accounts for a new client from the ClientObserver into a dedicated command class (GenerateBaseAccountCommand). This improves code organization and reusability. - The command is now used in the ClientObserver::created method. - The command is also made available as a manual action in the AccountsRelationManager table header, allowing admins to generate base accounts for existing clients that lack them. - Added necessary imports to the CreateSale page, though the command is not directly used there in this diff, suggesting preparatory work for future integration.
This commit is contained in:
64
app/Commands/Clients/GenerateBaseAccountCommand.php
Normal file
64
app/Commands/Clients/GenerateBaseAccountCommand.php
Normal file
@@ -0,0 +1,64 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace App\Commands\Clients;
|
||||||
|
|
||||||
|
use App\Commands\Command;
|
||||||
|
use Illuminate\Support\Facades\DB;
|
||||||
|
|
||||||
|
class GenerateBaseAccountCommand
|
||||||
|
{
|
||||||
|
/**
|
||||||
|
* Create a new class instance.
|
||||||
|
*/
|
||||||
|
public function __construct()
|
||||||
|
{
|
||||||
|
//
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Execute the command.
|
||||||
|
*/
|
||||||
|
public function execute($client): void
|
||||||
|
{
|
||||||
|
DB::transaction(function () use ($client) {
|
||||||
|
$client->accounts()->createMany([
|
||||||
|
[
|
||||||
|
'account_type_id' => 1,
|
||||||
|
'account' => 'Cash',
|
||||||
|
'normal_balance' => 'debit',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'account_type_id' => 1,
|
||||||
|
'account' => 'Input Tax',
|
||||||
|
'normal_balance' => 'debit',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'account_type_id' => 1,
|
||||||
|
'account' => 'Creditable Withholding Tax',
|
||||||
|
'normal_balance' => 'debit',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'account_type_id' => 2,
|
||||||
|
'account' => 'Output Tax',
|
||||||
|
'normal_balance' => 'credit',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'account_type_id' => 2,
|
||||||
|
'account' => 'Payable Withholding Tax',
|
||||||
|
'normal_balance' => 'credit',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'account_type_id' => 5,
|
||||||
|
'account' => 'Vat Exempt Revenue',
|
||||||
|
'normal_balance' => 'credit',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'account_type_id' => 4,
|
||||||
|
'account' => 'Sales Discount',
|
||||||
|
'normal_balance' => 'debit',
|
||||||
|
],
|
||||||
|
]);
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
@@ -2,6 +2,7 @@
|
|||||||
|
|
||||||
namespace App\Filament\Resources\ClientResource\RelationManagers;
|
namespace App\Filament\Resources\ClientResource\RelationManagers;
|
||||||
|
|
||||||
|
use App\Commands\Clients\GenerateBaseAccountCommand;
|
||||||
use App\Filament\Exports\ClientAccountsExporter;
|
use App\Filament\Exports\ClientAccountsExporter;
|
||||||
use App\Models\Account;
|
use App\Models\Account;
|
||||||
use Filament\Actions\Exports\Enums\ExportFormat;
|
use Filament\Actions\Exports\Enums\ExportFormat;
|
||||||
@@ -43,6 +44,20 @@ class AccountsRelationManager extends RelationManager
|
|||||||
//
|
//
|
||||||
])
|
])
|
||||||
->headerActions([
|
->headerActions([
|
||||||
|
Tables\Actions\Action::make('generate-base-accounts')
|
||||||
|
->requiresConfirmation()
|
||||||
|
->label('Generate Base Accounts')
|
||||||
|
->action(function () {
|
||||||
|
$client = $this->getOwnerRecord();
|
||||||
|
if (! $client ) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if($client->accounts()->count() > 0) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
app(GenerateBaseAccountCommand::class)->execute($client);
|
||||||
|
}),
|
||||||
Tables\Actions\ExportAction::make('Export Accounts')->exporter(ClientAccountsExporter::class)->formats([ExportFormat::Csv]),
|
Tables\Actions\ExportAction::make('Export Accounts')->exporter(ClientAccountsExporter::class)->formats([ExportFormat::Csv]),
|
||||||
Tables\Actions\CreateAction::make()->label('New Account')->icon('heroicon-o-plus')->slideOver(),
|
Tables\Actions\CreateAction::make()->label('New Account')->icon('heroicon-o-plus')->slideOver(),
|
||||||
])
|
])
|
||||||
|
|||||||
@@ -5,12 +5,15 @@ namespace App\Filament\Resources\SaleResource\Pages;
|
|||||||
use App\Actions\Sales\CreateSaleAction;
|
use App\Actions\Sales\CreateSaleAction;
|
||||||
use App\Actions\Sales\SyncAccountsAction;
|
use App\Actions\Sales\SyncAccountsAction;
|
||||||
use App\Actions\Transactions\CreateRecordTransactionsAction;
|
use App\Actions\Transactions\CreateRecordTransactionsAction;
|
||||||
|
use App\Commands\Clients\GenerateBaseAccountCommand;
|
||||||
use App\Filament\Resources\ClientResource;
|
use App\Filament\Resources\ClientResource;
|
||||||
use App\Filament\Resources\SaleResource;
|
use App\Filament\Resources\SaleResource;
|
||||||
use App\Models\Branch;
|
use App\Models\Branch;
|
||||||
use App\Models\Client;
|
use App\Models\Client;
|
||||||
use App\Models\Sale;
|
use App\Models\Sale;
|
||||||
use App\Services\Sales\SaleService;
|
use App\Services\Sales\SaleService;
|
||||||
|
use Filament\Actions\Action;
|
||||||
|
use Filament\Forms\Components\Actions;
|
||||||
use Filament\Resources\Pages\CreateRecord;
|
use Filament\Resources\Pages\CreateRecord;
|
||||||
use Illuminate\Database\Eloquent\Model;
|
use Illuminate\Database\Eloquent\Model;
|
||||||
use Illuminate\Support\Arr;
|
use Illuminate\Support\Arr;
|
||||||
|
|||||||
@@ -2,6 +2,7 @@
|
|||||||
|
|
||||||
namespace App\Observers;
|
namespace App\Observers;
|
||||||
|
|
||||||
|
use App\Commands\Clients\GenerateBaseAccountCommand;
|
||||||
use App\Models\Client;
|
use App\Models\Client;
|
||||||
use Illuminate\Support\Facades\DB;
|
use Illuminate\Support\Facades\DB;
|
||||||
|
|
||||||
@@ -12,45 +13,7 @@ class ClientObserver
|
|||||||
*/
|
*/
|
||||||
public function created(Client $client): void
|
public function created(Client $client): void
|
||||||
{
|
{
|
||||||
DB::transaction(function () use ($client) {
|
app(GenerateBaseAccountCommand::class)->execute($client);
|
||||||
$client->accounts()->createMany([
|
|
||||||
[
|
|
||||||
'account_type_id' => 1,
|
|
||||||
'account' => 'Cash',
|
|
||||||
'normal_balance' => 'debit',
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'account_type_id' => 1,
|
|
||||||
'account' => 'Input Tax',
|
|
||||||
'normal_balance' => 'debit',
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'account_type_id' => 1,
|
|
||||||
'account' => 'Creditable Withholding Tax',
|
|
||||||
'normal_balance' => 'debit',
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'account_type_id' => 2,
|
|
||||||
'account' => 'Output Tax',
|
|
||||||
'normal_balance' => 'credit',
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'account_type_id' => 2,
|
|
||||||
'account' => 'Payable Withholding Tax',
|
|
||||||
'normal_balance' => 'credit',
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'account_type_id' => 5,
|
|
||||||
'account' => 'Vat Exempt Revenue',
|
|
||||||
'normal_balance' => 'credit',
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'account_type_id' => 4,
|
|
||||||
'account' => 'Sales Discount',
|
|
||||||
'normal_balance' => 'debit',
|
|
||||||
],
|
|
||||||
]);
|
|
||||||
});
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
Reference in New Issue
Block a user