Skip to:
Content

BuddyPress.org

Changeset 11044


Ignore:
Timestamp:
08/30/2016 05:29:28 PM (10 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.