marthym / juery Goto Github PK

It could be interesting to allow range pagination.

• choose query string parameter name for 'from' and 'to' element pagination
• Improve PageRequestFormatter to parse this parameters

If no parameters was passed in the query string the parser start offset to 1 instead of 0.

• Change lombok properties to generate getter without prefix. This is very less verbose.
• Remove all public class properties. This avoid problem with deprecation of renamed fields

On nested Criteria, the criteria of n-level was not applyed by the visitor

Currently only equals, startWith, endWith and contains was parsed from query string.

It could be interresting to have lower and greater than operation too

When parsing querystring, if a contains operation have for value t or f or off, value type is parsed as Boolean instead of string.

This can produce error, contains as no sense for boolean or numerical or temporal values

The Visitor pattern was not properly implemented. Need to replace visit method by accept.

Default values for pagination all was -1.

It is not very usefull, using 0 and Integer.MAX_VALUE allow using it directly in request

If no field in mongo are ObjectId and/or all the field in entity was the same as mongo schema, no additionnal configuration was need. So it can be a good feature to allow creating MongoFilterVisitor with parameters.

It is more usefull to use the mongo field name as ObjectId field instead of the criteria property name.

It's time to move the api class to record and remove deprecated api

The JooqPagination doen’t need all the PageRequest but only pagination data.

Refactor to group pagination data will allow to transfert only this data to JooqPagination.

To manage in operator from Query String juery must be able to parse array.

Allow users to define custom query string parameters

Need to improve documentation with real and complete examples of using Juery

Add main features section

Currently, only Number, Boolean and String was parsed from query string.

It could be interresting to have date and time temporal too.