Thanks everyone for your valuable inputs on it. I'll create logical sub tickets to establish this practice in OFBiz.
Regards, Ratnesh Upadhyay HotWax Systems | www.hotwaxsystems.com On Mon, Jun 20, 2016 at 3:04 PM, Scott Gray <scott.g...@hotwaxsystems.com> wrote: > +1, good idea thanks Ratnesh > > Regards > Scott > > On 18 June 2016 at 02:30, Ratnesh Upadhyay <upadhyay.ratn...@gmail.com> > wrote: > > > Hello community, > > > > There are lots of places in code where we have used > > UtilValidate.isNotEmpty() or UtilValidate.isEmpty() for GenericValue > object > > . GenericValue is never empty. Its always null or not null. So should we > > use directly null or not null check instead of UtilValidate.isNotEmpty() > > and UtilValidate.isEmpty() for GenericValue objects ? > > > > Basically these validation methods should preferably be used on Strings > or > > Lists and normal Maps. > > > > Thanks!! > > > > Regards, > > Ratnesh Upadhyay > > HotWax Systems | www.hotwaxsystems.com > > >