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

Reply via email to