| Rev |
Age |
Author |
Path |
Log message |
Diff |
| 37489 |
2 d 9 h |
amit |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
Restore Utils.html and Utils.htmlJson removed by r37477
r37477 reorganised Utils while consolidating the mail APIs and dropped its only two
instance methods. They have no Java callers -- they exist solely for Velocity, where
AppConfig binds new Utils() as $vmUtils -- so the compiler, IDE find-usages and any
grep over *.java all reported them dead.
With htmlJson gone, data-paramslist="$vmUtils.htmlJson(...)" stopped rendering usable
JSON on every report link (admin.vm:840, admin.vm:910, reports.vm:51). reports.js:7
then read $(this).data("paramslist") as undefined and returned true, letting the
browser follow the plain <a href> natively -- a GET against the @PostMapping
/reports/{projectName}/{fileName} (ReportsController:126), answered with GE_1007
"Request method 'GET' not supported". The modal POST path was the only way that
endpoint was ever reachable, so every partner report broke at once.
Confirmed against prod: zero occurrences in fofo.log for Aug 13/20/26/28/30, and 87
today starting 18:37:23, right after today's ROOT.war deploy. 13 distinct reports hit.
$vmUtils.html had two further callers that also come back:
offer_margin_detail_partner.vm:12 and :241 (offer description and notes).
Restored verbatim from r37476, and commented as template-only so the next refactor
does not read them as dead code again. |
|
| 37477 |
2 d 11 h |
amit |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/ |
mail: one way to send mail, and nothing sends inline
There were three overlapping APIs -- Utils.sendMail* (7 static overloads),
EmailServiceImpl.sendMail* (7 more) and MailOutboxService.queueMail* (20). The
first two built their own MimeMessage, hardcoded From: noreply@smartdukaan.com
in eleven places, and pushed straight down whichever JavaMailSender the caller
passed.
That From is why mail was refused: an authenticated Workspace session may send
only as the account it logged in as, and noreply@ is a different identity in the
same domain. Sending inline also meant a failed send was lost outright and took
the calling job down with it, which is how one expired credential came to mark
ten report jobs FAILED.
New in profitmandi-common:
MailQueue - the single entry point; queue() returns once the mail is
recorded, not once it is sent
MailRequest - one value object replacing the 34 overloads
MailSenderType - GOOGLE (authenticated, From rewritten) or RELAY (IP-authorised)
The interface lives in common while MailOutboxService implements it in dao,
because dao depends on common and not the reverse -- Utils and EmailService
could not otherwise reach the outbox at all.
Utils and EmailService keep their signatures so the ~38 call sites still
compile, but now delegate and ignore the JavaMailSender argument; choosing a
transport was never the caller's business. Both are marked deprecated.
MailQueueHolder bridges the static helpers to the bean and is documented as a
compromise, not a pattern. |
|
| 36623 |
104 d 14 h |
amit |
/trunk/profitmandi-common/ |
Remove ThriftConfig dependency and Thrift client wrappers
- Removed ThriftConfig from build.gradle dependency
- Deleted entire thrift/clients/ directory (16 RPC client files)
- Removed Thrift-dependent methods from Utils.java (getRechargeDisplayStatus, getOrderStatus, getWarehouseClient, etc.)
- Deleted dead code: HelperServiceUtils.java, OrderObject.java
- Moved to profitmandi-dao: WalletHistoryModel.java, ExcelUtils.java, CustomOrder.java |
|
| 35204 |
322 d 12 h |
amit |
/trunk/ |
Fixed changes related to downloading file at home page |
|
| 35102 |
335 d 20 h |
amit |
/trunk/ |
Added csvn report |
|
| 34568 |
475 d 10 h |
vikas.jangra |
/trunk/ |
Liquidation/Bidding push notification and Publish Email |
|
| 33354 |
845 d 16 h |
amit.gupta |
/trunk/ |
Enhanced Sidbi Sanctions |
|
| 33231 |
869 d 7 h |
amit.gupta |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
Fixed Utils removed Thrift dependencies |
|
| 33045 |
912 d 16 h |
amit.gupta |
/trunk/ |
Added sidbi and improvements in credit note |
|
| 32955 |
932 d 16 h |
amit.gupta |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
Fixed changes |
|
| 32724 |
995 d 9 h |
amit.gupta |
/trunk/ |
Added changes related to Loan Interest correction and ReturnOrder |
|
| 32199 |
1148 d 8 h |
amit.gupta |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
Fixed additional scheme logic |
|
| 31903 |
1216 d 17 h |
amit.gupta |
/trunk/ |
Added changes related to new scheme rollout policy |
|
| 31697 |
1272 d 13 h |
amit.gupta |
/trunk/ |
Fixed sales margin logic |
|
| 31177 |
1381 d 16 h |
tejbeer |
/trunk/ |
change |
|
| 30122 |
1642 d 13 h |
amit.gupta |
/trunk/ |
Fixed issue |
|
| 29552 |
1763 d 9 h |
amit.gupta |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
|
|
| 28987 |
1841 d 17 h |
amit.gupta |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
|
|
| 28925 |
1847 d 10 h |
amit.gupta |
/trunk/profitmandi-common/src/main/java/com/spice/profitmandi/common/util/ |
|
|
| 28395 |
1938 d 9 h |
amit.gupta |
/trunk/ |
Added changes for activation schemes |
|