I do not think it's a thing to be put into bugzilla so it goes here: while the "standard" woody form example is fully functional the flow version
does not allow you to add or remove a contact.
We've debugged this offline already (Marc and myself during a car ride this afternoon), and changing line 156-157 into:
if (validator != undefined) {
finished = validator(this) && finished;will solve this as far as we can see now. But since Sylvain was quite specific about this in his commit changing the original version some days ago, we've not patched this before getting the word out to him.
Anyway, the reverting patch is attached if you can't wait.
Cheers,
</Steven> -- Steven Noels http://outerthought.org/ Outerthought - Open Source, Java & XML Competence Support Center Read my weblog at http://blogs.cocoondev.org/stevenn/ stevenn at outerthought.org stevenn at apache.org
Index: src/blocks/woody/java/org/apache/cocoon/woody/flow/javascript/woody.js
===================================================================
RCS file:
/home/cvs/cocoon-2.1/src/blocks/woody/java/org/apache/cocoon/woody/flow/javascript/woody.js,v
retrieving revision 1.8
diff -u -r1.8 woody.js
--- src/blocks/woody/java/org/apache/cocoon/woody/flow/javascript/woody.js 16 Aug
2003 13:22:28 -0000 1.8
+++ src/blocks/woody/java/org/apache/cocoon/woody/flow/javascript/woody.js 20 Aug
2003 18:37:12 -0000
@@ -153,8 +153,8 @@
} else {
this.submitId = undefined;
}
- if (finished && validator != undefined) {
- finished = validator(this);
+ if (validator != undefined) {
+ finished = validator(this) && finished;
}
if (finished) {
break;
