diff --git a/Backend/ERPCore/Controllers/PurchaseOrdersController.cs b/Backend/ERPCore/Controllers/PurchaseOrdersController.cs index a1bf9b9..ff9dc71 100644 --- a/Backend/ERPCore/Controllers/PurchaseOrdersController.cs +++ b/Backend/ERPCore/Controllers/PurchaseOrdersController.cs @@ -57,6 +57,25 @@ public sealed class PurchaseOrdersController : ApiControllerBase return Ok(result.Value); } + /// Submit a Draft PO (Draft → Approved). 409 PO_NOT_EDITABLE if not Draft. + [HttpPost("{poId:int}/submit")] + [ProducesResponseType(typeof(PurchaseOrderDto), StatusCodes.Status200OK)] + [ProducesResponseType(StatusCodes.Status404NotFound)] + [ProducesResponseType(StatusCodes.Status409Conflict)] + public async Task> Submit(int poId, CancellationToken ct) + => Ok(await _pos.SubmitAsync(poId, ct)); + + /// Delete a PO — permitted only while Draft (409 PO_NOT_EDITABLE otherwise). + [HttpDelete("{poId:int}")] + [ProducesResponseType(StatusCodes.Status204NoContent)] + [ProducesResponseType(StatusCodes.Status404NotFound)] + [ProducesResponseType(StatusCodes.Status409Conflict)] + public async Task Delete(int poId, CancellationToken ct) + { + await _pos.DeleteAsync(poId, ct); + return NoContent(); + } + /// Approve — no-op in Phase 1 (POs auto-approve); transitions PendingApproval→Approved when enabled. [HttpPost("{poId:int}/approve")] [ProducesResponseType(typeof(PurchaseOrderDto), StatusCodes.Status200OK)] diff --git a/Backend/ERPCore/Domain/Entities/GrnLine.cs b/Backend/ERPCore/Domain/Entities/GrnLine.cs index 5805127..f6981d4 100644 --- a/Backend/ERPCore/Domain/Entities/GrnLine.cs +++ b/Backend/ERPCore/Domain/Entities/GrnLine.cs @@ -3,9 +3,13 @@ using ERPCore.Domain.Enums; namespace ERPCore.Domain.Entities; /// -/// GRN line (FR-GRN-04..08). is the PO-derived cost for -/// PO-based receipts (client cost ignored — 02-SECURITY C.3) or the entered cost -/// for direct receipts. = qty × unitCost. +/// GRN line (FR-GRN-04..08). is the gross cost received at: +/// entered on the line, defaulting to the PO price when omitted (a per-receipt price +/// override is now permitted — see docs/02-SECURITY C.3, revised). +/// snapshots the PO price at receipt so the variance survives later PO edits. +/// = unitCost after trade discount — this is what the FIFO layer +/// costs at (VAT never enters stock value; it is recoverable input tax). +/// = qty × netUnitCost (after discount, before VAT). /// gates issuability. Model: docs/10 Part C.3. /// public class GrnLine @@ -31,7 +35,30 @@ public class GrnLine public Batch? Batch { get; set; } public decimal Qty { get; set; } + + /// Gross unit cost received at (entered, or PO price when omitted). public decimal UnitCost { get; set; } + + /// Snapshot of the PO line price at receipt; null for direct receipts. + public decimal? PoUnitPrice { get; set; } + + /// Trade discount percentage (0–100), entered. + public decimal DiscountPct { get; set; } + + /// UnitCost × (1 − DiscountPct/100) — the inventory (FIFO layer) cost. + public decimal NetUnitCost { get; set; } + + /// VAT percentage (0–100), entered. Recoverable — does not affect stock value. + public decimal VatPct { get; set; } + + /// Qty × NetUnitCost × VatPct/100. + public decimal VatAmount { get; set; } + + /// Qty × NetUnitCost (after discount, before VAT). public decimal ReceivedValue { get; set; } + + /// Qty × NetUnitCost + VatAmount — payable to the vendor. + public decimal LineTotal { get; set; } + public HoldStatus HoldStatus { get; set; } = HoldStatus.Available; } diff --git a/Backend/ERPCore/Domain/Entities/PoLine.cs b/Backend/ERPCore/Domain/Entities/PoLine.cs index 1f38169..0a5f087 100644 --- a/Backend/ERPCore/Domain/Entities/PoLine.cs +++ b/Backend/ERPCore/Domain/Entities/PoLine.cs @@ -22,7 +22,7 @@ public class PoLine public Warehouse? Warehouse { get; set; } public decimal Qty { get; set; } - public decimal UnitPrice { get; set; } + public decimal UnitPrice { get; set; }// public decimal Tax { get; set; } public decimal QtyReceived { get; set; } } diff --git a/Backend/ERPCore/Dtos/Grn/GrnDtos.cs b/Backend/ERPCore/Dtos/Grn/GrnDtos.cs index 9051017..aec88c2 100644 --- a/Backend/ERPCore/Dtos/Grn/GrnDtos.cs +++ b/Backend/ERPCore/Dtos/Grn/GrnDtos.cs @@ -7,7 +7,10 @@ namespace ERPCore.Dtos.Grn; public sealed record GrnLineDto( int GrnLineId, int? PoLineId, int ItemId, int UomId, int? BinId, - decimal Qty, decimal UnitCost, decimal ReceivedValue, HoldStatus HoldStatus, int? BatchId); + decimal Qty, decimal UnitCost, decimal? PoUnitPrice, + decimal DiscountPct, decimal NetUnitCost, decimal VatPct, decimal VatAmount, + decimal ReceivedValue, decimal LineTotal, decimal PriceVariance, + HoldStatus HoldStatus, int? BatchId); public sealed record GrnDto( int GrnId, string DocNo, int? PoId, int VendorId, int WarehouseId, GrnStatus Status, @@ -44,8 +47,16 @@ public sealed class CreateGrnLineInput [Required] public int UomId { get; set; } public int? BinId { get; set; } [Range(0.0001, double.MaxValue)] public decimal Qty { get; set; } - /// Used only for direct (no-PO) receipts; ignored when is set. + /// + /// Gross unit cost. Required for direct (no-PO) receipts. For a PO line it is an optional + /// per-receipt price override — when 0/omitted the PO line price is used; when supplied it + /// wins and a variance is recorded against the PO snapshot (docs/02-SECURITY C.3, revised). + /// [Range(0, double.MaxValue)] public decimal UnitCost { get; set; } + /// Trade discount percentage (0–100). Reduces the inventory cost. + [Range(0, 100)] public decimal DiscountPct { get; set; } + /// VAT percentage (0–100). Recoverable — does not affect stock value. + [Range(0, 100)] public decimal VatPct { get; set; } [EnumDataType(typeof(HoldStatus))] public HoldStatus HoldStatus { get; set; } = HoldStatus.Available; public BatchInput? Batch { get; set; } } diff --git a/Backend/ERPCore/Dtos/Procurement/PurchaseOrderDtos.cs b/Backend/ERPCore/Dtos/Procurement/PurchaseOrderDtos.cs index c70e9a7..d062179 100644 --- a/Backend/ERPCore/Dtos/Procurement/PurchaseOrderDtos.cs +++ b/Backend/ERPCore/Dtos/Procurement/PurchaseOrderDtos.cs @@ -37,6 +37,13 @@ public sealed class CreatePurchaseOrderRequest [Required] public int VendorId { get; set; } public int? RequisitionId { get; set; } [Required, MinLength(1)] public List Lines { get; set; } = new(); + + /// + /// When true the PO is created in Draft (editable/deletable, not yet issued). + /// When false (default) it auto-approves on creation, preserving the Requisition→PO + /// and RFQ→PO flows unchanged (docs/11 §3.3). + /// + public bool SaveAsDraft { get; set; } } public sealed class UpdatePurchaseOrderRequest diff --git a/Backend/ERPCore/Infra/Persistence/Configurations/GrnConfiguration.cs b/Backend/ERPCore/Infra/Persistence/Configurations/GrnConfiguration.cs index 087ea19..0c47198 100644 --- a/Backend/ERPCore/Infra/Persistence/Configurations/GrnConfiguration.cs +++ b/Backend/ERPCore/Infra/Persistence/Configurations/GrnConfiguration.cs @@ -37,7 +37,13 @@ public sealed class GrnLineConfiguration : IEntityTypeConfiguration builder.Property(l => l.Qty).HasPrecision(18, 4); builder.Property(l => l.UnitCost).HasPrecision(18, 6); + builder.Property(l => l.PoUnitPrice).HasPrecision(18, 6); + builder.Property(l => l.DiscountPct).HasPrecision(9, 4); + builder.Property(l => l.NetUnitCost).HasPrecision(18, 6); + builder.Property(l => l.VatPct).HasPrecision(9, 4); + builder.Property(l => l.VatAmount).HasPrecision(18, 4); builder.Property(l => l.ReceivedValue).HasPrecision(18, 4); + builder.Property(l => l.LineTotal).HasPrecision(18, 4); builder.Property(l => l.HoldStatus).HasConversion().HasMaxLength(20).IsRequired(); builder.HasOne(l => l.Grn).WithMany(g => g.Lines).HasForeignKey(l => l.GrnId).OnDelete(DeleteBehavior.Cascade); diff --git a/Backend/ERPCore/Infra/Persistence/Configurations/PermissionConfiguration.cs b/Backend/ERPCore/Infra/Persistence/Configurations/PermissionConfiguration.cs index 35b2ede..065f5b8 100644 --- a/Backend/ERPCore/Infra/Persistence/Configurations/PermissionConfiguration.cs +++ b/Backend/ERPCore/Infra/Persistence/Configurations/PermissionConfiguration.cs @@ -41,7 +41,11 @@ public sealed class PermissionConfiguration : IEntityTypeConfiguration("BinId") .HasColumnType("integer"); + b.Property("DiscountPct") + .HasPrecision(9, 4) + .HasColumnType("numeric(9,4)"); + b.Property("GrnId") .HasColumnType("integer"); @@ -288,9 +292,21 @@ namespace ERPCore.Infra.Persistence.Migrations b.Property("ItemId") .HasColumnType("integer"); + b.Property("LineTotal") + .HasPrecision(18, 4) + .HasColumnType("numeric(18,4)"); + + b.Property("NetUnitCost") + .HasPrecision(18, 6) + .HasColumnType("numeric(18,6)"); + b.Property("PoLineId") .HasColumnType("integer"); + b.Property("PoUnitPrice") + .HasPrecision(18, 6) + .HasColumnType("numeric(18,6)"); + b.Property("Qty") .HasPrecision(18, 4) .HasColumnType("numeric(18,4)"); @@ -306,6 +322,14 @@ namespace ERPCore.Infra.Persistence.Migrations b.Property("UomId") .HasColumnType("integer"); + b.Property("VatAmount") + .HasPrecision(18, 4) + .HasColumnType("numeric(18,4)"); + + b.Property("VatPct") + .HasPrecision(9, 4) + .HasColumnType("numeric(9,4)"); + b.HasKey("GrnLineId"); b.HasIndex("BatchId"); @@ -828,6 +852,30 @@ namespace ERPCore.Infra.Persistence.Migrations PermissionId = 18, Code = "NAV:settings.users", SubNavItemId = 8 + }, + new + { + PermissionId = 19, + Code = "NAV:procurement.requisitions", + SubNavItemId = 9 + }, + new + { + PermissionId = 20, + Code = "NAV:procurement.rfqs", + SubNavItemId = 10 + }, + new + { + PermissionId = 21, + Code = "NAV:procurement.purchase-orders", + SubNavItemId = 11 + }, + new + { + PermissionId = 22, + Code = "NAV:procurement.purchase-returns", + SubNavItemId = 12 }); }); @@ -1916,6 +1964,46 @@ namespace ERPCore.Infra.Persistence.Migrations NavItemId = 9, SortOrder = 2, Status = "Active" + }, + new + { + SubNavItemId = 9, + Code = "procurement.requisitions", + Href = "/dashboard/procurement/requisitions", + Label = "Requisitions", + NavItemId = 4, + SortOrder = 1, + Status = "Active" + }, + new + { + SubNavItemId = 10, + Code = "procurement.rfqs", + Href = "/dashboard/procurement/rfqs", + Label = "RFQs", + NavItemId = 4, + SortOrder = 2, + Status = "Active" + }, + new + { + SubNavItemId = 11, + Code = "procurement.purchase-orders", + Href = "/dashboard/procurement/purchase-orders", + Label = "Purchase Orders", + NavItemId = 4, + SortOrder = 3, + Status = "Active" + }, + new + { + SubNavItemId = 12, + Code = "procurement.purchase-returns", + Href = "/dashboard/procurement/purchase-returns", + Label = "Purchase Returns", + NavItemId = 4, + SortOrder = 4, + Status = "Active" }); }); diff --git a/Backend/ERPCore/Services/GrnService.cs b/Backend/ERPCore/Services/GrnService.cs index 8af5783..3da76b1 100644 --- a/Backend/ERPCore/Services/GrnService.cs +++ b/Backend/ERPCore/Services/GrnService.cs @@ -138,8 +138,11 @@ public sealed class GrnService : IGrnService if (input.BinId is not null && !await _bins.Query().AnyAsync(b => b.BinId == input.BinId && b.WarehouseId == request.WarehouseId, ct)) throw new DomainException(ErrorCodes.Validation, $"Bin {input.BinId} is not in warehouse {request.WarehouseId}.", 422); - // Cost: PO-derived for PO lines (client cost ignored, C.3); entered for direct. + // Cost: for a PO line, the PO price is used unless an override is entered (then it + // wins and a variance is recorded against the PO snapshot — docs/02-SECURITY C.3, + // revised). Direct receipts always use the entered cost. decimal unitCost; + decimal? poUnitPrice = null; if (input.PoLineId is not null) { var poLine = po?.Lines.FirstOrDefault(l => l.PoLineId == input.PoLineId) @@ -152,13 +155,19 @@ public sealed class GrnService : IGrnService throw new DomainException(ErrorCodes.OverReceiptTolerance, $"Receiving {input.Qty} exceeds the open quantity {openQty} on PO line {input.PoLineId}.", 422); - unitCost = poLine.UnitPrice; + poUnitPrice = poLine.UnitPrice; + unitCost = input.UnitCost > 0 ? input.UnitCost : poLine.UnitPrice; } else { unitCost = input.UnitCost; } + // Derived figures are always computed server-side, never accepted from the client. + var netUnitCost = Math.Round(unitCost * (1 - input.DiscountPct / 100m), 6, MidpointRounding.AwayFromZero); + var receivedValue = Math.Round(input.Qty * netUnitCost, 4, MidpointRounding.AwayFromZero); + var vatAmount = Math.Round(receivedValue * input.VatPct / 100m, 4, MidpointRounding.AwayFromZero); + var batch = await ResolveBatchAsync(item, input.Batch, batchCache, ct); lines.Add(new GrnLine @@ -170,7 +179,13 @@ public sealed class GrnService : IGrnService Batch = batch, // navigation so EF fixes up BatchId once the batch is inserted Qty = input.Qty, UnitCost = unitCost, - ReceivedValue = Math.Round(input.Qty * unitCost, 4, MidpointRounding.AwayFromZero), + PoUnitPrice = poUnitPrice, + DiscountPct = input.DiscountPct, + NetUnitCost = netUnitCost, + VatPct = input.VatPct, + VatAmount = vatAmount, + ReceivedValue = receivedValue, + LineTotal = receivedValue + vatAmount, HoldStatus = input.HoldStatus }); } @@ -220,7 +235,9 @@ public sealed class GrnService : IGrnService foreach (var line in grn.Lines.OrderBy(l => l.GrnLineId)) { var item = await _items.Query().AsNoTracking().FirstAsync(i => i.ItemId == line.ItemId, token); - var (qtyBase, unitCostBase) = await ToBaseAsync(item, line.UomId, line.Qty, line.UnitCost, token); + // FIFO layer costs at the after-discount net price; VAT is recoverable and never + // enters stock value (docs/10 FR-GRN-06, revised). + var (qtyBase, unitCostBase) = await ToBaseAsync(item, line.UomId, line.Qty, line.NetUnitCost, token); var layer = await _fifo.CreateInboundLayerAsync( line.ItemId, grn.WarehouseId, line.BatchId, null, line.GrnLineId, @@ -380,5 +397,8 @@ public sealed class GrnService : IGrnService private static GrnDto Map(Grn g) => new( g.GrnId, g.DocNo, g.PoId, g.VendorId, g.WarehouseId, g.Status, g.CreatedBy, g.CreatedAt, g.PostedAt, g.Lines.OrderBy(l => l.GrnLineId).Select(l => new GrnLineDto( - l.GrnLineId, l.PoLineId, l.ItemId, l.UomId, l.BinId, l.Qty, l.UnitCost, l.ReceivedValue, l.HoldStatus, l.BatchId)).ToList()); + l.GrnLineId, l.PoLineId, l.ItemId, l.UomId, l.BinId, l.Qty, l.UnitCost, l.PoUnitPrice, + l.DiscountPct, l.NetUnitCost, l.VatPct, l.VatAmount, l.ReceivedValue, l.LineTotal, + l.PoUnitPrice is null ? 0m : Math.Round((l.UnitCost - l.PoUnitPrice.Value) * l.Qty, 4, MidpointRounding.AwayFromZero), + l.HoldStatus, l.BatchId)).ToList()); } diff --git a/Backend/ERPCore/Services/Interfaces/IPurchaseOrderService.cs b/Backend/ERPCore/Services/Interfaces/IPurchaseOrderService.cs index 898588b..a93bca2 100644 --- a/Backend/ERPCore/Services/Interfaces/IPurchaseOrderService.cs +++ b/Backend/ERPCore/Services/Interfaces/IPurchaseOrderService.cs @@ -16,4 +16,10 @@ public interface IPurchaseOrderService Task> UpdateAsync(int poId, UpdatePurchaseOrderRequest request, uint expectedRowVersion, CancellationToken ct = default); Task ApproveAsync(int poId, CancellationToken ct = default); Task CancelAsync(int poId, string? reason, CancellationToken ct = default); + + /// Submit a Draft PO (Draft → Approved). 409 PO_NOT_EDITABLE if not Draft. + Task SubmitAsync(int poId, CancellationToken ct = default); + + /// Delete a PO — permitted only while Draft, else 409 PO_NOT_EDITABLE. + Task DeleteAsync(int poId, CancellationToken ct = default); } diff --git a/Backend/ERPCore/Services/PurchaseOrderService.cs b/Backend/ERPCore/Services/PurchaseOrderService.cs index cae9d35..25c114e 100644 --- a/Backend/ERPCore/Services/PurchaseOrderService.cs +++ b/Backend/ERPCore/Services/PurchaseOrderService.cs @@ -92,9 +92,10 @@ public sealed class PurchaseOrderService : IPurchaseOrderService DocNo = docNo, VendorId = request.VendorId, RequisitionId = request.RequisitionId, - // Phase 1: approvalRequired defaults off → auto-approved on creation (FR-PROC-04). + // Phase 1: approvalRequired defaults off → auto-approved on creation (FR-PROC-04), + // unless the caller explicitly saves a Draft (editable/deletable until submitted). ApprovalRequired = false, - Status = PurchaseOrderStatus.Approved, + Status = request.SaveAsDraft ? PurchaseOrderStatus.Draft : PurchaseOrderStatus.Approved, CreatedBy = actor, CreatedAt = DateTime.UtcNow, Lines = request.Lines.Select(ToLine).ToList() @@ -182,8 +183,41 @@ public sealed class PurchaseOrderService : IPurchaseOrderService return Map(po); } - private static bool IsEditable(PurchaseOrderStatus status) => status is not ( - PurchaseOrderStatus.FullyReceived or PurchaseOrderStatus.Closed or PurchaseOrderStatus.Cancelled); + public async Task SubmitAsync(int poId, CancellationToken ct = default) + { + var po = await _pos.Query() + .Include(p => p.Lines) + .FirstOrDefaultAsync(p => p.PoId == poId, ct) + ?? throw new NotFoundException($"Purchase order {poId} was not found."); + + if (po.Status != PurchaseOrderStatus.Draft) + throw new DomainException(ErrorCodes.PoNotEditable, $"Purchase order {poId} is {po.Status} and cannot be submitted.", 409); + + // Phase 1: no value gate, so a submitted draft goes straight to Approved (FR-PROC-04). + po.Status = PurchaseOrderStatus.Approved; + po.UpdatedAt = DateTime.UtcNow; + await _uow.SaveChangesAsync(ct); + + return Map(po); + } + + public async Task DeleteAsync(int poId, CancellationToken ct = default) + { + var po = await _pos.Query() + .Include(p => p.Lines) + .FirstOrDefaultAsync(p => p.PoId == poId, ct) + ?? throw new NotFoundException($"Purchase order {poId} was not found."); + + if (po.Status != PurchaseOrderStatus.Draft) + throw new DomainException(ErrorCodes.PoNotEditable, $"Purchase order {poId} is {po.Status} and cannot be deleted; only a Draft can be deleted.", 409); + + _pos.Remove(po); + await _uow.SaveChangesAsync(ct); + } + + // FR-PROC-05 (revised): a PO is editable/deletable only while Draft. Submitting locks it. + // Supersedes Phase-1 Option B "freely editable while open" — see docs/10 FR-PROC-05. + private static bool IsEditable(PurchaseOrderStatus status) => status is PurchaseOrderStatus.Draft; private static PoLine ToLine(CreatePoLineInput l) => new() { diff --git a/Backend/PROGRESS.md b/Backend/PROGRESS.md index b9baf5f..4cbffea 100644 --- a/Backend/PROGRESS.md +++ b/Backend/PROGRESS.md @@ -55,7 +55,7 @@ Convention: `docs/01-DOC-GUIDE.md §6`. Update this file in the **same commit** > Requisition/RFQ/PO implemented 2026-07-10. **Live smoke test PASSED** (docNo `PR/RFQ/PO-2026-#####` gap-controlled + incrementing, requestedBy/createdBy = seeded system user, RFQ comparison matrix, duplicate quotation→409, PO auto-approve + server-computed totals matching the spec example `112100/20178/132278`, edit-while-open with recomputed totals + ETag 200/412, PO_NOT_EDITABLE→409 on cancelled, cancel→200, bad reference→422). Same `[~]` reason as §1: the §6 security gate (auth + audit) is not yet wired. - [x] Requisition (+ lines) + submit (`POST /requisitions`, `/{id}/submit`, list, get) - [x] RFQ + quotations + comparison (`POST /rfqs`, `/{id}/quotations` [one per vendor], `GET /{id}/comparison` matrix) -- [x] Purchase Order: create (auto-approve, `approvalRequired` flag), edit-while-open (If-Match), approve (no-op), cancel +- [x] Purchase Order: create (auto-approve **or `saveAsDraft`**, `approvalRequired` flag), edit **Draft-only** (If-Match), **submit** (Draft→Approved), **delete** (Draft-only), approve (no-op), cancel — see the 2026-07-20 entry (FR-PROC-05 revised: draft-lock supersedes edit-while-open) - [x] Purchase Return (outbound movement, reason code) — `POST /purchase-returns` auto-posts an outbound FIFO consume via shared `StockMutator`; mandatory Return-context reason (`REASON_CODE_REQUIRED`→400, wrong context→422), references the GRN line for traceability, over-return→`409 STOCK_NEGATIVE_BLOCKED`. Verified. (Cumulative return-vs-received cap still relies on the stock-availability guard.) > **Deviation (recorded):** `VendorQuotation` is modelled as header + `VendorQuotationLine` (per-item pricing) to satisfy the API contract (docs/11 §3.2); docs/10 Part C.2's scalar `VENDOR_QUOTATION(unit_price, lead_days)` with no item ref cannot represent it. Update the ER model doc to match. @@ -74,7 +74,7 @@ Convention: `docs/01-DOC-GUIDE.md §6`. Update this file in the **same commit** ## 3. Goods Receipt > Implemented + **smoke test PASSED** 2026-07-13 (see §4 note for the shared stock verification). Same `[~]` reason as §1/§2: the §6 auth+audit gate. -- [x] GRN create (against PO / direct), over-receipt tolerance — `unitCost` **PO-derived server-side** (client `999` verified ignored → PO price used, 02-SECURITY C.3); direct receipt requires `vendorId` + entered cost (AR-04); over-receipt → `422 OVER_RECEIPT_TOLERANCE` (verified at open-qty boundary); batch created/reused per (item, batchNo). Serial capture deferred. +- [x] GRN create (against PO / direct), over-receipt tolerance — `unitCost` **defaults to the PO price but is now overridable per line** (variance recorded vs `poUnitPrice` snapshot — 02-SECURITY C.3 revised 2026-07-20; the old "client cost ignored" block is gone); direct receipt requires `vendorId` + entered cost (AR-04); over-receipt → `422 OVER_RECEIPT_TOLERANCE` (verified at open-qty boundary); batch created/reused per (item, batchNo). Serial capture deferred. **Discount/VAT added** — see the 2026-07-20 entry. - [x] GRN confirm → FIFO layer + ledger + PO `qtyReceived` (single UoW txn) — verified: layers+ledger posted, running balance, PO → PartiallyReceived/FullyReceived, UOM→base conversion (10 Box-12 → 120 base @10). **Idempotent** re-confirm verified (no double-post). Note: `Idempotency-Key` accepted but idempotency is resource-state based (already-Confirmed replays existing result); a keyed idempotency store is deferred. - [x] Inspection hold release / reject — Release fully verified (OnHold excluded from `available`, then released). Reject removes on-hand + posts a reversing ledger entry; formal link to a Purchase Return is deferred (§3.4). @@ -111,6 +111,16 @@ Convention: `docs/01-DOC-GUIDE.md §6`. Update this file in the **same commit** ## Done +### 2026-07-20 — PO draft lifecycle + GRN discount/VAT/price-override (migration `AddGrnPricingAndPoDraft`) +- **PO draft lifecycle (FR-PROC-05 revised).** `CreatePurchaseOrderRequest.SaveAsDraft` (default `false` → auto-approve unchanged; `true` → `Draft`). New `POST /purchase-orders/{id}/submit` (Draft→Approved, else `409 PO_NOT_EDITABLE`) and `DELETE /purchase-orders/{id}` (Draft-only, else 409). `IsEditable` narrowed from "not FullyReceived/Closed/Cancelled" to **`Draft` only** — so `PUT` now 409s on any submitted PO. **Option B ("freely edit while open") is superseded**; docs/10 FR-PROC-05, docs/11 §3.3 updated. ⚠️ **Every pre-existing PO is `Approved` and therefore now uneditable/undeletable** — intended, not a regression. No schema change (reuses the existing `Draft` enum value). +- **GRN discount + VAT + price override.** `GrnLine` gained `PoUnitPrice`(nullable snapshot), `DiscountPct`, `NetUnitCost`, `VatPct`, `VatAmount`, `LineTotal`. All derived figures **server-computed**, never client-supplied. FIFO layer + ledger now cost at **`NetUnitCost`** (after discount) — VAT is recoverable and never enters stock value (docs/10 FR-GRN-06 revised). For a PO line, `unitCost` defaults to the PO price but an entered override wins and a **variance** is recorded against `PoUnitPrice` (02-SECURITY C.3 revised — the "client cost ignored, decision locked" control is deliberately loosened; the variance trail + audit log are the compensating control). Multi-GRN-per-PO at differing prices (the 20/50/30 case) already worked via `openQty`/`QtyReceived` and is untouched. +- **Migration** `AddGrnPricingAndPoDraft` — hand-added a data backfill (`UPDATE grn_lines SET NetUnitCost = UnitCost, LineTotal = ReceivedValue`) so existing GRN lines stay consistent with their already-posted FIFO layers; `PoUnitPrice` left NULL for historical rows (no retroactive variance). `Down()` drops the six columns cleanly. +- **Verified:** `dotnet build` clean (0/0); migration **Up and Down** exercised against the live DB (rollback to `AddRolesNavPermissions` then re-apply — both `Done`). **Runtime end-to-end PASSED — 22/22 assertions** (Node script, register→cookie session): PO draft→edit→submit→edit/delete-locked (409), draft delete (204→404), plain create still auto-approves; **costing proof** (100 @10, 10% disc, 18% VAT → net 9.00, receivedValue 900, VAT 162, lineTotal 1062, **FIFO layer @9.00, valuation 900 — VAT absent from stock**); multi-GRN 20@10/50@11/30@12 → variances +50/+60, PO FullyReceived, blended valuation 2010. + +### 2026-07-20 (2) — Procurement sidebar submenu (migration `AddProcurementSubNav`) +- The sidebar submenu is driven by seeded `SubNavItem` rows + `GET /auth/me` navCodes; only Products/Settings had children, so **Purchase Orders had no sidebar section**. Added 4 `SubNavItem`s (ids 9–12, `NavItemId 4`) + 4 `Permission`s (ids 19–22) for Requisitions/RFQs/Purchase Orders/Purchase Returns via `AddProcurementSubNav`. The migration also grants the 4 to any role already holding the parent `NAV:procurement` (raw SQL, `ON CONFLICT DO NOTHING`); `Down()` removes the grants then the rows. +- **Found:** the `Admin` role (`RoleId 2`) was never granted `NAV:procurement` at all (nor Vendors), so its whole Procurement branch was hidden — granted the parent + 4 children directly. **Verified:** `/auth/me` for Admin returns `procurement` + all 4 children; frontend `tsc`/`eslint` clean. + ### 2026-07-09 — Bootstrap verified + Master Data (§1) implemented - Bootstrap scaffolding confirmed against 00-CORE §5 (solution, packages, `Program.cs` wiring, UoW, generic repo, `ICurrentUser`, ProblemDetails handler). Added enum-as-string JSON (`JsonStringEnumConverter`) and registered the 5 master-data services. - Domain: 3 enums (`ItemType`, `TrackingMode`, `EntityStatus`) + 8 entities (Category, Uom, UomConversion, Item, ItemReorder, Vendor, Warehouse, Bin) with one `IEntityTypeConfiguration` each; FKs `Restrict` (masters deactivate, not cascade-delete), unique indexes (SKU, vendor/warehouse code, uom name, bin code per-warehouse), decimal precision, `xmin` concurrency token on Item/Vendor. diff --git a/Frontend/PROGRESS.md b/Frontend/PROGRESS.md index 859af32..1b76f10 100644 --- a/Frontend/PROGRESS.md +++ b/Frontend/PROGRESS.md @@ -39,12 +39,12 @@ Convention: `docs/01-DOC-GUIDE.md §6`. Update this file in the **same commit** ## 3. Procurement screens - [~] Requisition (`app/dashboard/procurement/requisitions` list, `/new` create, `/[id]` detail + Submit) — FR-PROC-01 - [~] RFQ + quotations + comparison view (`.../rfqs` list, `/new` create with vendor multi-invite, `/[id]` detail: lines, comparison matrix, record-quotation form, "Create PO from vendor") — FR-PROC-02 -- [~] Purchase Order (`.../purchase-orders` list, `/new` create — auto-approved, prefillable from a Requisition or an RFQ+vendor quotation via query params — `/[id]` detail: edit-while-open with ETag/If-Match, Cancel with reason) — FR-PROC-03..07 +- [~] Purchase Order (`.../purchase-orders` list, `/new` create — **Save as draft** or **Create & submit** (auto-approve), prefillable from a Requisition or an RFQ+vendor quotation via query params — `/[id]` detail: **Draft** is editable (ETag/If-Match) with **Submit** + **Delete**; a submitted/open PO is read-only with **Cancel** (with reason)) — FR-PROC-03..07. **2026-07-20:** rewired to the draft lifecycle — `isPoEditable` is now `Draft`-only, `submit`/`remove` added to `lib/api/purchase-orders.ts`, `saveAsDraft` on the create request. See the 2026-07-20 entry. - [~] Purchase Return (`.../purchase-returns` list, `/new` create against a Confirmed/Closed GRN's lines) — FR-PROC-08; also reachable from a `Rejected` GRN line via a "Create Return" button on the GRN detail page ## 4. Receiving screens - [~] GRN list (`app/dashboard/receiving/grn/page.tsx`) — loading/empty/error states, links to detail -- [~] GRN create (`app/dashboard/receiving/grn/new/page.tsx`) — "Against PO" (picks an Approved/PartiallyReceived PO, prefills open lines) and "Direct receipt" (vendor + manual lines) modes; per-line bin, qty, unit cost, hold status, and batch (batchNo+expiryDate) or serial-number-list capture driven by the item's `trackingMode` +- [~] GRN create (`app/dashboard/receiving/grn/new/page.tsx`) — "Against PO" (picks an Approved/PartiallyReceived PO, prefills open lines) and "Direct receipt" (vendor + manual lines) modes; per-line bin, qty, unit cost, **discount % / VAT %** (with live after-discount/after-VAT line total + a per-row PO-price variance hint and a document total), hold status, and batch (batchNo+expiryDate) or serial-number-list capture driven by the item's `trackingMode`. **2026-07-20:** discount/VAT/variance added — see the 2026-07-20 entry. - [~] GRN confirm (`app/dashboard/receiving/grn/[id]/page.tsx`) — renders returned `createdLayers`/`ledgerRefs`/`poStatus` as a confirmation panel (20-FRONTEND §4); sends a stable `Idempotency-Key` per detail-page session - [~] Inspection hold release / reject — Release/Reject buttons per on-hold line, shown once the GRN is `Confirmed` - Sidebar: added "Receiving" nav entry (`components/Layouts/AppSidebar.tsx`) → `/dashboard/receiving/grn` @@ -90,6 +90,14 @@ Convention: `docs/01-DOC-GUIDE.md §6`. Update this file in the **same commit** ## Done +### 2026-07-20 — PO draft lifecycle + GRN discount/VAT/price-override (backend + frontend, same pass) +- **PO draft lifecycle.** `lib/api/purchase-orders.ts`: `isPoEditable` narrowed to `status === "Draft"` (was the three-status exclusion); added `submit(poId)` and `remove(poId)`. `types/procurement.ts`: `saveAsDraft?` on `CreatePurchaseOrderRequest`; `UpdatePurchaseOrderRequest` now `Omit`s it. `/new`: the single "Create PO" button split into **Save as draft** / **Create & submit**. `/[id]`: **Draft** shows the editable line grid + **Submit** + **Delete draft**; an issued-but-open PO (`Approved`/`PartiallyReceived`) is read-only with **Cancel PO** (with reason, disabled once receipts exist); closed/cancelled are terminal. The old Cancel button was gated on `isPoEditable` — that would have shown Cancel only for Draft, so the affordances were re-split into `editable` (Draft) vs `cancellable` (Approved/PartiallyReceived). +- **GRN discount/VAT/variance.** `types/grn.ts`: `discountPct`/`vatPct` on `CreateGrnLineInput`; `poUnitPrice`/`discountPct`/`netUnitCost`/`vatPct`/`vatAmount`/`lineTotal`/`priceVariance` on `GrnLine`. `/new`: per-line Disc %/VAT % inputs, a client mirror of the server arithmetic (`computeLine`, display-only — server stays authoritative) driving a live Line total + document total, and a PO-price variance hint when the entered unit cost differs from the prefilled PO price. `/[id]`: renders the full cost breakdown (unit cost + variance, disc %, net cost, received value, VAT, line total) and a stock-value/VAT/document-total footer. `lib/validations/grn.ts`: 0–100 range checks on the two percentages. +- **Deliberately not touched:** the item picker already showed `sku — name` (the request's stated need); multi-GRN-per-PO and adding a non-PO item to a PO GRN already worked. Vendor stays PO-derived (non-selectable) for a PO-based GRN — selecting a different vendor than the PO's would be wrong. +- **Select trigger showed the id, not the label (global fix).** Base UI's `Select.Value` renders the raw selected value unless the `Select.Root` is given an `items` map — the popup items unmount when closed, so their text isn't available to the trigger (confirmed in `@base-ui/react`'s `resolveSelectedLabel`, which `find`s `items` by value and only falls back to stringifying the value when none is supplied). Fixed once in the shared wrapper (`components/ui/select.tsx`): `Select` now walks its own `SelectItem` children and derives the `items` array automatically, so all ~60 `` call sites across 26 files show the selected label without any per-site change. `tsc`/`eslint` clean; verified against Base UI's label-resolution source. +- **Procurement sidebar submenu.** The sidebar builds submenus from backend-seeded `SubNavItem` rows filtered by `GET /auth/me`'s `navCodes`; only Products and Settings had children, so Purchase Orders had no sidebar section (only reachable via the Procurement hub card). Added a `children` array to the Procurement nav entry (`components/Layouts/AppSidebar.tsx`) — Requisitions, RFQs, Purchase Orders, Purchase Returns — matching new backend sub-nav codes. Also found the Admin role (`RoleId 2`) was never granted `NAV:procurement` at all, so the whole Procurement branch was hidden for it; granted the parent + 4 children. **Verified:** `/auth/me` for Admin now returns all five procurement codes → submenu renders. Stale PO hub-card copy ("freely editable while open") updated to the draft/submit wording. +- **Verified:** `tsc --noEmit` clean; `eslint` unchanged from baseline (7 pre-existing `set-state-in-effect` on the PO/GRN screens before and after — 0 new issues, confirmed by stashing and re-counting). Runtime browser verification is the next step in this pass. + ### 2026-07-17 — connected to the real API (mock-data.ts deleted) **The app now talks to ERPCore.** Every `lib/api/*.ts` module calls the backend; `lib/api/mock-data.ts` is gone. This is the pass the 2026-07-15 note anticipated. diff --git a/Frontend/erp-system/app/dashboard/procurement/page.tsx b/Frontend/erp-system/app/dashboard/procurement/page.tsx index dd9f1ad..cc3a0c4 100644 --- a/Frontend/erp-system/app/dashboard/procurement/page.tsx +++ b/Frontend/erp-system/app/dashboard/procurement/page.tsx @@ -18,7 +18,7 @@ const areas: { title: string; description: string; href: string; icon: LucideIco }, { title: "Purchase Orders", - description: "Auto-approved on creation, freely editable while open, cancellable before receipt.", + description: "Save as draft (editable/deletable) or submit to lock; cancel an issued PO before receipt.", href: "/dashboard/procurement/purchase-orders", icon: ShoppingCart, }, diff --git a/Frontend/erp-system/app/dashboard/procurement/purchase-orders/[id]/page.tsx b/Frontend/erp-system/app/dashboard/procurement/purchase-orders/[id]/page.tsx index a205486..cf3db2a 100644 --- a/Frontend/erp-system/app/dashboard/procurement/purchase-orders/[id]/page.tsx +++ b/Frontend/erp-system/app/dashboard/procurement/purchase-orders/[id]/page.tsx @@ -3,7 +3,7 @@ import { useEffect, useState } from "react" import { useParams, useRouter } from "next/navigation" import Link from "next/link" -import { AlertTriangle, ArrowLeft, Ban, Plus, Save, Trash2 } from "lucide-react" +import { AlertTriangle, ArrowLeft, Ban, Plus, Save, Send, Trash2 } from "lucide-react" import { isPoEditable, purchaseOrdersApi } from "@/lib/api/purchase-orders" import { warehousesApi } from "@/lib/api/warehouses" @@ -65,6 +65,8 @@ export default function PurchaseOrderDetailPage() { const [showCancelForm, setShowCancelForm] = useState(false) const [cancelReason, setCancelReason] = useState("") const [cancelling, setCancelling] = useState(false) + const [submitting, setSubmitting] = useState(false) + const [deleting, setDeleting] = useState(false) function toDraftLines(order: PurchaseOrder): DraftLine[] { return order.lines.map((l) => ({ @@ -185,6 +187,39 @@ export default function PurchaseOrderDetailPage() { } } + async function handleSubmitPo() { + if (!po) return + setSaveError(null) + setSubmitting(true) + try { + const updated = await purchaseOrdersApi.submit(po.poId) + setPo(updated) + setLines(toDraftLines(updated)) + toast.success("Purchase order submitted", `${updated.docNo} — now ${updated.status} and locked for editing.`) + } catch (err) { + setSaveError(errorMessage(err)) + toast.error("Could not submit purchase order", errorMessage(err)) + } finally { + setSubmitting(false) + } + } + + async function handleDelete() { + if (!po) return + if (!window.confirm(`Delete draft ${po.docNo}? This cannot be undone.`)) return + setSaveError(null) + setDeleting(true) + try { + await purchaseOrdersApi.remove(po.poId) + toast.success("Draft deleted", po.docNo) + router.push("/dashboard/procurement/purchase-orders") + } catch (err) { + setSaveError(errorMessage(err)) + toast.error("Could not delete purchase order", errorMessage(err)) + setDeleting(false) + } + } + async function handleCancel() { if (!po) return if (!cancelReason.trim()) { @@ -226,6 +261,9 @@ export default function PurchaseOrderDetailPage() { const editable = isPoEditable(po.status) && !conflict const hasReceipts = po.lines.some((l) => l.qtyReceived > 0) + // A submitted-but-still-open PO (issued to the vendor) is cancellable with a reason; + // a Draft is deleted instead, and closed/cancelled POs are terminal. + const cancellable = po.status === "Approved" || po.status === "PartiallyReceived" return (
@@ -245,18 +283,32 @@ export default function PurchaseOrderDetailPage() {
- {isPoEditable(po.status) && !showCancelForm && ( - - )} +
+ {po.status === "Draft" && ( + <> + + + + )} + {cancellable && !showCancelForm && ( + + )} +
{showCancelForm && ( diff --git a/Frontend/erp-system/app/dashboard/procurement/purchase-orders/new/page.tsx b/Frontend/erp-system/app/dashboard/procurement/purchase-orders/new/page.tsx index c7e6592..411799d 100644 --- a/Frontend/erp-system/app/dashboard/procurement/purchase-orders/new/page.tsx +++ b/Frontend/erp-system/app/dashboard/procurement/purchase-orders/new/page.tsx @@ -43,8 +43,12 @@ function newKey() { return `poline-${keySeq}` } +// Unit price and tax are no longer entered at PO creation — pricing is captured at GRN +// receipt (with discount/VAT there). They default to 0 here and stay off the form, but +// remain on the payload because the backend line DTO still requires them; a PO prefilled +// from an RFQ keeps its negotiated price (below). function emptyLine(): DraftLine { - return { key: newKey(), itemId: null, uomId: null, warehouseId: null, qty: "", unitPrice: "", tax: "0.18" } + return { key: newKey(), itemId: null, uomId: null, warehouseId: null, qty: "", unitPrice: "0", tax: "0" } } function NewPurchaseOrderContent() { @@ -98,8 +102,8 @@ function NewPurchaseOrderContent() { uomId: null, warehouseId: null, qty: String(l.qty), - unitPrice: "", - tax: "0.18", + unitPrice: "0", + tax: "0", }) ) ) @@ -124,8 +128,8 @@ function NewPurchaseOrderContent() { uomId: null, warehouseId: null, qty: String(l.qty), - unitPrice: cell ? String(cell.unitPrice) : "", - tax: "0.18", + unitPrice: cell ? String(cell.unitPrice) : "0", + tax: "0", } }) ) @@ -151,7 +155,7 @@ function NewPurchaseOrderContent() { return items?.find((i) => i.itemId === itemId) ?? null } - async function handleSubmit() { + async function handleSubmit(saveAsDraft: boolean) { setHeaderError(null) setSubmitError(null) @@ -197,8 +201,12 @@ function NewPurchaseOrderContent() { vendorId, requisitionId: requisitionId ?? (rfqId ? undefined : null), lines: payloadLines, + saveAsDraft, }) - toast.success("Purchase order created", `${po.docNo} — auto-approved (FR-PROC-04).`) + toast.success( + "Purchase order created", + saveAsDraft ? `${po.docNo} — saved as draft.` : `${po.docNo} — auto-approved (FR-PROC-04).` + ) router.push(`/dashboard/procurement/purchase-orders/${po.poId}`) } catch (err) { setSubmitError(errorMessage(err)) @@ -274,8 +282,6 @@ function NewPurchaseOrderContent() { UOM Warehouse Qty - Unit price - Tax @@ -348,30 +354,6 @@ function NewPurchaseOrderContent() { /> - - updateLine(line.key, { unitPrice: e.target.value })} - className="h-11 text-base" - /> - - - - updateLine(line.key, { tax: e.target.value })} - className="h-11 text-base" - /> - - + diff --git a/Frontend/erp-system/app/dashboard/receiving/grn/[id]/page.tsx b/Frontend/erp-system/app/dashboard/receiving/grn/[id]/page.tsx index 4928aef..cab3278 100644 --- a/Frontend/erp-system/app/dashboard/receiving/grn/[id]/page.tsx +++ b/Frontend/erp-system/app/dashboard/receiving/grn/[id]/page.tsx @@ -158,6 +158,7 @@ export default function GrnDetailPage() { )} +
@@ -165,8 +166,12 @@ export default function GrnDetailPage() { UOM Bin Qty - Unit cost - Received value + Unit cost + Disc % + Net cost + Received value + VAT + Line total Hold status {grn.status === "Confirmed" && Actions} @@ -180,8 +185,22 @@ export default function GrnDetailPage() { {uomFor(line.uomId)} {binFor(line.binId)} {line.qty} - {line.unitCost.toFixed(2)} - {line.receivedValue.toFixed(2)} + + {line.unitCost.toFixed(2)} + {line.poUnitPrice !== null && line.priceVariance !== 0 && ( + + PO {line.poUnitPrice.toFixed(2)} · var {line.priceVariance > 0 ? "+" : ""}{line.priceVariance.toFixed(2)} + + )} + + {line.discountPct.toFixed(2)} + {line.netUnitCost.toFixed(2)} + {line.receivedValue.toFixed(2)} + + {line.vatAmount.toFixed(2)} + {line.vatPct.toFixed(2)}% + + {line.lineTotal.toFixed(2)} @@ -228,6 +247,22 @@ export default function GrnDetailPage() { })}
+
+ +
+
+ Stock value (excl. VAT) + {grn.lines.reduce((s, l) => s + l.receivedValue, 0).toFixed(2)} +
+
+ VAT + {grn.lines.reduce((s, l) => s + l.vatAmount, 0).toFixed(2)} +
+
+ Document total + {grn.lines.reduce((s, l) => s + l.lineTotal, 0).toFixed(2)} +
+
) } diff --git a/Frontend/erp-system/app/dashboard/receiving/grn/new/page.tsx b/Frontend/erp-system/app/dashboard/receiving/grn/new/page.tsx index b230d3a..99ce3ef 100644 --- a/Frontend/erp-system/app/dashboard/receiving/grn/new/page.tsx +++ b/Frontend/erp-system/app/dashboard/receiving/grn/new/page.tsx @@ -37,12 +37,28 @@ interface DraftLine { binId: number | null qty: string unitCost: string + /** PO line price when prefilled from a PO; drives the variance hint. */ + poUnitPrice: number | null + discountPct: string + vatPct: string holdStatus: HoldStatus batchNo: string expiryDate: string serialNumbersText: string } +/** Mirror of the server's line arithmetic — display only (docs/20 §3, server stays authoritative). */ +function computeLine(l: DraftLine) { + const qty = Number(l.qty) || 0 + const gross = Number(l.unitCost) || 0 + const disc = Number(l.discountPct) || 0 + const vat = Number(l.vatPct) || 0 + const netUnitCost = gross * (1 - disc / 100) + const receivedValue = qty * netUnitCost + const vatAmount = receivedValue * (vat / 100) + return { netUnitCost, receivedValue, vatAmount, lineTotal: receivedValue + vatAmount } +} + let keySeq = 0 function newKey() { keySeq += 1 @@ -58,6 +74,9 @@ function emptyLine(): DraftLine { binId: null, qty: "", unitCost: "", + poUnitPrice: null, + discountPct: "0", + vatPct: "0", holdStatus: "Available", batchNo: "", expiryDate: "", @@ -152,6 +171,9 @@ export default function NewGrnPage() { binId: null, qty: String(l.qty - l.qtyReceived), unitCost: String(l.unitPrice), + poUnitPrice: l.unitPrice, + discountPct: "0", + vatPct: "0", holdStatus: "Available", batchNo: "", expiryDate: "", @@ -211,6 +233,8 @@ export default function NewGrnPage() { uomId: line.uomId, qty: line.qty, unitCost: line.unitCost, + discountPct: line.discountPct, + vatPct: line.vatPct, trackingMode: itemFor(line.itemId)?.trackingMode ?? null, batchNo: line.batchNo, serialNumbersText: line.serialNumbersText, @@ -232,6 +256,8 @@ export default function NewGrnPage() { binId: l.binId, qty: Number(l.qty), unitCost: Number(l.unitCost), + discountPct: Number(l.discountPct) || 0, + vatPct: Number(l.vatPct) || 0, holdStatus: l.holdStatus, batch: trackingMode === "Batch" ? { batchNo: l.batchNo.trim(), expiryDate: l.expiryDate || null } : null, serialNumbers: trackingMode === "Serial" ? splitSerials(l.serialNumbersText) : null, @@ -371,16 +397,20 @@ export default function NewGrnPage() { {poLoading && } {!poLoading && lines.length > 0 && ( +
Item - UOM - Bin - Qty + UOM + Bin + Qty Unit cost - Hold status - Batch / Serial + Disc % + VAT % + Line total + Hold status + Batch / Serial @@ -473,6 +503,50 @@ export default function NewGrnPage() { className="h-11 text-base" /> + {line.poUnitPrice !== null && Number(line.unitCost) !== line.poUnitPrice && ( +

+ PO price {line.poUnitPrice.toFixed(2)} — variance recorded +

+ )} + + + updateLine(line.key, { discountPct: e.target.value })} + className="h-11 text-base" + /> + + + + updateLine(line.key, { vatPct: e.target.value })} + className="h-11 text-base" + /> + + + + {(() => { + const c = computeLine(line) + return ( +
+ {c.lineTotal.toFixed(2)} + + net {c.receivedValue.toFixed(2)} + VAT {c.vatAmount.toFixed(2)} + +
+ ) + })()} @@ -533,6 +607,16 @@ export default function NewGrnPage() { })}
+
+ )} + + {!poLoading && lines.length > 0 && ( +
+ Document total (incl. VAT) + + {lines.reduce((sum, l) => sum + computeLine(l).lineTotal, 0).toFixed(2)} + +
)} diff --git a/Frontend/erp-system/components/Layouts/AppSidebar.tsx b/Frontend/erp-system/components/Layouts/AppSidebar.tsx index c612b7e..a11487c 100644 --- a/Frontend/erp-system/components/Layouts/AppSidebar.tsx +++ b/Frontend/erp-system/components/Layouts/AppSidebar.tsx @@ -8,12 +8,14 @@ import { Building2, ChevronRight, ClipboardList, + FileText, HelpCircle, LayoutGrid, ListTree, Menu, Package, PackageCheck, + PackageX, Ruler, Settings, ShieldCheck, @@ -56,7 +58,19 @@ const navItems: { ], }, { title: "Vendors", code: "vendors", href: "/dashboard/vendors", icon: Truck, chevron: true }, - { title: "Procurement", code: "procurement", href: "/dashboard/procurement", icon: ClipboardList, chevron: true }, + { + title: "Procurement", + code: "procurement", + href: "/dashboard/procurement", + icon: ClipboardList, + chevron: true, + children: [ + { title: "Requisitions", code: "procurement.requisitions", href: "/dashboard/procurement/requisitions", icon: ClipboardList }, + { title: "RFQs", code: "procurement.rfqs", href: "/dashboard/procurement/rfqs", icon: FileText }, + { title: "Purchase Orders", code: "procurement.purchase-orders", href: "/dashboard/procurement/purchase-orders", icon: ShoppingCart }, + { title: "Purchase Returns", code: "procurement.purchase-returns", href: "/dashboard/procurement/purchase-returns", icon: PackageX }, + ], + }, { title: "Receiving", code: "receiving", href: "/dashboard/receiving/grn", icon: PackageCheck, chevron: true }, { title: "Stock", code: "stock", href: "/dashboard/stock", icon: Warehouse, chevron: true }, { title: "Warehouses", code: "warehouses", href: "/dashboard/warehouse", icon: Building2, chevron: true }, @@ -91,18 +105,36 @@ function SidebarContent({ pathname: string isMobile: boolean }) { + const iconOnly = !isMobile && collapsed + + // Which parent menus are open. Starts with the parent that owns the active + // route auto-expanded; user toggles are preserved across navigation. + const [expanded, setExpanded] = useState>({}) + + useEffect(() => { + const parent = items.find((i) => + i.children?.some((c) => pathname === c.href || pathname.startsWith(`${c.href}/`)) + ) + if (parent) { + setExpanded((prev) => (prev[parent.code] ? prev : { ...prev, [parent.code]: true })) + } + }, [pathname, items]) + + const toggleExpand = (code: string) => + setExpanded((prev) => ({ ...prev, [code]: !prev[code] })) + return (