Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

#3576 closed defect (bug) (fixed)

Problem with HTML code on Activity updates

Reported by: jconti's profile j.conti Owned by:
Milestone: 1.6 Priority: minor
Severity: minor Version: 1.5
Component: Activity Keywords:
Cc:

Description

Hi,

There is a problem with HTML code on BuddyPress Activity updates.

If you paste something like this on Actity update:

<div id="wrapper">

<div id="header">

<div id="access">
</div>
</div>

<div id="main">

<div id="content" align="left">
<div class="padder">

You will get something like attaced screenshot.

Attachments (1)

div.jpg​ (69.3 KB) - added by j.conti 13 years ago.
div on activity update

Download all attachments as: .zip

Change History (6)

@j.conti
13 years ago

div on activity update

#1 @boonebgorges
13 years ago

  • Milestone changed from Awaiting Review to 1.5.1
  • Priority changed from normal to minor
  • Severity changed from normal to minor

We should probably be stripping div tags.

#2 @johnjamesjacoby
13 years ago

Surprised we aren't already. Possible this is a super admin issue with unfiltered html?

#3 @boonebgorges
13 years ago

Apparently we do allow divs through. See ​http://buddypress.trac.wordpress.org/changeset/2088

Seems wrong to me. What do divs have to do with activity commenting (Andy's commit message)?

#4 @boonebgorges
13 years ago

  • Milestone changed from 1.5.2 to 1.6

Moving to 1.6, since this is not a regression, and I don't want to anger anyone who is using this oddity to some advantage in the midst of a dev cycle.

#5 @boonebgorges
13 years ago

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

(In [5361]) Don't allow divs in activity items by default. Fixes #3576

Note: See TracTickets for help on using tickets.