ppapou commented on a change in pull request #770:
URL: https://github.com/apache/incubator-dlab/pull/770#discussion_r438502880
##########
File path:
services/self-service/src/main/java/com/epam/dlab/backendapi/util/BillingUtils.java
##########
@@ -212,12 +227,20 @@ public static String printLine(BillingReportLine line,
boolean isFull) {
return CSVFormatter.formatLine(lines, CSVFormatter.SEPARATOR);
}
- public static String getTotal(Double total, String currency) {
+ /**
+ *
+ * @param total monetary amount
+ * @param currency user's currency
+ * @param stringOfAdjustedHeader filtered fields of report header
+ * @return line with cost of resources
+ */
+ public static String getTotal(Double total, String currency, String
stringOfAdjustedHeader) {
Review comment:
As BillingUtilils.getHeader() returns the report header as a string,it
should be split by the CSVFormatter.SEPARATOR to retrieve the length of the
header.
As CSVFormatted is used in BillingUtils class, I decided to encapsulate the
the split procedure
in the getTotal method
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@dlab.apache.org
For additional commands, e-mail: dev-h...@dlab.apache.org
