Merge branch 'tenant/smep' into homo

Reviewed-on: https://gitea.quo.ar/tbianchini/shopit-back/pulls/11

app/Domains/Commerce/Catalog/Enums/InventoryMovementOperation.php

@@ -1,18 +0,0 @@
-<?php
-
-namespace App\Domains\Commerce\Catalog\Enums;
-
-enum InventoryMovementOperation: string
-{
-    case StockInitialized = 'stock_initialized';
-    case StockIncreased = 'stock_increased';
-    case StockDecreased = 'stock_decreased';
-    case StockReserved = 'stock_reserved';
-    case StockReleased = 'stock_released';
-    case PurchaseCommitted = 'purchase_committed';
-    case StockRefunded = 'stock_refunded';
-    case EntryStockReserved = 'entry_stock_reserved';
-    case EntryStockReleased = 'entry_stock_released';
-    case InventoryReset = 'inventory_reset';
-    case InventoryTransferred = 'inventory_transferred';
-}

app/Domains/Commerce/Catalog/Models/CatalogItem.php

@@ -19,7 +19,6 @@ use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use Illuminate\Support\Collection;
-use Illuminate\Support\Facades\Schema;
 
 #[Fillable([
     'tenant_code',
@@ -214,14 +213,10 @@ class CatalogItem extends Model
     /** @param Builder<CatalogItem> $query */
     public function scopeWhereAvailable(Builder $query): Builder
     {
-        $availableInventory = static fn (Builder $inventoryQuery): Builder => Schema::hasColumn('inventories', 'available_stock')
-            ? $inventoryQuery->where('inventories.available_stock', '>', 0)
-            : $inventoryQuery->whereRaw('inventories.real_stock > inventories.reserved_stock + inventories.entry_reserved_stock');
-
         return $query->where(function (Builder $query): void {
             $query->whereNull('catalog_items.sales_end_at')
                 ->orWhere('catalog_items.sales_end_at', '>', now());
-        })->where(function (Builder $query) use ($availableInventory): void {
+        })->where(function (Builder $query): void {
             $query
                 ->where(function (Builder $unlimitedQuery): void {
                     $unlimitedQuery
@@ -241,18 +236,20 @@ class CatalogItem extends Model
                         ->whereNull('replaced_by_variant_id')
                         ->whereHas(
                             'inventory',
-                            $availableInventory
+                            fn (Builder $inventoryQuery): Builder => $inventoryQuery
+                                ->whereRaw('inventories.real_stock > inventories.reserved_stock + inventories.entry_reserved_stock')
                         )
                 )
-                ->orWhere(function (Builder $directItemQuery) use ($availableInventory): void {
+                ->orWhere(function (Builder $directItemQuery): void {
                     $directItemQuery
                         ->whereDoesntHave('variants')
-                        ->where(function (Builder $inventoryQuery) use ($availableInventory): void {
+                        ->where(function (Builder $inventoryQuery): void {
                             $inventoryQuery
                                 ->whereNull('catalog_items.inventory_id')
                                 ->orWhereHas(
                                     'inventory',
-                                    $availableInventory
+                                    fn (Builder $availableInventoryQuery): Builder => $availableInventoryQuery
+                                        ->whereRaw('inventories.real_stock > inventories.reserved_stock + inventories.entry_reserved_stock')
                                 );
                         });
                 });

app/Domains/Commerce/Catalog/Models/Inventory.php

@@ -2,17 +2,11 @@
 
 namespace App\Domains\Commerce\Catalog\Models;
 
-use App\Domains\Commerce\Catalog\Enums\InventoryMovementOperation;
-use App\Domains\Core\Auth\Models\User;
 use Illuminate\Database\Eloquent\Attributes\Fillable;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\Relations\HasOne;
-use Illuminate\Support\Facades\Auth;
-use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Schema;
-use Illuminate\Validation\ValidationException;
 
 #[Fillable([
     'sold_units',
@@ -44,7 +38,6 @@ class Inventory extends Model
             'reserved_stock' => 'integer',
             'entry_reserved_stock' => 'integer',
             'real_stock' => 'integer',
-            'available_stock' => 'integer',
         ];
     }
 
@@ -66,295 +59,70 @@ class Inventory extends Model
         return $this->hasMany(StockReservationLine::class);
     }
 
-    /** @return HasMany<InventoryMovement, $this> */
-    public function movements(): HasMany
-    {
-        return $this->hasMany(InventoryMovement::class);
-    }
-
     public function availableStock(): int
     {
-        if (! array_key_exists('available_stock', $this->attributes)) {
+        return max(0, $this->real_stock - $this->reserved_stock - $this->entry_reserved_stock);
-            return max(0, $this->real_stock - $this->reserved_stock - ($this->entry_reserved_stock ?? 0));
-        }
-
-        return (int) $this->getAttribute('available_stock');
     }
 
     public function reserveEntry(int $amount, bool $tracksInventory): void
     {
-        $this->mutate(
+        if ($amount < 1 || ($tracksInventory && $this->availableStock() < $amount)) {
-            InventoryMovementOperation::EntryStockReserved,
-            ['entry_reserved_stock' => $amount],
-            function (self $inventory) use ($amount, $tracksInventory): void {
-                if ($amount < 1 || ($tracksInventory && $inventory->availableStock() < $amount)) {
             throw new \InvalidArgumentException('No hay stock disponible para la reserva de entradas.');
         }
-            },
+        $this->entry_reserved_stock += $amount;
-        );
+        $this->save();
     }
 
     public function releaseEntry(int $amount): void
     {
-        $this->mutate(
+        if ($amount < 1 || $this->entry_reserved_stock < $amount) {
-            InventoryMovementOperation::EntryStockReleased,
-            ['entry_reserved_stock' => -$amount],
-            function (self $inventory) use ($amount): void {
-                if ($amount < 1 || $inventory->entry_reserved_stock < $amount) {
             throw new \InvalidArgumentException('La cantidad de entradas reservadas no es válida.');
         }
-            },
+
-        );
+        $this->entry_reserved_stock -= $amount;
+        $this->save();
     }
 
     public function reserve(int $amount, bool $tracksInventory): void
     {
-        $this->mutate(
-            InventoryMovementOperation::StockReserved,
-            ['reserved_stock' => $amount],
-            function (self $inventory) use ($amount, $tracksInventory): void {
         if ($amount < 0) {
             throw new \InvalidArgumentException('La cantidad a reservar debe ser positiva.');
         }
-                if ($tracksInventory && $inventory->availableStock() < $amount) {
+
+        if ($tracksInventory && $this->availableStock() < $amount) {
             throw new \InvalidArgumentException('No hay suficiente stock disponible para reservar.');
         }
-            },
+
-        );
+        $this->reserved_stock += $amount;
+        $this->save();
     }
 
     public function release(int $amount): void
     {
-        $this->mutate(
+        if ($amount < 0 || $this->reserved_stock < $amount) {
-            InventoryMovementOperation::StockReleased,
-            ['reserved_stock' => -$amount],
-            function (self $inventory) use ($amount): void {
-                if ($amount < 0 || $inventory->reserved_stock < $amount) {
             throw new \InvalidArgumentException('La cantidad reservada no es válida.');
         }
-            },
+
-        );
+        $this->reserved_stock -= $amount;
+        $this->save();
     }
 
     public function buy(int $amount, bool $tracksInventory): void
     {
-        $counterDeltas = ['reserved_stock' => -$amount, 'sold_units' => $amount];
+        if ($amount < 0 || $this->reserved_stock < $amount) {
-        if (! array_key_exists('available_stock', $this->attributes) && $tracksInventory) {
-            $counterDeltas['real_stock'] = -$amount;
-        }
-
-        $this->mutate(
-            InventoryMovementOperation::PurchaseCommitted,
-            $counterDeltas,
-            function (self $inventory) use ($amount, $tracksInventory): void {
-                if ($amount < 0 || $inventory->reserved_stock < $amount) {
             throw new \InvalidArgumentException('La cantidad reservada no alcanza para confirmar la compra.');
         }
-                if ($tracksInventory && $inventory->availableStock() < 0) {
+
+        if ($tracksInventory && $this->real_stock - $this->entry_reserved_stock < $amount) {
             throw new \InvalidArgumentException('No hay suficiente stock real para confirmar la compra.');
         }
-            },
+
-        );
+        if ($tracksInventory) {
+            $this->real_stock -= $amount;
         }
 
-    public function adjustAvailableStock(
+        $this->reserved_stock -= $amount;
-        int $delta,
+        $this->sold_units += $amount;
-        ?User $responsibleUser = null,
-        ?string $idempotencyKey = null,
-    ): void {
-        if ($delta === 0) {
-            return;
-        }
-
-        $operation = $delta > 0
-            ? InventoryMovementOperation::StockIncreased
-            : InventoryMovementOperation::StockDecreased;
-
-        $this->mutate(
-            $operation,
-            ['real_stock' => $delta],
-            function (self $inventory) use ($delta): void {
-                if ($inventory->real_stock + $delta < 0 || $inventory->availableStock() + $delta < 0) {
-                    throw ValidationException::withMessages([
-                        'stock_difference' => ['El ajuste dejaría el stock en un valor inválido.'],
-                    ]);
-                }
-            },
-            $responsibleUser,
-            $idempotencyKey,
-        );
-    }
-
-    public function refundStock(int $amount, ?User $responsibleUser = null): void
-    {
-        $counterDeltas = ['refunded_units' => $amount];
-        if (! array_key_exists('available_stock', $this->attributes)) {
-            $counterDeltas['real_stock'] = $amount;
-        }
-
-        $this->mutate(
-            InventoryMovementOperation::StockRefunded,
-            $counterDeltas,
-            function () use ($amount): void {
-                if ($amount < 1) {
-                    throw new \InvalidArgumentException('La cantidad devuelta debe ser positiva.');
-                }
-            },
-            $responsibleUser,
-        );
-    }
-
-    public function commitDirectSale(int $amount, bool $tracksInventory = true): void
-    {
-        $this->mutate(
-            InventoryMovementOperation::PurchaseCommitted,
-            ['sold_units' => $amount],
-            function (self $inventory) use ($amount, $tracksInventory): void {
-                if ($amount < 1 || ($tracksInventory && $inventory->availableStock() < $amount)) {
-                    throw new \InvalidArgumentException('No hay suficiente stock para confirmar la compra.');
-                }
-            },
-        );
-    }
-
-    public function recordInitialization(?User $responsibleUser = null): void
-    {
-        $this->recordCurrentState(InventoryMovementOperation::StockInitialized, $responsibleUser);
-    }
-
-    public function recordTransferInitialization(?User $responsibleUser = null): void
-    {
-        $this->recordCurrentState(InventoryMovementOperation::InventoryTransferred, $responsibleUser);
-    }
-
-    /** @param array<string, int> $counterDeltas */
-    public function transferCounters(array $counterDeltas, ?User $responsibleUser = null): void
-    {
-        $counterDeltas = array_filter($counterDeltas, fn (int $delta): bool => $delta !== 0);
-        if ($counterDeltas === []) {
-            return;
-        }
-
-        $this->mutate(
-            InventoryMovementOperation::InventoryTransferred,
-            $counterDeltas,
-            function (self $inventory) use ($counterDeltas): void {
-                foreach ($counterDeltas as $counter => $delta) {
-                    if ((int) $inventory->getAttribute($counter) + $delta < 0) {
-                        throw new \LogicException("El traslado dejaría {$counter} en un valor negativo.");
-                    }
-                }
-            },
-            $responsibleUser,
-        );
-    }
-
-    private function recordCurrentState(
-        InventoryMovementOperation $operation,
-        ?User $responsibleUser,
-    ): void {
-        if (! Schema::hasTable('inventory_movements')) {
-            return;
-        }
-
-        $this->refresh();
-        $counterDeltas = array_filter([
-            'real_stock' => $this->real_stock,
-            'sold_units' => $this->sold_units,
-            'reserved_stock' => $this->reserved_stock,
-            'entry_reserved_stock' => $this->entry_reserved_stock,
-            'refunded_units' => $this->refunded_units,
-        ], fn (int $delta): bool => $delta !== 0);
-
-        $this->movements()->create([
-            'operation' => $operation,
-            'available_stock_delta' => $this->availableStock(),
-            'available_stock_before' => 0,
-            'available_stock_after' => $this->availableStock(),
-            'counter_deltas' => $counterDeltas,
-            'responsible_user_id' => $responsibleUser?->getKey() ?? Auth::id(),
-        ]);
-    }
-
-    public function resetTransactionCounters(?User $responsibleUser = null): void
-    {
-        $this->refresh();
-        $counterDeltas = array_filter([
-            'sold_units' => -$this->sold_units,
-            'reserved_stock' => -$this->reserved_stock,
-            'entry_reserved_stock' => -$this->entry_reserved_stock,
-            'refunded_units' => -$this->refunded_units,
-        ], fn (int $delta): bool => $delta !== 0);
-
-        if ($counterDeltas === []) {
-            return;
-        }
-
-        $this->mutate(
-            InventoryMovementOperation::InventoryReset,
-            $counterDeltas,
-            static function (): void {},
-            $responsibleUser,
-        );
-    }
-
-    /**
-     * @param  array<string, int>  $counterDeltas
-     * @param  callable(self): void  $validate
-     */
-    private function mutate(
-        InventoryMovementOperation $operation,
-        array $counterDeltas,
-        callable $validate,
-        ?User $responsibleUser = null,
-        ?string $idempotencyKey = null,
-    ): void {
-        DB::transaction(function () use ($operation, $counterDeltas, $validate, $responsibleUser, $idempotencyKey): void {
-            /** @var self $inventory */
-            $inventory = self::query()->lockForUpdate()->findOrFail($this->getKey());
-            $this->setRawAttributes($inventory->getAttributes(), true);
-
-            $recordsMovements = Schema::hasTable('inventory_movements');
-            if ($recordsMovements && $idempotencyKey !== null) {
-                $existing = $this->movements()
-                    ->where('idempotency_key', $idempotencyKey)
-                    ->first();
-                if ($existing !== null) {
-                    if ($existing->counter_deltas !== $counterDeltas) {
-                        throw ValidationException::withMessages([
-                            'stock_adjustment_id' => ['La operación ya fue utilizada con otro ajuste.'],
-                        ]);
-                    }
-
-                    return;
-                }
-            }
-
-            $validate($this);
-            $counterDeltas = array_filter($counterDeltas, fn (int $delta): bool => $delta !== 0);
-            if ($counterDeltas === []) {
-                return;
-            }
-
-            $availableBefore = $this->availableStock();
-            foreach ($counterDeltas as $counter => $delta) {
-                $this->setAttribute($counter, (int) $this->getAttribute($counter) + $delta);
-            }
         $this->save();
-            $this->refresh();
-
-            $availableAfter = $this->availableStock();
-            if ($recordsMovements) {
-                $this->movements()->create([
-                    'operation' => $operation,
-                    'available_stock_delta' => $availableAfter - $availableBefore,
-                    'available_stock_before' => $availableBefore,
-                    'available_stock_after' => $availableAfter,
-                    'counter_deltas' => $counterDeltas,
-                    'responsible_user_id' => $responsibleUser?->getKey() ?? Auth::id(),
-                    'idempotency_key' => $idempotencyKey,
-                ]);
-            }
-        });
     }
 }

app/Domains/Commerce/Catalog/Models/InventoryMovement.php

@@ -1,45 +0,0 @@
-<?php
-
-namespace App\Domains\Commerce\Catalog\Models;
-
-use App\Domains\Commerce\Catalog\Enums\InventoryMovementOperation;
-use App\Domains\Core\Auth\Models\User;
-use Illuminate\Database\Eloquent\Attributes\Fillable;
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Database\Eloquent\Relations\BelongsTo;
-
-#[Fillable([
-    'inventory_id',
-    'operation',
-    'available_stock_delta',
-    'available_stock_before',
-    'available_stock_after',
-    'counter_deltas',
-    'responsible_user_id',
-    'idempotency_key',
-])]
-class InventoryMovement extends Model
-{
-    protected function casts(): array
-    {
-        return [
-            'operation' => InventoryMovementOperation::class,
-            'available_stock_delta' => 'integer',
-            'available_stock_before' => 'integer',
-            'available_stock_after' => 'integer',
-            'counter_deltas' => 'array',
-        ];
-    }
-
-    /** @return BelongsTo<Inventory, $this> */
-    public function inventory(): BelongsTo
-    {
-        return $this->belongsTo(Inventory::class);
-    }
-
-    /** @return BelongsTo<User, $this> */
-    public function responsibleUser(): BelongsTo
-    {
-        return $this->belongsTo(User::class, 'responsible_user_id');
-    }
-}

app/Domains/Commerce/Catalog/Services/CatalogInventoryService.php

@@ -64,7 +64,6 @@ class CatalogInventoryService
             $selection->loadMissing('variants.inventory');
 
             return $selection->variants
-                ->each(fn (Variant $variant) => $variant->setRelation('catalogItem', $selection))
                 ->filter(fn (Variant $variant): bool => $variant->isSellable())
                 ->unique(fn (Variant $variant): string => $variant->inventory_id === null
                     ? 'object:'.spl_object_id($variant->inventory)
@@ -154,7 +153,7 @@ class CatalogInventoryService
 
                 if ($operation === 'commit'
                     && $requirement['tracks_inventory']
-                    && $inventory->availableStock() < 0) {
+                    && $inventory->real_stock - $inventory->entry_reserved_stock < $requiredQuantity) {
                     throw new \InvalidArgumentException('No hay suficiente stock real para confirmar la compra.');
                 }
             }

app/Domains/Commerce/Catalog/Services/CatalogService.php

@@ -370,12 +370,9 @@ class CatalogService
 
     private function createInventory(int $realStock): Inventory
     {
-        $inventory = Inventory::query()->create([
+        return Inventory::query()->create([
             'real_stock' => $realStock,
         ]);
-        $inventory->recordInitialization();
-
-        return $inventory;
     }
 
     /**

app/Domains/Commerce/Catalog/Services/StockReservationService.php

@@ -225,7 +225,7 @@ class StockReservationService
                 $inventory = $inventories->get($line->inventory_id)
                     ?? throw new \InvalidArgumentException('No se encontró el inventario reservado.');
                 if ($inventory->reserved_stock < $line->quantity
-                    || ($line->tracks_inventory && $inventory->availableStock() < 0)) {
+                    || ($line->tracks_inventory && $inventory->real_stock - $inventory->entry_reserved_stock < $line->quantity)) {
                     throw new \InvalidArgumentException('La reserva de stock no alcanza para confirmar la compra.');
                 }
             }

app/Domains/Commerce/Catalog/Services/VariantReplacementService.php

@@ -194,7 +194,6 @@ class VariantReplacementService
             'entry_reserved_stock' => $sourceInventory->entry_reserved_stock,
             'real_stock' => $sourceInventory->real_stock,
         ]);
-        $replacementInventory->recordTransferInitialization();
 
         if ($activeLines->isNotEmpty()) {
             StockReservationLine::query()
@@ -203,10 +202,7 @@ class VariantReplacementService
         }
         EntryReservation::query()->where('inventory_id', $sourceInventory->id)
             ->update(['inventory_id' => $replacementInventory->id]);
-        $sourceInventory->transferCounters([
+        $sourceInventory->update(['reserved_stock' => 0, 'entry_reserved_stock' => 0]);
-            'reserved_stock' => -$sourceInventory->reserved_stock,
-            'entry_reserved_stock' => -$sourceInventory->entry_reserved_stock,
-        ]);
 
         return $replacementInventory;
     }
@@ -240,14 +236,13 @@ class VariantReplacementService
             throw new \LogicException('El inventario reservado de la variante es inconsistente.');
         }
 
-        $transferredCounters = [
+        $destinationInventory->update([
-            'real_stock' => $sourceInventory->real_stock,
+            'real_stock' => $destinationInventory->real_stock + $sourceInventory->real_stock,
-            'reserved_stock' => $sourceInventory->reserved_stock,
+            'reserved_stock' => $destinationInventory->reserved_stock + $sourceInventory->reserved_stock,
-            'entry_reserved_stock' => $sourceInventory->entry_reserved_stock,
+            'entry_reserved_stock' => $destinationInventory->entry_reserved_stock + $sourceInventory->entry_reserved_stock,
-            'sold_units' => $sourceInventory->sold_units,
+            'sold_units' => $destinationInventory->sold_units + $sourceInventory->sold_units,
-            'refunded_units' => $sourceInventory->refunded_units,
+            'refunded_units' => $destinationInventory->refunded_units + $sourceInventory->refunded_units,
-        ];
+        ]);
-        $destinationInventory->transferCounters($transferredCounters);
         if ($activeLines->isNotEmpty()) {
             StockReservationLine::query()
                 ->whereKey($activeLines->modelKeys())
@@ -255,10 +250,13 @@ class VariantReplacementService
         }
         EntryReservation::query()->where('inventory_id', $sourceInventory->id)
             ->update(['inventory_id' => $destinationInventory->id]);
-        $sourceInventory->transferCounters(array_map(
+        $sourceInventory->update([
-            fn (int $value): int => -$value,
+            'real_stock' => 0,
-            $transferredCounters,
+            'reserved_stock' => 0,
-        ));
+            'entry_reserved_stock' => 0,
+            'sold_units' => 0,
+            'refunded_units' => 0,
+        ]);
     }
 
     /** @return list<string> */

app/Domains/Commerce/Purchase/Services/TenantTransactionResetService.php

@@ -2,7 +2,6 @@
 
 namespace App\Domains\Commerce\Purchase\Services;
 
-use App\Domains\Commerce\Catalog\Models\Inventory;
 use App\Domains\Commerce\Purchase\Models\Purchase;
 use Illuminate\Database\Query\Builder;
 use Illuminate\Support\Collection;
@@ -70,12 +69,15 @@ class TenantTransactionResetService
                 'users_preserved' => DB::table('users')->where('tenant_codigo', $tenantCode)->count(),
             ];
 
-            Inventory::query()
+            DB::table('inventories')
-                ->whereKey($scope['inventory_ids'])
+                ->whereIn('id', $scope['inventory_ids'])
-                ->orderBy('id')
+                ->update([
-                ->lockForUpdate()
+                    'real_stock' => DB::raw('real_stock + sold_units - refunded_units'),
-                ->get()
+                    'reserved_stock' => 0,
-                ->each(fn (Inventory $inventory) => $inventory->resetTransactionCounters());
+                    'entry_reserved_stock' => 0,
+                    'sold_units' => 0,
+                    'refunded_units' => 0,
+                ]);
 
             return $summary;
         });

app/Domains/Ticketing/Desfile/Services/InvitationPurchaseProvisioner.php

@@ -2,7 +2,6 @@
 
 namespace App\Domains\Ticketing\Desfile\Services;
 
-use App\Domains\Commerce\Catalog\Models\Inventory;
 use DateTimeInterface;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Hash;
@@ -394,22 +393,14 @@ class InvitationPurchaseProvisioner
 
         $inventory = DB::table('inventories')->where('id', $variant->inventory_id)->lockForUpdate()->first();
 
-        $available = $inventory?->available_stock
+        if ($inventory === null || $inventory->real_stock < 1 || $inventory->reserved_stock > 0 || ($inventory->entry_reserved_stock ?? 0) > 0) {
-            ?? ($inventory === null ? 0 : $inventory->real_stock - $inventory->reserved_stock - ($inventory->entry_reserved_stock ?? 0));
-        if ($available < 1) {
             throw new RuntimeException("El asiento {$variant->descripcion} ya no está disponible.");
         }
 
-        if (property_exists($inventory, 'available_stock')) {
-            Inventory::query()->findOrFail($inventory->id)->commitDirectSale(1);
-        } else {
-            // This provisioner is also used by historical migrations that run
-            // before generated availability and the movement ledger exist.
         DB::table('inventories')->where('id', $inventory->id)->update([
             'real_stock' => $inventory->real_stock - 1,
             'sold_units' => $inventory->sold_units + 1,
         ]);
-        }
 
         $reservationId = DB::table('compras')->where('id', $purchaseId)->value('stock_reservation_id');
         if ($reservationId === null) {

app/Domains/Ticketing/FiestaFutbolInfantil/Controllers/AccommodationController.php

@@ -28,7 +28,6 @@ class AccommodationController extends Controller
             $this->accommodationService->upsertMany(
                 $tenant,
                 $request->validated('variants'),
-                $request->validated('stock_adjustment_id'),
             )
         );
     }

app/Domains/Ticketing/FiestaFutbolInfantil/Controllers/EntryController.php

@@ -29,7 +29,6 @@ class EntryController extends Controller
         $entries = $this->entryService->upsertMany(
             $tenant,
             $request->validated('entries'),
-            $request->validated('stock_adjustment_id'),
         );
 
         return EntryResource::collection($entries)

app/Domains/Ticketing/FiestaFutbolInfantil/Controllers/FoodController.php

@@ -29,7 +29,6 @@ class FoodController extends Controller
             $this->foodService->upsertMany(
                 $tenant,
                 $request->validated('variants'),
-                $request->validated('stock_adjustment_id'),
             )
         );
     }
@@ -40,7 +39,6 @@ class FoodController extends Controller
             $this->foodService->updateHistoricalStock(
                 $request->user()->tenant()->firstOrFail(),
                 $request->validated('variants'),
-                $request->validated('stock_adjustment_id'),
             )
         );
     }

app/Domains/Ticketing/FiestaFutbolInfantil/Controllers/MerchandiseController.php

@@ -28,7 +28,6 @@ class MerchandiseController extends Controller
         $items = $this->merchandiseService->upsertMany(
             $tenant,
             $request->validated('items'),
-            $request->validated('stock_adjustment_id'),
         );
 
         return MerchandiseResource::collection($items)

app/Domains/Ticketing/FiestaFutbolInfantil/Requests/UpdateHistoricalFoodStockRequest.php

@@ -16,10 +16,9 @@ class UpdateHistoricalFoodStockRequest extends FormRequest
     {
         return [
             'variants' => ['required', 'array', 'min:1', 'max:500'],
-            'stock_adjustment_id' => ['nullable', 'uuid'],
+            'variants.*' => ['required', 'array:id,stock'],
-            'variants.*' => ['required', 'array:id,stock_difference'],
             'variants.*.id' => ['required', 'integer', 'distinct'],
-            'variants.*.stock_difference' => ['required', 'integer'],
+            'variants.*.stock' => ['required', 'integer', 'min:0'],
         ];
     }
 }

app/Domains/Ticketing/FiestaFutbolInfantil/Requests/UpsertAccommodationVariantsRequest.php

@@ -3,7 +3,6 @@
 namespace App\Domains\Ticketing\FiestaFutbolInfantil\Requests;
 
 use Illuminate\Foundation\Http\FormRequest;
-use Illuminate\Validation\Validator;
 
 class UpsertAccommodationVariantsRequest extends FormRequest
 {
@@ -17,42 +16,12 @@ class UpsertAccommodationVariantsRequest extends FormRequest
     {
         return [
             'variants' => ['required', 'array', 'min:1', 'max:500'],
-            'stock_adjustment_id' => ['nullable', 'uuid'],
+            'variants.*' => ['required', 'array:id,title,description,stock,price'],
-            'variants.*' => ['required', 'array:id,title,description,stock,stock_difference,price'],
             'variants.*.id' => ['sometimes', 'nullable', 'integer', 'distinct'],
             'variants.*.title' => ['required', 'string', 'max:255'],
             'variants.*.description' => ['sometimes', 'nullable', 'string'],
-            'variants.*.stock' => ['sometimes', 'integer', 'min:0'],
+            'variants.*.stock' => ['required', 'integer', 'min:0'],
-            'variants.*.stock_difference' => ['sometimes', 'integer'],
             'variants.*.price' => ['required', 'numeric', 'min:0', 'max:99999999.99'],
         ];
     }
-
-    /** @return array<int, callable> */
-    public function after(): array
-    {
-        return [
-            function (Validator $validator): void {
-                foreach ($this->input('variants', []) as $index => $variant) {
-                    if (! is_array($variant)) {
-                        continue;
-                    }
-
-                    if (isset($variant['id']) && ! array_key_exists('stock_difference', $variant)) {
-                        $validator->errors()->add(
-                            "variants.{$index}.stock_difference",
-                            'La diferencia de stock es obligatoria al actualizar una variante.',
-                        );
-                    }
-
-                    if (! isset($variant['id']) && ! array_key_exists('stock', $variant)) {
-                        $validator->errors()->add(
-                            "variants.{$index}.stock",
-                            'El stock inicial es obligatorio al crear una variante.',
-                        );
-                    }
-                }
-            },
-        ];
-    }
 }

app/Domains/Ticketing/FiestaFutbolInfantil/Requests/UpsertEntriesRequest.php

@@ -20,8 +20,7 @@ class UpsertEntriesRequest extends FormRequest
 
         return [
             'entries' => ['required', 'array', 'min:1', 'max:100'],
-            'stock_adjustment_id' => ['nullable', 'uuid'],
+            'entries.*' => ['required', 'array:id,title,description,event_date_ids,stock,price'],
-            'entries.*' => ['required', 'array:id,title,description,event_date_ids,stock,stock_difference,price'],
             'entries.*.id' => [
                 'sometimes',
                 'nullable',
@@ -47,8 +46,7 @@ class UpsertEntriesRequest extends FormRequest
                     fn ($query) => $query->where('tenant_code', $tenantCode)
                 ),
             ],
-            'entries.*.stock' => ['sometimes', 'integer', 'min:0'],
+            'entries.*.stock' => ['required', 'integer', 'min:0'],
-            'entries.*.stock_difference' => ['sometimes', 'integer'],
             'entries.*.price' => ['required', 'numeric', 'min:0', 'max:99999999.99'],
         ];
     }
@@ -63,20 +61,6 @@ class UpsertEntriesRequest extends FormRequest
                         continue;
                     }
 
-                    if (isset($entry['id']) && ! array_key_exists('stock_difference', $entry)) {
-                        $validator->errors()->add(
-                            "entries.{$index}.stock_difference",
-                            'La diferencia de stock es obligatoria al actualizar una entrada.',
-                        );
-                    }
-
-                    if (! isset($entry['id']) && ! array_key_exists('stock', $entry)) {
-                        $validator->errors()->add(
-                            "entries.{$index}.stock",
-                            'El stock inicial es obligatorio al crear una entrada.',
-                        );
-                    }
-
                     $dateIds = $entry['event_date_ids'] ?? [];
 
                     if (! is_array($dateIds)) {

app/Domains/Ticketing/FiestaFutbolInfantil/Requests/UpsertFoodVariantsRequest.php

@@ -20,8 +20,7 @@ class UpsertFoodVariantsRequest extends FormRequest
 
         return [
             'variants' => ['required', 'array', 'min:1', 'max:500'],
-            'stock_adjustment_id' => ['nullable', 'uuid'],
+            'variants.*' => ['required', 'array:id,event_date_id,schedule,service,description,stock,price'],
-            'variants.*' => ['required', 'array:id,event_date_id,schedule,service,description,stock,stock_difference,price'],
             'variants.*.id' => ['sometimes', 'nullable', 'integer', 'distinct'],
             'variants.*.event_date_id' => [
                 'required',
@@ -33,8 +32,7 @@ class UpsertFoodVariantsRequest extends FormRequest
             'variants.*.schedule' => ['required', 'string', 'max:255'],
             'variants.*.service' => ['required', 'string', 'max:255'],
             'variants.*.description' => ['sometimes', 'nullable', 'string'],
-            'variants.*.stock' => ['sometimes', 'integer', 'min:0'],
+            'variants.*.stock' => ['required', 'integer', 'min:0'],
-            'variants.*.stock_difference' => ['sometimes', 'integer'],
             'variants.*.price' => ['required', 'numeric', 'min:0', 'max:99999999.99'],
         ];
     }
@@ -51,20 +49,6 @@ class UpsertFoodVariantsRequest extends FormRequest
                         continue;
                     }
 
-                    if (isset($variant['id']) && ! array_key_exists('stock_difference', $variant)) {
-                        $validator->errors()->add(
-                            "variants.{$index}.stock_difference",
-                            'La diferencia de stock es obligatoria al actualizar una variante.',
-                        );
-                    }
-
-                    if (! isset($variant['id']) && ! array_key_exists('stock', $variant)) {
-                        $validator->errors()->add(
-                            "variants.{$index}.stock",
-                            'El stock inicial es obligatorio al crear una variante.',
-                        );
-                    }
-
                     $key = implode('|', [
                         $variant['event_date_id'] ?? '',
                         mb_strtolower(trim((string) ($variant['schedule'] ?? ''))),

app/Domains/Ticketing/FiestaFutbolInfantil/Requests/UpsertMerchandiseRequest.php

@@ -4,7 +4,6 @@ namespace App\Domains\Ticketing\FiestaFutbolInfantil\Requests;
 
 use Illuminate\Foundation\Http\FormRequest;
 use Illuminate\Validation\Rule;
-use Illuminate\Validation\Validator;
 
 class UpsertMerchandiseRequest extends FormRequest
 {
@@ -40,49 +39,12 @@ class UpsertMerchandiseRequest extends FormRequest
             'items.*.description' => ['sometimes', 'nullable', 'string'],
             'items.*.max_units_per_user' => ['required', 'integer', 'min:1'],
             'items.*.variants' => ['required', 'array', 'min:1', 'max:500'],
-            'stock_adjustment_id' => ['nullable', 'uuid'],
+            'items.*.variants.*' => ['required', 'array:id,color,size,stock,price'],
-            'items.*.variants.*' => ['required', 'array:id,color,size,stock,stock_difference,price'],
             'items.*.variants.*.id' => ['sometimes', 'nullable', 'integer', 'distinct'],
             'items.*.variants.*.color' => ['required', 'string', 'max:255'],
             'items.*.variants.*.size' => ['required', 'string', 'max:255'],
-            'items.*.variants.*.stock' => ['sometimes', 'integer', 'min:0'],
+            'items.*.variants.*.stock' => ['required', 'integer', 'min:0'],
-            'items.*.variants.*.stock_difference' => ['sometimes', 'integer'],
             'items.*.variants.*.price' => ['required', 'numeric', 'min:0', 'max:99999999.99'],
         ];
     }
-
-    /** @return array<int, callable> */
-    public function after(): array
-    {
-        return [
-            function (Validator $validator): void {
-                foreach ($this->input('items', []) as $itemIndex => $item) {
-                    if (! is_array($item)) {
-                        continue;
-                    }
-
-                    foreach ($item['variants'] ?? [] as $variantIndex => $variant) {
-                        if (! is_array($variant)) {
-                            continue;
-                        }
-
-                        $prefix = "items.{$itemIndex}.variants.{$variantIndex}";
-                        if (isset($variant['id']) && ! array_key_exists('stock_difference', $variant)) {
-                            $validator->errors()->add(
-                                "{$prefix}.stock_difference",
-                                'La diferencia de stock es obligatoria al actualizar una variante.',
-                            );
-                        }
-
-                        if (! isset($variant['id']) && ! array_key_exists('stock', $variant)) {
-                            $validator->errors()->add(
-                                "{$prefix}.stock",
-                                'El stock inicial es obligatorio al crear una variante.',
-                            );
-                        }
-                    }
-                }
-            },
-        ];
-    }
 }

app/Domains/Ticketing/FiestaFutbolInfantil/Resources/AccommodationResource.php

@@ -37,7 +37,7 @@ class AccommodationResource extends JsonResource
                     'title' => $options->get($value)?->label ?? $value,
                     'value' => $value,
                     'description' => $variant->descripcion,
-                    'stock' => $variant->inventory->availableStock(),
+                    'stock' => $variant->inventory->real_stock,
                     'price' => number_format($variant->getPrice(), 2, '.', ''),
                 ];
             })->values(),

app/Domains/Ticketing/FiestaFutbolInfantil/Resources/EntryResource.php

@@ -33,7 +33,7 @@ class EntryResource extends JsonResource
             'title' => $this->nombre,
             'description' => $this->descripcion,
             'event_date_ids' => $variant->selectedEventDates()->pluck('id')->values(),
-            'stock' => $variant->inventory->availableStock(),
+            'stock' => $variant->inventory->real_stock,
             'price' => $this->precio,
         ];
     }

app/Domains/Ticketing/FiestaFutbolInfantil/Resources/FoodResource.php

@@ -81,7 +81,7 @@ class FoodResource extends JsonResource
             'schedule' => $values->get('horario'),
             'service' => $values->get('servicio'),
             'description' => $variant->descripcion,
-            'stock' => $variant->inventory->availableStock(),
+            'stock' => $variant->inventory->real_stock,
             'price' => number_format($variant->getPrice(), 2, '.', ''),
         ];
     }

app/Domains/Ticketing/FiestaFutbolInfantil/Resources/MerchandiseResource.php

@@ -44,7 +44,7 @@ class MerchandiseResource extends JsonResource
                     'color_value' => $colorValue,
                     'size' => $sizeOptions->get($sizeValue)?->label ?? $sizeValue,
                     'size_value' => $sizeValue,
-                    'stock' => $variant->inventory->availableStock(),
+                    'stock' => $variant->inventory->real_stock,
                     'price' => number_format($variant->getPrice(), 2, '.', ''),
                 ];
             })->values(),

app/Domains/Ticketing/FiestaFutbolInfantil/Services/AccommodationService.php

@@ -39,9 +39,9 @@ class AccommodationService
     /**
      * @param  array<int, array<string, mixed>>  $variants
      */
-    public function upsertMany(Tenant $tenant, array $variants, ?string $stockAdjustmentId = null): CatalogItem
+    public function upsertMany(Tenant $tenant, array $variants): CatalogItem
     {
-        return DB::transaction(function () use ($tenant, $variants, $stockAdjustmentId): CatalogItem {
+        return DB::transaction(function () use ($tenant, $variants): CatalogItem {
             $attribute = $this->attribute($tenant);
             $accommodation = $this->accommodation($tenant, $variants);
             $itemAttribute = $accommodation->itemAttributes()->firstOrCreate(
@@ -70,7 +70,7 @@ class AccommodationService
                 if ($variant === null) {
                     $this->createVariant($attribute, $accommodation, $itemAttribute, $data);
                 } else {
-                    $this->updateVariant($attribute, $variant, $itemAttribute, $data, $index, $stockAdjustmentId);
+                    $this->updateVariant($attribute, $variant, $itemAttribute, $data, $index);
                 }
             }
 
@@ -169,7 +169,7 @@ class AccommodationService
             'title' => trim($variant['title']),
             'value' => $this->valueCode($variant['title']),
             'description' => $variant['description'] ?? null,
-            ...(array_key_exists('stock', $variant) ? ['stock' => (int) $variant['stock']] : []),
+            'stock' => (int) $variant['stock'],
         ])->all();
     }
 
@@ -212,7 +212,6 @@ class AccommodationService
         $this->createOption($attribute, $data['value'], $data['title']);
 
         $inventory = Inventory::query()->create(['real_stock' => $data['stock']]);
-        $inventory->recordInitialization();
         $variant = $accommodation->variants()->create([
             'inventory_id' => $inventory->id,
             'descripcion' => $data['description'],
@@ -231,13 +230,20 @@ class AccommodationService
         ItemAttribute $itemAttribute,
         array $data,
         int $index,
-        ?string $stockAdjustmentId,
     ): void {
         $inventory = Inventory::query()
             ->whereKey($variant->inventory_id)
             ->lockForUpdate()
             ->firstOrFail();
 
+        if ($data['stock'] < $inventory->reserved_stock) {
+            throw ValidationException::withMessages([
+                "variants.{$index}.stock" => [
+                    'El stock no puede ser menor que la cantidad actualmente reservada.',
+                ],
+            ]);
+        }
+
         $definition = $variant->definitions
             ->firstWhere('item_attribute_id', $itemAttribute->id);
         $option = $definition === null
@@ -257,10 +263,7 @@ class AccommodationService
             'descripcion' => $data['description'],
             'precio' => $data['price'],
         ]);
-        $inventory->adjustAvailableStock(
+        $inventory->update(['real_stock' => $data['stock']]);
-            (int) $data['stock_difference'],
-            idempotencyKey: $stockAdjustmentId,
-        );
         $variant->definitions()->updateOrCreate(
             ['item_attribute_id' => $itemAttribute->id],
             ['value' => $data['value']],

app/Domains/Ticketing/FiestaFutbolInfantil/Services/EntryService.php

@@ -43,18 +43,18 @@ class EntryService
      * @param  array<int, array<string, mixed>>  $entries
      * @return Collection<int, CatalogItem>
      */
-    public function upsertMany(Tenant $tenant, array $entries, ?string $stockAdjustmentId = null): Collection
+    public function upsertMany(Tenant $tenant, array $entries): Collection
     {
-        return DB::transaction(function () use ($tenant, $entries, $stockAdjustmentId): Collection {
+        return DB::transaction(function () use ($tenant, $entries): Collection {
             $reservedSlugs = [];
             $category = Category::query()->firstOrCreate([
                 'tenant_code' => $tenant->codigo,
                 'nombre' => 'Entradas',
             ]);
 
-            return collect($entries)->map(function (array $entry, int $index) use ($tenant, $category, $stockAdjustmentId, &$reservedSlugs): CatalogItem {
+            return collect($entries)->map(function (array $entry, int $index) use ($tenant, $category, &$reservedSlugs): CatalogItem {
                 if (isset($entry['id'])) {
-                    return $this->update($tenant, $category, $entry, $index, $stockAdjustmentId);
+                    return $this->update($tenant, $category, $entry, $index);
                 }
 
                 $slug = $this->uniqueSlug($tenant, $entry['title'], $reservedSlugs);
@@ -92,7 +92,7 @@ class EntryService
     }
 
     /** @param array<string, mixed> $entry */
-    private function update(Tenant $tenant, Category $category, array $entry, int $index, ?string $stockAdjustmentId): CatalogItem
+    private function update(Tenant $tenant, Category $category, array $entry, int $index): CatalogItem
     {
         $catalogItem = CatalogItem::query()
             ->whereKey($entry['id'])
@@ -121,6 +121,14 @@ class EntryService
             ->lockForUpdate()
             ->firstOrFail();
 
+        if ((int) $entry['stock'] < $inventory->reserved_stock) {
+            throw ValidationException::withMessages([
+                "entries.{$index}.stock" => [
+                    'El stock no puede ser menor que la cantidad actualmente reservada.',
+                ],
+            ]);
+        }
+
         $eventDateIds = collect($entry['event_date_ids'])
             ->map(fn ($id): int => (int) $id)
             ->unique()
@@ -141,10 +149,7 @@ class EntryService
         $catalogItem->itemAttributes()
             ->whereHas('attribute', fn ($query) => $query->where('codigo', 'event_date'))
             ->update(['allow_multi_select' => true]);
-        $inventory->adjustAvailableStock(
+        $inventory->update(['real_stock' => $entry['stock']]);
-            (int) $entry['stock_difference'],
-            idempotencyKey: $stockAdjustmentId,
-        );
 
         return $catalogItem->load([
             'variants.inventory',

app/Domains/Ticketing/FiestaFutbolInfantil/Services/FoodService.php

@@ -11,9 +11,9 @@ use App\Domains\Commerce\Catalog\Models\Inventory;
 use App\Domains\Commerce\Catalog\Models\ItemAttribute;
 use App\Domains\Commerce\Catalog\Models\Variant;
 use App\Domains\Commerce\Catalog\Services\CatalogService;
-use App\Domains\Core\Tenant\Models\Tenant;
 use App\Domains\Ticketing\Event\Enums\EventDateStatus;
 use App\Domains\Ticketing\Event\Models\EventDate;
+use App\Domains\Core\Tenant\Models\Tenant;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Validation\ValidationException;
@@ -50,9 +50,9 @@ class FoodService
     /**
      * @param  array<int, array<string, mixed>>  $variants
      */
-    public function upsertMany(Tenant $tenant, array $variants, ?string $stockAdjustmentId = null): CatalogItem
+    public function upsertMany(Tenant $tenant, array $variants): CatalogItem
     {
-        return DB::transaction(function () use ($tenant, $variants, $stockAdjustmentId): CatalogItem {
+        return DB::transaction(function () use ($tenant, $variants): CatalogItem {
             $attributes = $this->attributes($tenant);
             $food = $this->food($tenant, $variants);
             $itemAttributes = $this->itemAttributes($food, $attributes);
@@ -85,7 +85,7 @@ class FoodService
                 if ($variant === null) {
                     $this->createVariant($food, $itemAttributes, $data);
                 } else {
-                    $this->updateVariant($variant, $itemAttributes, $data, $index, $stockAdjustmentId);
+                    $this->updateVariant($variant, $itemAttributes, $data, $index);
                 }
             }
 
@@ -113,11 +113,11 @@ class FoodService
     }
 
     /**
-     * @param  array<int, array{id: int, stock_difference: int}>  $variants
+     * @param  array<int, array{id: int, stock: int}>  $variants
      */
-    public function updateHistoricalStock(Tenant $tenant, array $variants, ?string $stockAdjustmentId = null): CatalogItem
+    public function updateHistoricalStock(Tenant $tenant, array $variants): CatalogItem
     {
-        return DB::transaction(function () use ($tenant, $variants, $stockAdjustmentId): CatalogItem {
+        return DB::transaction(function () use ($tenant, $variants): CatalogItem {
             $food = CatalogItem::query()
                 ->forTenantCatalog($tenant)
                 ->where('slug', 'comida')
@@ -142,11 +142,17 @@ class FoodService
                     ]);
                 }
 
+                $stock = (int) $data['stock'];
                 $inventory = $this->inventoryForHistoricalStockUpdate($variant);
-                $inventory->adjustAvailableStock(
+                if ($stock < $inventory->reserved_stock) {
-                    (int) $data['stock_difference'],
+                    throw ValidationException::withMessages([
-                    idempotencyKey: $stockAdjustmentId,
+                        "variants.{$index}.stock" => [
-                );
+                            'El stock no puede ser menor que la cantidad actualmente reservada.',
+                        ],
+                    ]);
+                }
+
+                $inventory->update(['real_stock' => $stock]);
             }
 
             return $this->current($tenant) ?? $food;
@@ -175,7 +181,6 @@ class FoodService
             'reserved_stock' => 0,
             'real_stock' => $inventory->real_stock,
         ]);
-        $historicalInventory->recordTransferInitialization();
         $variant->update(['inventory_id' => $historicalInventory->getKey()]);
 
         return $historicalInventory;
@@ -296,7 +301,7 @@ class FoodService
                 'schedule' => $schedule->value,
                 'service' => $service->value,
                 'description' => (string) ($variant['description'] ?? ''),
-                ...(array_key_exists('stock', $variant) ? ['stock' => (int) $variant['stock']] : []),
+                'stock' => (int) $variant['stock'],
             ];
         })->all();
     }
@@ -379,7 +384,6 @@ class FoodService
     private function createVariant(CatalogItem $food, Collection $itemAttributes, array $data): void
     {
         $inventory = Inventory::query()->create(['real_stock' => $data['stock']]);
-        $inventory->recordInitialization();
         $variant = $food->variants()->create([
             'event_date_id' => $data['event_date_id'],
             'inventory_id' => $inventory->id,
@@ -396,23 +400,27 @@ class FoodService
         Collection $itemAttributes,
         array $data,
         int $index,
-        ?string $stockAdjustmentId,
     ): void {
         $inventory = Inventory::query()
             ->whereKey($variant->inventory_id)
             ->lockForUpdate()
             ->firstOrFail();
 
+        if ($data['stock'] < $inventory->reserved_stock) {
+            throw ValidationException::withMessages([
+                "variants.{$index}.stock" => [
+                    'El stock no puede ser menor que la cantidad actualmente reservada.',
+                ],
+            ]);
+        }
+
         $variant->update([
             'event_date_id' => $data['event_date_id'],
             'descripcion' => $data['description'],
             'precio' => $data['price'],
         ]);
         $variant->eventDates()->sync([$data['event_date_id']]);
-        $inventory->adjustAvailableStock(
+        $inventory->update(['real_stock' => $data['stock']]);
-            (int) $data['stock_difference'],
-            idempotencyKey: $stockAdjustmentId,
-        );
         $this->syncDefinitions($variant, $itemAttributes, $data);
     }
 

app/Domains/Ticketing/FiestaFutbolInfantil/Services/MerchandiseService.php

@@ -43,9 +43,9 @@ class MerchandiseService
      * @param  array<int, array<string, mixed>>  $items
      * @return Collection<int, CatalogItem>
      */
-    public function upsertMany(Tenant $tenant, array $items, ?string $stockAdjustmentId = null): Collection
+    public function upsertMany(Tenant $tenant, array $items): Collection
     {
-        return DB::transaction(function () use ($tenant, $items, $stockAdjustmentId): Collection {
+        return DB::transaction(function () use ($tenant, $items): Collection {
             $attributes = $this->attributes($tenant);
             $category = Category::query()->firstOrCreate([
                 'tenant_code' => $tenant->codigo,
@@ -57,7 +57,6 @@ class MerchandiseService
                 $tenant,
                 $attributes,
                 $category,
-                $stockAdjustmentId,
                 &$reservedSlugs,
             ): CatalogItem {
                 $item = isset($data['id'])
@@ -102,7 +101,7 @@ class MerchandiseService
                     if ($variant === null) {
                         $this->createVariant($item, $itemAttributes, $variantData);
                     } else {
-                        $this->updateVariant($variant, $itemAttributes, $variantData, $index, $variantIndex, $stockAdjustmentId);
+                        $this->updateVariant($variant, $itemAttributes, $variantData, $index, $variantIndex);
                     }
                 }
 
@@ -245,7 +244,7 @@ class MerchandiseService
                 ...$variant,
                 'color' => $color->value,
                 'size' => $size->value,
-                ...(array_key_exists('stock', $variant) ? ['stock' => (int) $variant['stock']] : []),
+                'stock' => (int) $variant['stock'],
             ];
         })->all();
     }
@@ -343,7 +342,6 @@ class MerchandiseService
         array $data,
     ): void {
         $inventory = Inventory::query()->create(['real_stock' => $data['stock']]);
-        $inventory->recordInitialization();
         $variant = $item->variants()->create([
             'inventory_id' => $inventory->id,
             'precio' => $data['price'],
@@ -361,18 +359,22 @@ class MerchandiseService
         array $data,
         int $itemIndex,
         int $variantIndex,
-        ?string $stockAdjustmentId,
     ): void {
         $inventory = Inventory::query()
             ->whereKey($variant->inventory_id)
             ->lockForUpdate()
             ->firstOrFail();
 
+        if ($data['stock'] < $inventory->reserved_stock) {
+            throw ValidationException::withMessages([
+                "items.{$itemIndex}.variants.{$variantIndex}.stock" => [
+                    'El stock no puede ser menor que la cantidad actualmente reservada.',
+                ],
+            ]);
+        }
+
         $variant->update(['precio' => $data['price']]);
-        $inventory->adjustAvailableStock(
+        $inventory->update(['real_stock' => $data['stock']]);
-            (int) $data['stock_difference'],
-            idempotencyKey: $stockAdjustmentId,
-        );
         $this->syncDefinitions($variant, $itemAttributes, $data);
     }
 

app/Domains/Ticketing/Ticket/Services/AdminAppTicketService.php

@@ -228,13 +228,13 @@ class AdminAppTicketService
                 'amount' => number_format($refundAmount, 2, '.', ''),
             ]);
 
-            $this->restoreInventory($ticket, $purchaseItem, $createdBy);
+            $this->restoreInventory($ticket, $purchaseItem);
 
             return $ticket->refresh()->load(self::RELATIONS);
         });
     }
 
-    private function restoreInventory(Ticket $ticket, PurchaseItem $purchaseItem, ?User $createdBy): void
+    private function restoreInventory(Ticket $ticket, PurchaseItem $purchaseItem): void
     {
         $catalogItem = $ticket->sourceCatalogItem;
         if ($catalogItem === null) {
@@ -272,7 +272,11 @@ class AdminAppTicketService
             throw ValidationException::withMessages(['ticket' => 'No se encontró el inventario del ticket.']);
         }
 
-        $inventory->refundStock(1, $createdBy);
+        if ($catalogItem->inventory_policy === InventoryPolicy::Tracked) {
+            $inventory->real_stock++;
+        }
+        $inventory->refunded_units++;
+        $inventory->save();
     }
 
     private function refundedAmountForPurchaseItem(PurchaseItem $purchaseItem): float

app/Domains/Ticketing/Ticket/Services/BackfillRefundedUnitsService.php

@@ -2,7 +2,6 @@
 
 namespace App\Domains\Ticketing\Ticket\Services;
 
-use App\Domains\Commerce\Catalog\Models\Inventory;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Log;
 use RuntimeException;
@@ -65,9 +64,14 @@ class BackfillRefundedUnitsService
                 }, 'refunds.id', 'id');
 
             foreach ($counts as $inventoryId => $count) {
-                $inventory = Inventory::query()->lockForUpdate()->find($inventoryId)
+                $updates = ['refunded_units' => DB::raw('refunded_units + '.$count['refunded'])];
-                    ?? throw new RuntimeException("No se encontró el inventario {$inventoryId} para reponerlo.");
+                if (($count['stock'] ?? 0) > 0) {
-                $inventory->refundStock($count['refunded']);
+                    $updates['real_stock'] = DB::raw('real_stock + '.$count['stock']);
+                }
+
+                if (DB::table('inventories')->where('id', $inventoryId)->update($updates) !== 1) {
+                    throw new RuntimeException("No se encontró el inventario {$inventoryId} para reponerlo.");
+                }
             }
 
             $refundedUnitsAdded = array_sum(array_column($counts, 'refunded'));
@@ -78,8 +82,7 @@ class BackfillRefundedUnitsService
                 'bundles_skipped' => $bundlesSkipped,
                 'inventories_updated' => count($counts),
                 'refunded_units_added' => $refundedUnitsAdded,
-                'real_stock_added' => 0,
+                'real_stock_added' => array_sum(array_column($counts, 'stock')),
-                'available_stock_added' => $refundedUnitsAdded,
             ];
         });
 

database/migrations/2026_08_26_000000_set_website_type_favicon.php

@@ -41,7 +41,7 @@ return new class extends Migration
             return;
         }
 
-        $sourcePath = public_path('images/website_types/onticket/'.self::FILENAME);
+        $sourcePath = public_path('images/website_types/'.self::FILENAME);
 
         if (! is_file($sourcePath)) {
             throw new RuntimeException("Favicon not found at path: {$sourcePath}");

database/migrations/2026_09_25_000000_generate_available_inventory_stock.php

@@ -1,89 +0,0 @@
-<?php
-
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Schema;
-
-return new class extends Migration
-{
-    public $withinTransaction = false;
-
-    private const FORMULA = 'CAST(real_stock AS SIGNED) - CAST(sold_units AS SIGNED) - CAST(reserved_stock AS SIGNED) - CAST(entry_reserved_stock AS SIGNED) + CAST(refunded_units AS SIGNED)';
-
-    public function up(): void
-    {
-        $conversion = 'CAST(real_stock AS SIGNED) + CAST(sold_units AS SIGNED) - CAST(refunded_units AS SIGNED)';
-        $this->assertNonNegative($conversion, 'la conversión produciría un stock real negativo');
-        $this->assertNonNegative(
-            "({$conversion}) - CAST(sold_units AS SIGNED) - CAST(reserved_stock AS SIGNED) - CAST(entry_reserved_stock AS SIGNED) + CAST(refunded_units AS SIGNED)",
-            'el stock disponible actual es negativo',
-        );
-
-        if (DB::getDriverName() === 'sqlite') {
-            Schema::table('inventories', function (Blueprint $table): void {
-                $table->bigInteger('available_stock')->virtualAs(self::FORMULA);
-            });
-            DB::table('inventories')->update(['real_stock' => DB::raw($conversion)]);
-        } else {
-            Schema::table('inventories', function (Blueprint $table): void {
-                $table->unsignedBigInteger('real_stock')->default(0)->change();
-                $table->bigInteger('available_stock')->storedAs(self::FORMULA);
-            });
-            DB::table('inventories')->update(['real_stock' => DB::raw($conversion)]);
-        }
-    }
-
-    public function down(): void
-    {
-        $conversion = 'CAST(real_stock AS SIGNED) - CAST(sold_units AS SIGNED) + CAST(refunded_units AS SIGNED)';
-        $this->assertNonNegative($conversion, 'el rollback produciría un stock real negativo');
-
-        if (DB::getDriverName() === 'sqlite') {
-            DB::table('inventories')->update(['real_stock' => DB::raw($conversion)]);
-            Schema::table('inventories', fn (Blueprint $table) => $table->dropColumn('available_stock'));
-
-            return;
-        }
-
-        DB::table('inventories')->update(['real_stock' => DB::raw($conversion)]);
-        Schema::table('inventories', fn (Blueprint $table) => $table->dropColumn('available_stock'));
-    }
-
-    private function assertNonNegative(string $expression, string $reason): void
-    {
-        $invalidInventories = DB::table('inventories')
-            ->select([
-                'id',
-                'real_stock',
-                'sold_units',
-                'refunded_units',
-                'reserved_stock',
-                'entry_reserved_stock',
-            ])
-            ->selectRaw("({$expression}) as calculated_stock")
-            ->whereRaw("({$expression}) < 0")
-            ->orderBy('id')
-            ->limit(20)
-            ->get();
-
-        if ($invalidInventories->isNotEmpty()) {
-            $details = $invalidInventories
-                ->map(static fn (object $inventory): string => sprintf(
-                    'id=%d [real=%d, vendidas=%d, reintegradas=%d, reservadas=%d, reservas_entradas=%d, resultado=%d]',
-                    $inventory->id,
-                    $inventory->real_stock,
-                    $inventory->sold_units,
-                    $inventory->refunded_units,
-                    $inventory->reserved_stock,
-                    $inventory->entry_reserved_stock,
-                    $inventory->calculated_stock,
-                ))
-                ->implode('; ');
-
-            throw new RuntimeException(
-                "Hay inventarios inconsistentes: {$reason}. Inventarios detectados (máximo 20): {$details}."
-            );
-        }
-    }
-};

database/migrations/2026_09_25_010000_create_inventory_movements_table.php

@@ -1,33 +0,0 @@
-<?php
-
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\Schema;
-
-return new class extends Migration
-{
-    public function up(): void
-    {
-        Schema::create('inventory_movements', function (Blueprint $table): void {
-            $table->id();
-            $table->foreignId('inventory_id')->constrained('inventories')->restrictOnDelete();
-            $table->string('operation', 40);
-            $table->bigInteger('available_stock_delta');
-            $table->bigInteger('available_stock_before');
-            $table->bigInteger('available_stock_after');
-            $table->json('counter_deltas');
-            $table->foreignId('responsible_user_id')->nullable()->constrained('users')->restrictOnDelete();
-            $table->uuid('idempotency_key')->nullable();
-            $table->timestamps();
-
-            $table->unique(['inventory_id', 'idempotency_key']);
-            $table->index(['inventory_id', 'created_at']);
-            $table->index(['operation', 'created_at']);
-        });
-    }
-
-    public function down(): void
-    {
-        Schema::dropIfExists('inventory_movements');
-    }
-};

database/migrations/2026_09_28_000000_update_shopit_admin_assets.php

@@ -1,47 +0,0 @@
-<?php
-
-use App\Domains\Core\Tenant\Models\AdminWebsiteType;
-use App\Domains\Core\Tenant\Services\AdminWebsiteTypeService;
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Http\UploadedFile;
-use Illuminate\Support\Facades\Storage;
-
-return new class extends Migration
-{
-    public function up(): void
-    {
-        if (app()->environment('testing')) {
-            Storage::fake('s3');
-        }
-
-        if (! AdminWebsiteType::query()->where('codigo', 'shopit')->exists()) {
-            return;
-        }
-
-        app(AdminWebsiteTypeService::class)->updateOrCreate(
-            ['codigo' => 'shopit'],
-            [
-                'primary_color' => '#2FD3AC',
-                'site_logo' => $this->uploadedImage('shopit_logo.png', 'image/png'),
-                'footer_logo' => $this->uploadedImage('shopit_footer_logo.png', 'image/png'),
-                'favicon' => $this->uploadedImage('shopit-favicon.svg', 'image/svg+xml'),
-            ],
-        );
-    }
-
-    public function down(): void
-    {
-        // Brand assets are operational data and are intentionally preserved.
-    }
-
-    private function uploadedImage(string $filename, string $mimeType): UploadedFile
-    {
-        $path = public_path("images/website_types/shopit/{$filename}");
-
-        if (! is_file($path)) {
-            throw new RuntimeException("ShopIt image not found at path: {$path}");
-        }
-
-        return new UploadedFile($path, $filename, $mimeType, null, true);
-    }
-};

database/seeders/WebsiteTypeSeeder.php

@@ -40,7 +40,7 @@ class WebsiteTypeSeeder extends Seeder
 
     public function run(): void
     {
-        $this->websiteTypeService->updateOrCreate(
+        $shopIt = $this->websiteTypeService->updateOrCreate(
             ['codigo' => 'shopit'],
             [
                 'nombre' => 'ShopIt',
@@ -48,10 +48,9 @@ class WebsiteTypeSeeder extends Seeder
                 'scanner_domain' => 'scanner.localhost',
                 'site_title' => 'ShopIt',
                 ...self::PRESENTATION,
-                'primary_color' => '#2FD3AC',
+                'site_logo' => $this->onTicketLogo(),
-                'site_logo' => $this->shopItLogo(),
+                'footer_logo' => $this->onTicketFooterLogo(),
-                'footer_logo' => $this->shopItFooterLogo(),
+                'favicon' => $this->onTicketFavicon(),
-                'favicon' => $this->shopItFavicon(),
             ],
         );
 
@@ -84,7 +83,7 @@ class WebsiteTypeSeeder extends Seeder
                 ...self::PRESENTATION,
                 'site_logo' => $this->onTicketLogo(),
                 'footer_logo' => $this->onTicketFooterLogo(),
-                'favicon' => $this->onTicketFavicon(),
+                'favicon' => $shopIt->favicon()->firstOrFail()->key,
             ],
         );
 
@@ -205,7 +204,7 @@ class WebsiteTypeSeeder extends Seeder
 
     private function onTicketLogo(): UploadedFile
     {
-        $path = public_path('images/website_types/onticket/onticket_logo.png');
+        $path = public_path('images/website_types/onticket_logo.png');
 
         if (! file_exists($path)) {
             throw new RuntimeException("OnTicket logo not found at path: {$path}");
@@ -222,7 +221,7 @@ class WebsiteTypeSeeder extends Seeder
 
     private function onTicketFooterLogo(): UploadedFile
     {
-        $path = public_path('images/website_types/onticket/onticket_footer_logo.png');
+        $path = public_path('images/website_types/onticket_footer_logo.png');
 
         if (! file_exists($path)) {
             throw new RuntimeException("OnTicket footer logo not found at path: {$path}");
@@ -239,7 +238,7 @@ class WebsiteTypeSeeder extends Seeder
 
     private function onTicketFavicon(): UploadedFile
     {
-        $path = public_path('images/website_types/onticket/onticket_favicon.svg');
+        $path = public_path('images/website_types/onticket_favicon.svg');
 
         if (! file_exists($path)) {
             throw new RuntimeException("OnTicket favicon not found at path: {$path}");
@@ -253,30 +252,4 @@ class WebsiteTypeSeeder extends Seeder
             true,
         );
     }
-
-    private function shopItLogo(): UploadedFile
-    {
-        return $this->uploadedImage('shopit', 'shopit_logo.png', 'image/png');
-    }
-
-    private function shopItFooterLogo(): UploadedFile
-    {
-        return $this->uploadedImage('shopit', 'shopit_footer_logo.png', 'image/png');
-    }
-
-    private function shopItFavicon(): UploadedFile
-    {
-        return $this->uploadedImage('shopit', 'shopit-favicon.svg', 'image/svg+xml');
-    }
-
-    private function uploadedImage(string $websiteType, string $filename, string $mimeType): UploadedFile
-    {
-        $path = public_path("images/website_types/{$websiteType}/{$filename}");
-
-        if (! file_exists($path)) {
-            throw new RuntimeException("Website type image not found at path: {$path}");
-        }
-
-        return new UploadedFile($path, $filename, $mimeType, null, true);
-    }
 }

public/images/website_types/shopit/shopit-favicon.svg

@@ -1,10 +0,0 @@
-<svg width="41" height="41" viewBox="0 0 41 41" fill="none" xmlns="http://www.w3.org/2000/svg" xmlns:se="http://svg-edit.googlecode.com" se:swatches="W3siaWQiOiJCMDBEREFFMy0zMjUwLTRERDgtOEU0RS0zMDYxREJGNUE3QTMiLCJuYW1lIjoiIzVCNUI1QiIsImhleCI6IiM1QjVCNUIifSx7ImlkIjoiRjI4QjU4NTYtNUM2OC00QkY4LTk3MDMtOTk5NDJCOTgzNTVBIiwibmFtZSI6IiMyRkQzQUMiLCJoZXgiOiIjMkZEM0FDIn1d">
-
-
-
-
-
-
-
-
-<g><se:title>Layer 1</se:title><path d="m16.32,27.13c2.29,0 3.13,-0.37 3.73,-0.76c0.65,-0.42 1.17,-1.06 1.24,-1.97c0.06,-0.76 -0.29,-1.4 -0.81,-1.78c-0.7,-0.5 -2.05,-0.81 -3.21,-1.05c-1.13,-0.24 -2.14,-0.6 -3.03,-1.22c-1.06,-0.73 -1.61,-1.47 -1.96,-2.52c-0.45,-1.32 -0.36,-3.12 -0.11,-4.02c0.43,-1.54 1.35,-2.86 2.59,-3.84001c1.6,-1.26 3.58,-2.01999 6.46,-2.17999c0.3,-0.02 0.71,-0.04 1.21,-0.04c0,0 0.16,0 0.21,0h13.3c-3.68,-4.71 -9.41,-7.75 -15.85,-7.75c-11.1,0 -20.09,9 -20.09,20.09c0,2.47 0.45,4.84 1.27,7.03h15.06l-0.01,0.01z" fill="#2FD3AC" id="svg_7"/><path d="m23.11,12.59c-0.65,0 -1.64,0.03 -1.95,0.07c-1.3,0.17 -2.25,0.55 -2.82,1.31c-0.3199,0.43 -0.48,0.99 -0.29,1.66c0.31,1.07 2.1501,1.44 3.4001,1.7c1.49,0.31 2.5599,0.57 3.5899,1.08c1.06,0.53 2.03,1.41 2.51,2.43c0.46,0.98 0.6001,1.96 0.5201,3.08c-0.11,1.56 -0.4701,2.72 -0.9901,3.69c-0.49,0.91 -1.32,1.87 -2.03,2.44c-0.74,0.59 -1.81,1.38 -3.15,1.83c-1.35,0.45 -3.1399,0.63 -5.2999,0.63h-12.30005c3.68,4.68 9.39005,7.68 15.80005,7.68c11.1,0 20.0899,-9 20.0899,-20.09c0,-2.65 -0.5199,-5.18 -1.4499,-7.5h-15.6201l-0.01,-0.01z" fill="#5B5B5B" id="svg_8"/></g></svg>

tests/Feature/FiestaFutbolInfantil/AccommodationControllerTest.php

@@ -97,8 +97,6 @@ class AccommodationControllerTest extends TestCase
 
         $updated = $this->variantPayload('Casa Rodante Premium', 'Con electricidad', 15, 50000);
         $updated['id'] = $variantId;
-        unset($updated['stock']);
-        $updated['stock_difference'] = -5;
 
         $this->postJson('/api/v1/adminapp/tenant/accommodations', [
             'variants' => [

tests/Feature/FiestaFutbolInfantil/EntryControllerTest.php

@@ -194,7 +194,7 @@ class EntryControllerTest extends TestCase
                     'title' => 'Abono actualizado',
                     'description' => 'Ahora incluye ambas fechas',
                     'event_date_ids' => [$firstDate->id, $secondDate->id],
-                    'stock_difference' => 10,
+                    'stock' => 20,
                     'price' => 250,
                 ],
                 [
@@ -268,7 +268,7 @@ class EntryControllerTest extends TestCase
                 'id' => $entryId,
                 'title' => 'Abono editado',
                 'event_date_ids' => [$date->id, $otherDate->id],
-                'stock_difference' => 10,
+                'stock' => 30,
                 'price' => 200,
             ]],
         ])->assertOk()
@@ -329,7 +329,7 @@ class EntryControllerTest extends TestCase
                 'title' => 'Entrada inválida',
                 'description' => null,
                 'event_date_ids' => [$foreignDate->id],
-                'stock_difference' => 0,
+                'stock' => 10,
                 'price' => 100,
             ]],
         ])
@@ -371,7 +371,7 @@ class EntryControllerTest extends TestCase
                 'title' => 'Entrada inválida',
                 'description' => null,
                 'event_date_ids' => [$eventDate->id],
-                'stock_difference' => 0,
+                'stock' => 10,
                 'price' => 100,
             ]],
         ])

tests/Feature/FiestaFutbolInfantil/FoodControllerTest.php

@@ -124,8 +124,6 @@ class FoodControllerTest extends TestCase
 
         $updated = $this->variantPayload($secondDate->id, 'Cena', 'Vianda', 80, 8500);
         $updated['id'] = $variantId;
-        unset($updated['stock']);
-        $updated['stock_difference'] = -20;
         $updated['description'] = 'Cena para llevar';
 
         $this->postJson('/api/v1/adminapp/tenant/foods', [
@@ -242,7 +240,7 @@ class FoodControllerTest extends TestCase
         Inventory::query()->whereKey($replacementInventoryId)->delete();
 
         $updated = $this->patchJson('/api/v1/adminapp/tenant/foods/history-stock', [
-            'variants' => [['id' => $historicalVariantId, 'stock_difference' => -55]],
+            'variants' => [['id' => $historicalVariantId, 'stock' => 45]],
         ])
             ->assertOk()
             ->assertJsonPath('data.history.0.variants.0.id', $historicalVariantId)
@@ -260,7 +258,7 @@ class FoodControllerTest extends TestCase
         ]);
 
         $this->patchJson('/api/v1/adminapp/tenant/foods/history-stock', [
-            'variants' => [['id' => $activeVariantId, 'stock_difference' => -60]],
+            'variants' => [['id' => $activeVariantId, 'stock' => 20]],
         ])
             ->assertUnprocessable()
             ->assertJsonValidationErrors(['variants.0.id']);

tests/Feature/FiestaFutbolInfantil/MerchandiseControllerTest.php

@@ -112,8 +112,6 @@ class MerchandiseControllerTest extends TestCase
 
         $updatedVariant = $this->variantPayload('Blanco', 'M', 80, 12500);
         $updatedVariant['id'] = $variantId;
-        unset($updatedVariant['stock']);
-        $updatedVariant['stock_difference'] = -20;
         $updatedItem = $this->itemPayload('Camiseta oficial', 2, [
             $updatedVariant,
             $this->variantPayload('Verde', 'L', 60, 15000),
@@ -236,8 +234,6 @@ class MerchandiseControllerTest extends TestCase
         $secondItemVariantId = $created->json('data.1.variants.0.id');
         $foreignVariant = $this->variantPayload('Azul Marino', 'XL', 20, 30000);
         $foreignVariant['id'] = $secondItemVariantId;
-        unset($foreignVariant['stock']);
-        $foreignVariant['stock_difference'] = 0;
         $firstItem = $this->itemPayload('Camiseta', 3, [$foreignVariant]);
         $firstItem['id'] = $firstItemId;
 

tests/Feature/Migrations/InventoryStockMovementMigrationTest.php

@@ -1,139 +0,0 @@
-<?php
-
-namespace Tests\Feature\Migrations;
-
-use App\Domains\Commerce\Catalog\Enums\InventoryMovementOperation;
-use App\Domains\Commerce\Catalog\Models\Inventory;
-use App\Domains\Commerce\Catalog\Models\InventoryMovement;
-use App\Domains\Core\Auth\Models\User;
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Schema;
-use Tests\TestCase;
-
-class InventoryStockMovementMigrationTest extends TestCase
-{
-    protected function setUp(): void
-    {
-        parent::setUp();
-
-        Schema::create('users', function (Blueprint $table): void {
-            $table->id();
-            $table->softDeletes();
-        });
-        Schema::create('inventories', function (Blueprint $table): void {
-            $table->id();
-            $table->unsignedBigInteger('sold_units')->default(0);
-            $table->unsignedInteger('reserved_stock')->default(0);
-            $table->unsignedInteger('real_stock')->default(0);
-            $table->unsignedBigInteger('refunded_units')->default(0);
-            $table->unsignedInteger('entry_reserved_stock')->default(0);
-        });
-    }
-
-    protected function tearDown(): void
-    {
-        Schema::disableForeignKeyConstraints();
-        Schema::dropIfExists('inventory_movements');
-        Schema::dropIfExists('inventories');
-        Schema::dropIfExists('users');
-        Schema::enableForeignKeyConstraints();
-
-        parent::tearDown();
-    }
-
-    public function test_migration_and_rollback_preserve_the_available_balance(): void
-    {
-        DB::table('inventories')->insert([
-            'id' => 17,
-            'real_stock' => 10,
-            'sold_units' => 3,
-            'refunded_units' => 1,
-            'reserved_stock' => 2,
-            'entry_reserved_stock' => 1,
-        ]);
-
-        $stockMigration = $this->stockMigration();
-        $movementMigration = $this->movementMigration();
-        $stockMigration->up();
-        $movementMigration->up();
-
-        $inventory = Inventory::findOrFail(17);
-        $this->assertSame(12, $inventory->real_stock);
-        $this->assertSame(7, $inventory->availableStock());
-        $this->assertTrue(Schema::hasTable('inventory_movements'));
-
-        $inventory->adjustAvailableStock(2);
-        $this->assertSame(9, $inventory->availableStock());
-
-        $movementMigration->down();
-        $stockMigration->down();
-
-        $this->assertFalse(Schema::hasColumn('inventories', 'available_stock'));
-        $this->assertFalse(Schema::hasTable('inventory_movements'));
-        $this->assertSame(12, Inventory::findOrFail(17)->real_stock);
-        $this->assertSame(9, Inventory::findOrFail(17)->availableStock());
-    }
-
-    public function test_movements_capture_counters_actor_and_idempotency(): void
-    {
-        $this->stockMigration()->up();
-        $this->movementMigration()->up();
-        DB::table('users')->insert(['id' => 5]);
-        $user = User::query()->findOrFail(5);
-
-        $inventory = Inventory::query()->create(['real_stock' => 10]);
-        $inventory->recordInitialization($user);
-        $inventory->adjustAvailableStock(3, $user, '955d2afb-1e36-46df-a9db-d9ee31ed5eb4');
-        $inventory->adjustAvailableStock(3, $user, '955d2afb-1e36-46df-a9db-d9ee31ed5eb4');
-        $inventory->reserve(2, true);
-        $inventory->buy(2, true);
-        $inventory->refundStock(1, $user);
-
-        $this->assertSame(12, $inventory->availableStock());
-        $this->assertCount(5, $inventory->movements()->get());
-
-        $adjustment = InventoryMovement::query()
-            ->where('operation', InventoryMovementOperation::StockIncreased->value)
-            ->firstOrFail();
-        $this->assertSame(3, $adjustment->available_stock_delta);
-        $this->assertSame(10, $adjustment->available_stock_before);
-        $this->assertSame(13, $adjustment->available_stock_after);
-        $this->assertSame(['real_stock' => 3], $adjustment->counter_deltas);
-        $this->assertSame(5, $adjustment->responsible_user_id);
-
-        $purchase = InventoryMovement::query()
-            ->where('operation', InventoryMovementOperation::PurchaseCommitted->value)
-            ->firstOrFail();
-        $this->assertSame(0, $purchase->available_stock_delta);
-        $this->assertSame(['reserved_stock' => -2, 'sold_units' => 2], $purchase->counter_deltas);
-    }
-
-    public function test_migration_rejects_an_inventory_with_negative_available_stock(): void
-    {
-        DB::table('inventories')->insert([
-            'id' => 23,
-            'real_stock' => 1,
-            'reserved_stock' => 2,
-        ]);
-
-        $this->expectException(\RuntimeException::class);
-        $this->expectExceptionMessage(
-            'stock disponible actual es negativo. Inventarios detectados (máximo 20): '
-            .'id=23 [real=1, vendidas=0, reintegradas=0, reservadas=2, reservas_entradas=0, resultado=-1]'
-        );
-
-        $this->stockMigration()->up();
-    }
-
-    private function stockMigration(): Migration
-    {
-        return require database_path('migrations/2026_09_25_000000_generate_available_inventory_stock.php');
-    }
-
-    private function movementMigration(): Migration
-    {
-        return require database_path('migrations/2026_09_25_010000_create_inventory_movements_table.php');
-    }
-}

tests/Feature/Migrations/SetWebsiteTypeFaviconTest.php

@@ -34,7 +34,7 @@ class SetWebsiteTypeFaviconTest extends TestCase
         $this->assertSame(1, DB::table('attachments')->where('filename', 'onticket_favicon.svg')->count());
         Storage::disk('s3')->assertExists($attachment->path);
         $this->assertSame(
-            file_get_contents(public_path('images/website_types/onticket/onticket_favicon.svg')),
+            file_get_contents(public_path('images/website_types/onticket_favicon.svg')),
             Storage::disk('s3')->get($attachment->path),
         );
     }

tests/Feature/Seeders/WebsiteTypeSeederTest.php

@@ -23,7 +23,7 @@ class WebsiteTypeSeederTest extends TestCase
 
         $this->assertSame(2, AdminWebsiteType::query()->count());
         $this->assertSame(3, StorefrontWebsiteType::query()->count());
-        $this->assertSame(6, Attachment::query()->count());
+        $this->assertSame(5, Attachment::query()->count());
 
         $expectedPresentation = [
             'primary_color' => '#FF7006',
@@ -45,15 +45,12 @@ class WebsiteTypeSeederTest extends TestCase
         $this->assertSame('ShopIt', $shopIt->nombre);
         $this->assertSame('localhost', $shopIt->dominio);
         $this->assertSame('scanner.localhost', $shopIt->scanner_domain);
-        $this->assertSame(
+        $this->assertSame($expectedPresentation, $shopIt->only(array_keys($expectedPresentation)));
-            [...$expectedPresentation, 'primary_color' => '#2FD3AC'],
+        $this->assertSame('onticket_logo.png', $shopIt->siteLogo->filename);
-            $shopIt->only(array_keys($expectedPresentation)),
-        );
-        $this->assertSame('shopit_logo.png', $shopIt->siteLogo->filename);
         Storage::disk('s3')->assertExists($shopIt->siteLogo->path);
-        $this->assertSame('shopit_footer_logo.png', $shopIt->footerLogo->filename);
+        $this->assertSame('onticket_footer_logo.png', $shopIt->footerLogo->filename);
         Storage::disk('s3')->assertExists($shopIt->footerLogo->path);
-        $this->assertSame('shopit-favicon.svg', $shopIt->favicon->filename);
+        $this->assertSame('onticket_favicon.svg', $shopIt->favicon->filename);
         Storage::disk('s3')->assertExists($shopIt->favicon->path);
         $shopItStorefront = StorefrontWebsiteType::query()->where('codigo', 'shopit')->with('extras')->sole();
         $this->assertSame('standard', $shopItStorefront->header_type);
@@ -87,7 +84,7 @@ class WebsiteTypeSeederTest extends TestCase
         Storage::disk('s3')->assertExists($onTicket->footerLogo->path);
         $this->assertNotSame($shopIt->site_logo, $onTicket->site_logo);
         $this->assertNotSame($shopIt->footer_logo, $onTicket->footer_logo);
-        $this->assertNotSame($shopIt->favicon_id, $onTicket->favicon_id);
+        $this->assertSame($shopIt->favicon_id, $onTicket->favicon_id);
         $this->assertSame('onticket_favicon.svg', $onTicket->favicon->filename);
         $onTicketStorefront = StorefrontWebsiteType::query()->where('codigo', 'onticket')->with('extras')->sole();
         $this->assertSame('standard', $onTicketStorefront->header_type);