Project

General

Profile

Actions

Bug #15144

closed

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

Added by Raymond Hoh about 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
Performance
Target version:
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 #1

Updated by Raymond Hoh about 2 years ago

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

Updated by Boone Gorges about 2 years ago

Thanks for catching this!

Actions #3

Updated by Boone Gorges about 2 years ago

  • Status changed from Staged for Production Release to Resolved
Actions

Also available in: Atom PDF