Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
77ca887d
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,发现更多精彩内容 >>
未验证
提交
77ca887d
编写于
12月 29, 2019
作者:
R
Rafael França
提交者:
Rafael Mendonça França
12月 29, 2019
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Merge pull request #38117 from nicolasdular/nicolasdular/fix-frozen-strings-in-to-sentence
Return unfrozen strings from to_sentence
上级
6d654cc6
变更
3
显示空白变更内容
内联
并排
Showing
3 changed file
with
23 addition
and
4 deletion
+23
-4
activesupport/CHANGELOG.md
activesupport/CHANGELOG.md
+12
-0
activesupport/lib/active_support/core_ext/array/conversions.rb
...esupport/lib/active_support/core_ext/array/conversions.rb
+4
-4
activesupport/test/core_ext/array/conversions_test.rb
activesupport/test/core_ext/array/conversions_test.rb
+7
-0
未找到文件。
activesupport/CHANGELOG.md
浏览文件 @
77ca887d
*
`Array#to_sentence`
no longer returns a frozen string.
Before:
['one', 'two'].to_sentence.frozen?
# => true
After:
['one', 'two'].to_sentence.frozen?
# => false
*Nicolas Dular*
*
Update
`ActiveSupport::Messages::Metadata#fresh?`
to work for cookies with expiry set when
`ActiveSupport.parse_json_times = true`
.
...
...
activesupport/lib/active_support/core_ext/array/conversions.rb
浏览文件 @
77ca887d
...
...
@@ -74,13 +74,13 @@ def to_sentence(options = {})
case
length
when
0
""
+
""
when
1
"
#{
self
[
0
]
}
"
+
"
#{
self
[
0
]
}
"
when
2
"
#{
self
[
0
]
}#{
options
[
:two_words_connector
]
}#{
self
[
1
]
}
"
+
"
#{
self
[
0
]
}#{
options
[
:two_words_connector
]
}#{
self
[
1
]
}
"
else
"
#{
self
[
0
...-
1
].
join
(
options
[
:words_connector
])
}#{
options
[
:last_word_connector
]
}#{
self
[
-
1
]
}
"
+
"
#{
self
[
0
...-
1
].
join
(
options
[
:words_connector
])
}#{
options
[
:last_word_connector
]
}#{
self
[
-
1
]
}
"
end
end
...
...
activesupport/test/core_ext/array/conversions_test.rb
浏览文件 @
77ca887d
...
...
@@ -68,6 +68,13 @@ def test_always_returns_string
assert_instance_of
String
,
[
ActiveSupport
::
SafeBuffer
.
new
(
"one"
),
"two"
].
to_sentence
assert_instance_of
String
,
[
ActiveSupport
::
SafeBuffer
.
new
(
"one"
),
"two"
,
"three"
].
to_sentence
end
def
test_returns_no_frozen_string
assert_not
[].
to_sentence
.
frozen?
assert_not
[
"one"
].
to_sentence
.
frozen?
assert_not
[
"one"
,
"two"
].
to_sentence
.
frozen?
assert_not
[
"one"
,
"two"
,
"three"
].
to_sentence
.
frozen?
end
end
class
ToSTest
<
ActiveSupport
::
TestCase
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录