> > this is a hackish addition I made to views. It is enabled by
> > overriding validate-object-form-view.
>
> What exactly are you referring to? The basic whole form view
> validation code has made it into upstream at some point.
>

Maciej wrote of
(defun validate-age (&rest args &key age age-field &allow-other-keys)

afaik, the age-field is not supported by weblocks view-validators, but
perhaps it's made it into upstream since last I checked (I'm not on
dev).

> I think it's approximately the same as yours on paste.lisp.org.

That would be news to me if the view-validator were passed the
dataform-data object and the foo-field args.

> No fancy things like setting form values or anything though.

I don't understand. Setting what form values? Or do you mean passing
the dataform-data object to the validator?

I looked at weblocks-dev, it has the following lines:

        (push parsed-value fields-values)
       (push (weblocks::symbol-to-keyword (view-field-slot-name
field)) fields-values)

in validate-object-form-view. The code I pasted has the following
additional lines (amongst other stuff).

                        ; 2 lines added by nunb: allow a symbol like 
:phone2-field to
refer to the field itself in the argslist to form-validator
                        (push field fields-values)
                        (push (weblocks::symbol-to-keyword 
(nunb-make-error-field-symbol
(view-field-slot-name field))) fields-values)

It seemed to me that Maciej's code was trying to use foo-field, and as
he mentioned looking at something I put in contrib, I assumed he was
using some of that code. Anyway, I've just realized that since his
validator has &allow-other-keys it should not have affected anything,
that is, Maciej's error is unrelated. Sorry for any confusion.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"weblocks" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/weblocks?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to