Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
李少辉-开发者
gitlab-foss
提交
84c14ac0
G
gitlab-foss
项目概览
李少辉-开发者
/
gitlab-foss
通知
15
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
G
gitlab-foss
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
84c14ac0
编写于
2月 14, 2018
作者:
K
Kamil Trzciński
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Improve validation message and add changelog
上级
48db60e4
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
7 addition
and
2 deletion
+7
-2
app/validators/variable_duplicates_validator.rb
app/validators/variable_duplicates_validator.rb
+1
-1
changelogs/unreleased/fix-validation-of-environment-scope-for-variables.yml
...sed/fix-validation-of-environment-scope-for-variables.yml
+5
-0
spec/support/features/variable_list_shared_examples.rb
spec/support/features/variable_list_shared_examples.rb
+1
-1
未找到文件。
app/validators/variable_duplicates_validator.rb
浏览文件 @
84c14ac0
...
...
@@ -20,7 +20,7 @@ class VariableDuplicatesValidator < ActiveModel::EachValidator
def
validate_duplicates
(
record
,
attribute
,
values
)
duplicates
=
values
.
reject
(
&
:marked_for_destruction?
).
group_by
(
&
:key
).
select
{
|
_
,
v
|
v
.
many?
}.
map
(
&
:first
)
if
duplicates
.
any?
error_message
=
"ha
s duplicate variabl
es (
#{
duplicates
.
join
(
", "
)
}
)"
error_message
=
"ha
ve duplicate valu
es (
#{
duplicates
.
join
(
", "
)
}
)"
error_message
+=
" for
#{
values
.
first
.
send
(
options
[
:scope
])
}
scope"
if
options
[
:scope
]
# rubocop:disable GitlabSecurity/PublicSend
record
.
errors
.
add
(
attribute
,
error_message
)
end
...
...
changelogs/unreleased/fix-validation-of-environment-scope-for-variables.yml
0 → 100644
浏览文件 @
84c14ac0
---
title
:
Fix validation of environment scope of variables
merge_request
:
author
:
type
:
fixed
spec/support/features/variable_list_shared_examples.rb
浏览文件 @
84c14ac0
...
...
@@ -263,7 +263,7 @@ shared_examples 'variable list' do
# We check the first row because it re-sorts to alphabetical order on refresh
page
.
within
(
'.js-ci-variable-list-section'
)
do
expect
(
find
(
'.js-ci-variable-error-box'
)).
to
have_content
(
/Validation failed Variables ha
s duplicate variabl
es \(.+\)/
)
expect
(
find
(
'.js-ci-variable-error-box'
)).
to
have_content
(
/Validation failed Variables ha
ve duplicate valu
es \(.+\)/
)
end
end
end
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录