[KSS-checkins] r49572 - kukit/kukit.js/branch/finish-closures/kukit
gotcha at codespeak.net
gotcha at codespeak.net
Sun Dec 9 17:45:35 CET 2007
Author: gotcha
Date: Sun Dec 9 17:45:34 2007
New Revision: 49572
Modified:
kukit/kukit.js/branch/finish-closures/kukit/commandprocessor.js
Log:
class closure
Modified: kukit/kukit.js/branch/finish-closures/kukit/commandprocessor.js
==============================================================================
--- kukit/kukit.js/branch/finish-closures/kukit/commandprocessor.js (original)
+++ kukit/kukit.js/branch/finish-closures/kukit/commandprocessor.js Sun Dec 9 17:45:34 2007
@@ -26,114 +26,111 @@
*/
cp.CommandProcessor = function() {
this.commands = new Array();
-};
-
-var CommandProcessor = cp.CommandProcessor.prototype;
-CommandProcessor.parseCommands =
- function(commands, transport) {
-;;;kukit.log('Parsing commands.');
-;;;kukit.logDebug('Number of commands : ' + commands.length + '.');
- for (var y=0;y < commands.length;y++) {
- var command = commands[y];
- this.parseCommand(command, transport);
- // If we receive an error command, we handle that separately.
- // We abort immediately and let the processError handler do its job.
- // This means that although no other commands should be in commands,
- // we make sure we execute none of them.
- var lastcommand = this.commands[this.commands.length-1];
- if (lastcommand.name == 'error') {
- // We have to throw an explicitError always, since we want
- // error fallbacks work both in production and development mode.
- throw kukit.err.explicitError(lastcommand);
+ this.parseCommands = function(commands, transport) {
+;;; kukit.log('Parsing commands.');
+;;; kukit.logDebug('Number of commands : ' + commands.length + '.');
+ for (var y=0;y < commands.length;y++) {
+ var command = commands[y];
+ this.parseCommand(command, transport);
+ // If we receive an error command, we handle that separately.
+ // We abort immediately and let the processError handler do its job.
+ // This means that although no other commands should be in commands,
+ // we make sure we execute none of them.
+ var lastcommand = this.commands[this.commands.length-1];
+ if (lastcommand.name == 'error') {
+ // We have to throw an explicitError always, since we want
+ // error fallbacks work both in production and development mode.
+ throw kukit.err.explicitError(lastcommand);
+ }
}
- }
-};
+ };
-CommandProcessor.parseCommand =
- function(command, transport) {
- var selector = "";
- var params = {};
- var name = "";
-
- selector = command.getAttribute("selector");
- name = command.getAttribute("name");
- type = command.getAttribute("selectorType");
- if (name == null)
- name = "";
- var childNodes = command.childNodes;
- for (var n=0;n < childNodes.length;n++) {
- var childNode = childNodes[n];
- if (childNode.nodeType != 1)
- continue;
- if (childNode.localName) {
- if (childNode.localName.toLowerCase() != "param") {
- throw new Error('Bad payload, expected param');
- }
- } else {
- //IE does not know DOM2
- if (childNode.nodeName.toLowerCase() != "param") {
- throw new Error('Bad payload, expected param (IE)');
- }
- }
- data = childNode.getAttribute('name');
- if (data != null) {
- // Decide if we have a string or a dom parameter
- var childCount = childNode.childNodes.length;
- var result;
- if (childCount == 0) {
- result = '';
+ this.parseCommand = function(command, transport) {
+ var selector = "";
+ var params = {};
+ var name = "";
+
+ selector = command.getAttribute("selector");
+ name = command.getAttribute("name");
+ type = command.getAttribute("selectorType");
+ if (name == null)
+ name = "";
+ var childNodes = command.childNodes;
+ for (var n=0;n < childNodes.length;n++) {
+ var childNode = childNodes[n];
+ if (childNode.nodeType != 1)
+ continue;
+ if (childNode.localName) {
+ if (childNode.localName.toLowerCase() != "param") {
+ throw new Error('Bad payload, expected param');
+ }
+ } else {
+ //IE does not know DOM2
+ if (childNode.nodeName.toLowerCase() != "param") {
+ throw new Error('Bad payload, expected param (IE)');
+ }
+ }
+ data = childNode.getAttribute('name');
+ if (data != null) {
+ // Decide if we have a string or a dom parameter
+ var childCount = childNode.childNodes.length;
+ var result;
+ if (childCount == 0) {
+ result = '';
+ } else {
+ // (we do not interpret html inline content any more)
+ // we have a single text node
+ // OR
+ // we have a single CDATA node (HTML parameter CDATA-style)
+;;; var isTextNode = childNode.firstChild.nodeType == 3;
+;;; var isCData = childNode.firstChild.nodeType == 4;
+;;; if (! (childCount == 1 && (isTextNode || isCData))) {
+;;; kukit.E = 'Bad payload, expected a text or a CDATA node';
+;;; throw new Error(kukit.E);
+;;; }
+ // we consider this as html payload
+ // The result is always a string from here.
+ result = childNode.firstChild.nodeValue;
+ }
+ params[data] = result;
} else {
- // (we do not interpret html inline content any more)
- // we have a single text node
- // OR
- // we have a single CDATA node (HTML parameter CDATA-style)
-;;; var isTextNode = childNode.firstChild.nodeType == 3;
-;;; var isCData = childNode.firstChild.nodeType == 4;
-;;; if (! (childCount == 1 && (isTextNode || isCData))) {
-;;; kukit.E = 'Bad payload, expected a text or a CDATA node';
-;;; throw new Error(kukit.E);
-;;; }
- // we consider this as html payload
- // The result is always a string from here.
- result = childNode.firstChild.nodeValue;
+ throw new Error('Bad payload, expected attribute "name"');
}
- params[data] = result;
- } else {
- throw new Error('Bad payload, expected attribute "name"');
}
- }
- var command = new kukit.cr.makeCommand(selector, name, type, params,
- transport);
- this.addCommand(command);
-};
+ var command = new kukit.cr.makeCommand(selector, name, type, params,
+ transport);
+ this.addCommand(command);
+ };
-CommandProcessor.addCommand = function(command) {
+ this.addCommand = function(command) {
this.commands[this.commands.length] = command;
};
-CommandProcessor.executeCommands = function(oper) {
- kukit.engine.beginSetupEventsCollection();
- // node, eventRule, binder are given on oper, in case
- // the command was called up from an event
- if (typeof(oper) == 'undefined' || oper == null) {
- oper = new kukit.op.Oper();
- }
- var commands = this.commands;
- for (var y=0;y < commands.length;y++) {
- var command = commands[y];
-;;; try {
- command.execute(oper);
-;;; } catch (e) {
-;;; if (e.name == 'RuleMergeError' || e.name == 'EventBindError') {
-;;; throw(e);
-;;; } else {
-;;; // augment the error message
-;;; throw kukit.err.commandExecutionError(e, command);
+ this.executeCommands = function(oper) {
+ kukit.engine.beginSetupEventsCollection();
+ // node, eventRule, binder are given on oper, in case
+ // the command was called up from an event
+ if (typeof(oper) == 'undefined' || oper == null) {
+ oper = new kukit.op.Oper();
+ }
+ var commands = this.commands;
+ for (var y=0;y < commands.length;y++) {
+ var command = commands[y];
+;;; try {
+ command.execute(oper);
+;;; } catch (e) {
+;;; if (e.name == 'RuleMergeError' || e.name == 'EventBindError') {
+;;; throw(e);
+;;; } else {
+;;; // augment the error message
+;;; throw kukit.err.commandExecutionError(e, command);
+;;; }
;;; }
-;;; }
- }
- kukit.engine.finishSetupEventsCollection();
+ }
+ kukit.engine.finishSetupEventsCollection();
+ };
+
};
}(); /// MODULE END
More information about the Kukit-checkins
mailing list