Bind expressions contain their data; even when added to a new query as an expression

Author: LaravelFreelancerNL

src/Expressions/BindExpression.php

@@ -4,9 +4,42 @@
 
 namespace LaravelFreelancerNL\FluentAQL\Expressions;
 
+use LaravelFreelancerNL\FluentAQL\QueryBuilder;
+
 /**
  * AQL literal expression.
  */
 class BindExpression extends LiteralExpression implements ExpressionInterface
 {
+    protected string $bindVariable;
+
+    protected mixed $data = null;
+
+    public function __construct(string $bindVariable, mixed $data = null)
+    {
+        $this->bindVariable = $bindVariable;
+
+        $this->data = $data;
+    }
+
+    /**
+     * Compile expression output.
+     *
+     * @param QueryBuilder $queryBuilder
+     * @return string
+     */
+    public function compile(QueryBuilder $queryBuilder): string
+    {
+        return $this->bindVariable;
+    }
+
+    public function getBindVariable(): string
+    {
+        return $this->bindVariable;
+    }
+
+    public function getData(): mixed
+    {
+        return $this->data;
+    }
 }

src/Expressions/LiteralExpression.php

@@ -17,7 +17,7 @@ class LiteralExpression extends Expression implements ExpressionInterface
      * @param  QueryBuilder  $queryBuilder
      * @return string
      */
-    public function compile(QueryBuilder $queryBuilder = null): string
+    public function compile(QueryBuilder $queryBuilder): string
     {
         return (string) $this->expression;
     }

src/QueryBuilder.php

@@ -203,7 +203,7 @@ public function bind(
 
         $to = $this->grammar->formatBind($to, false);
 
-        return new BindExpression($to);
+        return new BindExpression($to, $data);
     }
 
     /**

src/Traits/NormalizesExpressions.php

@@ -18,7 +18,6 @@
 
 trait NormalizesExpressions
 {
-
     /**
      * @param object|array<mixed>|string|int|float|bool|null $data
      */
@@ -36,6 +35,8 @@ public function normalizeArgument(
         array|string $allowedExpressionTypes = null
     ): Expression {
         if ($argument instanceof Expression) {
+            $argument = $this->processBindExpression($argument);
+
             return $argument;
         }
 
@@ -266,4 +267,16 @@ protected function normalizeObject(
 
         return new ObjectExpression($this->normalizeIterable((array) $argument, $allowedExpressionTypes));
     }
+
+    public function processBindExpression(Expression $argument): Expression
+    {
+        if ($argument instanceof BindExpression) {
+            $bindKey = ltrim($argument->getBindVariable(), '@');
+
+            if (!isset($this->binds[$bindKey])) {
+                $this->binds[$bindKey] = $argument->getData();
+            }
+        }
+        return $argument;
+    }
 }

tests/Unit/Expressions/BindExpressionTest.php

@@ -0,0 +1,40 @@
+<?php
+
+namespace Tests\Unit\Expressions;
+
+use LaravelFreelancerNL\FluentAQL\Expressions\BindExpression;
+use LaravelFreelancerNL\FluentAQL\QueryBuilder;
+use LaravelFreelancerNL\FluentAQL\Tests\TestCase;
+
+/**
+ * Class QueryExpressionTest
+ *
+ * @covers \LaravelFreelancerNL\FluentAQL\Expressions\NullExpression
+ */
+class BindExpressionTest extends TestCase
+{
+    public function testBindExpression()
+    {
+        $bindVar = '@myBind';
+        $data = 'myData';
+        $qb = new QueryBuilder();
+        $expression = new BindExpression($bindVar, $data);
+        $result = $expression->compile($qb);
+
+        self::assertEquals($bindVar, $result);
+        self::assertEquals($data, $expression->getData());
+    }
+
+    public function testBindExpressionIsAddedToNewQuery()
+    {
+        $bindVar = '@myBind';
+        $data = 'test';
+        $bindExpression = new BindExpression($bindVar, $data);
+
+        $qb = (new QueryBuilder())->filter('test', '==', $bindExpression);
+        $qb->get();
+
+        $this->assertCount(1, $qb->binds);
+        $this->assertSame($data, array_shift($qb->binds));
+    }
+}

tests/Unit/SubqueryTest.php

@@ -44,6 +44,7 @@ public function testSubQueryWithBinds()
             ->filter('u.active', '==', 'something to bind')
             ->return('u._key')
             ->get();
+
         self::assertEquals(
             'FOR u IN users FILTER u.active == @' . $subQuery->getQueryId() . '_1 RETURN u._key',
             $subQuery->query