Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#6003 closed defect (bug) (fixed)

variable firstrow is used in multiple methods that can be called simultaiously

Reported by: andemann Owned by: boonebgorges
Priority: normal Milestone: 2.2
Component: Appearance - Template Pack Version: 2.1
Severity: normal Keywords:
Cc:

Description

In buddypress/bp-templates/bp-legacy/jsbuddypress.js the variable firstrow is used both in
jq( document ).on( 'heartbeat-send.buddypress', on line 1605 to hold the timestamp for last updated

and in jq('#aw-whats-new-submit').on( 'click', function() { on line 101
it is used to hold a reference towards the first li in the activity list.

This causes and error if the wordpress heartbeat is triggered while a new activity is posted.

Change History (2)

#1 @boonebgorges
12 years ago

  • Component Core → Template Pack
  • Milestone Awaiting Review → 2.2
  • Owner set to boonebgorges
  • Status new → accepted

Nice find. Thanks for the report.

#2 @boonebgorges
12 years ago

  • Resolution → fixed
  • Status accepted → closed

In 9129:

Use a unique variable name in activate heartbeat JS.

The name 'firstrow' was colliding with the "What's new" JS.

Props andemann.
Fixes #6003.

Note: See TracTickets for help on using tickets.