Author: jmorliaguet Date: Sun Mar 5 15:53:07 2006 New Revision: 2529 Modified: cpsskins/branches/jmo-perspectives/ui/framework/cpsskins.js Log:
- fixed event comparisons Modified: cpsskins/branches/jmo-perspectives/ui/framework/cpsskins.js ============================================================================== --- cpsskins/branches/jmo-perspectives/ui/framework/cpsskins.js (original) +++ cpsskins/branches/jmo-perspectives/ui/framework/cpsskins.js Sun Mar 5 15:53:07 2006 @@ -93,7 +93,7 @@ /* Comparison */ compare: function(a, b) { - if (a == undefined && b == undefined) { + if (a == undefined || b == undefined) { return true; } if (typeof a == 'object' && typeof b == 'object') { @@ -115,8 +115,9 @@ this._subscribers[eventid] = []; } if (this._subscribers[eventid].findAll(function(e) { + if (event == undefined) return true; return (CPSSkins.compare(event.subscriber, e.subscriber) && - CPSSkins.compare(event.publisher, e.publisher)) + CPSSkins.compare(event.publisher, e.publisher)) }).length == 0) { this._subscribers[eventid].push(event); } @@ -137,13 +138,9 @@ var subscribers = this._subscribers; var publisher = event.publisher; (subscribers[eventid] || []).findAll(function(e) { - if (e.publisher == null) { - return true; - } - if (CPSSkins.compare(e.publisher, publisher)) { - return true; - } - return false; + if (event == undefined) return true; + return (CPSSkins.compare(event.subscriber, e.subscriber) && + CPSSkins.compare(event.publisher, e.publisher)) }).each(function(e) { var handler = CPSSkins.getEventHandler(eventid, e.subscriber); if (handler) { -- http://lists.nuxeo.com/mailman/listinfo/z3lab-checkins