don't abuse link_to: do it right

diff --git a/app/views/browse/_common.rhtml b/app/views/browse/_common.rhtml
index ec590bda0319381a88987a2fde017593b0c30909..83dadd64172fab9b48877789dd9b1c3e758cf086 100644 (file)
--- a/app/views/browse/_common.rhtml
+++ b/app/views/browse/_common.rhtml
@@ -5,8 +5,8 @@ Deleted
 <% end %>
 <ul>
 <% if obj.visible %>
 <% end %>
 <ul>
 <% if obj.visible %>

-<li><%= link_to h("API"), :controller => "api/#{API_VERSION}/"+type, :id => obj.id %></li>
+<li><%= link_to h("API"), :controller => type, :action => "read", :id => obj.id %></li>

 <% end %>
 <% end %>

-<li><%= link_to h("History"), :controller => "api/#{API_VERSION}/"+type, :id => obj.id, :action => "history" %></li>
+<li><%= link_to h("History"), :controller => "old_" + type, :action => "history", :id => obj.id %></li>

 </ul>
 <%= render :partial => 'common_editedby', :locals => { :obj => obj } %>
 </ul>
 <%= render :partial => 'common_editedby', :locals => { :obj => obj } %>

diff --git a/app/views/browse/_member.rhtml b/app/views/browse/_member.rhtml
index 2b9ee2b31833e49cc436ba9b9fe7c8070dcd4c63..0082bc4db3a6a7b4f8266b6e0f60e347c6b3285b 100644 (file)
--- a/app/views/browse/_member.rhtml
+++ b/app/views/browse/_member.rhtml
@@ -1,5 +1,5 @@
 <tr>
 <tr>

-  <td><%= link_to member.member_id.to_s, :controller => member.member_type, :id => member.member_id %></td>
+  <td><%= link_to member.member_id.to_s, :action => member.member_type+'_view', :id => member.member_id %></td>

   <td><%= h(member.member_type) %></td>
   <td><%= h(member.member_role) %></td>
 </tr> 
   <td><%= h(member.member_type) %></td>
   <td><%= h(member.member_role) %></td>
 </tr>