Page MenuHomePhabricator

preload can leave UNIQ
Closed, ResolvedPublic

Description

Preloading a text with pre gives UNIQ inside the preloaded text (see url).

Thanks.

related bug 5210


Version: 1.17.x
Severity: major
URL: http://test.wikipedia.org/w/index.php?title=Preload&action=edit&section=new&preload=Preload

Details

Reference
bz27467

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:21 PM
bzimport set Reference to bz27467.
bzimport added a subscriber: Unknown Object (MLST).

Patch: add unstripBoth to Parser::getPreloadText()

Simple fix attached

Attached:

howard wrote:

See also bug 27511.

(In reply to comment #1)

Created attachment 8159 [details]
Patch: add unstripBoth to Parser::getPreloadText()

Simple fix attached

Applied in r82473

Attached:

oh, marking resolved, but would like Umherirrender to confirm.

(In reply to comment #4)

oh, marking resolved, but would like Umherirrender to confirm.

Looks good.

*** Bug 27519 has been marked as a duplicate of this bug. ***

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

folengo wrote:

Looks good for Bug 27519 too.

Moving blockers from bugs that were duped to this.

Tests added with r100599