Using literals for these criterions actually had some unexpected performance implications, and they weren't super useful anyways. This PR removes them and updates a few things to replace them:
Passing a null criterion to methods like deleteWhere or update is now equivalent to "delete all" and "update all". Accordingly, new convenience methods deleteAll and updateAll have been added to SquidDatabase (refs #16, which was closed but will now be better off than it was when 2.0 launches :) )
Passing null for a Criterion value is handled more gracefully in several places. It is ignored where possible (e.g. query.where(null) is a no-op) and throws an exception where it is very bad (e.g. Criterion.negate(null) is nonsensical, so throws an IllegalArgumentException).
Users who still want the old behavior of Criterion.all (i.e. "WHERE 1") can use Criterion.literal(1) instead.
jdkoren
commented
9 years ago
Using literals for these criterions actually had some unexpected performance implications, and they weren't super useful anyways. This PR removes them and updates a few things to replace them:
Users who still want the old behavior of Criterion.all (i.e. "WHERE 1") can use Criterion.literal(1) instead.