Changeset 12793 for trunk/src/bp-groups/classes/class-bp-groups-invitation-manager.php

Timestamp:

11/19/2020 12:50:44 PM (4 years ago)

Author:

imath

Message:

Groups: Avoid a PHP 8.0 deprecated notice about optional function arg.

The BP_Groups_Invitation_Manager->run_acceptance_action() method was wrongly using one optional arguments before a required one. As PHP 8.0 no longer accepts this practice, all arguments of the method will be required.

Fixes #8392 (trunk)

File:

• trunk/src/bp-groups/classes/class-bp-groups-invitation-manager.php (modified) (1 diff)

trunk/src/bp-groups/classes/class-bp-groups-invitation-manager.php

@@ -68 +68 @@
      * @return bool True on success, false on failure.
      */
-    public function run_acceptance_action( $type = 'invite', $r  ) {
+    public function run_acceptance_action( $type, $r ) {
+        if ( ! $type || ! in_array( $type, array( 'request', 'invite' ), true ) ) {
+            return false;
+        }
+
         // If the user is already a member (because BP at one point allowed two invitations to
         // slip through), return early.