configures document views for new document type schema

Author: madoleary

app/views/documents/_form.html.erb

@@ -1,11 +1,18 @@
 <%= simple_form_for [document] do |f| %>
+
   <%= f.association :service, collection: services, hint: "If your service isn't on the list, please contact the team" %>
-  
-  <%= f.input :name, collection: document_names, hint: 'Name of the document', selected: 'Privacy Policy'%>
+
+  <% if document.name.present? %>
+    <label class="sr-only" for="inputHelpBlock">Name</label>
+    <input type="text" id="inputHelpBlock" class="form-control" aria-describedby="helpBlock" placeholder="<%= document.name%>" readonly>
+    <span id="helpBlock" class="help-block">To change the document name, please select a new document type.</span>
+  <% end %>
+
+  <%= f.association :document_type, collection: document_names, hint: "Inspect the document types and their descriptions #{link_to 'here', document_types_path}".html_safe %>
 
   <%= f.input :url, hint: "Where we should fetch this document", placeholder: "e.g. \"https://www.facebook.com/about/privacy\"" %>
 
-  <%= f.input :xpath, input_html: { value: (f.object.xpath.present?) ? f.object.xpath : '//body' }, placeholder: "e.g. \"//*[@id='content']//div[@class='_5tkp _3m9']\"", hint: "The location of the terms on the page (Using <a href=\"https://en.wikipedia.org/wiki/XPath\" title=\"Wikipedia explanation of XPath\" target=\"_blank\">XPath</a>.)".html_safe %>
+  <%= f.input :xpath, input_html: { value: (f.object.xpath.present?) ? f.object.xpath : '//body' }, placeholder: "e.g. \"//*[@id='content']//div[@class='_5tkp _3m9']\"", hint: "The location of the terms on the page using <a href=\"https://en.wikipedia.org/wiki/XPath\" title=\"Wikipedia explanation of XPath\" target=\"_blank\">XPath</a>".html_safe %>
 
   <%= f.input :crawler_server, collection: crawlers, label_method: :second, value_method: :first, hint: "<a href=\"https://to.tosdr.org/U98u1\">Select from which country the crawler should be used, or a specific crawler. Useful if blocked by EU. (Optional)</a>".html_safe, selected: document.crawler_server || "eu.crawler.api.tosdr.org" %>
 

app/views/documents/_table_documents.html.erb

@@ -13,7 +13,7 @@
       <% @documents.each do |d| %>
         <tr class="toSort">
           <td><%= link_to d.service.name, service_path(d.service), title: "View the service connected to this document" %></td>
-          <td><%= link_to truncate(d.name, length: 30), document_path(d), title: "Inspect this document" %></td>
+          <td><%= link_to truncate(d.document_type ? d.document_type.name : d.name, length: 30), document_path(d), title: "Inspect this document" %></td>
           <td><%= truncate(d.url, length: 30) %></td>
           <td></td>
         </tr>

app/views/documents/edit.html.erb

@@ -1,4 +1,4 @@
 <% content_for :title do %>
-  <title>Editing Document <%= @document.service.name %> <%= @document.name %> (ToS;DR Phoenix)</title>
+  <title>Editing <%= @document.service.name %> <%= @document.document_type ? @document.document_type.name : @document.name %></title>
 <% end %>
 <%= render 'form', document: @document, document_names: @document_names, crawlers: @crawlers, services: @services %>

app/views/documents/new.html.erb

@@ -1,5 +1,5 @@
 <% content_for :title do %>
-  <title>New Document (ToS;DR Phoenix)</title>
+  <title>New document</title>
 <% end %>
 <div class="container">
   <div>

app/views/documents/show.html.erb

@@ -1,13 +1,13 @@
 <% content_for :title do %>
-  <title>Document <%= @document.service.name %> <%= @document.name %> (ToS;DR Phoenix)</title>
+  <title><%= @document.service.name %> <%= @name %></title>
 <% end %>
 
 <div class="card-inline">
   <div class="card-inline-item card-inline-service">
     <div class="row">
       <div class="col-lg-6">
         <h1><%= link_to @document.service.name, annotate_service_path(@document.service) %></h1>
-        <h5><%= @document.name %></h5>
+        <h5><%= @name %></h5>
       </div>
     </div>