Skip to:
Content

BuddyPress.org

Opened 10 years ago

Closed 9 years ago

#6495 closed defect (bug) (fixed)

Group - allowed class tag is coded twice

Reported by: danbp's profile danbp Owned by: r-a-y's profile r-a-y
Milestone: 2.4 Priority: low
Severity: minor Version: 1.2
Component: Groups Keywords: good-first-bug has-patch
Cc: danco38@…

Description

in bp-groups-filters.php:90

	// Add our own tags allowed in group descriptions
	$allowed_tags['a']['class']    = array();
	$allowed_tags['img']           = array();
	$allowed_tags['img']['src']    = array();
	$allowed_tags['img']['alt']    = array();
$allowed_tags['img']['class']  = array();
	$allowed_tags['img']['width']  = array();
	$allowed_tags['img']['height'] = array();
$allowed_tags['img']['class']  = array();
	$allowed_tags['img']['id']     = array();
	$allowed_tags['code']          = array();

there is one to many !

Attachments (1)

trunk.patch (548 bytes) - added by svenl77 9 years ago.
remove allowed class tag it was coded twice

Download all attachments as: .zip

Change History (4)

#1 @r-a-y
9 years ago

  • Keywords good-first-bug added
  • Milestone changed from 2.3.2 to 2.4
  • Owner set to r-a-y
  • Priority changed from normal to low
  • Severity changed from normal to minor
  • Status changed from new to assigned
  • Version changed from 2.3.1 to 1.2

Nice catch, danbp!

Since this was introduced in v1.2.0 (r2663), we'll get to this in 2.4.0.

@svenl77
9 years ago

remove allowed class tag it was coded twice

#2 @svenl77
9 years ago

  • Keywords has-patch added

I have attached a patch now to remove allowed class tag it was coded twice.

#3 @djpaul
9 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 9975:

Groups: remove duplicated filter in KSES whitelist.

Fixes #6495, props svenl77 for the patch (WordCamp Europe 2015).

Note: See TracTickets for help on using tickets.