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.

Reply via email to