invert if statement order

app/views/notify/pipeline_failed_email.html.haml

@@ -121,12 +121,12 @@
                                           %td{style: "font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:15px;line-height:1.4;vertical-align:middle;padding-right:5px;"}
                                             %img.avatar{height: "24", src: avatar_icon(commit.author || commit.author_email, 24), style: "display:block;border-radius:12px;margin:-2px 0;", width: "24"}/
                                           %td{style: "font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:15px;line-height:1.4;vertical-align:middle;"}
-                                            - if commit.author.nil?
-                                              %span
-                                                = commit.author_name
-                                            - else
+                                            - if commit.author
                                               %a.muted{href: user_url(commit.author), style: "color:#333333;text-decoration:none;"}
                                                 = commit.author.name
+                                            - else
+                                              %span
+                                                = commit.author_name
                         %tr.spacer
                           %td{style: "font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;height:18px;font-size:18px;line-height:18px;"}
                              

app/views/notify/pipeline_failed_email.text.erb

@@ -9,10 +9,10 @@ Merge Request: <%= @merge_request.to_reference %> ( <%= merge_request_url(@merge
 Commit: <%= @pipeline.short_sha %> ( <%= namespace_project_commit_url(@project.namespace, @project, @pipeline.sha) %> )
 Commit Message: <%= @pipeline.git_commit_message.truncate(50) %>
 <% commit = @pipeline.commit -%>
-<% if commit.author.nil? -%>
-Commit Author: <%= commit.author_name %>
-<% else -%>
+<% if commit.author -%>
 Commit Author: <%= commit.author.name %> ( <%= user_url(commit.author) %> )
+<% else -%>
+Commit Author: <%= commit.author_name %>
 <% end -%>
 <% failed = @pipeline.statuses.latest.failed -%>

app/views/notify/pipeline_success_email.html.haml

@@ -121,12 +121,12 @@
                                           %td{style: "font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:15px;line-height:1.4;vertical-align:middle;padding-right:5px;"}
                                             %img.avatar{height: "24", src: avatar_icon(commit.author || commit.author_email, 24), style: "display:block;border-radius:12px;margin:-2px 0;", width: "24"}/
                                           %td{style: "font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:15px;line-height:1.4;vertical-align:middle;"}
-                                            - if commit.author.nil?
-                                              %span
-                                                = commit.author_name
-                                            - else
+                                            - if commit.author
                                               %a.muted{href: user_url(commit.author), style: "color:#333333;text-decoration:none;"}
                                                 = commit.author.name
+                                            - else
+                                              %span
+                                                = commit.author_name
                         %tr.spacer
                           %td{style: "font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;height:18px;font-size:18px;line-height:18px;"}
                              

app/views/notify/pipeline_success_email.text.erb

@@ -9,10 +9,10 @@ Merge Request: <%= @merge_request.to_reference %> ( <%= merge_request_url(@merge
 Commit: <%= @pipeline.short_sha %> ( <%= namespace_project_commit_url(@project.namespace, @project, @pipeline.sha) %> )
 Commit Message: <%= @pipeline.git_commit_message.truncate(50) %>
 <% commit = @pipeline.commit -%>
-<% if commit.author.nil? -%>
-Commit Author: <%= commit.author_name %>
-<% else -%>
+<% if commit.author -%>
 Commit Author: <%= commit.author.name %> ( <%= user_url(commit.author) %> )
+<% else -%>
+Commit Author: <%= commit.author_name %>
 <% end -%>
 <% build_count = @pipeline.statuses.latest.size -%>