Merge pull request #2207 from zendesk/grosser/author

do not blow up on missing author url and display missing author nicely
zendesk · Aug 22, 2017 · 1049803 · 1049803
2 parents 384ef67 + 53cc6f6
commit 1049803
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/app/views/changeset/_commits.html.erb b/app/views/changeset/_commits.html.erb
@@ -10,7 +10,10 @@
             <%= image_tag url, width: 20, height: 20 %>
           <% end %>
         </td>
-        <td><%= link_to_if commit.author.present?, commit.author_name, commit.author&.url %></td>
+        <td>
+          <% name = commit.author_name.presence || "Unknown" %>
+          <% url = commit.author&.url %>
+          <%= link_to_if(url, name, url) %></td>
         <td><%= link_to commit.summary, commit.url %></td>
         <td align="right"><code><%= link_to commit.short_sha, commit.url %></code></td>
       </tr>