[jifty-devel] Patch for more handlers (including onchange)
Jesse Vincent
jesse at bestpractical.com
Mon Aug 13 15:32:41 EDT 2007
Hey Edward,
Thanks very much for the patch! I've been talking it through with a
couple folks and we have a few style questions/requests. It seems
like there's quite a bit of duplicated code in handler_allowed. Is
there a base class that stuff could move to?
Also, the handler: named block should likely be updated. Named blocks
are generally all uppercase in jifty code.
Can you send me the output of htpasswd -n under separate cover so I
can set you up a commit bit for once the code is cleaned up?
Best,
Jesse
On Aug 11, 2007, at 10:24 PM, Edward Funnekotter wrote:
> Hi,
>
> I noticed that onchange (and other handlers) wasn't implemented, so
> I decided to try to add support for it. Here is a patch that adds
> that support for most of the missing handlers.
>
> Since I am fairly new at Jifty internals, I suspect that the style
> may not be completely inline with what you guys like. Please
> either change what I have or suggest changes and I will make them.
>
> If you prefer me to commit this stuff, lob me a commit bit and I
> will do it.
>
> Thanks,
> Edward
>
>
> <handlers_patch.gz>
> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
Url : http://lists.bestpractical.com/pipermail/jifty-devel/attachments/20070813/8cc7dd07/PGP.pgp
More information about the jifty-devel
mailing list