Project

General

Profile

Actions
Copy link

Bug #15144

closed

Move logged-in user checks for bbPress BP Notification Grabber later down the page

Added by Raymond Hoh almost 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Raymond Hoh
Category name:
Performance
Target version:
1.19.1
Start date:
2022-01-04
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

The logged-in user check in the bbPress BP Notification Grabber plugin here: https://github.com/cuny-academic-commons/cac/blob/a58ed0b5f7b5b907fbc5631c46e502cfa7a73806/wp-content/plugins/bbp-bp-notification-grabber/bbp-bp-notification-grabber.php#L21-L24

Is causing bbPress to run its roles initialization routine, which causes some DB queries to run on the 'plugins_loaded' hook.

I'm going to move the logged-in user check to the callbacks, which will stop these DB queries from running.

Actions
Copy link
 #1

Updated by Raymond Hoh almost 3 years ago

  • Status changed from New to Staged for Production Release
Actions
Copy link
 #2

Updated by Boone Gorges almost 3 years ago

Thanks for catching this!

Actions
Copy link
 #3

Updated by Boone Gorges almost 3 years ago

  • Status changed from Staged for Production Release to Resolved
Actions
Copy link

Also available in: Atom PDF