Merge pull request 'use the correct table alias in joins 'on clause'' (#17) from bugfix/fix-multiple-level-join-table-alias into master
All checks were successful
soko-web/pipeline/head This commit looks good
All checks were successful
soko-web/pipeline/head This commit looks good
Reviewed-on: #17
This commit is contained in:
commit
585d469b69
@ -150,7 +150,7 @@ class PersistentDataManager implements IPersistentDataManager
|
||||
|
||||
$columns = array_merge($columns, $this->getRelationColumns($table, $relations, $withRelations));
|
||||
|
||||
$this->leftJoinRelations($select, $table, $relations, $withRelations);
|
||||
$this->leftJoinRelations($select, $table, $table, $relations, $withRelations);
|
||||
$select->columns($columns);
|
||||
} else {
|
||||
$select->columns($columns);
|
||||
@ -180,19 +180,19 @@ class PersistentDataManager implements IPersistentDataManager
|
||||
return $columns;
|
||||
}
|
||||
|
||||
private function leftJoinRelations(Select $select, string $table, array $relations, array $withRelations): void
|
||||
private function leftJoinRelations(Select $select, string $table, string $tableAlias, array $relations, array $withRelations): void
|
||||
{
|
||||
foreach ($relations as $relation => $relationType) {
|
||||
$relationTableAlias = $table . '__' . $relation;
|
||||
$relationTable = call_user_func([$relationType, 'getTable']);
|
||||
$select->setTableAliases([$relationTableAlias => $relationTable]);
|
||||
$select->leftJoin($relationTableAlias, [$relationTableAlias, 'id'], '=', [$table, $relation . '_id']);
|
||||
$select->leftJoin($relationTableAlias, [$relationTableAlias, 'id'], '=', [$tableAlias, $relation . '_id']);
|
||||
|
||||
$relationsOfRelation = call_user_func([$relationType, 'getRelations']);
|
||||
if (count($withRelations)) {
|
||||
$relationsOfRelation = array_intersect_key($relationsOfRelation, array_flip($withRelations));
|
||||
}
|
||||
$this->leftJoinRelations($select, $relationTable, $relationsOfRelation, $withRelations);
|
||||
$this->leftJoinRelations($select, $relationTable, $relationTableAlias, $relationsOfRelation, $withRelations);
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user