[Solar-talk] Working with hashes (MD5) and column updates

Antti Holvikari anttih at gmail.com
Thu Jan 3 12:12:21 CST 2008


On Jan 3, 2008 8:04 PM, Rodrigo Moraes <rodrigo.moraes at gmail.com> wrote:
> On Jan 3, 2008 3:43 PM, Antti Holvikari wrote:
> > [1] http://git.anttih.com/?p=solar.git;a=commitdiff;h=9ab282c9c77e7f88dc80caa5f766e682fa3246c1
>
> Hmm, ok, this partially solves the problem, but I don't see other way
> to mix both validations. The problem is that with this approach, you
> never get model and non-model validation errors at the same time,
> which is kind of strange or annoying from an user point of view. :-/

Hmmm, I think you should get normal invalidation feedback.
Solar_Form_Load_Model should load filters, shouldn't it?

Oh, it seems I have a patch for that too:

http://git.anttih.com/?p=solar.git;a=commitdiff;h=0802060e9b5e6d11f87e15b90318bff434df3fb6

That is an actual bug with Solar_Form_Load_Model.

> Now, patching your patch:
>
> -        if ($form && $form instanceof Solar_Form) {
> +        if ($spec && $spec instanceof Solar_Form) {
>
> thanks, antti.

Oh, it seems I haven't tested that one at all. Thanks a lot Rodrigo! :-)


-- 
Antti Holvikari


More information about the Solar-talk mailing list