nyphbl8d wrote:

I'm fairly sure that braceSubstitution is the best function for this hook. I'm beginning to think that the proper location is just before this line:
wfProfileOut( METHOD . '-loadtpl' );
I'm still undecided on what exactly should get passed to the hook other than the text. Also, I need to check to see if the text there is in DOM format or not. I'll see what I can hammer out this weekend if no one has a better suggestion.

nyphbl8d wrote:

Transclude Hooks

I'm sure this patch is wrong in some way, but I'd appreciate some hints on how to improve it. Right now, it attempts to cover both local and interwiki transcludes.

Attached:

nyphbl8d wrote:

Also, I'm currently using the ParserAfterStrip hook. I'm not sure if I want to make a new hook or use this one. It's open to debate.

nyphbl8d wrote:

Setting this to need-review

audreyt wrote:

Hi Kinsey, thank you for the patch!

As you may already know, MediaWiki is currently revamping its PHP-based parser
into a "Parsoid" prototype component, to support the rich-text Visual Editor
project:

https://www.mediawiki.org/wiki/Parsoid
https://www.mediawiki.org/wiki/Visual_editor

Folks interested in enhancing the parser's capabilities are very much welcome
to join the Parsoid project, and contribute patches as Git branches:

https://www.mediawiki.org/wiki/Git/Tutorial#How_to_submit_a_patch

Compared to .diff attachments in Bugzilla tickets, Git branches are much easier
for us to review, refine and merge features together.

Each change set has a distinct URL generated by the "git review" tool, which
can be referenced in Bugzilla by pasting its gerrit.wikimedia.org URL as a
comment.

If you run into any issues with the patch process, please feel free to ask on
irc.freenode.net #wikimedia-dev and the wikitext-l mailing list. Thank you!

Change 103238 had a related patch set uploaded by Cptanu:
Included a hook to have access to templates and other transcluded pages.

https://gerrit.wikimedia.org/r/103238