Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
11/08/2013 04:12:50 AM (9 years ago)
Author:
johnjamesjacoby
Message:

Clean up bp-notifications-classes.php. See #5148.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/bp-notifications/bp-notifications-classes.php

    r7521 r7525  
    493493        ) );
    494494
    495         $bp = buddypress();
    496 
     495        $bp         = buddypress();
     496        $pag_sql    = '';
    497497        $select_sql = "SELECT *";
    498498        $from_sql   = "FROM {$bp->notifications->table_name}";
    499 
    500         $where_args = array(
     499        $where_sql  = self::get_where_sql( array(
    501500            'id'                => $r['id'],
    502501            'user_id'           => $r['user_id'],
     
    507506            'is_new'            => $r['is_new'],
    508507            'search_terms'      => $r['search_terms'],
    509         );
    510 
    511         $where_sql = self::get_where_sql( $where_args );
    512 
    513         $pag_sql = '';
     508        ) );
     509
    514510        if ( ! empty( $r['page'] ) && ! empty( $r['per_page'] ) ) {
    515             $page     = absint( $r['page'] );
     511            $page     = absint( $r['page']     );
    516512            $per_page = absint( $r['per_page'] );
    517513            $offset   = $per_page * ( $page - 1 );
    518             $pag_sql = $wpdb->prepare( "LIMIT %d, %d", $offset, $per_page );
     514            $pag_sql  = $wpdb->prepare( "LIMIT %d, %d", $offset, $per_page );
    519515        }
    520516
     
    538534        global $wpdb;
    539535
    540         $bp = buddypress();
    541 
     536        $bp         = buddypress();
    542537        $select_sql = "SELECT COUNT(*)";
    543538        $from_sql   = "FROM {$bp->notifications->table_name}";
    544539        $where_sql  = self::get_where_sql( $args );
    545 
    546         $sql = "{$select_sql} {$from_sql} {$where_sql}";
     540        $sql        = "{$select_sql} {$from_sql} {$where_sql}";
    547541
    548542        return $wpdb->get_var( $sql );
     
    567561    public static function update( $update_args = array(), $where_args = array() ) {
    568562        $update = self::get_query_clauses( $update_args );
    569         $where  = self::get_query_clauses( $where_args );
     563        $where  = self::get_query_clauses( $where_args  );
    570564
    571565        return self::_update( $update['data'], $where['data'], $update['format'], $where['format'] );
     
    591585    }
    592586
    593 
    594     /** Convenience methods **********************************************/
     587    /** Convenience methods ***************************************************/
    595588
    596589    /**
     
    623616     */
    624617    public static function get_all_for_user( $user_id, $status = 'is_new' ) {
    625         $args = array(
     618        return self::get( array(
    626619            'user_id' => $user_id,
    627620            'is_new'  => 'is_new' === $status,
    628         );
    629         return self::get( $args );
     621        ) );
    630622    }
    631623
     
    640632     */
    641633    public static function get_unread_for_user( $user_id = 0 ) {
    642         $args = array(
     634        return self::get( array(
    643635            'user_id' => $user_id,
    644636            'is_new'  => true,
    645         );
    646         return self::get( $args );
     637        ) );
    647638    }
    648639
     
    657648     */
    658649    public static function get_read_for_user( $user_id = 0 ) {
    659         $args = array(
     650        return self::get( array(
    660651            'user_id' => $user_id,
    661652            'is_new'  => false,
    662         );
    663         return self::get( $args );
     653        ) );
    664654    }
    665655
     
    708698    }
    709699
    710     /** Mark Read *************************************************************/
     700    /** Mark ******************************************************************/
    711701
    712702    /**
     
    731721        );
    732722
    733         if ( $component_name ) {
     723        if ( !empty( $item_id ) ) {
     724            $where_args['item_id'] = $item_id;
     725        }
     726
     727        if ( !empty( $component_name ) ) {
    734728            $where_args['component_name'] = $component_name;
    735729        }
    736730
    737         if ( $component_action ) {
     731        if ( !empty( $component_action ) ) {
    738732            $where_args['component_action'] = $component_action;
    739733        }
    740734
    741         if ( $secondary_item_id ) {
     735        if ( !empty( $secondary_item_id ) ) {
    742736            $where_args['secondary_item_id'] = $secondary_item_id;
    743737        }
     
    766760        );
    767761
    768         if ( $component_name ) {
     762        if ( !empty( $component_name ) ) {
    769763            $where_args['component_name'] = $component_name;
    770764        }
    771765
    772         if ( $component_action ) {
     766        if ( !empty( $component_action ) ) {
    773767            $where_args['component_action'] = $component_action;
    774768        }
    775769
    776         if ( $secondary_item_id ) {
     770        if ( !empty( $secondary_item_id ) ) {
    777771            $where_args['secondary_item_id'] = $secondary_item_id;
    778772        }
     
    808802        );
    809803
    810         if ( $component_name ) {
     804        if ( !empty( $component_name ) ) {
    811805            $where_args['component_name'] = $component_name;
    812806        }
    813807
    814         if ( $component_action ) {
     808        if ( !empty( $component_action ) ) {
    815809            $where_args['component_action'] = $component_action;
    816810        }
    817811
    818         if ( $secondary_item_id ) {
     812        if ( !empty( $secondary_item_id ) ) {
    819813            $where_args['secondary_item_id'] = $secondary_item_id;
    820814        }
Note: See TracChangeset for help on using the changeset viewer.