Remove property passthru

Author: stevebauman

src/Query/Model/Builder.php

@@ -4,6 +4,7 @@
 
 use Closure;
 use DateTime;
+use Exception;
 use LdapRecord\Models\Attributes\Guid;
 use LdapRecord\Models\Collection;
 use LdapRecord\Models\Model;
@@ -109,15 +110,6 @@ public function __call(string $method, array $parameters): mixed
         return $this;
     }
 
-    /**
-     * Dynamically access properties on the underlying query builder.
-     */
-    public function __get(string $name): mixed
-    {
-        // TODO: Fix all property passthru.
-        return $this->query->{$name};
-    }
-
     /**
      * Force a clone of the underlying query builder when cloning.
      */

tests/Unit/Models/ActiveDirectory/UserTest.php

@@ -142,15 +142,15 @@ public function test_reject_computer_object_class_is_a_default_scope()
 
     public function test_scope_where_has_mailbox_is_applied()
     {
-        $filters = User::whereHasMailbox()->filters;
+        $filters = User::whereHasMailbox()->getQuery()->filters;
 
         $this->assertEquals('msExchMailboxGuid', $filters['and'][4]['attribute']);
         $this->assertEquals('*', $filters['and'][4]['operator']);
     }
 
     public function test_scope_where_has_lockout_is_applied()
     {
-        $filters = User::whereHasLockout()->filters;
+        $filters = User::whereHasLockout()->getQuery()->filters;
 
         $this->assertEquals('lockoutTime', $filters['and'][4]['attribute']);
         $this->assertEquals('>=', $filters['and'][4]['operator']);

tests/Unit/Query/Model/ActiveDirectoryBuilderTest.php

@@ -42,7 +42,7 @@ public function test_where_member_of_substitutes_base_dn()
         $b = $this->newBuilder();
         $b->setBaseDn('dc=org,dc=acme');
         $b->whereMemberOf('cn=Accounting,{base}');
-        $where = $b->filters['and'][0];
+        $where = $b->getQuery()->filters['and'][0];
         $this->assertEquals('memberof', $where['attribute']);
         $this->assertEquals('=', $where['operator']);
         $this->assertEquals(
@@ -74,7 +74,7 @@ public function test_where_member_of_nested_substitutes_base_dn()
         $b = $this->newBuilder();
         $b->setBaseDn('dc=org,dc=acme');
         $b->whereMemberOf('cn=Accounting,{base}', nested: true);
-        $where = $b->filters['and'][0];
+        $where = $b->getQuery()->filters['and'][0];
         $this->assertEquals('memberof:1.2.840.113556.1.4.1941:', $where['attribute']);
         $this->assertEquals('=', $where['operator']);
         $this->assertEquals(
@@ -111,7 +111,7 @@ public function test_or_where_member_of_substitutes_base_dn()
         $b->setBaseDn('dc=org,dc=acme');
         $b->orWhereEquals('cn', 'John Doe');
         $b->orWhereMemberOf('cn=Accounting,{base}');
-        $where = $b->filters['or'][1];
+        $where = $b->getQuery()->filters['or'][1];
         $this->assertEquals('memberof', $where['attribute']);
         $this->assertEquals('=', $where['operator']);
         $this->assertEquals(