[GitHub] [fineract] vorburger commented on a change in pull request #1123: FINERACT-854 Removed string concatenated SQL from CenterReadPlatformS…

[GitBox <gi...@apache.org>]

vorburger commented on a change in pull request #1123:
URL: https://github.com/apache/fineract/pull/1123#discussion_r449616181



##########
File path: fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/utils/SQLBuilder.java
##########
@@ -110,6 +111,39 @@ public void addNonNullCriteria(String criteria, Object argument) {
         }
     }
 
+    /**
+     * Adds the limit statement in the end of WHERE clause
+     *
+     * @param limit
+     *            The value that will be used as limit
+     */
+    public void setLimit(Object limit) {
+        sb.append(" LIMIT ? ");
+        args.add(limit);
+    }
+
+    /**
+     * Adds the offset statement in the end of WHERE clause
+     *
+     * @param offset
+     *            The value that will be used as offset
+     */
+    public void setOffset(Object offset) {
+        sb.append(" OFFSET ? ");
+        args.add(offset);
+    }
+
+    /**
+     * Adds the orderBy statement in the end of WHERE clause
+     *
+     * @param orderBy
+     *            The value that will be used as orderBy
+     */
+    public void addOrderBY(Object orderBy) {

Review comment:
       lower-case _y_:
   
   ```suggestion
       public void addOrderBy(Object orderBy) {
   ```




----------------------------------------------------------------
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:
users@infra.apache.org