Author: jmorliaguet Date: Sun Feb 19 13:23:17 2006 New Revision: 2431 Modified: cpsskins/branches/jmo-perspectives/ui/framework/cpsskins.js Log:
- renamed private variables using the js styleguide 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 Feb 19 13:23:17 2006 @@ -36,18 +36,13 @@ var CPSSkins = { Version: "0.7", - Subscribers: $H({}), - - Models: $H({}), - - Controllers: $H({}), - - Handlers: $H({}), - - Perspectives: $H({}), - - CurrentPerspective: "", - PreviousPerspective: "", + _subscribers: $H({}), + _models: $H({}), + _controllers: $H({}), + _handlers: $H({}), + _perspectives: $H({}), + _currentPerspective: "", + _previousPerspective: "", init: function() { CPSSkins.parse(document); @@ -55,7 +50,7 @@ getModelById: function(id) { var model = null; - $A(CPSSkins.Models).each(function(m) { + $A(CPSSkins._models).each(function(m) { var def = m[1].def; if (def.id == id) { model = m[1] @@ -66,7 +61,7 @@ getControllerById: function(id) { var controller = null; - $A(CPSSkins.Controllers).each(function(c) { + $A(CPSSkins._controllers).each(function(c) { var def = c[1].def; if (def.id == id) { controller = c[1] @@ -85,29 +80,29 @@ /* Public events */ registerHandlers: function(handlers) { - Object.extend(CPSSkins.Handlers, handlers); + Object.extend(CPSSkins._handlers, handlers); }, /* Internal events */ subscribe: function(eventid, event) { - if (!(eventid in CPSSkins.Subscribers)) { - CPSSkins.Subscribers[eventid] = $A([]); + if (!(eventid in CPSSkins._subscribers)) { + CPSSkins._subscribers[eventid] = $A([]); } - CPSSkins.Subscribers[eventid].push(event); + CPSSkins._subscribers[eventid].push(event); }, unsubscribe: function(eventid, event) { new_subscribers = $A([]); - CPSSkins.Subscribers[eventid].each(function(e) { + CPSSkins._subscribers[eventid].each(function(e) { if (!(event.subscriber == e.subscriber && event.publisher == e.publisher)) { new_subscribers.push(e); } }); - CPSSkins.Subscribers[eventid] = new_subscribers; + CPSSkins._subscribers[eventid] = new_subscribers; }, notify: function(eventid, event) { - var subscribers = CPSSkins.Subscribers[eventid]; + var subscribers = CPSSkins._subscribers[eventid]; var publisher = event.publisher; $A(subscribers).each(function(e) { var event_publisher = e.publisher; @@ -217,7 +212,7 @@ var factory = Controllers[controller_type]; if (factory) { var controller = factory(el, def); - CPSSkins.Controllers[name] = controller; + CPSSkins._controllers[name] = controller; CPSSkins.notify("registered controller " + def.id, {'publisher': controller}); } @@ -226,7 +221,7 @@ case "model": { var model = new CPSSkins.Model(el, def); - CPSSkins.Models[name] = model; + CPSSkins._models[name] = model; CPSSkins.notify("registered model " + def.id, {'publisher': model}); break; } @@ -294,10 +289,10 @@ var perspectives = def.perspectives || []; $A(perspectives).each(function(p) { if (!p) return; - if (!(p in CPSSkins.Perspectives)) { - CPSSkins.Perspectives[p] = $A([]); + if (!(p in CPSSkins._perspectives)) { + CPSSkins._perspectives[p] = $A([]); } - CPSSkins.Perspectives[p].push(view); + CPSSkins._perspectives[p].push(view); }); //CPSSkins.notify("registered view " + def.id, view); } @@ -308,7 +303,7 @@ }, _resolvePerspective: function(perspective) { - var path = CPSSkins.CurrentPerspective.split("/"); + var path = CPSSkins._currentPerspective.split("/"); var base = perspective; var ext = ""; @@ -319,7 +314,7 @@ } if (perspective.substr(0,2) == "./") { - base = CPSSkins.CurrentPerspective; + base = CPSSkins._currentPerspective; if (base) { base += "/"; } @@ -336,7 +331,7 @@ } if (perspective == '-') { - base = CPSSkins.PreviousPerspective; + base = CPSSkins._previousPerspective; } return base + ext; @@ -347,7 +342,7 @@ var to_hide = []; perspective = CPSSkins._resolvePerspective(perspective); - CPSSkins.Perspectives.each(function(s) { + CPSSkins._perspectives.each(function(s) { var elements = s[1]; var path = perspective.split("/"); path.each(function(p, index) { @@ -368,8 +363,8 @@ $A(to_hide).each(function(el) { el.hide(); }); $A(to_show).each(function(el) { el.show(); }); - CPSSkins.PreviousPerspective = CPSSkins.CurrentPerspective; - CPSSkins.CurrentPerspective = perspective; + CPSSkins._previousPerspective = CPSSkins._currentPerspective; + CPSSkins._currentPerspective = perspective; } } @@ -433,7 +428,7 @@ /* the event handler calls the controller's registered handler */ var handler_id = controller.def.handlers[action]; if (handler_id) { - var handler = CPSSkins.Handlers[handler_id]; + var handler = CPSSkins._handlers[handler_id]; if (handler) handler(event); } }); @@ -458,7 +453,7 @@ $H(r[1]).each(function(s) { var event_name = s[0]; var handler_name = s[1]; - var handler = CPSSkins.Handlers[handler_name]; + var handler = CPSSkins._handlers[handler_name]; selector.each(function(el) { Event.observe(el, event_name, handler.bindAsEventListener(controller)); }); @@ -625,7 +620,7 @@ var model_node = this._getModelNode(node); if (model_node) { var name = model_node.getAttribute("name"); - return CPSSkins.Models[name]; + return CPSSkins._models[name]; } return null; }, -- http://lists.nuxeo.com/mailman/listinfo/z3lab-checkins