Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
081eec4b
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,发现更多精彩内容 >>
提交
081eec4b
编写于
6月 21, 2014
作者:
M
Matthew Draper
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #15764 from arthurnn/dependencies_name_error
Make dependencies.rb add a name to NameError
上级
c2d5b310
bd3fde09
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
11 addition
and
5 deletion
+11
-5
activesupport/CHANGELOG.md
activesupport/CHANGELOG.md
+3
-0
activesupport/lib/active_support/dependencies.rb
activesupport/lib/active_support/dependencies.rb
+4
-4
activesupport/test/dependencies_test.rb
activesupport/test/dependencies_test.rb
+4
-1
未找到文件。
activesupport/CHANGELOG.md
浏览文件 @
081eec4b
*
Make Dependencies pass a name to NameError error.
*arthurnn*
*
Fixed
`ActiveSupport::Cache::FileStore`
exploding with long paths.
*Adam Panzer / Michael Grosser*
...
...
activesupport/lib/active_support/dependencies.rb
浏览文件 @
081eec4b
...
...
@@ -187,7 +187,7 @@ def const_missing(const_name)
# top-level constant.
def
guess_for_anonymous
(
const_name
)
if
Object
.
const_defined?
(
const_name
)
raise
NameError
,
"
#{
const_name
}
cannot be autoloaded from an anonymous class or module"
raise
NameError
.
new
"
#{
const_name
}
cannot be autoloaded from an anonymous class or module"
,
const_name
.
to_s
else
Object
end
...
...
@@ -516,9 +516,9 @@ def load_missing_constant(from_mod, const_name)
end
end
raise
NameError
,
"uninitialized constant
#{
qualified_name
}
"
,
caller
.
reject
{
|
l
|
l
.
starts_with?
__FILE__
}
name_error
=
NameError
.
new
(
"uninitialized constant
#{
qualified_name
}
"
,
qualified_name
)
name_error
.
set_backtrace
(
caller
.
reject
{
|
l
|
l
.
starts_with?
__FILE__
})
raise
name_error
end
# Remove the constants that have been autoloaded, and those that have been
...
...
activesupport/test/dependencies_test.rb
浏览文件 @
081eec4b
...
...
@@ -367,9 +367,11 @@ def test_non_existing_const_raises_name_error_with_fully_qualified_name
with_autoloading_fixtures
do
e
=
assert_raise
(
NameError
)
{
A
::
DoesNotExist
.
nil?
}
assert_equal
"uninitialized constant A::DoesNotExist"
,
e
.
message
assert_equal
"A::DoesNotExist"
,
e
.
name
e
=
assert_raise
(
NameError
)
{
A
::
B
::
DoesNotExist
.
nil?
}
assert_equal
"uninitialized constant A::B::DoesNotExist"
,
e
.
message
assert_equal
"A::B::DoesNotExist"
,
e
.
name
end
end
...
...
@@ -537,6 +539,7 @@ def test_const_missing_in_anonymous_modules_raises_if_the_constant_belongs_to_Ob
mod
=
Module
.
new
e
=
assert_raise
(
NameError
)
{
mod
::
E
}
assert_equal
'E cannot be autoloaded from an anonymous class or module'
,
e
.
message
assert_equal
'E'
,
e
.
name
end
end
...
...
@@ -954,7 +957,7 @@ def test_access_unloaded_constants_for_reload
assert_kind_of
Class
,
A
::
B
# Necessary to load A::B for the test
ActiveSupport
::
Dependencies
.
mark_for_unload
(
A
::
B
)
ActiveSupport
::
Dependencies
.
remove_unloadable_constants!
A
::
B
# Make sure no circular dependency error
end
end
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录