Code improvement on BuddyPress detection

[Mte90]

As I can see in every page there is this query, but there isn't Buddypress in my installation.

AS I can see on

lifterlms/includes/class.llms.integrations.php

Line 52 in d824132

$load_integrations = apply_filters( 'lifterlms_integrations', array(

the class are defined maybe they are executed on every page.

There is the request https://github.com/gocodebox/lifterlms/blob/master/includes/abstracts/llms.abstract.integration.php#L125 maybe we can add a check in the classes with detect the plugin like for

lifterlms/includes/class.llms.widgets.php

Line 34 in 0eb9568

if ( class_exists( 'bbPress' ) && 'yes' === get_option( 'llms_integration_bbpress_enabled', 'no' ) ) {

inside every class.
In the meantime we can disable with an hook and save a query in every page, in all the pages of the site.

[thomasplevy]

This are the queries that happen in all the pages (in this case in the website homepage) maybe we can do something to do only when required?

Let's track this as a separate issue please

[thomasplevy]

I see that also this query run in all the pages on

lifterlms/includes/class.llms.query.php

Line 121 in 2a36f21

  	'person-logout' => get_option( 'lifterlms_logout_endpoint', 'person-logout' ), 

but this value is required in the code?

Please open this as a new issue as well please

[Mte90]

Opened other two tickets and cleaned this one

[thomasplevy]

Opened other two tickets and cleaned this one

Thanks!

[thomasplevy]

@Mte90 check #515 for me please, this will flip the is_installed() and is_enabled() checks on integration classes which will prevent the DB call (made by is_enabled()) when the integration is loaded by first using is_installed() to check for the existence of the integration dependencies

With BuddyPress: https://github.com/gocodebox/lifterlms/blob/master/includes/integrations/class.llms.integration.buddypress.php#L108

With bbPress: https://github.com/gocodebox/lifterlms/blob/master/includes/integrations/class.llms.integration.bbpress.php#L229

Etc...

Let me know if that helps!