Changeset 13495 for trunk/src/bp-notifications/classes/class-bp-notifications-component.php

Timestamp:

06/02/2023 02:05:08 AM (3 years ago)

Author:

imath

Message:

Prevent potential issues with late_includes() in Component classes

Use require_once instead of require to load action & screen files.

See #4954
Fixes #8910
Closes ​https://github.com/buddypress/buddypress/pull/112

File:

• trunk/src/bp-notifications/classes/class-bp-notifications-component.php (modified) (1 diff)

trunk/src/bp-notifications/classes/class-bp-notifications-component.php

@@ -78 +78 @@
                 // Actions.
                 if ( bp_is_post_request() ) {
-                        require $this->path . 'bp-notifications/actions/bulk-manage.php';
+                        require_once $this->path . 'bp-notifications/actions/bulk-manage.php';
                 } elseif ( bp_is_get_request() ) {
-                        require $this->path . 'bp-notifications/actions/delete.php';
+                        require_once $this->path . 'bp-notifications/actions/delete.php';
                 }
 
                 // Screens.
-                require $this->path . 'bp-notifications/screens/unread.php';
+                require_once $this->path . 'bp-notifications/screens/unread.php';
                 if ( bp_is_current_action( 'read' ) ) {
-                        require $this->path . 'bp-notifications/screens/read.php';
+                        require_once $this->path . 'bp-notifications/screens/read.php';
                 }
         }