Skip to:
Content

BuddyPress.org

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#3047 closed defect (bug) (fixed)

jQuery not properly enqueued in all circumstances

Reported by: johnpbloch's profile johnpbloch Owned by: johnpbloch's profile johnpbloch
Milestone: 1.5 Priority: normal
Severity: normal Version:
Component: Core Keywords: has-patch
Cc:

Description

In the function bp_core_confirmation_js(), BuddyPress echoes inline javascript with jQuery directly to the <head> section without properly checking to make sure jQuery has been loaded. Since BuddyPress doesn't normally run without jQuery by default, this isn't generally a problem. However, if using the BP Template Pack, one has the option of turning off the default javascript and CSS. If you do that, the core confirmation script still runs without the jQuery dependency in place.

bp_core_confirmation_js() should be checking if jQuery has been loaded, and should load it if it hasn't been. I have a working patch which I'll attach momentarily.

Attachments (1)

3047.patch​ (666 bytes) - added by johnpbloch 14 years ago.
If jQuery hasn't been enqueued, do so. If it hasn't been printed, do so.

Download all attachments as: .zip

Change History (5)

@johnpbloch
14 years ago

If jQuery hasn't been enqueued, do so. If it hasn't been printed, do so.

#1 @johnpbloch
14 years ago

  • Keywords has-patch added
  • Owner set to johnpbloch
  • Status changed from new to assigned

#2 @johnjamesjacoby
14 years ago

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

(In [4018]) Make sure jQuery is loaded in bp_core_confirmation_js(). This is primarily for themes that do not include jQuery independently. Fixes #3047. Props johnploch.

#3 @johnjamesjacoby
14 years ago

Props johnpbloch even.

#4 @johnjamesjacoby
13 years ago

  • Milestone changed from Awaiting Review to 1.5
  • Severity set to normal
  • Version 1.5 deleted

Moving closed ticket out of Awaiting Review.

Note: See TracTickets for help on using tickets.