Author: jmorliaguet
Date: Sat May 13 11:14:26 2006
New Revision: 3114

Modified:
   cpsskins/branches/paris-sprint-2006/lib/cpsskins/src/cpsskins.js

Log:

- added a warning message in case an action handler isn't registered
  instead of failing silently.



Modified: cpsskins/branches/paris-sprint-2006/lib/cpsskins/src/cpsskins.js
==============================================================================
--- cpsskins/branches/paris-sprint-2006/lib/cpsskins/src/cpsskins.js    
(original)
+++ cpsskins/branches/paris-sprint-2006/lib/cpsskins/src/cpsskins.js    Sat May 
13 11:14:26 2006
@@ -551,10 +551,15 @@
           var event_name = s.key;
           var action_id = s.value;
           var handler = CPSSkins.getAction(action_id);
-          handlers[action_id] = handler.bindAsEventListener(info)
-          selector.each(function(el) {
-            Event.observe(el, event_name, handler);
-          });
+          if (handler == null) {
+            CPSSkins.warn(controller.def.id + ": no handler registered for "
+                          + action_id, view.widget);
+          } else {
+            handlers[action_id] = handler.bindAsEventListener(info)
+            selector.each(function(el) {
+              Event.observe(el, event_name, handler);
+            });
+          }
         })
       }
     })
@@ -571,9 +576,11 @@
           var event_name = s.key;
           var action_id = s.value;
           var handler = handlers[action_id];
-          selector.each(function(el) {
-            Event.stopObserving(el, event_name, handler);
-          });
+          if (handler != null) {
+            selector.each(function(el) {
+              Event.stopObserving(el, event_name, handler);
+            });
+          }
         })
       }
     })
-- 
http://lists.nuxeo.com/mailman/listinfo/z3lab-checkins

Reply via email to