Skip to content

Commit

Permalink
[webui] Don't provide a 'view' link for files inside tarballs in diff
Browse files Browse the repository at this point in the history
view.

At least until we support 'viewing' such files.
  • Loading branch information
saschpe committed Oct 25, 2011
1 parent 930b6dd commit 6f2b0e6
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/webui/app/views/request/show.html.erb
Expand Up @@ -322,7 +322,10 @@
<% end %>
</td>
<td><%= file_element.value('state').capitalize %></td>
<td><%= link_to_unless(is_binary(filename), filename, :controller => 'package', :action => 'view_file', :project => diff[:action].source.project, :package => diff[:action].source.package, :file => filename, :rev => diff[:action].source.rev) %></td>
<td>
<%# TODO: drop this check for '/' once we can 'view' files inside tarballs: %>
<%= link_to_unless(is_binary(filename) || filename.include?('/'), filename, :controller => 'package', :action => 'view_file', :project => diff[:action].source.project, :package => diff[:action].source.package, :file => filename, :rev => diff[:action].source.rev) %>
</td>
</tr>
<% if file_diff_text.length > 0 %>
<% if is_expanded %>
Expand Down

0 comments on commit 6f2b0e6

Please sign in to comment.