Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 15 years ago

#3576 closed defect (bug) (fixed)

Problem with HTML code on Activity updates

Reported by: j.conti Owned by:
Priority: minor Milestone: 1.6
Component: Activity Version: 1.5
Severity: minor 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 15 years ago.
div on activity update

Download all attachments as: .zip

Change History (6)

@j.conti
15 years ago

div on activity update

#1 @boonebgorges
15 years ago

  • Milestone Awaiting Review → 1.5.1
  • Priority normal → minor
  • Severity normal → minor

We should probably be stripping div tags.

#2 @johnjamesjacoby
15 years ago

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

#3 @boonebgorges
15 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
15 years ago

  • Milestone 1.5.2 → 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
15 years ago

  • Resolution → fixed
  • Status new → closed

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

Note: See TracTickets for help on using tickets.