From fc95ece6cbc4f9d6e633e025b8efb97525175783 Mon Sep 17 00:00:00 2001 From: Abhinandan Prateek Date: Mon, 3 Aug 2015 14:15:31 +0530 Subject: [PATCH] quota: fixes --- .../api/response/QuotaResponseBuilderImpl.java | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/plugins/database/quota/src/org/apache/cloudstack/api/response/QuotaResponseBuilderImpl.java b/plugins/database/quota/src/org/apache/cloudstack/api/response/QuotaResponseBuilderImpl.java index bea740f1f38..104aa6b3c1f 100644 --- a/plugins/database/quota/src/org/apache/cloudstack/api/response/QuotaResponseBuilderImpl.java +++ b/plugins/database/quota/src/org/apache/cloudstack/api/response/QuotaResponseBuilderImpl.java @@ -110,18 +110,15 @@ public class QuotaResponseBuilderImpl implements QuotaResponseBuilder { QuotaBalanceResponse resp = new QuotaBalanceResponse(); BigDecimal lastCredits = new BigDecimal(0); - boolean consecutive = true; for (Iterator it = quotaBalance.iterator(); it.hasNext();) { QuotaBalanceVO entry = it.next(); s_logger.info("createQuotaBalanceResponse: Date=" + entry.getUpdatedOn().toGMTString() + " balance=" + entry.getCreditBalance() + " credit=" + entry.getCreditsId()); if (entry.getCreditsId() > 0) { - if (consecutive) { - lastCredits = lastCredits.add(entry.getCreditBalance()); - } + lastCredits = lastCredits.add(entry.getCreditBalance()); resp.addCredits(entry); it.remove(); } else { - consecutive = false; + break; } } @@ -145,7 +142,7 @@ public class QuotaResponseBuilderImpl implements QuotaResponseBuilder { @Override public QuotaStatementResponse createQuotaStatementResponse(final List quotaUsage) { if (quotaUsage == null || quotaUsage.size() == 0) { - new InvalidParameterValueException("There is no uage data for period mentioned."); + throw new InvalidParameterValueException("There is no uage data for period mentioned."); } final short opendb = TransactionLegacy.currentTxn().getDatabaseId(); TransactionLegacy.open(TransactionLegacy.USAGE_DB).close();