Showing AI reviews for tenant-service
Project AI Score ?
67%
Quality Avg ?
69%
Security Avg ?
64%
Reviews ?
143

Review Result ?

solutionbowl/tenant-service · 173e6e98
This commit removes previously commented out logic related to payout calculation and replaces it with a simpler calculation. The change is partial and lacks context, reducing clarity. The commented-out code should either be removed cleanly or fully restored with clear explanations to avoid confusion. The commit message is vague and uninformative.
Quality ?
40%
Security ?
80%
Business Value ?
30%
Maintainability ?
40%
Issues & Suggested Fixes ?
1AI detects the issue
2Shows file, line, or evidence
3Suggests the fix to apply
Lacks Clarity And Detail
Where commit message
Issue / Evidence lacks clarity and detail
Suggested Fix provide a clear, descriptive message explaining what was fixed and why
Unclear Commented Code Mixed With Active C...
Where src/controllers/horseracing/result.controller.js:153-161
Issue / Evidence unclear commented code mixed with active code
Suggested Fix remove unused comments or restore full logic with comments to improve maintainability and clarity
Code Change Preview · src/controllers/horseracing/result.controller.js ?
Removed / Before Commit
- diff --git a/src/controllers/horseracing/result.controller.js b/src/controllers/horseracing/result.controller.js
- index 0223724..ca23659 100644
- }
- } else if (oddsType === "lay") {
- // Lay only on IndiaManual
-           deductionAmount = (Number(bet.bet_amount || 0) * deductionPercent) / 100;
-           finalCreditAmount = Number(bet.bet_amount || 0) - deductionAmount;
-           payout = Number(bet.liability_amount || 0) + finalCreditAmount;
-           payoutAdded = Number(bet.liability_amount || 0) + finalCreditAmount;
- }
- 
- betBulkOps.push({
Added / After Commit
+ diff --git a/src/controllers/horseracing/result.controller.js b/src/controllers/horseracing/result.controller.js
+ index 0223724..ca23659 100644
+ }
+ } else if (oddsType === "lay") {
+ // Lay only on IndiaManual
+           // if (isPlaced) {
+           //   deductionAmount = (Number(bet.bet_amount || 0) * deductionPercent) / 100;
+           //   finalCreditAmount = Number(bet.bet_amount || 0) - deductionAmount;
+           //   payout = Number(bet.liability_amount || 0) + finalCreditAmount;
+           //   payoutAdded = Number(bet.liability_amount || 0) + finalCreditAmount;
+           // } else {
+           payout = Number(bet.liability_amount || 0) + bet.bet_amount;
+           payoutAdded = Number(bet.liability_amount || 0) + bet.bet_amount;
+           // }
+ }
+ 
+ betBulkOps.push({