Merge pull request #2816 from Arne1303/fix-has-one-through

[9.x] Fix HasOneThrough (added tests)

Author: yajra

src/EloquentDataTable.php

@@ -189,16 +189,18 @@ protected function joinEagerLoadedColumn($relation, $relationColumn)
 
                 case $model instanceof HasOneThrough:
                     $pivot    = explode('.', $model->getQualifiedParentKeyName())[0]; // extract pivot table from key
-                    $pivotPK  = $pivot . '.' . $model->getLocalKeyName();
+                    $pivotPK  = $pivot . '.' . $model->getFirstKeyName();
                     $pivotFK  = $model->getQualifiedLocalKeyName();
                     $this->performJoin($pivot, $pivotPK, $pivotFK);
 
                     $related = $model->getRelated();
                     $table   = $related->getTable();
-                    $tablePK = $related->getForeignKey();
+                    $tablePK = $model->getSecondLocalKeyName();
                     $foreign = $pivot . '.' . $tablePK;
                     $other   = $related->getQualifiedKeyName();
 
+                    $lastQuery->addSelect($lastQuery->getModel()->getTable().'.*');
+
                     break;
 
                 case $model instanceof HasOneOrMany:

tests/Integration/HasOneThroughTest.php

@@ -26,6 +26,31 @@ public function it_returns_all_records_with_the_relation_when_called_without_par
         $this->assertCount(60, $response->json()['data']);
     }
 
+    /** @test */
+    public function it_can_search_has_one_through_relation()
+    {
+        $response = $this->call('GET', '/relations/hasOneThroughSearchRelation', [
+            'columns' => [
+                [
+                    'data'       => 'heart.size',
+                    'searchable' => true,
+                    'search'     => [
+                        'value' => 'heart-1',
+                    ],
+                ],
+            ],
+        ]);
+
+        $response->assertJson([
+            'draw'            => 0,
+            'recordsTotal'    => 60,
+            'recordsFiltered' => 33,
+        ]);
+
+        $this->assertArrayHasKey('heart', $response->json()['data'][0]);
+        $this->assertCount(33, $response->json()['data']);
+    }
+
     /** @test */
     public function it_returns_all_records_with_the_deleted_relation_when_called_with_withtrashed_parameter()
     {
@@ -101,6 +126,10 @@ protected function setUp(): void
             return $datatables->eloquent(Post::with('heart')->select('posts.*'))->toJson();
         });
 
+        $this->app['router']->get('/relations/hasOneThroughSearchRelation', function (DataTables $datatables) {
+            return $datatables->eloquent(Post::with('heart'))->addColumns(['hearts.size'])->toJson();
+        });
+
         $this->app['router']->get('/relations/hasOneThroughWithTrashed', function (DataTables $datatables) {
             return $datatables->eloquent(Post::with(['heart' => function ($query) {
                 $query->withTrashed();