From 3fc3228c8f5bc55f266d8a1fa2681f126df72154 Mon Sep 17 00:00:00 2001
From: Michael Richey <github@richeyweb.com>
Date: Fri, 1 Dec 2017 10:38:48 -0600
Subject: [PATCH] getData bugfix and onDependencyChanged feature

getData bombs out on an item list with empty elements.  I added several tests for "s" before additional tests on properties of "s".  This allows dependency changes to continue even on item lists with empty elements.

onDependencyChanged, I passed the source variable from the function input vars to the schema resolver.  This allows the resolver to calculate which of an item list is making the modification.
---
 src/js/brutusin-json-forms.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/js/brutusin-json-forms.js b/src/js/brutusin-json-forms.js
index 49ee5ea..f8d30c5 100644
--- a/src/js/brutusin-json-forms.js
+++ b/src/js/brutusin-json-forms.js
@@ -834,7 +834,7 @@ if (typeof brutusin === "undefined") {
                 if (s === null) {
                     s = SCHEMA_ANY;
                 }
-                if (s.$ref) {
+                if (s && s.$ref) {
                     s = getDefinition(s.$ref);
                 }
                 if (object instanceof Array) {
@@ -856,15 +856,15 @@ if (typeof brutusin === "undefined") {
                             continue;
                         }
                         var ss = null;
-                        if (s.hasOwnProperty("properties") && s.properties.hasOwnProperty(prop)) {
+                        if (s && s.hasOwnProperty("properties") && s.properties.hasOwnProperty(prop)) {
                             ss = s.properties[prop];
                         }
-                        if (ss === null && s.hasOwnProperty("additionalProperties")) {
+                        if (ss === null && s && s.hasOwnProperty("additionalProperties")) {
                             if (typeof s.additionalProperties === 'object') {
                                 ss = s.additionalProperties;
                             }
                         }
-                        if (ss === null && s.hasOwnProperty("patternProperties")) {
+                        if (ss === null && s && s.hasOwnProperty("patternProperties")) {
                             for (var p in s.patternProperties) {
                                 var r = RegExp(p);
                                 if (prop.search(r) !== -1) {
@@ -879,7 +879,7 @@ if (typeof brutusin === "undefined") {
                             nonEmpty = true;
                         }
                     }
-                    if (nonEmpty || s.required) {
+                    if (nonEmpty || s && s.required) {
                         return clone;
                     } else {
                         return null;
@@ -1346,7 +1346,7 @@ if (typeof brutusin === "undefined") {
                 BrutusinForms.onResolutionFinished(source);
             };
             BrutusinForms.onResolutionStarted(source);
-            obj.schemaResolver(arr, obj.getData(), cb);
+            obj.schemaResolver(arr, obj.getData(), cb, source);
 
 
         }