com.mysema.query.types.expr.EString API contains methods with differing signatures

Bug #624618 reported by Vesa Marttila
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Querydsl
Fix Released
Undecided
Unassigned

Bug Description

For instance the methods contains, startsWith and endsWith accept a boolean
value to enable case insensitive querying, but for equality
com.mysema.query.types.expr.EString has two separate methods equals and
equalsIgnoreCase.

In my opinion it would be better to follow the latter style for all the methods
because

where(person.username.startsWithIgnoreCase(username))

is a lot more readable than

where(person.username.startsWith(username, false))

java.lang.String follows the equalsIgnoreCase style as well.

Changed in querydsl:
milestone: none → 2.0
Revision history for this message
Timo Westkämper (timo-westkamper) wrote :

Fixed like described in SVN trunk. The old methods are deprecated for now and will be removed in 2.0.

Changed in querydsl:
status: New → Fix Committed
Revision history for this message
Timo Westkämper (timo-westkamper) wrote :

Released in 1.9.2

Changed in querydsl:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.