tbianchini/shopit-back
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: implement BankAccount management with CRUD operations and routes for tenant selection

Browse Source
This commit is contained in:
ncoronel
2026-07-03 13:58:21 -03:00
parent c1b7b12fe1
commit 0d4c6a4492
13 changed files with 120 additions and 77 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
app/Domains/Tenant/Controllers/BankAccountController.php → app/Domains/BankAccount/Controllers/BankAccountController.php
View File

@@ -1,12 +1,12 @@
<?php <?php
namespace App\Domains\Tenant\Controllers; namespace App\Domains\BankAccount\Controllers;
use App\Domains\Tenant\Models\BankAccount; use App\Domains\BankAccount\Models\BankAccount;
use App\Domains\Tenant\Models\Tenant; use App\Domains\Tenant\Models\Tenant;
use App\Domains\Tenant\Requests\StoreBankAccountRequest; use App\Domains\BankAccount\Requests\StoreBankAccountRequest;
use App\Domains\Tenant\Requests\UpdateBankAccountRequest; use App\Domains\BankAccount\Requests\UpdateBankAccountRequest;
use App\Domains\Tenant\Resources\BankAccountResource; use App\Domains\BankAccount\Resources\BankAccountResource;
use App\Domains\Tenant\Resources\TenantResource; use App\Domains\Tenant\Resources\TenantResource;
use App\Http\Controllers\Controller; use App\Http\Controllers\Controller;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
@@ -57,19 +57,19 @@ class BankAccountController extends Controller
public function select(Tenant $tenant, BankAccount $bankAccount): TenantResource public function select(Tenant $tenant, BankAccount $bankAccount): TenantResource
{ {
abort_if($bankAccount->tenant_code !== $tenant->codigo, 400, 'Bank account does not belong to this tenant.'); abort_if($bankAccount->tenant_code !== $tenant->codigo, 400, 'La cuenta bancaria no pertenece a este tenant.');
$tenant->selected_bank_account_id = $bankAccount->id; $tenant->selected_bank_account_id = $bankAccount->id;
$tenant->save(); $tenant->save();
return TenantResource::make($tenant->loadMissing(['headerLogo', 'footerLogo', 'selectedBankAccount'])); return TenantResource::make($tenant->loadMissing(['headerLogo', 'footerLogo']));
} }
public function selected(Tenant $tenant): JsonResponse public function selected(Tenant $tenant): JsonResponse
{ {
$account = $tenant->selectedBankAccount; $account = $tenant->selectedBankAccount;
abort_if(!$account, 404, 'No bank account is currently selected for this tenant.'); abort_if(!$account, 404, 'No hay ninguna cuenta bancaria seleccionada para este tenant.');
return BankAccountResource::make($account)->response(); return BankAccountResource::make($account)->response();
} }

3
app/Domains/Tenant/Models/BankAccount.php → app/Domains/BankAccount/Models/BankAccount.php
View File

@@ -1,7 +1,8 @@
<?php <?php
namespace App\Domains\Tenant\Models; namespace App\Domains\BankAccount\Models;
use App\Domains\Tenant\Models\Tenant;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsTo;

47
app/Domains/BankAccount/Requests/StoreBankAccountRequest.php Normal file
View File

@@ -0,0 +1,47 @@
<?php
namespace App\Domains\BankAccount\Requests;
use Illuminate\Foundation\Http\FormRequest;
class StoreBankAccountRequest extends FormRequest
{
public function authorize(): bool
{
return true;
}
/**
* @return array<string, mixed>
*/
public function rules(): array
{
return [
'titular' => ['required', 'string', 'max:255'],
'entidad' => ['required', 'string', 'max:255'],
'alias' => ['required', 'string', 'max:255'],
'cvu' => ['required', 'string', 'max:255'],
];
}
/**
* @return array<string, string>
*/
public function messages(): array
{
return [
'titular.required' => 'El campo titular es obligatorio.',
'titular.string' => 'El campo titular debe ser una cadena de texto.',
'titular.max' => 'El campo titular no debe superar los 255 caracteres.',
'entidad.required' => 'El campo entidad es obligatorio.',
'entidad.string' => 'El campo entidad debe ser una cadena de texto.',
'entidad.max' => 'El campo entidad no debe superar los 255 caracteres.',
'alias.required' => 'El campo alias es obligatorio.',
'alias.string' => 'El campo alias debe ser una cadena de texto.',
'alias.max' => 'El campo alias no debe superar los 255 caracteres.',
'cvu.required' => 'El campo cvu es obligatorio.',
'cvu.string' => 'El campo cvu debe ser una cadena de texto.',
'cvu.max' => 'El campo cvu no debe superar los 255 caracteres.',
];
}
}

43
app/Domains/BankAccount/Requests/UpdateBankAccountRequest.php Normal file
View File

@@ -0,0 +1,43 @@
<?php
namespace App\Domains\BankAccount\Requests;
use Illuminate\Foundation\Http\FormRequest;
class UpdateBankAccountRequest extends FormRequest
{
public function authorize(): bool
{
return true;
}
/**
* @return array<string, mixed>
*/
public function rules(): array
{
return [
'titular' => ['sometimes', 'string', 'max:255'],
'entidad' => ['sometimes', 'string', 'max:255'],
'alias' => ['sometimes', 'string', 'max:255'],
'cvu' => ['sometimes', 'string', 'max:255'],
];
}
/**
* @return array<string, string>
*/
public function messages(): array
{
return [
'titular.string' => 'El campo titular debe ser una cadena de texto.',
'titular.max' => 'El campo titular no debe superar los 255 caracteres.',
'entidad.string' => 'El campo entidad debe ser una cadena de texto.',
'entidad.max' => 'El campo entidad no debe superar los 255 caracteres.',
'alias.string' => 'El campo alias debe ser una cadena de texto.',
'alias.max' => 'El campo alias no debe superar los 255 caracteres.',
'cvu.string' => 'El campo cvu debe ser una cadena de texto.',
'cvu.max' => 'El campo cvu no debe superar los 255 caracteres.',
];
}
}

4
app/Domains/Tenant/Resources/BankAccountResource.php → app/Domains/BankAccount/Resources/BankAccountResource.php
View File

@@ -1,12 +1,12 @@
<?php <?php
namespace App\Domains\Tenant\Resources; namespace App\Domains\BankAccount\Resources;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/** /**
* @mixin \App\Domains\Tenant\Models\BankAccount * @mixin \App\Domains\BankAccount\Models\BankAccount
*/ */
class BankAccountResource extends JsonResource class BankAccountResource extends JsonResource
{ {

13
app/Domains/BankAccount/routes/api.php Normal file
View File

@@ -0,0 +1,13 @@
<?php
use App\Domains\BankAccount\Controllers\BankAccountController;
use Illuminate\Support\Facades\Route;
Route::prefix('tenants/{tenant:codigo}')->group(function (): void {
Route::middleware('auth:sanctum')->group(function (): void {
Route::get('bank-accounts/selected', [BankAccountController::class, 'selected']);
Route::post('bank-accounts/{bank_account}/select', [BankAccountController::class, 'select']);
Route::apiResource('bank-accounts', BankAccountController::class)
->parameters(['bank-accounts' => 'bank_account']);
});
});

26
app/Domains/Tenant/Requests/StoreBankAccountRequest.php
View File

@@ -1,26 +0,0 @@
<?php
namespace App\Domains\Tenant\Requests;
use Illuminate\Foundation\Http\FormRequest;
class StoreBankAccountRequest extends FormRequest
{
public function authorize(): bool
{
return true;
}
/**
* @return array<string, mixed>
*/
public function rules(): array
{
return [
'titular' => ['required', 'string', 'max:255'],
'entidad' => ['required', 'string', 'max:255'],
'alias' => ['required', 'string', 'max:255'],
'cvu' => ['required', 'string', 'max:255'],
];
}
}

26
app/Domains/Tenant/Requests/UpdateBankAccountRequest.php
View File

@@ -1,26 +0,0 @@
<?php
namespace App\Domains\Tenant\Requests;
use Illuminate\Foundation\Http\FormRequest;
class UpdateBankAccountRequest extends FormRequest
{
public function authorize(): bool
{
return true;
}
/**
* @return array<string, mixed>
*/
public function rules(): array
{
return [
'titular' => ['sometimes', 'string', 'max:255'],
'entidad' => ['sometimes', 'string', 'max:255'],
'alias' => ['sometimes', 'string', 'max:255'],
'cvu' => ['sometimes', 'string', 'max:255'],
];
}
}

1
app/Domains/Tenant/Resources/TenantResource.php
View File

@@ -29,7 +29,6 @@ class TenantResource extends JsonResource
// 1 day // 1 day
'header_logo' => $this->headerLogo?->getTemporaryUrl(1440), 'header_logo' => $this->headerLogo?->getTemporaryUrl(1440),
'footer_logo' => $this->footerLogo?->getTemporaryUrl(1440 ), 'footer_logo' => $this->footerLogo?->getTemporaryUrl(1440 ),
'selected_bank_account_id' => $this->selected_bank_account_id,
]; ];
} }
} }

9
app/Domains/Tenant/routes/api.php
View File

@@ -8,12 +8,3 @@ Route::get('tenants/bootstrap/{dominio}', BootstrapTenantController::class)
->where('dominio', '.*'); ->where('dominio', '.*');
Route::apiResource('tenants', TenantController::class); Route::apiResource('tenants', TenantController::class);
Route::prefix('tenants/{tenant:codigo}')->group(function (): void {
Route::middleware('auth:sanctum')->group(function (): void {
Route::get('bank-accounts/selected', [\App\Domains\Tenant\Controllers\BankAccountController::class, 'selected']);
Route::post('bank-accounts/{bank_account}/select', [\App\Domains\Tenant\Controllers\BankAccountController::class, 'select']);
Route::apiResource('bank-accounts', \App\Domains\Tenant\Controllers\BankAccountController::class)
->parameters(['bank-accounts' => 'bank_account']);
});
});

6
database/seeders/TenantSeeder.php
View File

@@ -86,7 +86,7 @@ class TenantSeeder extends Seeder
'footer_logo' => $footerLogo, 'footer_logo' => $footerLogo,
]); ]);
$bankAccount1 = \App\Domains\Tenant\Models\BankAccount::query()->create([ $bankAccount1 = \App\Domains\BankAccount\Models\BankAccount::query()->create([
'tenant_code' => $tenant->codigo, 'tenant_code' => $tenant->codigo,
'titular' => 'Sonder S.A. (Principal)', 'titular' => 'Sonder S.A. (Principal)',
'entidad' => 'Banco de la Nación Argentina', 'entidad' => 'Banco de la Nación Argentina',
@@ -94,7 +94,7 @@ class TenantSeeder extends Seeder
'cvu' => '0110065420006540987654', 'cvu' => '0110065420006540987654',
]); ]);
\App\Domains\Tenant\Models\BankAccount::query()->create([ \App\Domains\BankAccount\Models\BankAccount::query()->create([
'tenant_code' => $tenant->codigo, 'tenant_code' => $tenant->codigo,
'titular' => 'Sonder S.A. (Secundaria)', 'titular' => 'Sonder S.A. (Secundaria)',
'entidad' => 'Banco de Galicia y Buenos Aires', 'entidad' => 'Banco de Galicia y Buenos Aires',
@@ -102,7 +102,7 @@ class TenantSeeder extends Seeder
'cvu' => '0070012345678901234567', 'cvu' => '0070012345678901234567',
]); ]);
\App\Domains\Tenant\Models\BankAccount::query()->create([ \App\Domains\BankAccount\Models\BankAccount::query()->create([
'tenant_code' => $tenant->codigo, 'tenant_code' => $tenant->codigo,
'titular' => 'Sonder S.A. (Mercado Pago)', 'titular' => 'Sonder S.A. (Mercado Pago)',
'entidad' => 'Mercado Pago', 'entidad' => 'Mercado Pago',

1
routes/api.php
View File

@@ -9,3 +9,4 @@ require __DIR__.'/../app/Domains/Cart/routes/api.php';
require __DIR__.'/../app/Domains/StorageTest/routes/api.php'; require __DIR__.'/../app/Domains/StorageTest/routes/api.php';
require __DIR__.'/../app/Domains/Purchase/routes/api.php'; require __DIR__.'/../app/Domains/Purchase/routes/api.php';
require __DIR__.'/../app/Domains/Tenant/routes/api.php'; require __DIR__.'/../app/Domains/Tenant/routes/api.php';
require __DIR__.'/../app/Domains/BankAccount/routes/api.php';

2
tests/Feature/Tenant/BankAccountControllerTest.php
View File

@@ -2,7 +2,7 @@
namespace Tests\Feature\Tenant; namespace Tests\Feature\Tenant;
use App\Domains\Tenant\Models\BankAccount; use App\Domains\BankAccount\Models\BankAccount;
use App\Domains\Tenant\Models\Tenant; use App\Domains\Tenant\Models\Tenant;
use App\Domains\Auth\Models\User; use App\Domains\Auth\Models\User;
use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\RefreshDatabase;