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