Alias fabric.Observable.fire to trigger, similar to jQuery, Backbone, etc. Closes #401

This commit is contained in:
kangax 2013-02-11 13:22:05 +01:00
parent 97fa81aa77
commit 160e7e5ded
2 changed files with 22 additions and 1 deletions

View file

@ -51,6 +51,7 @@ fabric.Observable = {
/**
* Fires event with an optional options object
* @deprecated since 1.0.7
* @method fire
* @param {String} eventName
* @param {Object} [options]
@ -80,4 +81,11 @@ fabric.Observable.on = fabric.Observable.observe;
* @method off
* @type function
*/
fabric.Observable.off = fabric.Observable.stopObserving;
fabric.Observable.off = fabric.Observable.stopObserving;
/**
* Alias for fire
* @method trigger
* @type function
*/
fabric.Observable.trigger = fabric.Observable.fire;

View file

@ -98,4 +98,17 @@ test('event options', function() {
foo.fire('foo:bar', { value: 'sekret' });
equal('sekret', someValue);
});
test('trigger', function() {
var foo = { };
fabric.util.object.extend(foo, fabric.Observable);
var eventFired = false;
foo.on('bar:baz', function() {
eventFired = true;
});
foo.trigger('bar:baz');
equal(true, eventFired);
});