Skip to content

Commit 924b6b5

Browse files
committed
Changing the subquery builder for Oracle
Signed-off-by: Andrey Pyzhikov <[email protected]>
1 parent c93b967 commit 924b6b5

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

system/Database/BaseBuilder.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2846,7 +2846,7 @@ protected function buildSubquery($builder, bool $wrapped = false, string $alias
28462846
$alias = trim($alias);
28472847

28482848
if ($alias !== '') {
2849-
$subquery .= ' AS ' . ($this->db->protectIdentifiers ? $this->db->escapeIdentifiers($alias) : $alias);
2849+
$subquery .= ' ' . ($this->db->protectIdentifiers ? $this->db->escapeIdentifiers($alias) : $alias);
28502850
}
28512851
}
28522852

tests/system/Database/Builder/FromTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,19 +103,19 @@ public function testFromReset()
103103

104104
public function testFromSubquery()
105105
{
106-
$expectedSQL = 'SELECT * FROM (SELECT * FROM "users") AS "alias"';
106+
$expectedSQL = 'SELECT * FROM (SELECT * FROM "users") "alias"';
107107
$subquery = new BaseBuilder('users', $this->db);
108108
$builder = $this->db->newQuery()->fromSubquery($subquery, 'alias');
109109

110110
$this->assertSame($expectedSQL, str_replace("\n", ' ', $builder->getCompiledSelect()));
111111

112-
$expectedSQL = 'SELECT * FROM (SELECT "id", "name" FROM "users") AS "users_1"';
112+
$expectedSQL = 'SELECT * FROM (SELECT "id", "name" FROM "users") "users_1"';
113113
$subquery = (new BaseBuilder('users', $this->db))->select('id, name');
114114
$builder = $this->db->newQuery()->fromSubquery($subquery, 'users_1');
115115

116116
$this->assertSame($expectedSQL, str_replace("\n", ' ', $builder->getCompiledSelect()));
117117

118-
$expectedSQL = 'SELECT * FROM (SELECT * FROM "users") AS "alias", "some_table"';
118+
$expectedSQL = 'SELECT * FROM (SELECT * FROM "users") "alias", "some_table"';
119119
$subquery = new BaseBuilder('users', $this->db);
120120
$builder = $this->db->newQuery()->fromSubquery($subquery, 'alias')->from('some_table');
121121

@@ -145,7 +145,7 @@ public function testFromSubqueryWithSQLSRV()
145145

146146
$builder->fromSubquery($subquery, 'users_1');
147147

148-
$expectedSQL = 'SELECT * FROM "test"."dbo"."jobs", (SELECT * FROM "test"."dbo"."users") AS "users_1"';
148+
$expectedSQL = 'SELECT * FROM "test"."dbo"."jobs", (SELECT * FROM "test"."dbo"."users") "users_1"';
149149

150150
$this->assertSame($expectedSQL, str_replace("\n", ' ', $builder->getCompiledSelect()));
151151
}

tests/system/Database/Builder/SelectTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ public function testSelectSubquery()
270270
$subquery->select('name')->where('id', 1);
271271
$builder->select('name')->selectSubquery($subquery, 'country');
272272

273-
$expected = 'SELECT "name", (SELECT "name" FROM "countries" WHERE "id" = 1) AS "country" FROM "users"';
273+
$expected = 'SELECT "name", (SELECT "name" FROM "countries" WHERE "id" = 1) "country" FROM "users"';
274274

275275
$this->assertSame($expected, str_replace("\n", ' ', $builder->getCompiledSelect()));
276276
}

0 commit comments

Comments
 (0)