add version checking for Laravel 5.1+ where the lists() functionality has changed

Author: cmgmyr

phpunit.xml

@@ -15,4 +15,10 @@
             <directory suffix="Test.php">./tests/</directory>
         </testsuite>
     </testsuites>
+    <php>
+        <env name="APP_ENV" value="testing"/>
+        <env name="CACHE_DRIVER" value="array"/>
+        <env name="SESSION_DRIVER" value="array"/>
+        <env name="QUEUE_DRIVER" value="sync"/>
+    </php>
 </phpunit>

src/Cmgmyr/Messenger/Traits/Messagable.php

@@ -45,6 +45,18 @@ public function threadsWithNewMessages()
         $threadsWithNewMessages = [];
         $participants = Participant::where('user_id', $this->id)->lists('last_read', 'thread_id');
 
+        /**
+         * @todo: see if we can fix this more in the future.
+         * Illuminate\Foundation is not available through composer, only in laravel/framework which
+         * I don't want to include as a dependency for this package...it's overkill. So let's
+         * exclude this check in the testing environment.
+         */
+        if (getenv('APP_ENV') != 'testing') {
+            if (!str_contains(\Illuminate\Foundation\Application::VERSION, '5.0')) {
+                $participants = $participants->all();
+            }
+        }
+
         if ($participants) {
             $threads = Thread::whereIn('id', array_keys($participants))->get();