Cleanup and tests

CHANGELOG.md

@@ -3,9 +3,23 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+- Add classes to cast `ObjectId` and `UUID` instances [#1](https://github.com/GromNaN/laravel-mongodb-private/pull/1) by [@alcaeus](https://github.com/alcaeus).
+- Add `Query\Builder::toMql()` to simplify comprehensive query tests [#6](https://github.com/GromNaN/laravel-mongodb-private/pull/6) by [@GromNaN](https://github.com/GromNaN).
+- Fix `Query\Builder::whereNot` to use MongoDB [`$not`](https://www.mongodb.com/docs/manual/reference/operator/query/not/) operator [#13](https://github.com/GromNaN/laravel-mongodb-private/pull/13) by [@GromNaN](https://github.com/GromNaN).
+- Fix `Query\Builder::whereBetween` to accept `Carbon\Period` object [#10](https://github.com/GromNaN/laravel-mongodb-private/pull/10) by [@GromNaN](https://github.com/GromNaN).
+- Throw an exception for unsupported `Query\Builder` methods [#9](https://github.com/GromNaN/laravel-mongodb-private/pull/9) by [@GromNaN](https://github.com/GromNaN).
+- Throw an exception when `Query\Builder::orderBy()` is used with invalid direction [#7](https://github.com/GromNaN/laravel-mongodb-private/pull/7) by [@GromNaN](https://github.com/GromNaN).
+- Throw an exception when `Query\Builder::push()` is used incorrectly [#5](https://github.com/GromNaN/laravel-mongodb-private/pull/5) by [@GromNaN](https://github.com/GromNaN).
+- Remove public property `Query\Builder::$paginating` [#15](https://github.com/GromNaN/laravel-mongodb-private/pull/15) by [@GromNaN](https://github.com/GromNaN).
+- Remove call to deprecated `Collection::count` for `countDocuments` [#18](https://github.com/GromNaN/laravel-mongodb-private/pull/18) by [@GromNaN](https://github.com/GromNaN).
+- Accept operators prefixed by `$` in `Query\Builder::orWhere` [#20](https://github.com/GromNaN/laravel-mongodb-private/pull/20) by [@GromNaN](https://github.com/GromNaN).
+- Remove `Query\Builder::whereAll($column, $values)`. Use `Query\Builder::where($column, 'all', $values)` instead. [#16](https://github.com/GromNaN/laravel-mongodb-private/pull/16) by [@GromNaN](https://github.com/GromNaN).
+- Fix validation of unique values when the validated value is found as part of an existing value. [#21](https://github.com/GromNaN/laravel-mongodb-private/pull/21) by [@GromNaN](https://github.com/GromNaN).
+- Support `%` and `_` in `like` expression [#17](https://github.com/GromNaN/laravel-mongodb-private/pull/17) by [@GromNaN](https://github.com/GromNaN).
+
 ## [3.9.2] - 2022-09-01
 
-### Addded 
+### Added
 - Add single word name mutators [#2438](https://github.com/jenssegers/laravel-mongodb/pull/2438) by [@RosemaryOrchard](https://github.com/RosemaryOrchard) & [@mrneatly](https://github.com/mrneatly).
 
 ### Fixed

README.md

@@ -332,6 +332,12 @@ $users =
         ->get();
 ```
 
+**NOT statements**
+
+```php
+$users = User::whereNot('age', '>', 18)->get();
+```
+
 **whereIn**
 
 ```php
@@ -477,6 +483,17 @@ Car::where('weight', 300)
 
 ### MongoDB-specific operators
 
+In addition to the Laravel Eloquent operators, all available MongoDB query operators can be used with `where`:
+
+```php
+User::where($fieldName, $operator, $value)->get();
+```
+
+It generates the following MongoDB filter:
+```ts
+{ $fieldName: { $operator: $value } }
+```
+
 **Exists**
 
 Matches documents that have the specified field.

composer.json

@@ -19,6 +19,7 @@
     ],
     "license": "MIT",
     "require": {
+        "ext-mongodb": "^1.15",
         "illuminate/support": "^10.0",
         "illuminate/container": "^10.0",
         "illuminate/database": "^10.0",
@@ -36,11 +37,9 @@
         }
     },
     "autoload-dev": {
-        "classmap": [
-            "tests/TestCase.php",
-            "tests/models",
-            "tests/seeds"
-        ]
+        "psr-4": {
+            "Jenssegers\\Mongodb\\Tests\\": "tests/"
+        }
     },
     "suggest": {
         "jenssegers/mongodb-session": "Add MongoDB session support to Laravel-MongoDB",

src/Auth/DatabaseTokenRepository.php

@@ -18,7 +18,7 @@ protected function getPayload($email, $token)
         return [
             'email' => $email,
             'token' => $this->hasher->make($token),
-            'created_at' => new UTCDateTime(Date::now()->format('Uv')),
+            'created_at' => new UTCDateTime(Date::now()),
         ];
     }
 

src/Collection.php

@@ -6,6 +6,9 @@
 use MongoDB\BSON\ObjectID;
 use MongoDB\Collection as MongoCollection;
 
+/**
+ * @mixin MongoCollection
+ */
 class Collection
 {
     /**

src/Connection.php

@@ -12,6 +12,9 @@
 use MongoDB\Database;
 use Throwable;
 
+/**
+ * @mixin Database
+ */
 class Connection extends BaseConnection
 {
     use ManagesTransactions;

src/Eloquent/Casts/BinaryUuid.php

@@ -0,0 +1,63 @@
+<?php
+
+namespace Jenssegers\Mongodb\Eloquent\Casts;
+
+use function bin2hex;
+use function hex2bin;
+use Illuminate\Contracts\Database\Eloquent\CastsAttributes;
+use Jenssegers\Mongodb\Eloquent\Model;
+use MongoDB\BSON\Binary;
+use function str_replace;
+use function substr;
+
+class BinaryUuid implements CastsAttributes
+{
+    /**
+     * Cast the given value.
+     *
+     * @param  Model  $model
+     * @param  string  $key
+     * @param  mixed  $value
+     * @param  array  $attributes
+     * @return mixed
+     */
+    public function get($model, string $key, $value, array $attributes)
+    {
+        if (! $value instanceof Binary || $value->getType() !== Binary::TYPE_UUID) {
+            return $value;
+        }
+
+        $base16Uuid = bin2hex($value->getData());
+
+        return sprintf(
+            '%s-%s-%s-%s-%s',
+            substr($base16Uuid, 0, 8),
+            substr($base16Uuid, 8, 4),
+            substr($base16Uuid, 12, 4),
+            substr($base16Uuid, 16, 4),
+            substr($base16Uuid, 20, 12),
+        );
+    }
+
+    /**
+     * Prepare the given value for storage.
+     *
+     * @param  Model  $model
+     * @param  string  $key
+     * @param  mixed  $value
+     * @param  array  $attributes
+     * @return Binary
+     */
+    public function set($model, string $key, $value, array $attributes)
+    {
+        if ($value instanceof Binary) {
+            return $value;
+        }
+
+        if (is_string($value) && strlen($value) === 16) {
+            return new Binary($value, Binary::TYPE_UUID);
+        }
+
+        return new Binary(hex2bin(str_replace('-', '', $value)), Binary::TYPE_UUID);
+    }
+}

src/Eloquent/Casts/ObjectId.php

@@ -0,0 +1,46 @@
+<?php
+
+namespace Jenssegers\Mongodb\Eloquent\Casts;
+
+use Illuminate\Contracts\Database\Eloquent\CastsAttributes;
+use Jenssegers\Mongodb\Eloquent\Model;
+use MongoDB\BSON\ObjectId as BSONObjectId;
+
+class ObjectId implements CastsAttributes
+{
+    /**
+     * Cast the given value.
+     *
+     * @param  Model  $model
+     * @param  string  $key
+     * @param  mixed  $value
+     * @param  array  $attributes
+     * @return mixed
+     */
+    public function get($model, string $key, $value, array $attributes)
+    {
+        if (! $value instanceof BSONObjectId) {
+            return $value;
+        }
+
+        return (string) $value;
+    }
+
+    /**
+     * Prepare the given value for storage.
+     *
+     * @param  Model  $model
+     * @param  string  $key
+     * @param  mixed  $value
+     * @param  array  $attributes
+     * @return mixed
+     */
+    public function set($model, string $key, $value, array $attributes)
+    {
+        if ($value instanceof BSONObjectId) {
+            return $value;
+        }
+
+        return new BSONObjectId($value);
+    }
+}

src/Eloquent/EmbedsRelations.php

@@ -6,6 +6,9 @@
 use Jenssegers\Mongodb\Relations\EmbedsMany;
 use Jenssegers\Mongodb\Relations\EmbedsOne;
 
+/**
+ * Embeds relations for MongoDB models.
+ */
 trait EmbedsRelations
 {
     /**
@@ -23,9 +26,7 @@ protected function embedsMany($related, $localKey = null, $foreignKey = null, $r
         // the calling method's name and use that as the relationship name as most
         // of the time this will be what we desire to use for the relationships.
         if ($relation === null) {
-            [, $caller] = debug_backtrace(false);
-
-            $relation = $caller['function'];
+            $relation = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 2)[1]['function'];
         }
 
         if ($localKey === null) {
@@ -58,9 +59,7 @@ protected function embedsOne($related, $localKey = null, $foreignKey = null, $re
         // the calling method's name and use that as the relationship name as most
         // of the time this will be what we desire to use for the relationships.
         if ($relation === null) {
-            [, $caller] = debug_backtrace(false);
-
-            $relation = $caller['function'];
+            $relation = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 2)[1]['function'];
         }
 
         if ($localKey === null) {

src/Eloquent/HybridRelations.php

@@ -12,6 +12,10 @@
 use Jenssegers\Mongodb\Relations\MorphMany;
 use Jenssegers\Mongodb\Relations\MorphTo;
 
+/**
+ * Cross-database relationships between SQL and MongoDB.
+ * Use this trait in SQL models to define relationships with MongoDB models.
+ */
 trait HybridRelations
 {
     /**
@@ -134,9 +138,7 @@ public function belongsTo($related, $foreignKey = null, $otherKey = null, $relat
         // the calling method's name and use that as the relationship name as most
         // of the time this will be what we desire to use for the relationships.
         if ($relation === null) {
-            [$current, $caller] = debug_backtrace(false, 2);
-
-            $relation = $caller['function'];
+            $relation = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 2)[1]['function'];
         }
 
         // Check if it is a relation with an original model.
@@ -178,9 +180,7 @@ public function morphTo($name = null, $type = null, $id = null, $ownerKey = null
         // since that is most likely the name of the polymorphic interface. We can
         // use that to get both the class and foreign key that will be utilized.
         if ($name === null) {
-            [$current, $caller] = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 2);
-
-            $name = $caller['function'];
+            $name = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 2)[1]['function'];
         }
 
         [$type, $id] = $this->getMorphs(Str::snake($name), $type, $id);

src/Eloquent/Model.php

@@ -134,7 +134,7 @@ public function getDateFormat()
      */
     public function freshTimestamp()
     {
-        return new UTCDateTime(Date::now()->format('Uv'));
+        return new UTCDateTime(Date::now());
     }
 
     /**