Merge pull request #864 from NSSuresh11/OLE-8961

OLE-8961 : LOST NOTICE batch job billing patrons multiple times when run
openlibraryenvironment · Aug 12, 2016 · 5d29a91 · 5d29a91
2 parents c82d103 + a57415e
commit 5d29a91
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/ole-app/olefs/src/main/java/org/kuali/ole/deliver/service/LostNoticesExecutor.java b/ole-app/olefs/src/main/java/org/kuali/ole/deliver/service/LostNoticesExecutor.java
@@ -85,7 +85,9 @@ private BigDecimal getFineAmount(OleLoanDocument oleLoanDocument) {
             OLEDeliverNotice deliverNotice = iterator.next();
             if (deliverNotice.getNoticeType().equals(OLEConstants.NOTICE_LOST)) {
                 BigDecimal replacementFeeAmount = deliverNotice.getReplacementFeeAmount();
-                feeAmount = feeAmount.add(replacementFeeAmount);
+                if(replacementFeeAmount!=null) {
+                    feeAmount = feeAmount.add(replacementFeeAmount);
+                }
             }
         }
         return feeAmount;