sufia 4.0.0.beta2 → 4.0.0.beta3

data/app/views/catalog/_delete_partials/_default.html.erb

@@ -3,27 +3,24 @@
 <%- stylesheet_links << "../plugin_assets/hydra-head/javascripts/date-picker/css/datepicker" %>
 <%- stylesheet_links << "../plugin_assets/hydra-head/javascripts/fancybox/jquery.fancybox-1.3.1.css" %>
 <%- stylesheet_links << ["../plugin_assets/hydra-head/javascripts/select_to_ui_slider/css/ui.slider.extras.css"] %>
-<%- stylesheet_links << ["slider", {:plugin=>"hydra-head"}] %>
-
-
+<%- stylesheet_links << ["slider", { plugin: "hydra-head" }] %>
 
 <div class="document_viewer" id="cover_container">
 	<div id="delete_dialog">
 		<div id="delete_dialog_container">
-			
-				
-				<div>
+
+			<div>
 	        <span>Permanently delete <%= @document['id'] %> and its assets from the repository?</span><br/>
 	        <div id="deletable_assets"></div>
 	      </div>
-				
+
 					<div>
-					<%= button_to("Delete", {:action => :destroy, :controller => :assets, :id => @document['id'] }, :confirm => 'Are you sure?', :method => :delete )%>
-					 <%= button_to("Withdraw", :action=>"withdraw", :controller => :assets, :id => @document['id'] ) %>
-					 <%= button_to("Cancel", :action=>:edit, :controller => :catalog, :id => @document['id'] ) %>
-					
+					<%= button_to("Delete", { action: :destroy, controller: :assets, id: @document['id'] }, confirm: 'Are you sure?', method: :delete )%>
+					 <%= button_to("Withdraw", action: "withdraw", controller: :assets, id: @document['id'] ) %>
+					 <%= button_to("Cancel", action: :edit, controller: :catalog, id: @document['id'] ) %>
+
 	      </div>
-	      		 
+
 		 </div>
 	</div>
 </div>

data/app/views/catalog/_document.html.erb

@@ -1,18 +1,18 @@
     <% # container for a single doc -%>
     <div class="document <%= render_document_class document %>">
-      
+
       <% # header bar for doc items in index view -%>
       <div class="documentHeader">
-        
+
         <% # bookmark/folder functions for items/docs -%>
         <%= render_index_doc_actions document %>
-        
+
         <% # main title container for doc partial view -%>
         <div class="yui-u">
-          <h3 class="index_title"><%= document_counter + 1 + @response.params[:start].to_i %>. <%= link_to_document document, :label=>document_show_link_field, :counter => (document_counter + 1 + @response.params[:start].to_i) %></h3>
+          <h3 class="index_title"><%= document_counter + 1 + @response.params[:start].to_i %>. <%= link_to_document document, label: document_show_link_field, counter: (document_counter + 1 + @response.params[:start].tol_i) %></h3>
         </div>
       </div>
-      
+
       <% # main container for doc partial view -%>
       <%= render_document_partial document, :index %>
 

data/app/views/catalog/_document_list.html.erb

@@ -1,2 +1,2 @@
-<% # container for all documents in index view -%>
-    <%= render :partial => 'catalog/_index_partials/default_group', :locals => {:docs => @response.docs.map { |x| SolrDocument.new(x) }, :facet_name => nil, :facet_value => nil} %>
+<%# container for all documents in index view -%>
+    <%= render partial: 'catalog/_index_partials/default_group', locals: { docs: @response.docs.map { |x| SolrDocument.new(x) }, facet_name: nil, facet_value: nil } %>

data/app/views/catalog/_edit_partials/_default.html.erb

@@ -5,8 +5,6 @@
 <%- stylesheet_links << ["../plugin_assets/hydra-head/javascripts/select_to_ui_slider/css/ui.slider.extras.css"] %>
 <%- stylesheet_links << ["../plugin_assets/hydra-head/stylesheets/slider.css"] %>
 
-
-
 <% @fedora_object = ActiveFedora::Base.load_instance(params[:id])
    @datastreams = @fedora_object.datastreams
 %>
@@ -34,11 +32,11 @@
     <dt>datastreams:</dt>
         <dd>
           <ul>
-            <% @datastreams.keys.sort!.each do |dsid| %>
-                <li><a href="#<%=dsid%>" class="inline"><%=dsid%></a>
-                    <div style='display:none'>
-                      <div id='<%=dsid%>'>
-                        <h1><%=dsid%></h1>
+            <% @datastreams.keys.sort.each do |dsid| %>
+                <li><a href="#<%= dsid %>" class="inline"><%= dsid %></a>
+                    <div style='display: none'>
+                      <div id='<%= dsid %>'>
+                        <h1><%= dsid %></h1>
                         <dl>
                             <%= ("<dt>label:</dt><dd>" + @datastreams[dsid].dsLabel + "</dd>").html_safe unless @datastreams[dsid].dsLabel.blank? %>
                             <%= ("<dt>last modified:</dt><dd>" + @datastreams[dsid].lastModifiedDate + "</dd>").html_safe unless @datastreams[dsid].lastModifiedDate.blank? %>
@@ -52,7 +50,6 @@
         </dd>
   </dl>
 
-  <%= render :partial => "hydra/permissions/index", :locals => {:document => document, :asset_id=>params[:id]}%>
+  <%= render partial: "hydra/permissions/index", locals: { document: document, asset_id: params[:id] }%>
 
 </div>
-

data/app/views/catalog/_edit_partials/_default_details.html.erb

@@ -1,11 +1,9 @@
-
-
 <%# this should imitate the stuff in catalog/_show_partials/_default_details -%>
-<%= form_tag hydra_update_url_for(:action=>"update", :controller=>"hydra/assets"), :id=>"document_metadata"  "accept-charset"=>"utf-8" do %>
+<%= form_tag hydra_update_url_for(action: "update", controller: "hydra/assets"), id: "document_metadata"  "accept-charset" => "utf-8" do %>
 
   <div id="multipleEdit">
       <dl class="defList fl-theme-mist">
-      <%= editable_metadata_field(@document_fedora, "descMetadata", :title, :label => "Title:") %>
+      <%= editable_metadata_field(@document_fedora, "descMetadata", :title, label: "Title:") %>
       </dl>
     </div>
   </div>

data/app/views/catalog/_email_form.html.erb

@@ -1,5 +1,5 @@
 <h1>Email This</h1>
-<%= form_tag url_for(:controller => "catalog", :action => "email"), :id => 'email_form', :method => :post do %>
+<%= form_tag url_for(controller: "catalog", action: "email"), id: 'email_form', method: :post do %>
   <%= label_tag(:to, "Email:") %><br/>
   <%= text_field_tag :to %><br/>
   <%= label_tag(:message, "Message:") %><br/>

data/app/views/catalog/_facet_limit.html.erb

@@ -1,18 +1,17 @@
 <ul class="facet-values list-unstyled">
-  <% paginator = facet_paginator(facet_field, display_facet)
-  %>
+  <% paginator = facet_paginator(facet_field, display_facet) %>
   <% paginator.items.each do |item| -%>
     <li>
-    <% if facet_in_params?( solr_field, item.value ) %>              
-        <%= render_selected_facet_value(solr_field, item) %>              
-      <% else %>              
-        <%= render_facet_value(solr_field, item) %>              
+    <% if facet_in_params?(solr_field, item.value) %>
+        <%= render_selected_facet_value(solr_field, item) %>
+      <% else %>
+        <%= render_facet_value(solr_field, item) %>
     <% end -%>
     </li>
   <% end %>
 
   <% if paginator.has_next? and params[:action] != "facet" %>
-    <li class="more_facets_link"><%= link_to('more '+ facet_field.label.pluralize + '»', params.merge(:id => solr_field, :action=>"facet", :page => nil), :class => "more_facets_link") %></li>
+    <li class="more_facets_link"><%= link_to('more '+ facet_field.label.pluralize + '»', params.merge(id: solr_field, action: "facet", page: nil), class: "more_facets_link") %></li>
   <% end %>
 
 </ul>

data/app/views/catalog/_facet_selected.html.erb

@@ -2,10 +2,10 @@
 <% if params[:f] %>
   <% params[:f].each do |k,v| %>
     <% v.each do |selected_facet| %>
-      <% link_options = remove_facet_params(k, selected_facet, params).merge!Hash["controller" => "catalog", :action=> "index"] %>
+      <% link_options = remove_facet_params(k, selected_facet, params).merge!({ controller: "catalog", action: "index" }) %>
       <div class="alert alert-warning">
-        <%= link_to 'X', url_for(link_options), {:class=>'close', :'data-dismiss' => "alert" } %>
-        <i class="icon-ok icon-large"></i> <strong><%=  facet_display_value(k, selected_facet) %></strong>
+        <%= link_to 'X', url_for(link_options), {class: 'close', :'data-dismiss' => "alert" } %>
+        <i class="icon-ok icon-large"></i> <strong><%= facet_display_value(k, selected_facet) %></strong>
       </div>
     <% end %>
   <% end %>

data/app/views/catalog/_folder_control.html.erb

@@ -1,12 +1,11 @@
-<%-   
+<%-
   # pass in local :document with a SolrDocument
-  method = (item_in_folder?(document.id) ? "delete" : "put")
-  label = (item_in_folder?(document.id) ? "Unselect" : "Select")
-  cssClass = (item_in_folder?(document.id) ? "deleteFolder" : "addFolder")
+  method = item_in_folder?(document.id) ? "delete" : "put"
+  label = item_in_folder?(document.id) ? "Unselect" : "Select"
+  cssClass = item_in_folder?(document.id) ? "deleteFolder" : "addFolder"
 -%>
 
-<%= form_tag( folder_path(document), :method => method, :class=> "folder_toggle #{cssClass}", "data-doc-id" => document.id, :title=>h(document[document_show_link_field])) do -%>  
-  <%= hidden_field(:bookmark, :title, :value => document[document_show_link_field]) %>
-  <%= submit_tag(label, :class=>"folder_submit", :id => "folder_submit_#{document.id}") %>
+<%= form_tag(folder_path(document), method: method, class: "folder_toggle #{cssClass}", "data-doc-id" => document.id, title: h(document[document_show_link_field])) do -%>
+  <%= hidden_field(:bookmark, :title, value: document[document_show_link_field]) %>
+  <%= submit_tag(label, class: "folder_submit", id: "folder_submit_#{document.id}") %>
 <% end %>
-

data/app/views/catalog/_index_default.html.erb

@@ -3,8 +3,8 @@
 
   <% index_fields.each do |solr_fname, field| -%>
     <% if document.has? solr_fname %>
-	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_label :field => solr_fname %></dt>
-	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_value :document => document, :field => solr_fname %></dd>
+	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_label field: solr_fname %></dt>
+	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_index_field_value document: document, field: solr_fname %></dd>
     <% end -%>
   <% end -%>
 

data/app/views/catalog/_index_header_default.html.erb

@@ -0,0 +1,20 @@
+     
+      <% # header bar for doc items in index view -%>
+      <div class="documentHeader row">
+        
+        <%# main title container for doc partial view 
+            How many bootstrap columns need to be reserved
+            for bookmarks control depends on size. 
+        -%>
+         <h5 class="index_title col-sm-9 col-lg-10">
+         <% counter = document_counter_with_offset(document_counter) %>
+         <span class="document-counter">
+         <%= t('blacklight.search.documents.counter', counter: counter) if counter %>
+         </span>
+         <%= link_to_document document, label: document_show_link_field(document), counter: counter %>
+        </h5>
+
+        <% # bookmark functions for items/docs -%>
+        <%= render_index_doc_actions document, wrapping_class: "index-document-functions col-sm-3 col-lg-2" %>
+      </div>
+      

data/app/views/catalog/_index_partials/_default_group.html.erb

@@ -1,7 +1,7 @@
 <div id="search-results">
   <ol class="catalog" start="<%=1+((@response.current_page-1)*@response.limit_value)%>">
     <% docs.each_with_index do |document,counter| %>
-    <%= render :partial => 'catalog/_index_partials/list_files', :locals => {:document => document, :counter => counter} %>
+    <%= render partial: 'catalog/_index_partials/list_files', locals: {document: document, counter: counter} %>
   <% end %>
   </ol>
 </div>

data/app/views/catalog/_index_partials/_list_files.html.erb

@@ -1,11 +1,14 @@
 <li>
   <h2>
     <%= link_to document.title_or_label, sufia.generic_file_path(document.noid) %>
-    <small>[<%= link_to 'Download', sufia.download_path(document.noid), :target => '_new' %>]</small>
+    <small>[<%= link_to 'Download', sufia.download_path(document.noid), target: '_new' %>]</small>
   </h2>
   <table class="table">
     <tr>
-      <td rowspan="6" class="span20"> <%= render :partial => 'dashboard/_index_partials/thumbnail_display', :locals => {:document=>document, :height=>200, :width=>150} %>
+      <td rowspan="6" class="span20">
+        <%= link_to sufia.generic_file_path(document) do %>
+          <%= render_thumbnail_tag document, { width: 150 } %>
+        <% end %>
       </td>
       <th><h4>Title:</h4></th>
       <td><%= document.title %></td>

data/app/views/catalog/_refworks_form.html.erb

@@ -1,7 +1,7 @@
-<%= form_tag "http://www.refworks.com/express/expressimport.asp?vendor=#{application_name}&filter=MARC%20Format&encoding=65001", :name => "refworks", :target => "_blank" do %>
+<%= form_tag "http://www.refworks.com/express/expressimport.asp?vendor=#{application_name}&filter=MARC%20Format&encoding=65001", name: "refworks", target: "_blank" do %>
 
 <%= hidden_field_tag "ImportData", render_refworks_texts(documents) %>
-<%= submit_tag "Export to Refworks", :class => 'submit'%>
+<%= submit_tag "Export to Refworks", class: 'submit'%>
 <%= link_to_function "Export to Refworks", "document.refworks.submit();" %>
 <% end %>
 

data/app/views/catalog/_results_pagination.html.erb

@@ -1,6 +1,6 @@
 <% if @response.total_pages > 1 %>
     <div class="pager">
-      <%= paginate @response, :outer_window => 2, :theme => 'blacklight', :route_set=> sufia %>
+      <%= paginate @response, outer_window: 2, theme: 'blacklight', route_set: sufia %>
       <div class="clearfix"></div>
     </div><!-- /pager -->
 <% end %>

data/app/views/catalog/_search_form.html.erb

@@ -1,15 +1,30 @@
 <%# only display the q parameter if the controller is not the dashboard.  If they are in the dashboard controller, then the search is of the dashboard. %>
-<% q = params[:controller] == 'dashboard' ? nil : params[:q] %>
-    <%= form_tag catalog_index_path, :method => :get, :id => "search-form-header", :class => "col-xs-12", :role => "search" do %>
-      <%= render_hash_as_hidden_fields(params_for_search().except(:q, :search_field, :qt, :page, :utf8)) %>
-      <div class="input-group">
-        <label class="sr-only" for="search-field-header"><%= t('sufia.search.form.q.label') %></label>
-        <%= text_field_tag :q, q, :class => "q form-control", :id => "search-field-header", :placeholder => t('sufia.search.form.q.placeholder') %>
-        <span class="input-group-btn">
-          <button type="submit" class="btn btn-default" id="search-submit-header">
+    <%= form_tag catalog_index_path, method: :get, id: "search-form-header", class: "col-xs-12", role: "search" do %>
+      <%= render_hash_as_hidden_fields(params_for_search().except(:q, :search_field, :qt, :page, :utf8, :owner)) %>
+      <%= hidden_field_tag 'owner', params[:owner] %>
+
+      <div class="row">
+        <div class="col-lg-10">
+          <div class="input-group">
+            <% q = params[:controller].match(/^dashboard/) ? nil : params[:q] %>
+            <%= text_field_tag :q, q, class: "q form-control", id: "search-field-header", placeholder: t('sufia.search.form.q.placeholder') %>
+            <%# if this label goes before the search field, it'll mess up the rounded corners %>
+            <label class="sr-only" for="search-field-header"><%= t('sufia.search.form.q.label') %></label>
+
+            <div class="input-group-btn">
+              <button type="button" class="btn btn-default dropdown-toggle" data-toggle="dropdown"><span data-search-element="label"><%= t('sufia.search.form.option.all.label_long') %></span> <span class="caret"></span></button>
+              <ul class="dropdown-menu pull-right">
+                <li><a href="#" data-search-option="all" data-search-label="<%= t('sufia.search.form.option.all.label_short') %>"><%= t('sufia.search.form.option.all.label_long') %></a></li>
+                <li><a href="#" data-search-option="mine" data-search-label="<%= t('sufia.search.form.option.mine.label_short') %>"><%= t('sufia.search.form.option.mine.label_long') %></a></li>
+              </ul>
+            </div>
+          </div>
+        </div>
+        <div class="col-lg-2">
+          <button type="submit" class="btn btn-primary" id="search-submit-header">
             <i class="glyphicon glyphicon-search"></i> Go
           </button>
-        </span>
+        </div>
       </div>
     <% end %>
 

data/app/views/catalog/_show_default.html.erb

@@ -2,8 +2,8 @@
 <dl class="defList">
   <% document_show_fields.each do |solr_fname, field| -%>
     <% if document.has? solr_fname %>
-	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_label :field => solr_fname %></dt>
-	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_value :document => document, :field => solr_fname %></dd>
+	    <dt class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_label field: solr_fname %></dt>
+	    <dd class="blacklight-<%= solr_fname.parameterize %>"><%= render_document_show_field_value document: document, field: solr_fname %></dd>
     <% end -%>
   <% end -%>
 </dl>

data/app/views/catalog/_show_partials/_default.html.erb

@@ -1,10 +1,9 @@
 <%# default partial to display solr document fields in catalog show view -%>
 
-
 <div class="document_viewer" id="cover_container">
   <dl class="defList">
     <dt>ID: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"id") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "id") %></dd>
     <dt>Download:</dt>
     <dd id="downloads">
 	  <ul>
@@ -12,11 +11,11 @@
       </ul>
     </dd>
     <dt>Copyright: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"rights_t") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "rights_t") %></dd>
   </dl>
 </div>
-<div class="<%= params[:action]%>_details">
-  <%= render :partial => "#{params[:controller]}/_#{params[:action]}_partials/default_details", :locals => {:document => document}%>
+<div class="<%= params[:action] %>_details">
+  <%= render partial: "#{params[:controller]}/_#{params[:action]}_partials/default_details", locals: { document: document } %>
 </div>
 
 <%= async_load_tag(hydra_asset_downloads_path(params[:id]), 'downloads') %>

data/app/views/catalog/_show_partials/_default_details.html.erb

@@ -2,14 +2,14 @@
 <div>
   <dl class="defList">
     <dt>Date: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"date_t") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "date_t") %></dd>
     <dt>Title: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"title_t") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "title_t") %></dd>
     <dt>Document Type: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"medium_t") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "medium_t") %></dd>
     <dt>Location: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"location_t") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "location_t") %></dd>
     <dt>Date: </dt>
-    <dd><%= render_document_show_field_value(:document=>document, :field=>"date_t") %></dd>
+    <dd><%= render_document_show_field_value(document: document, field: "date_t") %></dd>
   </dl>
 </div>

data/app/views/catalog/_show_partials/_facets.html.erb

@@ -1,11 +1,11 @@
 <% facets_display_heading ||=  "" %>
-<% # main container for facets/limits menu -%>
+<%# main container for facets/limits menu -%>
 <div id="facets">
-  <h2><%=h facets_display_heading %></h2>
+  <h2><%= h(facets_display_heading) %></h2>
   <% facet_field_names.each do |solr_fname| %>
     <div>
       <% display_facet = @response.facets.detect {|f| f.name == solr_fname} -%>
-      <%#- raise solr_fname.inspect unless display_facet -%>
+      <%# raise solr_fname.inspect unless display_facet -%>
       <% if display_facet && display_facet.items.length > 0 %>
         <h3 class="facet_selected"><%= facet_field_labels[solr_fname] -%></h3>
         <ul>
@@ -16,16 +16,16 @@
               <% if facet_in_params? solr_fname, item.value %>
                 <li>
                   <span class="selected"><%= h(item.value) %> (<%= format_num facet_count_for_value %>)</span>
-                  <% link_options = remove_facet_params(solr_fname, item.value, params).merge!Hash["controller" => "catalog", :action=> "index"] %>
-                  [<%= link_to 'remove', url_for(link_options), :class=>'remove' %>]
+                  <% link_options = remove_facet_params(solr_fname, item.value, params).merge!({ controller: "catalog", action: "index" }) %>
+                  [<%= link_to 'remove', url_for(link_options), class: 'remove' %>]
                 </li>
                 <% else %>
-				<%- if item_counter > Blacklight.config[:facet_more_num]-1 -%>
+				<%- if item_counter > Blacklight.config[:facet_more_num] - 1 -%>
                   <li class="more">
 				<%- else -%>
 				  <li>
 				<%- end -%>
-                  <% link_options = add_facet_params(solr_fname, item.value).merge!Hash["controller" => "catalog", :action=> "index"] %>
+                  <% link_options = add_facet_params(solr_fname, item.value).merge!({ controller: "catalog", action: "index" }) %>
                   <%= link_to h(item.value), url_for(link_options) %> (<%= format_num facet_count_for_value %>)
                 </li>
               <% end -%>

data/app/views/catalog/_show_tools.html.erb

@@ -10,36 +10,36 @@
   
     <% if (@document.respond_to?(:export_as_mla_citation_txt) || @document.respond_to?(:export_as_apa_citation_txt)) %>
       <li class="cite">
-      <%= link_to "Cite This", citation_catalog_path(:id => @document), {:id => 'citeLink', :name => 'citation', :class => 'lightboxLink'} %>
+      <%= link_to "Cite This", citation_catalog_path(id: @document), {id: 'citeLink', name: 'citation', class: 'lightboxLink'} %>
       </li>
     <% end %>
     
     <% if @document.export_formats.keys.include?( :refworks_marc_txt ) %>
       <li class="refworks">
-      <%= link_to "Export to RefWorks", refworks_export_url(:id => @document) %>        
+      <%= link_to "Export to RefWorks", refworks_export_url(id: @document) %>        
       </li>
     <% end %>
     
     <% if @document.export_formats.keys.include?( :endnote ) %>
       <li class="endnote">
-        <%= link_to "Export to EndNote", catalog_path(@document, :format => 'endnote') %>
+        <%= link_to "Export to EndNote", catalog_path(@document, format: 'endnote') %>
       </li>
     <% end %>
     
     <% if @document.respond_to?( :to_email_text ) %>
       <li class="email">
-      <%= link_to "Email This", email_catalog_path(:id => @document), {:id => 'emailLink', :name => 'email', :class => 'lightboxLink'} %>
+      <%= link_to "Email This", email_catalog_path(id: @document), {id: 'emailLink', name: 'email', class: 'lightboxLink'} %>
       </li>
     <%- end -%>
     <%- if @document.respond_to?( :to_sms_text ) -%>
       <li class="sms">
-      <%= link_to "SMS This", sms_catalog_path(:id => @document), {:id => 'smsLink', :name => 'sms', :class => 'lightboxLink'} %>
+      <%= link_to "SMS This", sms_catalog_path(id: @document), {id: 'smsLink', name: 'sms', class: 'lightboxLink'} %>
       </li>
     <%- end -%>
     
     <% if @document.respond_to?(:to_marc) %>
       <li class="librarian_view">
-        <%= link_to "Librarian View", librarian_view_catalog_path(@document), {:id => 'librarianLink', :name => 'librarian_view', :class => 'lightboxLink'} %>
+        <%= link_to "Librarian View", librarian_view_catalog_path(@document), {id: 'librarianLink', name: 'librarian_view', class: 'lightboxLink'} %>
       </li>
     <% end %>