[Mediacloth-devel] [PATCH] Support category links

[Owen Fraser-Green]

Hi,


----- "Thiago Arrais" <thiago.arrais at gmail.com> wrote:
> Owen.
> 
> On Wed, Mar 26, 2008 at 6:04 PM, Owen Fraser-Green <owen at fraser-
> >  I agree. So in the new patch attached I've split it out into
> > CategoryLinkAST for the [:Category:Foo] links and CategoryAST
> > for [Category:Foo] directives
> 
> Spliting the two classes is clearly an improvent in my opinion, but
> this new patch didn't address the link handler problem. The handler
> still receives only one kind of message for page links and category
> links and no message for category directives.
> 
> I was thinking more along the lines of the patch that I attached to
> this message. There are two new methods for link handlers:
> `link_for_category` that is similar to `link_for_resource`, but deals
> with category links; and `category_add` for category directives.
> This last one maybe needs to be treated differently. The behavior
> with this patch is to generate inline text for category directives
> just
> like links, but maybe it needs to inject text at random locations
> depending on the application. Mediawiki, for example, places its
> category listing at the bottom of the page and other applications
> may want to place it somewhere else inside the emitted HTML
> code.

Ah, yes, OK. Will you commit the patch or can someone make me a member so I can?

Cheers,
Owen