<% service_response = holding.service_response %>
<% if current_user %>
  <%- # Link for "Request" action with class "ajax_window" for unobtrusive JS. -%>
  <%= link_to("Request", new_holding_request_url(service_response), 
    target: "_blank", class: "btn btn-primary request-link ajax_window") %>
<% else %>
  <%- # Link to login screen -%>
  <strong>
    <%= link_to("Login to Request",
      login_path({return_url: url_for_request(service_response.request)}),
      target: "_blank", class: "request-login-link") %>
  </strong>
<% end %>

<% service_response = holding.service_response %>
<% if current_user %>
  <%- # Link for "Request" action with class "ajax_window" for unobtrusive JS. -%>
  <%= link_to("Request", new_holding_request_url(service_response), 
    target: "_blank", class: "btn btn-primary request-link ajax_window") %>
<% else %>
  <%- # Link to login screen -%>
  <strong>
    <%= link_to("Login to Request",
      login_url({return_url: url_for_request(service_response.request)}),
      target: "_blank", class: "request-login-link") %>
  </strong>
<% end %>

Use the login_url named url method since the login_path link doesn't seem to work
Use the login_url named url method since the login_path link doesn't seem to work
