Remove HTMLDiff
Closed, ResolvedPublic


HTMLDiff is terribly broken. It's been an i18n nightmare since the start and doesn't function correctly a lot of the time. I think this experimental feature should be removed.

Bugs 18872, 17812 and 15328 can probably be INVALID if we nuke HTMLDiff.

Version: unspecified
Severity: enhancement

bzimport added a subscriber: Unknown Object (MLST).
bzimport set Reference to bz19859.
demon created this task.Via LegacyJul 21 2009, 1:30 PM
demon added a comment.Via ConduitJul 21 2009, 2:21 PM

Created attachment 6378
Remove HTMLDiff Entirely

  • Remove HTMLDiff and all dependent stuff, nuke UI, nuke $wgEnableHTMLDiff
    • Removed $htmldiff param in DifferenceEngine constructor, updated single caller that passed this
  • Remove now useless Node classes and children, update Autoloader
  • Nuke all diff-* messages except diff-multi, visualcomparison, visual-comparison and wikicodecomparison

Attached: nukehtmldiff.patch

demon added a comment.Via ConduitOct 28 2009, 6:21 PM

Done in r58267.

Add Comment