[HOTFIX] - Include bound parameters for joins within subqueries (#29)

* Include bound subquery join values

* Use latest actions/cache package

* Updated remaining actions/cache reference

Author: joshmcrae

.github/workflows/tests.yml

@@ -56,7 +56,7 @@ jobs:
 
       - name: Cache Composer packages
         id: composer-cache
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: vendor
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
@@ -119,7 +119,7 @@ jobs:
 
       - name: Cache Composer packages
         id: composer-cache
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: vendor
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}

lib/PicoDb/Table.php

@@ -646,8 +646,7 @@ public function joinSubquery(Table $subQuery, string $alias, string $foreign_col
 
         $this->joinValues = array_merge(
             $this->joinValues,
-            $subQuery->getConditionBuilder()->getValues(),
-            $subQuery->getAggregatedConditionBuilder()->getValues()
+            $subQuery->getValues()
         );
 
         return $this;
@@ -675,8 +674,7 @@ public function innerJoinSubquery(Table $subQuery, string $alias, string $foreig
 
         $this->joinValues = array_merge(
             $this->joinValues,
-            $subQuery->getConditionBuilder()->getValues(),
-            $subQuery->getAggregatedConditionBuilder()->getValues()
+            $subQuery->getValues()
         );
 
         return $this;