more informative error message for invalid components

clay · Sep 4, 2018 · 4641f94 · 4641f94
1 parent f93dc71
commit 4641f94
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/decorators/component-list.js b/lib/decorators/component-list.js
@@ -34,9 +34,9 @@ function updateOrder(el) {
 
     if (_.get(store, 'state.ui.currentFocus')) {
       // save an open child component's form before saving the parent component
-      return store.dispatch('unfocus').then(() => store.dispatch('saveComponent', { uri, data: { [path]: newList }}));
+      return store.dispatch('unfocus').then(() => store.dispatch('saveComponent', { uri, data: { [path]: newList } }));
     } else {
-      return store.dispatch('saveComponent', { uri, data: { [path]: newList }});
+      return store.dispatch('saveComponent', { uri, data: { [path]: newList } });
     }
   } else {
     // data is in a page area
@@ -178,7 +178,9 @@ function containsChildComponents(uri, path, el) {
       !child.classList.contains('kiln-inactive-placeholder') &&
       !child.classList.contains('mini-selector') &&
       !child.classList.contains('mini-selector-inactive')) {
-      log.error(`Template for ${getComponentName(uri)} → ${path} contains non-component elements! All direct children of a component list must be components.`, { action: 'containsChildComponents', nonChildEl: child });
+      log.error(`Template for ${getComponentName(uri)} → ${path} contains non-component elements! 
+        All direct children of a component list must be components.`, { action: 'containsChildComponents', nonChildEl: child });
+      log.debug('Ensure each child component has a valid handlebars template and a single root element with a data-uri property');
       result = false;
     }
   });