Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
李少辉-开发者
gitlab-foss
提交
378fe076
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,发现更多精彩内容 >>
提交
378fe076
编写于
10月 11, 2012
作者:
R
Robb Kidd
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Reduce complexity: replace case statement with method lookup.
上级
baf94bd7
变更
2
显示空白变更内容
内联
并排
Showing
2 changed file
with
6 addition
and
11 deletion
+6
-11
app/observers/note_observer.rb
app/observers/note_observer.rb
+5
-10
spec/observers/note_observer_spec.rb
spec/observers/note_observer_spec.rb
+1
-1
未找到文件。
app/observers/note_observer.rb
浏览文件 @
378fe076
...
...
@@ -16,16 +16,11 @@ class NoteObserver < ActiveRecord::Observer
protected
def
notify_team_of_new_note
(
note
)
notify_method
=
'note_'
+
note
.
noteable_type
.
underscore
+
'_email'
if
Notify
.
respond_to?
notify_method
team_without_note_author
(
note
).
map
do
|
u
|
case
note
.
noteable_type
when
"Commit"
;
Notify
.
note_commit_email
(
u
.
id
,
note
.
id
).
deliver
when
"Issue"
;
Notify
.
note_issue_email
(
u
.
id
,
note
.
id
).
deliver
when
"Wiki"
;
Notify
.
note_wiki_email
(
u
.
id
,
note
.
id
).
deliver
when
"MergeRequest"
;
Notify
.
note_merge_request_email
(
u
.
id
,
note
.
id
).
deliver
when
"Wall"
;
Notify
.
note_wall_email
(
u
.
id
,
note
.
id
).
deliver
when
"Snippet"
;
true
# no notifications for snippets?
else
true
Notify
.
send
(
notify_method
.
to_sym
,
u
.
id
,
note
.
id
).
deliver
end
end
end
...
...
spec/observers/note_observer_spec.rb
浏览文件 @
378fe076
...
...
@@ -90,7 +90,7 @@ describe NoteObserver do
it
'does nothing for a new note on a snippet'
do
note
.
stub
(
:noteable_type
).
and_return
(
'Snippet'
)
subject
.
send
(
:notify_team_of_new_note
,
note
).
should
==
[
true
,
true
]
subject
.
send
(
:notify_team_of_new_note
,
note
).
should
be_nil
end
end
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录