SmartDukaan – /

Rev 37310 – Scratch Offers now selects only active partners

vikas — Thu, 13 Aug 2026 20:53:52 +0530

vikas – 1 file(s) modified
Scratch Offers now selects only active partners

~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/scratch/ScratchController.java

Rev 37309 – Scratch Offers now selects only active partners

vikas — Thu, 13 Aug 2026 18:48:00 +0530

vikas – 3 file(s) modified
Scratch Offers now selects only active partners

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/auth/LocationTrackingRepository.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/auth/LocationTrackingRepositoryImpl.java
~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/scratch/ScratchController.java

Rev 37308 – DN receipt: reject on condition mismatch instead of silently reclassifying ...

amit — Thu, 13 Aug 2026 18:29:56 +0530

amit – 3 file(s) modified
DN receipt: reject on condition mismatch instead of silently reclassifying

The receive screen defaulted the Condition dropdown to GOOD and receiveDebitNoteItems
overwrote purchase_return_item.type with whatever was submitted. A DOA return received
without touching the dropdown was rewritten BAD->GOOD, so on reject -> acknowledge the
restoreInventory branch put the unit back into good (sellable) stock instead of bad.
UKHD948/57: IMEI 353243710802439 was declared DOA, rejected as 'wrong entry', restored
as good stock and resold at MOP 3h44m later.

- receiveDebitNoteItems now compares each submitted condition against the stored one and
rejects the WHOLE debit note on any mismatch (PRO + rejectReturn are both per-DN and
there is no model for a partial receipt). PRO is still persisted so the receipt attempt
stays on record, then stamped rejected; no scans, no refund, no IRN work.
- Removed the pri.setReturnType(rt) overwrite. Past the gate the submitted condition always
equals the stored one, and applyReceipt already reads it off the entity. Rejected returns
now keep the declared type, so restoreReturnedItems restores to the right bucket.
- New notifyReturnRejectedConditionMismatch: to partner, cc Logistics L2/L3, RBM L1/L2 and
Sales L1 + first populated level in L2..L4. Sales ladder is sparse (L2 262 partners,
L3 64, L4 1582 of 1524 open stores) so a fixed L2/L3 rule resolves to nobody for most.
- getTeamEmails now filters inactive users and dedupes. csService.getAuthUserByCategoryId
(2-arg) does not filter active unlike its 1-arg sibling; fixed here rather than in
CsServiceImpl because that overload has ~25 other call sites where dropping inactive
users would change report scoping.

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/inventory/PurchaseReturnNotificationService.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/inventory/PurchaseReturnNotificationServiceImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/inventory/PurchaseReturnServiceImpl.java

Rev 37307 – Block activation of item-less schemes (SELLIN/SELLOUT/SPECIAL_SUPPORT): validateSchemeHasItems guards activeSchemeById and ...

amit — Thu, 13 Aug 2026 18:04:41 +0530

amit – 1 file(s) modified
Block activation of item-less schemes (SELLIN/SELLOUT/SPECIAL_SUPPORT): validateSchemeHasItems guards activeSchemeById and activeSchemeByIds, fixes /todayOfferList NPE caused by scheme 7877

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/scheme/SchemeServiceImpl.java

Rev 37306 – Block activation of item-less SELLIN/SELLOUT/SPECIAL_SUPPORT schemes: add SCHM_1009 response code

amit — Thu, 13 Aug 2026 18:04:36 +0530

amit – 1 file(s) modified
Block activation of item-less SELLIN/SELLOUT/SPECIAL_SUPPORT schemes: add SCHM_1009 response code

~ /trunk/profitmandi-common/src/main/resources/response-codes.properties

Rev 37305 – carry bag for silver at 1

ranu — Thu, 13 Aug 2026 17:44:34 +0530

ranu – 2 file(s) modified
carry bag for silver at 1

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/cart/CartServiceImpl.java
~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/V2CartController.java

Rev 37304 – v2 version some fixes

ranu — Thu, 13 Aug 2026 17:39:13 +0530

ranu – 1 file(s) modified
v2 version some fixes

~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/V2CartController.java

Rev 37303 – one assist ew at 99 up to 20k

ranu — Thu, 13 Aug 2026 15:52:50 +0530

ranu – 1 file(s) modified
one assist ew at 99 up to 20k

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/service/ScratchService.java

Rev 37302 – Split refurbished (margin scheme) items into their own order at ...

amit — Thu, 13 Aug 2026 13:55:10 +0530

amit – 3 file(s) modified
Split refurbished (margin scheme) items into their own order at placement

Refurbished lines are invoiced on the RF- series, but only the partner checkout path
split them out. Internal POs and bulk orders raised a single mixed transaction, which
billed as one invoice on whichever series its first line happened to resolve to
(NSPJB2757, NSPJB3212, NSUPDL3898/3899/4365/4366).

- PurchaseOrderServiceImpl: split a mixed internal PO into a regular PO and a margin
scheme PO, each with its own PO number, PDF and internal transaction. External vendor
POs raise no transaction, so they are left intact.
- BulkOrderService: raise one transaction per billing group, each with its own loan
sized to the wallet balance left to it. Pricing, combo, credit availability and the
first-PO gates still validate the order as a whole, so a split first PO cannot block
its own second half.
- TransactionServiceImpl: drop the float narrowing of walletUsed in
createTransactionForCartLines - the field is a double and the cart total is validated
within 0.001, so any total carrying paise was rejected.

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/order/BulkOrderService.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/transaction/TransactionServiceImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/warehouse/PurchaseOrderServiceImpl.java

Rev 37301 – one assist ew at 99 up to 20k

ranu — Thu, 13 Aug 2026 13:34:58 +0530

ranu – 1 file(s) modified
one assist ew at 99 up to 20k

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/service/ScratchService.java

Rev 37300 – one assist ew at 99 up to 20k

ranu — Wed, 12 Aug 2026 23:56:14 +0530

ranu – 1 file(s) modified
one assist ew at 99 up to 20k

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/integrations/oneassist/OneAssistService.java

Rev 37299 – scratch offer live

ranu — Wed, 12 Aug 2026 23:32:17 +0530

ranu – 1 file(s) modified
scratch offer live

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/service/ScratchService.java

Rev 37298 – Default subCategoryId to 0 on endpoints that declare it required=false ...

amit — Wed, 12 Aug 2026 22:10:01 +0530

amit – 4 file(s) modified
Default subCategoryId to 0 on endpoints that declare it required=false

subCategoryId is a primitive int, so a required=false declaration with no
defaultValue makes Spring throw when the param is omitted:

Optional int parameter 'subCategoryId' is present but cannot be translated
into a null value due to being declared as a primitive type.

v1 DealsController and GuestController already default it to 0; their v2
wrappers dropped that, and StoreController never had it on either version. Any
caller omitting the param got a 500 with a message that does not name the
parameter at fault.

Adding the default only removes that failure path — every currently working
caller already sends a value, so no existing behaviour changes.

~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/controller/StoreController.java
~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/V2DealsController.java
~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/V2GuestController.java
~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/V2StoreController.java

Rev 37297 – 13 to 16 ew at 99 rs u to 20k

ranu — Wed, 12 Aug 2026 15:32:19 +0530

ranu – 1 file(s) modified
13 to 16 ew at 99 rs u to 20k

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/integrations/oneassist/OneAssistService.java

Rev 37296 – external api v2: docs - color removed from all record ...

amit — Wed, 12 Aug 2026 15:26:22 +0530

amit – 1 file(s) modified
external api v2: docs - color removed from all record shapes

~ /trunk/profitmandi-web/src/main/resources/external-api-docs.html

Rev 37295 – external api v2: never expose item color on partner APIs ...

amit — Wed, 12 Aug 2026 15:26:21 +0530

amit – 4 file(s) modified
external api v2: never expose item color on partner APIs (filter at API layer; catalog.item untouched)

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/model/external/CatalogSkuModel.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/model/external/StoreSkuModel.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/external/ExternalFeedRepositoryImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/external/ExternalApiServiceImpl.java

Rev 37294 – all active brand in hot deals

ranu — Wed, 12 Aug 2026 13:07:21 +0530

ranu – 1 file(s) modified
all active brand in hot deals

~ /trunk/profitmandi-fofo/src/main/webapp/WEB-INF/views/ftl/partner-readonly-info.vm

Rev 37293 – Fix region-ALL (fofoId=0 sentinel) handling on RBM home active-offers: ...

amit — Wed, 12 Aug 2026 11:58:47 +0530

amit – 2 file(s) modified
Fix region-ALL (fofoId=0 sentinel) handling on RBM home

active-offers: regionId resolved via rows.get(0).getFofoId(), which is 0 for
region ALL, so selectByRetailerId(0) threw FOFOSTR_1000 'Fofo store not found'
(400) for every region-ALL user. Now filters fofoId>0, picks an active store,
and falls back to the caller's fofoId when a region resolves to nothing.

rbm-regions: dropdown now lists only regions holding an active retailer (with
activePartners count), which drops the ALL pseudo-region; users carrying the
sentinel get every live region as assignedRegionIds instead of an empty set.

rbm-dashboard: expand the sentinel like CsServiceImpl does (row region ->
its partners; region ALL -> all active stores) so the partners tile stops
reporting 0/1. Scope-wide batches limited to stores/targets/purchases/
last-billed; name, wallet and loan lookups now run only over the returned
slice. partners and alerts capped via ?limit (default 50, max 200) with
partnersReturned/totalAlerts/alertsReturned/limit added to summary. Partners
with no target this month rank after those actually missing a target, so the
capped slice is not filled by unset-target stores.

~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/fofo/V2FofoMonitorController.java
~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/fofo/V2FofoOfferController.java

Rev 37292 – Hot deals: javadoc matches reindex-based sync syncHotDealFlags no longer ...

amit — Tue, 11 Aug 2026 15:59:29 +0530

amit – 2 file(s) modified
Hot deals: javadoc matches reindex-based sync

syncHotDealFlags no longer mirrors model_hot_deal into tag_listing.hot_deals;
since r37276 hot_deal_b is derived from the active window at index time, so the
daily job re-indexes models whose window opened today or closed yesterday.
Comment-only, no behaviour change.

~ /trunk/profitmandi-cron/src/main/java/com/smartdukaan/cron/scheduled/ScheduledSkeleton.java
~ /trunk/profitmandi-cron/src/main/java/com/smartdukaan/cron/scheduled/ScheduledTasks.java

Rev 37291 – external api v2: docs - parameters-at-a-glance table, per-endpoint optional/required params, ...

amit — Tue, 11 Aug 2026 11:39:19 +0530

amit – 1 file(s) modified
external api v2: docs - parameters-at-a-glance table, per-endpoint optional/required params, clamp semantics

~ /trunk/profitmandi-web/src/main/resources/external-api-docs.html

Rev 37290 – Credit Note IRN: credit only the returned units, not the ...

amit — Tue, 11 Aug 2026 02:29:47 +0530

amit – 3 file(s) modified
Credit Note IRN: credit only the returned units, not the whole invoice

refundOrder persisted the CreditNote and its lines from the returned
inventory items, but then called generateCreditNoteIrn(invoiceNumber, ...),
which rebuilt the NIC payload by re-reading the original invoice - every
line at billed quantity. A 1-pc DOA return against NSDL37092 therefore
tried to register a CN covering the full invoice (2 phones + 27 carry
bags, Rs 26,664.27) against a Rs 13,332 wallet refund, and the legacy
4-digit HSN on the carry-bag line failed NIC validation with error 2311.

The wallet credit, the CreditNote row and its lines were always correct;
only the e-invoice payload was wrong.

- InvoiceService.getInvoicePdfModelForIrn: optional returnedQtyByOrderId
restricts the item list to the returned orders and prices each line at
the returned quantity. The existing single-arg method delegates with
null, so invoice PDF generation is unchanged. Margin-scheme, delivery
challan, IMEI-suffix and HSN handling apply to the reduced line as-is.
- GstProService.generateCreditNoteIrn: 4-arg overload taking the map; the
3-arg version delegates with null for whole-invoice returns.
- PurchaseReturnServiceImpl.refundOrder: passes orderReturnQtyMap, which
was already built for the ReturnOrderInfo rows.

Verified against the NIC sandbox using the real NSDL37092 rows: the old
payload is rejected with error 2311, the new payload is accepted
(DocTyp CRN, ItemCnt 1, MainHsnCode 85171300, TotInvVal 13331.99).

Not covered here: applyInvoiceReturnViaCreditNote still sends the whole
invoice, so an invoice partially returned earlier re-credits those units.

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/integrations/gstpro/GstProService.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/inventory/PurchaseReturnServiceImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/transaction/invoicing/InvoiceService.java

Rev 37289 – external api v2: space-separated ram/storage pair in catalog titles (refurb ...

amit — Mon, 10 Aug 2026 19:58:46 +0530

amit – 2 file(s) modified
external api v2: space-separated ram/storage pair in catalog titles (refurb naming)

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/catalog/CategorySpecDerivationService.java
~ /trunk/profitmandi-dao/src/test/java/com/spice/profitmandi/service/catalog/CategorySpecDerivationServiceTest.java

Rev 37288 – all active brand in hot deals

ranu — Mon, 10 Aug 2026 18:22:17 +0530

ranu – 1 file(s) modified
all active brand in hot deals

~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/HotDealController.java

Rev 37287 – warehouse access

ranu — Mon, 10 Aug 2026 18:00:04 +0530

ranu – 3 file(s) modified
warehouse access

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/AdminUser.java
~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/DashboardController.java
~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/LeadController.java

Rev 37286 – warehouse access

ranu — Mon, 10 Aug 2026 17:46:01 +0530

ranu – 2 file(s) modified
warehouse access

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/AdminUser.java
~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/DashboardController.java

Rev 37285 – warehouse access

ranu — Mon, 10 Aug 2026 17:19:08 +0530

ranu – 2 file(s) modified
warehouse access

~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/DashboardController.java
~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/InventoryController.java

Rev 37284 – external api v2: derive specs from catalog brand/model title (pair ...

amit — Mon, 10 Aug 2026 16:53:17 +0530

amit – 3 file(s) modified
external api v2: derive specs from catalog brand/model title (pair + lone-storage forms) as last resort; migration file marks applied state, legacy update() note

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/catalog/CategorySpecDerivationService.java
~ /trunk/profitmandi-dao/src/main/resources/sql/migration_external_api_v2.sql
~ /trunk/profitmandi-dao/src/test/java/com/spice/profitmandi/service/catalog/CategorySpecDerivationServiceTest.java

Rev 37283 – hot deals: cart/pricing availability reads date-windowed model_hot_deal instead of tag_listing.hot_deals

amit — Mon, 10 Aug 2026 16:50:54 +0530

amit – 2 file(s) modified
hot deals: cart/pricing availability reads date-windowed model_hot_deal instead of tag_listing.hot_deals

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/cart/CartServiceImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/pricing/PricingServiceImpl.java

Rev 37282 – hot deals: remove legacy tag_listing hot-deals endpoints and catalog-item UI ...

amit — Mon, 10 Aug 2026 16:50:53 +0530

amit – 3 file(s) modified
hot deals: remove legacy tag_listing hot-deals endpoints and catalog-item UI (replaced by model_hot_deal admin flow)

~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/IndentController.java
~ /trunk/profitmandi-fofo/src/main/webapp/resources/js/common.js
~ /trunk/profitmandi-fofo/src/main/webapp/WEB-INF/views/ftl/catalog-item.vm

Rev 37281 – external api v2: storeMaster visibility by fofo_type=FRANCHISE (drop internal flag ...

amit — Mon, 10 Aug 2026 16:22:16 +0530

amit – 4 file(s) modified
external api v2: storeMaster visibility by fofo_type=FRANCHISE (drop internal flag check); spec derivation supports MB sizes (feature phones)

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/external/ExternalFeedRepository.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/external/ExternalFeedRepositoryImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/catalog/CategorySpecDerivationService.java
~ /trunk/profitmandi-dao/src/test/java/com/spice/profitmandi/service/catalog/CategorySpecDerivationServiceTest.java

Rev 37280 – external api v2: document addedOn field for new-SKU partial sync

amit — Mon, 10 Aug 2026 15:06:42 +0530

amit – 1 file(s) modified
external api v2: document addedOn field for new-SKU partial sync

~ /trunk/profitmandi-web/src/main/resources/external-api-docs.html

Rev 37279 – external api v2: addedOn (item first-added epoch) in catalogSkuMaster/catalogSkus rows

amit — Mon, 10 Aug 2026 15:06:41 +0530

amit – 2 file(s) modified
external api v2: addedOn (item first-added epoch) in catalogSkuMaster/catalogSkus rows

Rev 37278 – warehouse access

ranu — Mon, 10 Aug 2026 15:01:03 +0530

ranu – 1 file(s) modified
warehouse access

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/AdminUser.java

Rev 37277 – Updated Escalation Level

vikas — Mon, 10 Aug 2026 12:26:59 +0530

vikas – 5 file(s) modified
Updated Escalation Level

~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/BeatJourneyController.java
~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/BeatPlanController.java
~ /trunk/profitmandi-fofo/src/main/webapp/resources/js/beat-plan-app.js
~ /trunk/profitmandi-fofo/src/main/webapp/WEB-INF/views/ftl/beat-journey.vm
~ /trunk/profitmandi-fofo/src/main/webapp/WEB-INF/views/ftl/beat-plan-window.vm

Rev 37276 – Hot deals Solr: derive hot_deal_b from model_hot_deal active windows at ...

amit — Mon, 10 Aug 2026 11:44:16 +0530

amit – 4 file(s) modified
Hot deals Solr: derive hot_deal_b from model_hot_deal active windows at index time (full push, single-catalog update, parent + child docs) - catalog.model_hot_deal is the only source, tag_listing.hot_deals no longer read; drop unused tag_listing write-through methods; syncHotDealFlags javadoc reflects reindex-based daily sync

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/catalog/TagListingRepository.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/catalog/TagListingRepositoryImpl.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/service/solr/FofoSolr.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/catalog/ModelHotDealService.java

Rev 37275 – Add ModelHotDeal window-boundary and active catalog-id lookups r37268 committed ...

amit — Mon, 10 Aug 2026 11:35:10 +0530

amit – 2 file(s) modified
Add ModelHotDeal window-boundary and active catalog-id lookups

r37268 committed the ModelHotDealServiceImpl call to
selectWindowBoundaryCatalogIds but not the repository declaration or
implementation, breaking the trunk build with 'cannot find symbol'.

Adds both methods:
- selectWindowBoundaryCatalogIds(today): catalog ids whose deal window opens
today or closed yesterday, i.e. the models whose hot_deal_b flips at
midnight and need a Solr re-index.
- selectActiveCatalogIds(today): catalog ids with a deal live on the given
date, the Solr hot_deal_b source (caller in FofoSolr still pending).

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/catalog/ModelHotDealRepository.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/catalog/ModelHotDealRepositoryImpl.java

Rev 37274 – Freeze margin-scheme purchase price on the invoice line Refurbished ...

amit — Mon, 10 Aug 2026 11:20:42 +0530

amit – 2 file(s) modified
Freeze margin-scheme purchase price on the invoice line

Refurbished (GST Rule 32(5)) lines re-derived their purchase price by serial on
every read, resolving to the LATEST warehouse.inventoryItem row. A returned
device gets re-GRNed, sometimes at a different price, so a credit note or a
regenerated PDF could reverse a margin the invoice never charged. 11 already-
billed lines resolved to a price they were not sold at; IMEI 353917853695268
has been sold on six invoices at two different purchase prices.

InvoiceService.freezeMarginPurchasePrices now stamps the value onto
transaction.lineitem.margin_purchase_price at billing time (saveInvoice step 0,
inside the billing transaction), and resolveMarginPurchasePrice reads the frozen
value thereafter, falling back to the live lookup only when null so pre-existing
lines behave exactly as before.

Requires the column from db-scripts/add_margin_purchase_price_column.sql, which
must be applied before this is deployed.

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/entity/transaction/LineItem.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/transaction/invoicing/InvoiceService.java

Rev 37273 – Apply same idempotent super-catalog mapping save to V2 controller ...

amit — Mon, 10 Aug 2026 11:12:34 +0530

amit – 1 file(s) modified
Apply same idempotent super-catalog mapping save to V2 controller

Keeps V2FofoSuperCatalogController in step with the fofo SuperCatalogController
fix (r37272) so the dormant endpoint cannot violate the new
UNIQUE(catalog_id, super_catalog_id) constraint if it is ever enabled.

~ /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/controller/fofo/V2FofoSuperCatalogController.java

Rev 37272 – Fix duplicate super-catalog mapping rows on save The add-row ...

amit — Mon, 10 Aug 2026 11:12:26 +0530

amit – 1 file(s) modified
Fix duplicate super-catalog mapping rows on save

The add-row flow in super_catalog.js always posts mapping id 0, so saving a
catalog that was already mapped inserted a second row. That duplicated the
catalog inside superCatalogVariants_s, which surfaced as repeated storage
variant chips on the partner catalog card.

processCatalogMapping now falls back to a (catalogId, superCatalogId) lookup
when no mapping id is supplied and updates the existing row instead of
inserting. Also removes a latent NPE when a stale mapping id was posted.

Backed by a new UNIQUE(catalog_id, super_catalog_id) on
catalog.super_catalog_mapping; 5 pre-existing surplus rows were removed.

~ /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/SuperCatalogController.java

Rev 37271 – Add selectMappingByCatalogIdAndSuperCatalogId to SuperCatalogMappingRepository Lookup by the (catalogId, superCatalogId) ...

amit — Mon, 10 Aug 2026 11:12:16 +0530

amit – 2 file(s) modified
Add selectMappingByCatalogIdAndSuperCatalogId to SuperCatalogMappingRepository

Lookup by the (catalogId, superCatalogId) pair so the super-catalog save
flow can detect an existing mapping instead of always inserting.

~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/catalog/SuperCatalogMappingRepository.java
~ /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/catalog/SuperCatalogMappingRepositoryImpl.java