Subversion Repositories SmartDukaan

(root)//trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/warehouse/InvoiceService.java – Rev 37769

Show changed files | Details | Compare with Previous | Blame | RSS feed

Filtering Options

From rev	To rev	Max revs

Search history for

Show All

Rev	Age	Author	Path	Log message	Diff
37769	12 d 19 h	amit	/trunk/profitmandi-dao/src/	fix(movement): receive a movement invoice only against its own PO, stop auto-closing movements, cancel reduces the PO line - GRN: movement invoice resolves invoice -> orders -> transaction -> its own movement PO (or a receive-only PO from the billing warehouse); supplier/warehouse keyed at entry must match; scanned IMEIs must be the billed ones - Cancelling an unbilled movement order takes its qty off the PO line; PO CLOSED/PRECLOSED once nothing is open - reopenedAt removed (movements no longer auto-close); PO records createdBy - SupplierStateResolver: a supplier's state is derived from its GSTIN on save - sql: add_po_created_by (applied), drop_po_reopened_at (run AFTER fofo/cron/web deploy)
35716	239 d 14 h	amit	/trunk/	Refactor GrnController and PurchaseOrderController: extract business logic into focused services and fix N+1 query performance issues - Create InvoiceService: invoice item CRUD, validation, vendor price matching - Create GrnRequestService: GRN request lifecycle, mismatch resolution state machine - Create DebitNoteService: debit note generation, PDF model building - Move PO amendment logic from PurchaseOrderController to PurchaseOrderService - Delegate generateDebitNote from PurchaseOrderServiceImpl to DebitNoteService - Thin GrnController from ~1215 to ~717 lines, remove 15 unused autowired dependencies - Thin PurchaseOrderController editPurchaseOrder from 40+ lines to 3 lines Performance fixes: - InvoiceServiceImpl.validateInvoiceDetail: batch item/vendor pricing/GST loading (4N+3 queries -> ~5) - DebitNoteServiceImpl: batch item loading in buildDebitNotePdfModel and generateDebitNote - GrnController.generateDebitNote: eliminate 2 redundant DB fetches - Replace 3x supplierRepository.selectAll() with targeted selectBySupplierIds()