tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

Backward compatible behaviour for SQLConditionalExpression::getJoins

Browse Source
This commit is contained in:
Serge Latyntcev 2019-05-02 14:08:23 +12:00
parent 523456ae09
commit 3d777cfb8a
2 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/ORM/Queries/SQLConditionalExpression.php
View File

@ -287,6 +287,13 @@ abstract class SQLConditionalExpression extends SQLExpression
continue; continue;
} }
if (preg_match('/AS\s+(?:"[^"]+"|[A-Z0-9_]+)\s*$/i', $join)) {
// custom aliases override the ones defined through array keys
// this is only meant to keep backward compatibility with SS <= 4.3,
// to be removed in SS5
continue;
}
$trimmedAlias = trim($alias, '"'); $trimmedAlias = trim($alias, '"');
if ($trimmedAlias !== trim($join, '"')) { if ($trimmedAlias !== trim($join, '"')) {

16
tests/php/ORM/SQLSelectTest.php
View File

@ -841,10 +841,22 @@ class SQLSelectTest extends SapphireTest
$sql = $query->sql(); $sql = $query->sql();
$this->assertSQLEquals( $this->assertSQLEquals(
'SELECT * 'SELECT *
FROM "MyTable" AS "MyTableAlias" FROM "MyTable" AS "MyTableAlias"
LEFT JOIN "OtherTable" AS "OtherTableAlias" ON "Thing" = "OtherThing"', LEFT JOIN "OtherTable" AS "OtherTableAlias" ON "Thing" = "OtherThing"',
$sql $sql
); );
$query = SQLSelect::create('*', [
'MyTableAlias' => '"MyTable"',
'ignoredAlias' => ', (SELECT * FROM "MyTable" where "something" = "whatever") as "CrossJoin"'
]);
$sql = $query->sql();
$this->assertSQLEquals(
'SELECT * FROM "MyTable" AS "MyTableAlias" , '.
'(SELECT * FROM "MyTable" where "something" = "whatever") as "CrossJoin"',
$sql
);
} }
} }