[Mediacloth-devel] [PATCH] Support category links
owen at fraser-green.com
Tue Apr 8 15:41:18 EDT 2008
----- "Thiago Arrais" <thiago.arrais at gmail.com> wrote:
> 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
> 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
Ah, yes, OK. Will you commit the patch or can someone make me a member so I can?
More information about the Mediacloth-devel