Changed the Sammy.Store KVO to send an object instead of multiple arg…

…uments [Thanks dpree!] This makes Sammy.Store better conform with Sammy.Application#bind and #trigger The object sent looks like {key: key, value:value}
mrb · Feb 25, 2010 · 16f0e38 · 16f0e38
1 parent cfcc03d
commit 16f0e38
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/plugins/sammy.storage.js b/lib/plugins/sammy.storage.js
@@ -96,7 +96,7 @@
       this.storage.set(key, string_value);
       if (key != this.meta_key) { 
         this._addKey(key); 
-        this.$element.trigger('set-' + this.name + '.' + key, [key, value]);
+        this.$element.trigger('set-' + this.name + '.' + key, {key: key, value: value});
       };
       return string_value;
     },

diff --git a/test/test_sammy_storage.js b/test/test_sammy_storage.js
@@ -79,8 +79,8 @@
         })
         .should('fire events on get and set', function() {
           var fired = false;
-          $('#main').bind('set-test_store.foo', function(e, key, value) {
-            fired = value;
+          $('#main').bind('set-test_store.foo', function(e, data) {
+            fired = data.value;
           });
           this.store.set('foo', 'bar');
           soon(function() {