Last modified: 2006-07-12 03:51:17 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 T8561, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 6561 - "frame" and "border" and "rules" listed twice as table attributes in includes/Sanitizer.php


Summary:	"frame" and "border" and "rules" listed twice as table attributes in includes...

Status:	RESOLVED FIXED

Product:	MediaWiki
Classification:	Unclassified
Component:	General/Unknown (Other open bugs)
Version:	1.7.x
Hardware:	All All

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

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-07-06 08:22 UTC by Nick Jenkins
Modified:	2006-07-12 03:51 UTC (History)
CC List:	0 users

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Nick Jenkins 2006-07-06 08:22:41 UTC

In includes/Sanitizer.php, "frame" and "border" and "rules" appear to be
accidentally listed twice in this section:
-----------------------------------------
            # 11.2.1
            'table'      => array_merge( $common,
                                array( 'summary', 'width', 'border', 'frame',
                                             'rules', 'cellspacing', 'cellpadding',
                                             'align', 'bgcolor', 'frame', 'rules',
                                             'border' ) ),
-----------------------------------------

Comment 1 Antoine "hashar" Musso (WMF) 2006-07-10 09:13:58 UTC

Fixed in trunk r15493

Comment 2 Nick Jenkins 2006-07-11 00:09:07 UTC

Thank you, and I'm really sorry to be a pain-in-the-butt, but I think "rules" is
still accidentally duped. ;-)

I.e.:
================================
- 'align', 'bgcolor', 'rules',
+ 'align', 'bgcolor'
================================

Comment 3 Antoine "hashar" Musso (WMF) 2006-07-11 16:40:07 UTC

Fixed in trunk r15543

I should pay attention :o)

Comment 4 Nick Jenkins 2006-07-12 03:51:17 UTC

Thank you!

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links