Thanks, I hadn't seen #874. The interface described there is similar
to what I proposed, though I think it might be better in a few
respects. E.g., the code from #874 includes two separate interfaces
(IRegistrationFieldProvider and IRegistrationManipulator), which makes
things a bit more general than what I did. I posted a link to my patch
along with a note about the interface differences on the ticket.
Also, I noticed in the original patch submitted by coderanger in #874
that there is code to support adding both optional and required
fields. This seems likes A Good Idea to me. Ideally, it would handle
cases where two plugins add fields of the same name, where one of them
is optional and the other is required. In this case the required field
would subsume the optional one.
Do you know what the present state of #874 is? Are there plans to roll
the patch into AccountManager?
Best,
-Nick
On Jun 20, 2008, at 3:33 PM, John Hampton wrote:
>
> Nicholas Bergson-Shilcock wrote:
> <snip>
>> What do folks think re: writing a stand-alone plugin vs. adding an
>> extension point to AccountManager and then hooking in to that? I'll
>> admit that the latter seems a better way to me, but as I said, I'm
>> very new to Trac development and I'm just getting my feet wet here.
>> I'd love to hear from some of the veteran developers and other
>> members
>> of the community to find out what path they think is the best to
>> pursue.
>
> FYI, I haven't looked at the patch yet, but will try to find some time
> to do so. If Matt or Noah are able to review it, that would be
> great too.
>
> I'm all for adding interfaces and extension points to account
> manager to
> allow other plugins to hook into. Wile not everyone will want a
> captcha, or even the same kind of captcha, having the proper
> interfaces
> in account manager to allow such things is a good thing.
>
> Will you please take a look at ticket #874 [1] on t-h.o and link to
> your
> patch, as well as specifying how your patch is different/same? Thanks
>
> -John
>
>
> [1] http://trac-hacks.org/ticket/874
>
> >
>
> !DSPAM:4032,485c063178891961014482!
>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Trac
Development" 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/trac-dev?hl=en
-~----------~----~----~----~------~----~------~--~---
