Skip to:
Content

BuddyPress.org

Changeset 11044


Ignore:
Timestamp:
08/30/2016 05:29:28 PM (7 years ago)
Author:
boonebgorges
Message:

Update Messages tests to work after the changes in [11042].

See #7234.

Location:
trunk/tests/phpunit/testcases/messages
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/testcases/messages/cache.php

    r9819 r11044  
    1414
    1515        // create the thread
    16         $t1 = $this->factory->message->create( array(
     16        $message_1 = $this->factory->message->create_and_get( array(
    1717            'sender_id'  => $u1,
    1818            'recipients' => array( $u2 ),
    … …  
    2121
    2222        // create a reply
    23         $this->factory->message->create( array(
    24             'thread_id'  => $t1,
     23        $message_2 = $this->factory->message->create_and_get( array(
     24            'thread_id'  => $message_1->thread_id,
    2525            'sender_id'  => $u2,
    2626            'recipients' => array( $u1 ),
    … …  
    2828        ) );
    2929
    30         // grab the message ids as individual variables
    31         list( $m1, $m2 ) = $this->get_message_ids( $t1 );
     30        $m1 = $message_1->id;
     31        $m2 = $message_2->id;
    3232
    3333        // add cache for each message
    … …  
    8686        $u2 = $this->factory->user->create();
    8787
    88         // create the thread
    89         $t1 = $this->factory->message->create( array(
     88        // create the message and thread
     89        $m = $this->factory->message->create_and_get( array(
    9090            'sender_id'  => $u1,
    9191            'recipients' => array( $u2 ),
    … …  
    9494
    9595        // add message cache
    96         list( $m1 ) = $this->get_message_ids( $t1 );
    97         bp_messages_update_meta( $m1, 'yolo', 'gah' );
     96        bp_messages_update_meta( $m->id, 'yolo', 'gah' );
    9897
    9998        // prime meta cache in message loop
    10099        bp_thread_has_messages( array(
    101             'thread_id' => $t1,
     100            'thread_id' => $m->thread_id,
    102101            'update_meta_cache' => true
    103102        ) );
    104103
    105         $this->assertNotEmpty( wp_cache_get( $m1, 'message_meta' ) );
     104        $this->assertNotEmpty( wp_cache_get( $m->id, 'message_meta' ) );
    106105    }
    107106
    … …  
    117116
    118117        // create the thread
    119         $t1 = $this->factory->message->create( array(
     118        $message_1 = $this->factory->message->create_and_get( array(
    120119            'sender_id'  => $u1,
    121120            'recipients' => array( $u2 ),
    … …  
    124123
    125124        // create a reply
    126         $this->factory->message->create( array(
    127             'thread_id'  => $t1,
     125        $message_2 = $this->factory->message->create_and_get( array(
     126            'thread_id'  => $message_1->thread_id,
    128127            'sender_id'  => $u2,
    129128            'recipients' => array( $u1 ),
    … …  
    131130        ) );
    132131
     132        $m1 = $message_1->id;
     133        $m2 = $message_2->id;
     134        $t1 = $message_1->thread_id;
     135
    133136        // add message meta
    134         list( $m1, $m2 ) = $this->get_message_ids( $t1 );
    135137        bp_messages_update_meta( $m1, 'yolo', 'gah' );
    136138        bp_messages_update_meta( $m2, 'yolo', 'bah' );

  • trunk/tests/phpunit/testcases/messages/class.bp-messages-thread.php

    r10038 r11044  
    33/**
    44 * @group BP_Messages_Thread
     5 * @group messages
    56 */
    67class BP_Tests_BP_Messages_Thread extends BP_UnitTestCase {
    … …  
    1314        $u2 = $this->factory->user->create();
    1415
    15         $t1 = $this->factory->message->create( array(
     16        $message = $this->factory->message->create_and_get( array(
    1617            'sender_id' => $u1,
    1718            'recipients' => array( $u2 ),
    … …  
    2021
    2122        // prime cache
    22         new BP_Messages_Thread( $t1 );
     23        new BP_Messages_Thread( $message->thread_id );
    2324
    2425        // Cache should exist
    2526        $this->assertThat(
    26             wp_cache_get( $t1, 'bp_messages_threads' ),
     27            wp_cache_get( $message->thread_id, 'bp_messages_threads' ),
    2728            $this->logicalNot( $this->equalTo( false ) ),
    2829            'Message thread cache should exist.'
    … …  
    3839
    3940        // create thread
    40         $t1 = $this->factory->message->create( array(
    41             'sender_id' => $u1,
    42             'recipients' => array( $u2 ),
    43             'subject' => 'Foo',
    44         ) );
    45         // save message ID
    46         $thread = new BP_Messages_Thread( $t1 );
    47         $m1 = wp_list_pluck( $thread->messages, 'id' );
    48         $m1 = array_pop( $m1 );
     41        $message_1 = $this->factory->message->create_and_get( array(
     42            'sender_id' => $u1,
     43            'recipients' => array( $u2 ),
     44            'subject' => 'Foo',
     45        ) );
     46        $m1 = $message_1->id;
    4947
    5048        // create reply
    51         $t2 = $this->factory->message->create( array(
    52             'thread_id' => $t1,
     49        $message_2 = $this->factory->message->create_and_get( array(
     50            'thread_id' => $message_1->thread_id,
    5351            'sender_id' => $u1,
    5452            'recipients' => array( $u2 ),
    5553            'content' => 'Bar'
    5654        ) );
    57         // save message ID
    58         $thread = new BP_Messages_Thread( $t1 );
    59         $m2 = wp_list_pluck( $thread->messages, 'id' );
    60         $m2 = array_pop( $m2 );
     55        $m2 = $message_2->id;
    6156
    6257        // now get thread by DESC
    63         $thread = new BP_Messages_Thread( $t1, 'DESC' );
     58        $thread = new BP_Messages_Thread( $message_1->thread_id, 'DESC' );
    6459
    6560        // assert!
    … …  
    7772        $u2 = $this->factory->user->create();
    7873
    79         $t1 = $this->factory->message->create( array(
    80             'sender_id' => $u1,
    81             'recipients' => array( $u2 ),
    82             'subject' => 'Foo',
    83         ) );
    84 
    85         $t2 = $this->factory->message->create( array(
     74        $message_1 = $this->factory->message->create_and_get( array(
     75            'sender_id' => $u1,
     76            'recipients' => array( $u2 ),
     77            'subject' => 'Foo',
     78        ) );
     79
     80        $message_2 = $this->factory->message->create_and_get( array(
    8681            'sender_id' => $u1,
    8782            'recipients' => array( $u2 ),
    … …  
    9489        ) );
    9590
    96         $expected = array( $t2 );
     91        $expected = array( $message_2->thread_id );
    9792        $found = wp_parse_id_list( wp_list_pluck( $threads['threads'], 'thread_id' ) );
    9893
    … …  
    107102        $u2 = $this->factory->user->create();
    108103
    109         $t1 = $this->factory->message->create( array(
    110             'sender_id' => $u1,
    111             'recipients' => array( $u2 ),
    112             'subject' => 'Foo',
    113         ) );
    114 
    115         $t2 = $this->factory->message->create( array(
     104        $message_1 = $this->factory->message->create_and_get( array(
     105            'sender_id' => $u1,
     106            'recipients' => array( $u2 ),
     107            'subject' => 'Foo',
     108        ) );
     109
     110        $message_2 = $this->factory->message->create_and_get( array(
    116111            'sender_id' => $u1,
    117112            'recipients' => array( $u2 ),
    … …  
    125120        ) );
    126121
    127         $expected = array( $t2 );
     122        $expected = array( $message_2->thread_id );
    128123        $found = wp_parse_id_list( wp_list_pluck( $threads['threads'], 'thread_id' ) );
    129124
    … …  
    139134        $u2 = $this->factory->user->create();
    140135
    141         $t1 = $this->factory->message->create( array(
    142             'sender_id' => $u1,
    143             'recipients' => array( $u2 ),
    144             'subject' => 'Foo',
    145         ) );
    146 
    147         $t2 = $this->factory->message->create( array(
     136        $message_1 = $this->factory->message->create_and_get( array(
     137            'sender_id' => $u1,
     138            'recipients' => array( $u2 ),
     139            'subject' => 'Foo',
     140        ) );
     141
     142        $message_2 = $this->factory->message->create_and_get( array(
    148143            'sender_id' => $u1,
    149144            'recipients' => array( $u2 ),
    … …  
    153148        $threads = BP_Messages_Thread::get_current_threads_for_user( $u1, 'sentbox', 'all', null, null, 'ar' );
    154149
    155         $expected = array( $t2 );
     150        $expected = array( $message_2->thread_id );
    156151        $found = wp_parse_id_list( wp_list_pluck( $threads['threads'], 'thread_id' ) );
    157152
    … …  
    169164        $u2 = $this->factory->user->create();
    170165
    171         $t1 = $this->factory->message->create( array(
    172             'sender_id' => $u1,
    173             'recipients' => array( $u2 ),
    174             'subject' => 'Foo',
    175         ) );
    176 
    177         $thread = new BP_Messages_Thread( $t1 );
     166        $message = $this->factory->message->create_and_get( array(
     167            'sender_id' => $u1,
     168            'recipients' => array( $u2 ),
     169            'subject' => 'Foo',
     170        ) );
     171
     172        $thread = new BP_Messages_Thread( $message->thread_id );
    178173        $recipients = $thread->get_recipients();
    179174
    … …  
    195190        $u2 = $this->factory->user->create();
    196191
    197         $t1 = $this->factory->message->create( array(
    198             'sender_id' => $u1,
    199             'recipients' => array( $u2 ),
    200             'subject' => 'Foo',
    201         ) );
    202 
    203         $thread = new BP_Messages_Thread( $t1 );
     192        $message = $this->factory->message->create_and_get( array(
     193            'sender_id' => $u1,
     194            'recipients' => array( $u2 ),
     195            'subject' => 'Foo',
     196        ) );
     197
     198        $thread = new BP_Messages_Thread( $message->thread_id );
    204199        $recipients = $thread->get_recipients();
    205200
    … …  
    211206        messages_new_message( array(
    212207            'sender_id' => $u2,
    213             'thread_id' => $t1,
     208            'thread_id' => $message->thread_id,
    214209            'recipients' => array( $u1 ),
    215210            'subject' => 'Bar',
    … …  
    233228        $u2 = $this->factory->user->create();
    234229
    235         $t1 = $this->factory->message->create( array(
    236             'sender_id' => $u1,
    237             'recipients' => array( $u2 ),
    238             'subject' => 'Foo',
    239         ) );
     230        $message = $this->factory->message->create_and_get( array(
     231            'sender_id' => $u1,
     232            'recipients' => array( $u2 ),
     233            'subject' => 'Foo',
     234        ) );
     235
     236        $t1 = $message->thread_id;
    240237
    241238        $thread = new BP_Messages_Thread( $t1 );
    … …  
    263260        $u2 = $this->factory->user->create();
    264261
    265         $t1 = $this->factory->message->create( array(
    266             'sender_id' => $u1,
    267             'recipients' => array( $u2 ),
    268             'subject' => 'Foo',
    269         ) );
     262        $message = $this->factory->message->create_and_get( array(
     263            'sender_id' => $u1,
     264            'recipients' => array( $u2 ),
     265            'subject' => 'Foo',
     266        ) );
     267
     268        $t1 = $message->thread_id;
    270269
    271270        $thread = new BP_Messages_Thread( $t1 );
    … …  
    293292        $u2 = $this->factory->user->create();
    294293
    295         $t1 = $this->factory->message->create( array(
    296             'sender_id' => $u1,
    297             'recipients' => array( $u2 ),
    298             'subject' => 'Foo',
    299         ) );
     294        $message = $this->factory->message->create_and_get( array(
     295            'sender_id' => $u1,
     296            'recipients' => array( $u2 ),
     297            'subject' => 'Foo',
     298        ) );
     299
     300        $t1 = $message->thread_id;
    300301
    301302        $thread = new BP_Messages_Thread( $t1 );
    … …  
    328329        $u2 = $this->factory->user->create();
    329330
    330         $t1 = $this->factory->message->create( array(
    331             'sender_id' => $u1,
    332             'recipients' => array( $u2 ),
    333             'subject' => 'Foo',
    334         ) );
     331        $message = $this->factory->message->create_and_get( array(
     332            'sender_id' => $u1,
     333            'recipients' => array( $u2 ),
     334            'subject' => 'Foo',
     335        ) );
     336
     337        $t1 = $message->thread_id;
    335338
    336339        $thread = new BP_Messages_Thread( $t1 );
    … …  
    360363        $u2 = $this->factory->user->create();
    361364
    362         $t1 = $this->factory->message->create( array(
    363             'sender_id' => $u1,
    364             'recipients' => array( $u2 ),
    365             'subject' => 'Foo',
    366         ) );
     365        $message = $this->factory->message->create_and_get( array(
     366            'sender_id' => $u1,
     367            'recipients' => array( $u2 ),
     368            'subject' => 'Foo',
     369        ) );
     370
     371        $t1 = $message->thread_id;
    367372
    368373        // save recipient ID
    … …  
    388393        $u2 = $this->factory->user->create();
    389394
    390         $t1 = $this->factory->message->create( array(
    391             'sender_id' => $u1,
    392             'recipients' => array( $u2 ),
    393             'subject' => 'Foo',
    394         ) );
     395        $message = $this->factory->message->create_and_get( array(
     396            'sender_id' => $u1,
     397            'recipients' => array( $u2 ),
     398            'subject' => 'Foo',
     399        ) );
     400
     401        $t1 = $message->thread_id;
    395402
    396403        $this->assertEquals( $t1, BP_Messages_Thread::is_valid( $t1 ) );
    … …  
    413420        $date = bp_core_current_time();
    414421
    415         $t1 = $this->factory->message->create( array(
     422        $message = $this->factory->message->create_and_get( array(
    416423            'sender_id' => $u1,
    417424            'recipients' => array( $u2 ),
    … …  
    420427            'content' => 'Bar and baz.',
    421428        ) );
     429
     430        $t1 = $message->thread_id;
    422431
    423432        $thread = new BP_Messages_Thread( $t1 );

  • trunk/tests/phpunit/testcases/messages/star.php

    r9951 r11044  
    1515
    1616        // create the thread
    17         $t1 = $this->factory->message->create( array(
     17        $message_1 = $this->factory->message->create_and_get( array(
    1818            'sender_id'  => $u1,
    1919            'recipients' => array( $u2 ),
    … …  
    2222
    2323        // create a reply
    24         $this->factory->message->create( array(
    25             'thread_id'  => $t1,
     24        $message_2 = $this->factory->message->create_and_get( array(
     25            'thread_id'  => $message_1->thread_id,
    2626            'sender_id'  => $u2,
    2727            'recipients' => array( $u1 ),
    … …  
    2929        ) );
    3030
    31         // grab the message ids as individual variables
    32         list( $m1, $m2 ) = $this->get_message_ids( $t1 );
     31        $m1 = $message_1->id;
     32        $m2 = $message_2->id;
    3333
    3434        // star the second message
    … …  
    6363
    6464        // create the thread
    65         $t1 = $this->factory->message->create( array(
     65        $message_1 = $this->factory->message->create_and_get( array(
    6666            'sender_id'  => $u1,
    6767            'recipients' => array( $u2 ),
    … …  
    7070
    7171        // create a reply
    72         $this->factory->message->create( array(
    73             'thread_id'  => $t1,
     72        $message_2 = $this->factory->message->create_and_get( array(
     73            'thread_id'  => $message_1->thread_id,
    7474            'sender_id'  => $u2,
    7575            'recipients' => array( $u1 ),
    … …  
    7777        ) );
    7878
    79         // grab the message ids as individual variables
    80         list( $m1, $m2 ) = $this->get_message_ids( $t1 );
     79        $m1 = $message_1->id;
     80        $m2 = $message_2->id;
     81        $t1 = $message_1->thread_id;
    8182
    8283        // star all messages
    … …  
    116117
    117118        // create three threads
    118         $t1 = $this->factory->message->create( array(
     119        $message_1 = $this->factory->message->create_and_get( array(
    119120            'sender_id'  => $u1,
    120121            'recipients' => array( $u2 ),
    121122            'subject'    => 'A',
    122123        ) );
    123         $t2 = $this->factory->message->create( array(
     124        $message_2 = $this->factory->message->create_and_get( array(
    124125            'sender_id'  => $u1,
    125126            'recipients' => array( $u2 ),
    126127            'subject'    => 'B',
    127128        ) );
    128         $t3 = $this->factory->message->create( array(
     129        $message_3 = $this->factory->message->create_and_get( array(
    129130            'sender_id'  => $u1,
    130131            'recipients' => array( $u2 ),
    … …  
    133134
    134135        // grab the message ids as individual variables
    135         list( $m1 ) = $this->get_message_ids( $t1 );
    136         list( $m2 ) = $this->get_message_ids( $t2 );
    137         list( $m3 ) = $this->get_message_ids( $t3 );
     136        $m1 = $message_1->id;
     137        $m2 = $message_2->id;
     138        $m3 = $message_3->id;
     139
     140        $t2 = $message_2->thread_id;
    138141
    139142        // star all threads

  • trunk/tests/phpunit/testcases/messages/template.php

    r9948 r11044  
    1414
    1515        // create initial thread
    16         $t1 = $this->factory->message->create( array(
     16        $message_1 = $this->factory->message->create_and_get( array(
    1717            'sender_id'  => $u1,
    1818            'recipients' => array( $u2 ),
    … …  
    2020
    2121        // create some replies to thread
    22         $this->factory->message->create( array(
    23             'thread_id' => $t1,
     22        $message_2 = $this->factory->message->create_and_get( array(
     23            'thread_id' => $message_1->thread_id,
    2424            'sender_id' => $u2,
    2525            'recipients' => array( $u1 ),
    2626        ) );
    27         $this->factory->message->create( array(
    28             'thread_id' => $t1,
     27        $message_3 = $this->factory->message->create_and_get( array(
     28            'thread_id' => $message_1->thread_id,
    2929            'sender_id' => $u2,
    3030            'recipients' => array( $u1 ),
    … …  
    3434
    3535        $this->assertEquals( 1, $messages_template->thread_count );
    36         $this->assertSame( array( $t1 ), wp_list_pluck( $messages_template->threads, 'thread_id' ) );
     36        $this->assertSame( array( $message_1->thread_id ), wp_list_pluck( $messages_template->threads, 'thread_id' ) );
    3737    }
    3838
    … …  
    4747
    4848        // create initial thread
    49         $t1 = $this->factory->message->create( array(
     49        $message_1 = $this->factory->message->create_and_get( array(
    5050            'sender_id'  => $u1,
    5151            'recipients' => array( $u2 ),
    … …  
    5353
    5454        // create some replies to thread
    55         $this->factory->message->create( array(
    56             'thread_id' => $t1,
     55        $message_2 = $this->factory->message->create_and_get( array(
     56            'thread_id' => $message_1->thread_id,
    5757            'sender_id' => $u2,
    5858            'recipients' => array( $u1 ),
    5959        ) );
    60         $this->factory->message->create( array(
    61             'thread_id' => $t1,
     60        $message_3 = $this->factory->message->create_and_get( array(
     61            'thread_id' => $message_1->thread_id,
    6262            'sender_id' => $u2,
    6363            'recipients' => array( $u1 ),
    … …  
    6767
    6868        $this->assertEquals( 1, $messages_template->thread_count );
    69         $this->assertSame( array( $t1 ), wp_list_pluck( $messages_template->threads, 'thread_id' ) );
     69        $this->assertSame( array( $message_1->thread_id ), wp_list_pluck( $messages_template->threads, 'thread_id' ) );
    7070    }
    7171
    … …  
    7979
    8080        // create some threads
    81         $t1 = $this->factory->message->create( array(
     81        $message_1 = $this->factory->message->create_and_get( array(
    8282            'sender_id'  => $u1,
    8383            'recipients' => array( $u2 ),
    8484            'subject'    => 'This is a knive',
    8585        ) );
    86         $t2 = $this->factory->message->create( array(
     86        $message_2 = $this->factory->message->create_and_get( array(
    8787            'sender_id'  => $u1,
    8888            'recipients' => array( $u2 ),
    … …  
    9696        ) );
    9797
     98        $t1 = $message_1->thread_id;
     99        $t2 = $message_2->thread_id;
     100
    98101        // create some replies for thread 1
    99         $this->factory->message->create( array(
     102        $message_3 = $this->factory->message->create_and_get( array(
    100103            'thread_id'  => $t1,
    101104            'sender_id'  => $u2,
    … …  
    103106            'content'    => "That's a spoon",
    104107        ) );
    105         $this->factory->message->create( array(
     108        $message_4 = $this->factory->message->create_and_get( array(
    106109            'thread_id'  => $t1,
    107110            'sender_id'  => $u1,
    … …  
    109112            'content'    => "I see you've played knivey-spooney before.",
    110113        ) );
    111         $this->factory->message->create( array(
     114        $message_5 = $this->factory->message->create_and_get( array(
    112115            'thread_id'  => $t1,
    113116            'sender_id'  => $u2,
    … …  
    117120
    118121        // create some replies for thread 2
    119         $this->factory->message->create( array(
     122        $message_6 = $this->factory->message->create_and_get( array(
    120123            'thread_id'  => $t2,
    121124            'sender_id'  => $u2,
    … …  
    124127        ) );
    125128
    126         // grab the message ids as individual variables for thread 1
    127         $thread = new BP_Messages_Thread( $t1 );
    128         $mids = wp_list_pluck( $thread->messages, 'id' );
    129         list( $m1, $m2, $m3, $m4 ) = $mids;
    130 
    131         // grab the message ids as individual variables for thread 2
    132         $thread = new BP_Messages_Thread( $t2 );
    133         $mids = wp_list_pluck( $thread->messages, 'id' );
    134         list( $m5, $m6 ) = $mids;
     129        // belong to $t1
     130        $m1 = $message_1->id;
     131        $m3 = $message_3->id;
     132        $m4 = $message_4->id;
     133        $m5 = $message_5->id;
     134
     135        // belong to $t2
     136        $m2 = $message_2->id;
     137        $m6 = $message_6->id;
    135138
    136139        // add meta for some of the messages
    … …  
    145148        bp_messages_update_meta( $m3, "starred_by_user_{$u2}", true );
    146149
    147         bp_messages_update_meta( $m5, "starred_by_user_{$u2}", true );
     150        bp_messages_update_meta( $m6, "starred_by_user_{$u2}", true );
    148151
    149152        // now, do the message thread loop query
    … …  
    171174
    172175        // create some threads
    173         $t1 = $this->factory->message->create( array(
     176        $message_1 = $this->factory->message->create_and_get( array(
    174177            'sender_id'  => $u1,
    175178            'recipients' => array( $u2 ),
    176179            'subject'    => 'This is a knive',
    177180        ) );
    178         $t2 = $this->factory->message->create( array(
     181        $message_2 = $this->factory->message->create_and_get( array(
    179182            'sender_id'  => $u1,
    180183            'recipients' => array( $u2 ),
    … …  
    188191        ) );
    189192
     193        $t1 = $message_1->thread_id;
     194        $t2 = $message_2->thread_id;
     195
    190196        // create some replies for thread 1
    191         $this->factory->message->create( array(
     197        $message_3 = $this->factory->message->create_and_get( array(
    192198            'thread_id'  => $t1,
    193199            'sender_id'  => $u2,
    … …  
    195201            'content'    => "That's a spoon",
    196202        ) );
    197         $this->factory->message->create( array(
     203        $message_4 = $this->factory->message->create_and_get( array(
    198204            'thread_id'  => $t1,
    199205            'sender_id'  => $u1,
    … …  
    201207            'content'    => "I see you've played knivey-spooney before.",
    202208        ) );
    203         $this->factory->message->create( array(
     209        $message_5 = $this->factory->message->create_and_get( array(
    204210            'thread_id'  => $t1,
    205211            'sender_id'  => $u2,
    … …  
    209215
    210216        // create some replies for thread 2
    211         $this->factory->message->create( array(
     217        $message_6 = $this->factory->message->create_and_get( array(
    212218            'thread_id'  => $t2,
    213219            'sender_id'  => $u2,
    … …  
    216222        ) );
    217223
    218         // grab the message ids as individual variables for thread 1
    219         $thread = new BP_Messages_Thread( $t1 );
    220         $mids = wp_list_pluck( $thread->messages, 'id' );
    221         list( $m1, $m2, $m3, $m4 ) = $mids;
    222 
    223         // grab the message ids as individual variables for thread 2
    224         $thread = new BP_Messages_Thread( $t2 );
    225         $mids = wp_list_pluck( $thread->messages, 'id' );
    226         list( $m5, $m6 ) = $mids;
     224        // belong to $t1
     225        $m1 = $message_1->id;
     226        $m3 = $message_3->id;
     227        $m4 = $message_4->id;
     228        $m5 = $message_5->id;
     229
     230        // belong to $t2
     231        $m2 = $message_2->id;
     232        $m6 = $message_6->id;
    227233
    228234        // add meta for some of the messages
    … …  
    237243        bp_messages_update_meta( $m3, "starred_by_user_{$u2}", true );
    238244
    239         bp_messages_update_meta( $m5, "starred_by_user_{$u2}", true );
     245        bp_messages_update_meta( $m6, "starred_by_user_{$u2}", true );
    240246
    241247        // now, do the message thread loop query
Note: See TracChangeset for help on using the changeset viewer.