diff options
| author | Rémy Coutable <remy@rymai.me> | 2016-05-04 14:12:51 +0000 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2016-05-04 14:12:51 +0000 |
| commit | b9d9a4d213e2f957f334e9a8d2edbfd325e3d899 (patch) | |
| tree | da086f6e4cbd23fb14503ff31da93696cbac35c4 | |
| parent | 466f6874af6d05f58bad35a81c08a9a199e98d52 (diff) | |
| parent | 41df76b028aa663711b8126b2aabd7f36f254153 (diff) | |
| download | gitlab-ce-b9d9a4d213e2f957f334e9a8d2edbfd325e3d899.tar.gz | |
Merge branch 'remove-unnecessary-method-call-in-events-view' into 'master'
Remove unnecessary method call in events view
Since `User#to_param` already returns `User#username`, we don't need to
pass in the user's username. Changing this also helps us obey LoD.
See merge request !3996
| -rw-r--r-- | app/views/events/_event.html.haml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/views/events/_event.html.haml b/app/views/events/_event.html.haml index 5d622582088..e4629bae0e6 100644 --- a/app/views/events/_event.html.haml +++ b/app/views/events/_event.html.haml @@ -5,7 +5,7 @@ = cache [event, current_application_settings, "v2.2"] do - if event.author - = link_to user_path(event.author.username) do + = link_to user_path(event.author) do = image_tag avatar_icon(event.author_email, 40), class: "avatar s40", alt:'' - else = image_tag avatar_icon(event.author_email, 40), class: "avatar s40", alt:'' |