Subversion Repositories SmartDukaan

Rev

Show changed files | Directory listing | RSS feed

Filtering Options

Rev Age Author Path Log message Diff
37760 3 h 11 m vikas /trunk/profitmandi-fofo/src/main/ Upload directory and Data correction for lead  
37759 3 h 14 m vikas /trunk/ LMS Call for App  
37758 3 h 19 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/model/ aging po approval process  
37757 3 h 20 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/model/ aging po approval process  
37756 3 h 22 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/warehouse/ aging po approval process  
37755 3 h 23 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/service/warehouse/ aging po approval process  
37754 3 h 24 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/entity/warehouse/ aging po approval process  
37753 3 h 25 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/warehouse/ aging po approval process  
37752 3 h 27 m ranu /trunk/profitmandi-dao/src/main/java/com/spice/profitmandi/dao/repository/warehouse/ today po rbm view showing only for l7 and above  
37751 3 h 30 m ranu /trunk/ aging po approval mail to some collected users  
37750 3 h 56 m ranu /trunk/ aging sku purchasing need to approval of niranjan kala sir  
37749 4 h 46 m vikas /trunk/ Upload directory and Data correction for lead  
37748 6 h 56 m ranu /trunk/profitmandi-fofo/src/main/ today po rbm view showing only for l7 and above  
37747 7 h 30 m ranu /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/monitors/ today po rbm view showing only for l7 and above  
37746 7 h 42 m amit /trunk/profitmandi-fofo/src/main/java/com/spice/profitmandi/web/controller/ feat(scheme): add Ear Buds category (10024) to scheme creation category list  
37745 7 h 42 m ranu /trunk/profitmandi-fofo/src/main/ today po rbm view showing only for l7 and above  
37744 9 h 58 m amit /trunk/profitmandi-fofo/src/test/java/com/spice/profitmandi/service/warehouse/ test(warehouse): raising a movement order stamps the line with the stock's cost and vendor

Covers the last untested link in the cost-layer chain: PurchaseOrderServiceImpl writing lineitem.origin_vendor_id and the line price, which is what GRN copies onto received stock.
 
37743 2 d 4 h vikas /trunk/ LMS checklist for call  
37742 2 d 5 h amit /trunk/profitmandi-web/src/main/java/com/spice/profitmandi/web/v2/exception/ v2: silence client disconnects and demote 4xx, matching the v1 taxonomy

r37728 added a client-disconnect handler to the v1 GlobalExceptionHandler and the
flood did not stop: #39 went on adding events, and every one of a 50-event sample
was from a /v2/ URL. V2GlobalExceptionHandler is scoped
basePackages="com.spice.profitmandi.web.v2", so v2 responses never reached the v1
handler at all and fell through to its own handleGenericException at ERROR --
~19,900 events, the second largest source on the board.

Worth recording because the first diagnosis was wrong. The theory was that an
exception raised while the response body is written cannot reach an @ExceptionHandler
because the response is already committed. It can: DispatcherServlet catches it out of
ha.handle() and still runs the resolvers. What it cannot do afterwards is WRITE the
substitute response. The handler runs and logs either way, which is precisely why
patching the wrong advice class changed nothing.

Adds @ExceptionHandler(IOException.class) here, delegating to the same
isClientDisconnect test the v1 handler uses -- duplicated rather than shared because
catalina is provided by the container and is not on this module's compile classpath,
so ClientAbortException cannot be imported and is matched on simple name plus the
messages a dead peer actually produces. A genuine IOException (full disk, a broken
pipe to something that is not the client) matches none of them and keeps its ERROR
and its 500. Returns null for a disconnect: the connection that would carry a body is
already gone, and writing to it is what raised this.

Also demotes five sibling handlers in the same file that were logging caller mistakes
at ERROR, which v1 has logged at WARN since the taxonomy work: business validation,
missing parameter, type mismatch, malformed body, method not supported, media type
not supported. v2 never received that pass. The business line also stops attaching a
stack trace -- a validation has nothing useful in one.

Left at ERROR deliberately, because they are real defects: IllegalArgument, NPE,
genuine IO failure, and the two catch-alls.
 
37741 2 d 7 h amit /trunk/profitmandi-fofo/src/main/webapp/WEB-INF/views/ftl/ fix(fofo): show Pine Labs monthly EMI in rupees, not paise

Plural sends Money.value in paise; the EMI offers modal printed it raw
(185959 INR instead of Rs 1,859.59). Converted at the view only - the
web API hands the same DTO to the app, which expects paise.
 

Show All