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
