Update mongodb/laravel-mongodb package to version 5.0 in composer.json (#27)

* Update mongodb/laravel-mongodb package to version 5.0 in composer.json

* Update MongoDB commands and improve session handling

Refactor the command to use `getDatabase()` instead of `getMongoDB()`. Update payload access in `MongoDbSessionHandler` from array syntax to object syntax for better readability. Adjust test assertions to match the new access methodology and ensure compatibility with Carbon instances for expiration and last activity timestamps. Clean up formatting inconsistencies throughout the code.

* Update MongoDB connection method to use getDatabase() instead of getMongoDB() in MongodbSessionIndex command

Author: DevquasarX9

composer.json

@@ -5,7 +5,7 @@
     "require": {
         "php": "^8.1",
         "illuminate/session": "^10.0",
-        "mongodb/laravel-mongodb": "^4.0",
+        "mongodb/laravel-mongodb": "^5.0",
         "ext-mongodb": "*"
     },
     "require-dev": {

src/MongoDbSessionHandler.php

@@ -2,20 +2,23 @@
 
 namespace ForFit\Session;
 
-use Illuminate\Database\ConnectionInterface;
-use Illuminate\Database\Query\Builder;
 use MongoDB\BSON\Binary;
 use MongoDB\BSON\UTCDateTime;
 use MongoDB\Driver\Exception\BulkWriteException;
 use SessionHandlerInterface;
 
 class MongoDbSessionHandler implements SessionHandlerInterface
 {
-    protected ConnectionInterface $connection;
-    protected int $minutes;
-    protected string $table;
+    protected $connection;
+    protected $minutes;
+    protected $table;
 
-    public function __construct(ConnectionInterface $connection, string $table = 'sessions', int $minutes = 60)
+    /**
+     * @param \Illuminate\Database\ConnectionInterface $connection
+     * @param string $table
+     * @param integer $minutes
+     */
+    public function __construct($connection, $table = 'sessions', $minutes = 60)
     {
         $this->connection = $connection;
         $this->minutes = (int)$minutes;
@@ -86,7 +89,7 @@ public function gc($max_lifetime): false|int
      * Returns the query builder
      *
      */
-    protected function query(): Builder
+    protected function query()
     {
         return $this->connection->table($this->table);
     }
@@ -97,7 +100,7 @@ protected function query(): Builder
      * @param string|null $data
      * @return array
      */
-    protected function buildPayload($data): array
+    protected function buildPayload($data)
     {
         return [
             'payload' => new Binary($data, Binary::TYPE_OLD_BINARY),

tests/Unit/MongoDbSessionHandlerTest.php

@@ -4,106 +4,106 @@
 
 use ForFit\Session\MongoDbSessionHandler;
 use ForFit\Session\Tests\TestCase;
+use Illuminate\Support\Carbon;
 use MongoDB\BSON\Binary;
-use MongoDB\BSON\UTCDateTime;
 
 class MongoDbSessionHandlerTest extends TestCase
 {
     /**
      * @var MongoDbSessionHandler
      */
     protected $handler;
-    
+
     /**
      * @var string
      */
     protected $sessionId;
-    
+
     /**
      * Set up the test environment.
      */
     protected function setUp(): void
     {
         parent::setUp();
-        
+
         $this->handler = $this->app['session.store']->getHandler();
         $this->sessionId = md5(uniqid('test_session'));
     }
-    
+
     /**
      * Test the open method
      */
     public function test_open_method(): void
     {
         $this->assertTrue($this->handler->open('path', 'name'));
     }
-    
+
     /**
      * Test the close method
      */
     public function test_close_method(): void
     {
         $this->assertTrue($this->handler->close());
     }
-    
+
     /**
      * Test reading non-existent session
      */
     public function test_read_non_existent_session(): void
     {
         $this->assertEquals('', $this->handler->read('non_existent_id'));
     }
-    
+
     /**
      * Test write and read session
      */
     public function test_write_and_read_session(): void
     {
         $data = 'test_data_' . time();
-        
+
         // Write session data
         $this->assertTrue($this->handler->write($this->sessionId, $data));
-        
+
         // Read it back
         $readData = $this->handler->read($this->sessionId);
-        
+
         $this->assertEquals($data, $readData);
-        
+
         // Check database directly
         $session = $this->app['db']->table(config('session.table'))
             ->where('_id', $this->sessionId)
             ->first();
-            
+
         $this->assertNotNull($session);
-        $this->assertInstanceOf(Binary::class, $session['payload']);
-        $this->assertInstanceOf(UTCDateTime::class, $session['expires_at']);
-        $this->assertInstanceOf(UTCDateTime::class, $session['last_activity']);
+        $this->assertInstanceOf(Binary::class, $session->payload);
+        $this->assertInstanceOf(Carbon::class, $session->expires_at);
+        $this->assertInstanceOf(Carbon::class, $session->last_activity);
     }
-    
+
     /**
      * Test destroy session
      */
     public function test_destroy_session(): void
     {
         // First write a session
         $this->handler->write($this->sessionId, 'test_data');
-        
+
         // Verify it exists
         $exists = $this->app['db']->table(config('session.table'))
             ->where('_id', $this->sessionId)
             ->exists();
         $this->assertTrue($exists);
-        
+
         // Now destroy it
         $this->assertTrue($this->handler->destroy($this->sessionId));
-        
+
         // Verify it's gone
         $exists = $this->app['db']->table(config('session.table'))
             ->where('_id', $this->sessionId)
             ->exists();
         $this->assertFalse($exists);
     }
-    
+
     /**
      * Test garbage collection
      */
@@ -112,4 +112,4 @@ public function test_garbage_collection(): void
         // gc should return a truthy value as it's handled by MongoDB TTL index
         $this->assertNotFalse($this->handler->gc(100));
     }
-} 
+}