8000 [9.x] Address confusion between PostgreSQL concepts "schema" and "search_path" by cbj4074 · Pull Request #35463 · laravel/framework · GitHub
[go: up one dir, main page]
Skip to content

[9.x] Address confusion between PostgreSQL concepts "schema" and "search_path" #35463

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Dec 4, 2020
Merged

[9.x] Address confusion between PostgreSQL concepts "schema" and "search_path" #35463

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
20b91b9
Fix terminology in respect to 'schema' vs 'search_path'
poppabear8883 Dec 16, 2017
b8e1aca
Added support for Comma separated search paths with added test
poppabear8883 Dec 16, 2017
54c3fd3
Added support for Postgres Variables in search paths with added test
poppabear8883 Dec 16, 2017
354784f
Fix schema quoting issue and update tests to verify behavior
cbj4074 Dec 2, 2020
380650d
Update method to use new "search_path" config key name
cbj4074 Dec 4, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update method to use new "search_path" config key name
  • Loading branch information
@cbj4074
cbj4074 committed Dec 4, 2020
commit 380650d461952fc5855aa8785a4e59c02b35d786
6 changes: 3 additions & 3 deletions src/Illuminate/Database/Schema/PostgresBuilder.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -164,12 +164,12 @@ protected function parseSchemaAndTable($table)
{
$table = explode('.', $table);

if (is_array($schema = $this->connection->getConfig('schema'))) {
if (in_array($table[0], $schema)) {
if (is_array($searchPath = $this->connection->getConfig('search_path'))) {
if (in_array($table[0], $searchPath)) {
return [array_shift($table), implode('.', $table)];
}

$schema = head($schema);
$schema = head($searchPath);
}

return [$schema ?: 'public', implode('.', $table)];
Expand Down
0