Page MenuHomePhabricator

AbortNewAccount hook was changed accidentially in I402c6beb
Closed, ResolvedPublic


In the past, hooks can set an HTML error message via the second argument $abortError. After I402c6beb, the message is treated as wikitext and parsed (again), causing HTML tags which are unacceptable in wikitext shown on pages, where old extensions like TitleBlacklist parse error messages themselves and send HTML output as $abortError.

Version: 1.21.x
Severity: normal
See Also:



Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 1:37 AM
bzimport set Reference to bz44718.
liangent created this task.Feb 6 2013, 4:26 PM

Sorry about that. That's my fault. The reason is that even though the HTML is being wrapped in a RawMessage, I forgot that it was being parsed when it got returned to the parent function. I'll have a quick patch fix shortly.

This is a temporary fix. I will make a more robust fix when I construct a better way for messages to be stored within messages and for statuses to handle message parameters.

(In reply to comment #2 by Tyler Romeo)

This is a temporary fix.

Patch in Gerrit still awaiting review & merge. :-/

Change 47821 had a related patch set uploaded by Matmarex:
Fix double-parsing of account creation messages.

  • Bug 38894 has been marked as a duplicate of this bug. ***
  • Bug 54336 has been marked as a duplicate of this bug. ***

Change 47821 merged by jenkins-bot:
Fix double-parsing of account creation messages.