Changeset 11844

Timestamp:

02/04/2018 05:51:58 PM (7 years ago)

Author:

espellcaste

Message:

Remove redundant handling of optional $user_id parameter from messages_check_thread_access()

On messages_check_thread_access(), we are checking if the $user_id parameter is set. The same check is being done on BP_Messages_Thread::check_access().

So we are keeping only the low level check, instead of both.

Props DjPaul.

Fixes #7668

File:

• trunk/src/bp-messages/bp-messages-functions.php (modified) (1 diff)

trunk/src/bp-messages/bp-messages-functions.php

@@ -316 +316 @@
  */
 function messages_check_thread_access( $thread_id, $user_id = 0 ) {
-    if ( empty( $user_id ) ) {
-        $user_id = bp_loggedin_user_id();
-    }
-
     return BP_Messages_Thread::check_access( $thread_id, $user_id );
 }