Changeset 12483 for branches/5.0/src/bp-xprofile/classes/class-bp-xprofile-component.php

Timestamp:

10/27/2019 04:53:16 AM (5 years ago)

Author:

imath

Message:

Only load BP REST API Avatar Controllers when avatar uploads are on

• Move the loading of the BP_REST_Attachments_Member_Avatar_Endpoint controller into the xProfile component's class. User Avatar is an xProfile feature.
• Make sure to check the community administrator allowed member avatar uploads before loading this controller.
• Make sure to check the community administrator allowed group avatar uploads before loading BP_REST_Attachments_Group_Avatar_Endpoint.

Fixes #8151 (Branch 5.0)

File:

• branches/5.0/src/bp-xprofile/classes/class-bp-xprofile-component.php (modified) (1 diff)

branches/5.0/src/bp-xprofile/classes/class-bp-xprofile-component.php

@@ -500 +500 @@
      */
     public function rest_api_init( $controllers = array() ) {
-        parent::rest_api_init( array(
+        $controllers = array(
             'BP_REST_XProfile_Fields_Endpoint',
             'BP_REST_XProfile_Field_Groups_Endpoint',
             'BP_REST_XProfile_Data_Endpoint',
-        ) );
+        );
+
+        // Only Load the Member's Avatar controller if Avatar uploads are allowed.
+        if ( ! bp_disable_avatar_uploads() ) {
+            $controllers[] = 'BP_REST_Attachments_Member_Avatar_Endpoint';
+        }
+
+        parent::rest_api_init( $controllers );
     }
 }