Skip to content

Commit 14d85a4

Browse files
authored
Removal of redundant SoftDelete check
1 parent 0b6f9c0 commit 14d85a4

File tree

1 file changed

+2
-19
lines changed

1 file changed

+2
-19
lines changed

src/EloquentDataTable.php

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ protected function isNotEagerLoaded($relation)
138138
protected function joinEagerLoadedColumn($relation, $relationColumn)
139139
{
140140
$table = '';
141-
$deletedAt = false;
142141
$lastQuery = $this->query;
143142
foreach (explode('.', $relation) as $eachRelation) {
144143
$model = $lastQuery->getRelation($eachRelation);
@@ -164,45 +163,33 @@ protected function joinEagerLoadedColumn($relation, $relationColumn)
164163
$table = $model->getRelated()->getTable();
165164
$foreign = $model->getQualifiedForeignKeyName();
166165
$other = $model->getQualifiedParentKeyName();
167-
$deletedAt = $this->checkSoftDeletesOnModel($model->getRelated());
168166
break;
169167

170168
case $model instanceof BelongsTo:
171169
$table = $model->getRelated()->getTable();
172170
$foreign = $model->getQualifiedForeignKey();
173171
$other = $model->getQualifiedOwnerKeyName();
174-
$deletedAt = $this->checkSoftDeletesOnModel($model->getRelated());
175172
break;
176173

177174
default:
178175
throw new Exception('Relation ' . get_class($model) . ' is not yet supported.');
179176
}
180-
$this->performJoin($table, $foreign, $other, $deletedAt);
177+
$this->performJoin($table, $foreign, $other);
181178
$lastQuery = $model->getQuery();
182179
}
183180

184181
return $table . '.' . $relationColumn;
185182
}
186183

187-
protected function checkSoftDeletesOnModel($model)
188-
{
189-
if (in_array('Illuminate\Database\Eloquent\SoftDeletes', class_uses($model))) {
190-
return $model->getQualifiedDeletedAtColumn();
191-
}
192-
193-
return false;
194-
}
195-
196184
/**
197185
* Perform join query.
198186
*
199187
* @param string $table
200188
* @param string $foreign
201189
* @param string $other
202-
* @param string $deletedAt
203190
* @param string $type
204191
*/
205-
protected function performJoin($table, $foreign, $other, $deletedAt = false, $type = 'left')
192+
protected function performJoin($table, $foreign, $other, $type = 'left')
206193
{
207194
$joins = [];
208195
foreach ((array) $this->getBaseQueryBuilder()->joins as $key => $join) {
@@ -212,9 +199,5 @@ protected function performJoin($table, $foreign, $other, $deletedAt = false, $ty
212199
if (! in_array($table, $joins)) {
213200
$this->getBaseQueryBuilder()->join($table, $foreign, '=', $other, $type);
214201
}
215-
216-
if ($deletedAt !== false) {
217-
$this->getBaseQueryBuilder()->whereNull($deletedAt);
218-
}
219202
}
220203
}

0 commit comments

Comments
 (0)