Changeset 13900 for trunk/src/bp-core/classes/class-bp-core-notification.php

Timestamp:

06/02/2024 07:12:25 PM (20 months ago)

Author:

espellcaste

Message:

WPCS: Part VIII: miscellaneous fixes for some of the files of the core component.

Follow-up to [13883], [13886], [13887], [13888], [13891], [13892], [13893]

See #9164 and #7228

File:

• trunk/src/bp-core/classes/class-bp-core-notification.php (modified) (12 diffs)

trunk/src/bp-core/classes/class-bp-core-notification.php

@@ -85 +85 @@
     public function __construct( $id = 0 ) {
 
-        // Bail if no ID
+        // Bail if no ID.
         if ( empty( $id ) ) {
             return;
@@ -99 +99 @@
      * @global wpdb $wpdb WordPress database object.
      *
-     * @return bool Success or failure.
+     * @return bool
      */
     public function save() {
@@ -111 +111 @@
             $sql   = $wpdb->prepare( $query, $this->item_id, $this->secondary_item_id, $this->user_id, $this->component_name, $this->component_action, $this->date_notified, $this->is_new, $this->id );
 
-        // Save.
+            // Save.
         } else {
             $query = "INSERT INTO {$bp->notifications->table_name} ( item_id, secondary_item_id, user_id, component_name, component_action, date_notified, is_new ) VALUES ( %d, %d, %d, %s, %s, %s, %d )";
@@ -145 +145 @@
 
         if ( ! empty( $result ) ) {
-            $this->item_id = $result->item_id;
+            $this->item_id           = $result->item_id;
             $this->secondary_item_id = $result->secondary_item_id;
             $this->user_id           = $result->user_id;
@@ -160 +160 @@
      * Check the access for a user.
      *
+     * @global wpdb $wpdb WordPress database object.
+     *
      * @param int $user_id         ID to check access for.
      * @param int $notification_id Notification ID to check for.
@@ -181 +183 @@
      * @global wpdb $wpdb WordPress database object
      *
-     * @static
-     *
      * @param int    $user_id User ID.
      * @param string $status 'is_new' or 'all'.
@@ -207 +207 @@
      *
      * @global wpdb $wpdb WordPress database object.
-     *
-     * @static
      *
      * @param int    $user_id          ID of the user to delet notification for.
@@ -232 +230 @@
      * @global wpdb $wpdb WordPress database object.
      *
-     * @static
-     *
-     * @param int      $user_id           The ID of the user who the notifications are for.
-     * @param int      $item_id           The item ID of the notifications we wish to delete.
-     * @param string   $component_name    The name of the component that the notifications we wish to delete.
-     * @param string   $component_action  The action of the component that the notifications we wish to delete.
-     * @param int      $secondary_item_id (optional) The secondary item id of the notifications that we wish to
-     *                                    use to delete.
+     * @param int    $user_id           The ID of the user who the notifications are for.
+     * @param int    $item_id           The item ID of the notifications we wish to delete.
+     * @param string $component_name    The name of the component that the notifications we wish to delete.
+     * @param string $component_action  The action of the component that the notifications we wish to delete.
+     * @param int    $secondary_item_id (optional) The secondary item id of the notifications that we wish to
+     *                                  use to delete.
      * @return mixed
      */
@@ -248 +244 @@
 
         $secondary_item_sql = ! empty( $secondary_item_id )
-            ? $wpdb->prepare( " AND secondary_item_id = %d", $secondary_item_id )
+            ? $wpdb->prepare( ' AND secondary_item_id = %d', $secondary_item_id )
             : '';
 
@@ -262 +258 @@
      *
      * @global wpdb $wpdb WordPress database object.
-     *
-     * @static
      *
      * @param int    $user_id          The ID of the user whose sent notifications we wish to delete.
@@ -287 +281 @@
      *
      * @global wpdb $wpdb WordPress database object.
-     *
-     * @static
      *
      * @param int    $item_id           The item id that they notifications are to be for.
@@ -300 +292 @@
 
         $component_action_sql = ! empty( $component_action )
-            ? $wpdb->prepare( "AND component_action = %s", $component_action )
+            ? $wpdb->prepare( 'AND component_action = %s', $component_action )
             : '';
 
         $secondary_item_sql = ! empty( $secondary_item_id )
-            ? $wpdb->prepare( "AND secondary_item_id = %d", $secondary_item_id )
+            ? $wpdb->prepare( 'AND secondary_item_id = %d', $secondary_item_id )
             : '';