--- Comment #2 from 2011-01-06 23:55:30 UTC ---
(In reply to comment #1)
> The reason i didn't do it in ParserOutput (well in addition to the parser 
> being
> scary), is that an extension can add stuff to that array in several different
> ways, which makes it hard to put the check directly in ParserOutput.
> Perhaps we could add a check in ParserOutput::addCategory where categories are
> added 99% of the time, and still keep the extra substring check in 
> LinksUpdate.
> Does that sound sane?

Yes, when the truncate inside LinksUpdate is needed, because of extensions,
than keep the extra substring.

Configure bugmail:
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

Wikibugs-l mailing list

Reply via email to