HomePhabricator

* Revert back to my parser pseudo-branch again. Note: if you feel like…

Description

  • Revert back to my parser pseudo-branch again. Note: if you feel like reverting this, reverting Parser.php alone to r40010 will work just fine.
  • Merged replaceFreeExternalLinks() with doMagicLinks(). Makes a lot of sense, very similar operations, doesn't break any parser tests. Stops free links from interacting with other parser stages, the same way ISBN links don't.
  • The pass order change fixes Brion's complaint in r39980. Early link expansion, triggered by having more than 1000 links in the page, was outputting URLs which were destroyed by RFEL. Added parser test.
  • Fixed an unrelated bug in LinkHolderArray::replace(): if a link to a redirect appears in two separate RLH calls, the second and subsequent calls do not add the mw-redirect class. Caused by an unmigrated LinkCache fetch.
  • Added a parser test for a pass interaction bug that the pass order change fixes.
  • The fuzzer told me to tell you that free external links in non-caption image parameters, which are and have always been invisible, are now not registered either.
  • Miscellaneous supporting updates to the test infrastructure.

Details

Committed
tstarlingAug 26 2008, 2:37 PM
Parents
rSVN40019: Commit a little bit of code for using a subclass of EditPage.
Branches
Unknown
Tags
Unknown
ChangeId
None