Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
ff42b219
R
rails
项目概览
张重言
/
rails
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
R
rails
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
未验证
提交
ff42b219
编写于
7月 08, 2019
作者:
W
Will Jessop
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
When DATABASE_URL is specified don't trample envs that use a url: key
fixes #36610
上级
7729dfc6
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
24 addition
and
5 deletion
+24
-5
activerecord/lib/active_record/database_configurations.rb
activerecord/lib/active_record/database_configurations.rb
+5
-5
activerecord/test/cases/connection_adapters/merge_and_resolve_default_url_config_test.rb
...ion_adapters/merge_and_resolve_default_url_config_test.rb
+19
-0
未找到文件。
activerecord/lib/active_record/database_configurations.rb
浏览文件 @
ff42b219
...
...
@@ -153,11 +153,11 @@ def build_db_config_from_hash(env_name, spec_name, config)
def
build_url_config
(
url
,
configs
)
env
=
ActiveRecord
::
ConnectionHandling
::
DEFAULT_ENV
.
call
.
to_s
if
original_config
=
configs
.
find
(
&
:for_current_env?
)
if
original_config
.
url_config?
configs
else
configs
.
map
do
|
config
|
if
configs
.
find
(
&
:for_current_env?
)
configs
.
map
do
|
config
|
if
config
.
url_config?
config
else
ActiveRecord
::
DatabaseConfigurations
::
UrlConfig
.
new
(
config
.
env_name
,
config
.
spec_name
,
url
,
config
.
config
)
end
end
...
...
activerecord/test/cases/connection_adapters/merge_and_resolve_default_url_config_test.rb
浏览文件 @
ff42b219
...
...
@@ -244,6 +244,25 @@ def test_url_sub_key_with_database_url
assert_equal
expected
,
actual
end
def
test_no_url_sub_key_with_database_url_doesnt_trample_other_envs
ENV
[
"DATABASE_URL"
]
=
"postgres://localhost/baz"
config
=
{
"default_env"
=>
{
"database"
=>
"foo"
},
"other_env"
=>
{
"url"
=>
"postgres://foohost/bardb"
}
}
actual
=
resolve_config
(
config
)
expected
=
{
"default_env"
=>
{
"database"
=>
"baz"
,
"adapter"
=>
"postgresql"
,
"host"
=>
"localhost"
},
"other_env"
=>
{
"adapter"
=>
"postgresql"
,
"database"
=>
"bardb"
,
"host"
=>
"foohost"
}
}
assert_equal
expected
,
actual
end
def
test_merge_no_conflicts_with_database_url
ENV
[
"DATABASE_URL"
]
=
"postgres://localhost/foo"
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录