In GenericDaoBase's findById and findByUuid if I added a check to return null if passed value is null to avoid potential exceptions, will this break any use case? No changes I found till now.
@@ -905,6 +905,8 @@ public abstract class GenericDaoBase<T, ID extends Serializable> implements Gene public T findById(final ID id) { + if (id == null) + return null; if (_cache != null) { final Element element = _cache.get(id); return element == null ? lockRow(id, null) : (T)element.getObjectValue(); @@ -916,6 +918,8 @@ public abstract class GenericDaoBase<T, ID extends Serializable> implements Gene public T findByUuid(final String uuid) { + if (uuid == null) + return null; SearchCriteria<T> sc = createSearchCriteria(); sc.addAnd("uuid", SearchCriteria.Op.EQ, uuid); Regards.