Last modified: 2011-10-24 10:52:21 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T29467, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 27467 - preload can leave UNIQ


Summary:	preload can leave UNIQ

Status:	VERIFIED FIXED

Product:	MediaWiki
Classification:	Unclassified
Component:	Page editing (Other open bugs)
Version:	1.17.x
Hardware:	All All

Importance:	Normal major (vote)
Target Milestone:	---
Assigned To:	Nobody - You can work on this!

URL:	http://test.wikipedia.org/w/index.php...
Whiteboard:
Keywords:	patch, patch-need-review

Duplicates:	27519 27623 (view as bug list)
Depends on:
Blocks:	UNIQ 26676 27339
	Show dependency tree / graph

Reported:	2011-02-16 19:48 UTC by Umherirrender
Modified:	2011-10-24 10:52 UTC (History)
CC List:	5 users (show)

See Also:
Web browser:	---
Mobile Platform:	---
Assignee Huggle Beta Tester:	---

Attachments
Patch: add unstripBoth to Parser::getPreloadText() (567 bytes, patch) 2011-02-17 22:24 UTC, P.Copp	Details
Add an attachment (proposed patch, testcase, etc.)

Description Umherirrender 2011-02-16 19:48:12 UTC

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

Thanks.

related bug 5210

Comment 1 P.Copp 2011-02-17 22:24:36 UTC

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

Simple fix attached

Comment 2 Howard Cheng 2011-02-17 22:49:17 UTC

See also bug 27511.

Comment 3 Mark A. Hershberger 2011-02-19 19:19:53 UTC

(In reply to comment #1)
> Created attachment 8159 [details]
> Patch: add unstripBoth to Parser::getPreloadText()
> 
> Simple fix attached

Applied in r82473

Comment 4 Mark A. Hershberger 2011-02-19 19:21:12 UTC

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

Comment 5 Umherirrender 2011-02-19 20:34:00 UTC

(In reply to comment #4)
> oh, marking resolved, but would like Umherirrender to confirm.

Looks good.

Comment 6 DieBuche 2011-02-20 13:29:12 UTC

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

Comment 7 db [inactive,noenotif] 2011-02-22 06:58:39 UTC

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

Comment 8 folengo 2011-02-23 21:39:10 UTC

Looks good for Bug 27519 too.

Comment 9 Krinkle 2011-02-24 09:12:18 UTC

The example [1] linked in bug 27623 works now as well.
Fully resolved, great.

[1] http://en.wikipedia.org/?title=User:Krinkle/Template:Top_icon2/testcases&action=edit&preload=Template:Documentation/preload-testcases

Comment 10 Krinkle 2011-06-05 10:41:46 UTC

Moving blockers from bugs that were duped to this.

Comment 11 Antoine "hashar" Musso (WMF) 2011-10-24 10:52:21 UTC

Tests added with r100599

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links