Skip to:
Content

BuddyPress.org

Opened 16 years ago

Closed 16 years ago

Last modified 10 years ago

#2151 closed enhancement (fixed)

don't use global variable j in bp-themes/bp-default/_inc/globals.js for jQuery [has-patch]

Reported by: andrew_s1 Owned by:
Priority: minor Milestone: 1.2.3
Component: Activity Version:
Severity: Keywords: has-patch
Cc: wordpress@…

Description

It's just asking for trouble. People are always going to use i and j as handy counter variables, so it's not wise to rely on having j in the global namespace as an untainted jQuery object

Attachments (1)

globaljs.patch​ (2.0 KB ) - added by andrew_s1 16 years ago.

Download all attachments as: .zip

Change History (4)

@andrew_s1
16 years ago

#1 @andrew_s1
16 years ago

  • Cc wordpress@… added
  • Component → Activity
  • Priority trivial → minor
  • Summary don't use global variable j in bp-themes/bp-default/_inc/globals.js for jQuery → don't use global variable j in bp-themes/bp-default/_inc/globals.js for jQuery [has-patch]

#2 @apeatling
16 years ago

  • Resolution → fixed
  • Status new → closed

(In [2861]) Change jQuery var from j to jq to avoid potential iterator conflicts. Fixes #2151 props andrew_s1

#3 @DJPaul
10 years ago

  • Type task → enhancement
Note: See TracTickets for help on using tickets.