Hi,

> You were right.  I've updated it to test for them.  Seems to be good now.

Not a biggie but try be be consistent  with the null checks ie  dataProvider vs 
values != null.

Also nothing wrong with doing something like this at the top of the function.

if (dataProvider == null || values == null)
        return [];

Thanks,
Justin


Reply via email to