vidakovic commented on code in PR #2455:
URL: https://github.com/apache/fineract/pull/2455#discussion_r933381461


##########
fineract-provider/src/main/java/org/apache/fineract/accounting/glaccount/data/GLAccountData.java:
##########
@@ -132,14 +135,6 @@ private GLAccountData(String name, Long parentId, String 
glCode, Boolean manualE
         this.allowedExpensesTagOptions = null;
     }
 
-    public Integer getRowIndex() {
-        return rowIndex;
-    }
-
-    public CodeValueData getTagId() {
-        return tagId;
-    }
-
     public GLAccountData(final Long id, final String name, final Long 
parentId, final String glCode, final boolean disabled,

Review Comment:
   @hmg65 you know that when you add the annotation for the constructor then 
you have to remove the existing? Other than that I don't think another error is 
possible... this is pretty straight forward...



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to