simplyflow 0.2.2 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "simplyflow",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Flow based programming in javascript, with signals and effects",
   "main": "src/flow.mjs",
   "type": "module",

package/src/bind.mjs

@@ -25,7 +25,7 @@ class SimplyBind {
         const getBindingAttribute = (el) => {
             const foundAttribute = bindAttributes.find(attr => el.hasAttribute(attr))
             if (!foundAttribute) {
-                console.error('No matching attribute found',el)
+                console.error('No matching attribute found',el,attr)
             }
             return foundAttribute
         }
@@ -119,7 +119,7 @@ class SimplyBind {
             updateBindings(changes)
         })
 
-        this.observer.observe(options.container, {
+        this.observer.observe(this.options.container, {
             subtree: true,
             childList: true
         })
@@ -364,7 +364,7 @@ export function defaultListTransformer(context) {
     const attribute      = this.options.attribute
 
     if (!Array.isArray(value)) {
-        console.error('Value is not an array.', el, value)
+        console.error('Value is not an array.', el, path, value)
     } else if (!templates?.length) {
         console.error('No templates found in', el)
     } else {
@@ -382,7 +382,7 @@ export function defaultMapTransformer(context) {
     const attribute      = this.options.attribute
 
     if (typeof value != 'object') {
-        console.error('Value is not an object.', el, value)
+        console.error('Value is not an object.', el, path, value)
     } else if (!templates?.length) {
         console.error('No templates found in', el)
     } else {

package/src/model.mjs

@@ -141,18 +141,19 @@ export function filter(options) {
 	if (!options?.name || typeof options.name!=='string') {
 		throw new Error('filter requires options.name to be a string')
 	}
-	if (this.state.options[options.name]) {
-		throw new Error('a filter with this name already exists on this model')
-	}
 	if (!options.matches || typeof options.matches!=='function') {
 		throw new Error('filter requires options.matches to be a function')
 	}
 	return function(data) {
+		if (this.state.options[options.name]) {
+			throw new Error('a filter with this name already exists on this model')
+		}
 		this.state.options[options.name] = options
 		return effect(() => {
 			if (this.state.options[options.name].enabled) {
-				return data.filter(this.state.options[options.name].matches)
+				return data.current.filter(this.state.options[options.name].matches.bind(this))
 			}
+			return data.current
 		})
 	}
 }