works_cited 0.1.14 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa5b9e63b43e8210682d2664ea515311f6e752fe7d2149b46dfeea134db70427
-  data.tar.gz: 81b60e1bcb0fbddb8fd370dc1878998cedb7d2d8f22e3f1bd8a7c73cc681a940
+  metadata.gz: 5db67f2a964c0433ad04e7e3fba13605d8bd38cf6d743a56f3eebf71e08a016d
+  data.tar.gz: f61b70f80049f28cbb897e2e4c9b1d107237838460266a71ae0c54b318d80cc5
 SHA512:
-  metadata.gz: '09189d70a8ab33d4d956ce4372495df050768fc96f844a9cdfd7ba0422bba2b710e96d08366fc899aef1f010cc0154209951eb3179835c3ceb62d44293e352d3'
-  data.tar.gz: 39f009806bde960422e5e192e3a36f6b40d34a616eea0f919f3cdc174fc4fa9ec26d5adc5731d5561fdeb2c3010372effcbaced13af46fb2f7da5f5618355fe9
+  metadata.gz: bab946554a8189670c7a8b5de7b15cd208ddc0df3c0d7539ad8200324cb2091e5b44452a3b3380b692e7a720f142cd4cec4b3c1a41b6118bb7630469f3be4d4f
+  data.tar.gz: 6989928293676bce8baf3332ad34c837394cb51e0b096e2997b980d8a54e51801536e259b355548f1a8a22fdc94338bdc9b22c7ee19212a47bca5f91127edd71

data/README.md

@@ -3,8 +3,6 @@ Works Cited allows you to add a list of the works cited in ActiveRecord objects,
 
 Works Cited uses CanCanCan to authorize the editing of citations. This makes it easy for you to control access.
 
-Works Cited is compatible with, but does not require, Rails Admin.
-
 ## Installation
 Add this line to your application's Gemfile:
 
@@ -85,6 +83,33 @@ Add the helpers to the relevant views
 = works_cited_list @record
 ```
 
+To add routes so that you can edit citations on their own pages:
+
+```ruby
+  mount WorksCited::Engine => '/works_cited'
+```
+
+To add the fields, nested inside your forms
+
+```haml
+= form_for(@doodad) do |f|
+  = works_cited_citations_fields f
+```
+
+Don't forget to add the controller concern to enable nested attributes with strong parameters
+
+```ruby
+class RecipesController < ApplicationController
+  include Cookbook::Params
+
+  #...
+    
+  def recipe_params
+    params.require(:doodad).permit(:name, :description, works_cited_params)
+  end
+end
+```
+
 ## Contributing
 * Check out the latest master to make sure the feature hasn't been implemented or the bug hasn't been fixed yet.
 * Check out the issue tracker to make sure someone already hasn't requested it and/or contributed it.

data/VERSION

@@ -1 +1 @@
-0.1.14
+0.1.15

data/app/assets/javascripts/works_cited/addFields.js

@@ -0,0 +1,44 @@
+class addFields {
+  // This executes when the function is instantiated.
+  constructor() {
+    this.links = document.querySelectorAll('.add_fields')
+    this.iterateLinks()
+    const event = new Event('works_cited:load')
+    window.dispatchEvent(event);
+  }
+
+  iterateLinks() {
+    // If there are no links on the page, stop the function from executing.
+    if (this.links.length === 0) return
+    // Loop over each link on the page. A page could have multiple nested forms.
+    this.links.forEach(link => {
+      link.addEventListener('click', e => {
+        this.handleClick(link, e)
+      })
+    })
+  }
+
+  handleClick(link, e) {
+    // Stop the function from executing if a link or event were not passed into the function.
+    if (!link || !e) return
+    // Prevent the browser from following the URL.
+    e.preventDefault()
+    // Save a unique timestamp to ensure the key of the associated array is unique.
+    let time = new Date().getTime()
+    // Save the data id attribute into a variable. This corresponds to `new_object.object_id`.
+    let linkId = link.dataset.id
+    // Create a new regular expression needed to find any instance of the `new_object.object_id` used in the fields data attribute if there's a value in `linkId`.
+    let regexp = linkId ? new RegExp(linkId, 'g') : null
+    // Replace all instances of the `new_object.object_id` with `time`, and save markup into a variable if there's a value in `regexp`.
+    let newFields = regexp ? link.dataset.fields.replace(regexp, time) : null
+    // Add the new markup to the form if there are fields to add.
+    newFields ? link.insertAdjacentHTML('beforebegin', newFields) : null
+    // Tell it new fields are there
+    const event = new Event('works_cited:add_fields')
+    window.dispatchEvent(event);
+  }
+}
+
+// Wait for turbolinks to load, otherwise `document.querySelectorAll()` won't work
+window.addEventListener('load', () => new addFields())
+window.addEventListener('works_cited:add_fields', () => new addFields())

data/app/assets/javascripts/works_cited/application.js

@@ -1 +1,6 @@
-//= require vanilla_nested
+//= require works_cited/shared.js
+//= require works_cited/addFields.js
+//= require works_cited/removeFields.js
+//= require works_cited/showFields.js
+//= require works_cited/loadPreview.js
+//= require works_cited/submitForm.js

data/app/assets/javascripts/works_cited/loadPreview.js

@@ -0,0 +1,45 @@
+function startPreview(url, id, citationLocator) {
+  var element = document.getElementById(id)
+  var fields = element.querySelectorAll('input, select');
+  var index = 0;
+
+  for (index = 0; index < fields.length; ++index) {
+    var textField = fields[index];
+    textField.addEventListener('change', (e) => loadPreview(url, citationLocator))
+  }
+  loadPreview(url, citationLocator)
+}
+
+function loadPreview(url, locator) {
+  // Build formData object.
+  var formData = new FormData(document.querySelector('form'))
+  var data = {}
+  data.citation = getFieldByLocator(formData, locator)
+  var authenticity_token = document.head.querySelector('meta[name="csrf-token"]').content
+  var preview = document.getElementById('preview')
+  var container = document.getElementById('preview-html')
+
+  fetch(url + '.json',
+    {
+      method: 'post',
+      headers: {
+        'X-Requested-With': 'XMLHttpRequest',
+        'X-CSRF-Token': authenticity_token,
+        'Content-type': 'application/json'
+      },
+      body: JSON.stringify(data),
+      credentials: 'same-origin'
+    })
+    .then(function(response) {
+      // When the page is loaded convert it to json
+      return response.json()
+    })
+    .then(function(json) {
+      // get the html value for it
+      return json.html
+    })
+    .then((html) => {
+      preview.style.display = 'block'
+      container.innerHTML = html
+    })
+}

data/app/assets/javascripts/works_cited/removeFields.js

@@ -0,0 +1,37 @@
+class removeFields {
+  // This executes when the function is instantiated.
+  constructor() {
+    this.iterateLinks()
+  }
+
+  iterateLinks() {
+    document.querySelectorAll('.remove_fields').forEach(link => {
+      link.addEventListener('click', e => {
+        this.handleClick(link, e)
+      })
+    })
+  }
+
+  handleClick(link, e) {
+    // Stop the function from executing if a link or event were not passed into the function.
+    if (!link || !e) return
+    // Prevent the browser from following the URL.
+    e.preventDefault()
+    // Find the parent wrapper for the set of nested fields.
+    let fieldParent = link.closest('.nested-fields')
+    // If there is a parent wrapper, find the hidden delete field.
+    let deleteField = fieldParent
+      ? fieldParent.querySelector('input[type="hidden"]')
+      : null
+    // If there is a delete field, update the value to `1` and hide the corresponding nested fields.
+    if (deleteField) {
+      deleteField.value = 1
+      fieldParent.style.display = 'none'
+    }
+  }
+}
+
+// Wait for turbolinks to load, otherwise `document.querySelectorAll()` won't work
+window.addEventListener('load', () => new removeFields())
+window.addEventListener('works_cited:load', () => new removeFields())
+window.addEventListener('works_cited:add_fields', () => new removeFields())

data/app/assets/javascripts/works_cited/shared.js

@@ -0,0 +1,59 @@
+function packInContainer(container, keyParts, value) {
+  var part = keyParts.shift()
+  var packed = value
+  if(keyParts.length > 0) {
+    var previous = container[part] || {}
+    packed = packInContainer(previous, keyParts, value)
+  }
+
+  return {...container, [part]: packed}
+}
+
+function getWritableField(key) {
+  var fields = document.getElementsByName(key)
+  var found
+
+  fields.forEach((field) => {
+    if(field.readonly === undefined || !field.readonly) {
+      if(field.value) {
+        found = field.value
+        return
+      }
+    }
+  })
+
+  return found || ''
+}
+
+function getFieldByLocator(data, locator) {
+  var regex = /^\[(.*)\]$/
+  var container = {}
+
+  // Display the key/value pairs
+  for(var pair of data.entries()) {
+    // We want to avoid using pair[1] even though it's there because some fields are readonly
+    var key = pair[0]
+    if(key.startsWith(locator)) {
+      var strippedKeyString = key.replace(locator, '').replace(regex, "$1")
+      var keyParts = strippedKeyString.split('][')
+      var value = getWritableField(key)
+      console.warn(strippedKeyString)
+      container = packInContainer(container, keyParts, value)
+    }
+  }
+
+  return container
+}
+
+function getFormDataPacked(data) {
+  var container = {}
+
+  // Display the key/value pairs
+  for(var pair of data.entries()) {
+    // We want to avoid using pair[1] even though it's there because some fields are readonly
+    var key = pair[0]
+    container[key] = getWritableField(key)
+  }
+
+  return container
+}

data/app/assets/javascripts/works_cited/showFields.js

@@ -0,0 +1,9 @@
+function showFields(citation_id, citation_type) {
+  citation = document.getElementById(`citation-${citation_id}`)
+  for (let element of citation.getElementsByClassName('fields')){
+    element.style.display='none'
+    element.readonly=true
+  }
+  console.warn(citation.getElementsByClassName(`fields-${citation_type}`)[0])
+  citation.getElementsByClassName(`fields-${citation_type}`)[0].style.display = 'block'
+}

data/app/assets/javascripts/works_cited/submitForm.js

@@ -0,0 +1,33 @@
+function post(path, params, method = 'post') {
+
+  // The rest of this code assumes you are not using a library.
+  // It can be made less verbose if you use one.
+  const form = document.createElement('form')
+  form.method = method
+  form.action = path
+
+  for (const key in params) {
+    if (params.hasOwnProperty(key)) {
+      const hiddenField = document.createElement('input')
+      hiddenField.type = 'hidden'
+      hiddenField.name = key
+      hiddenField.value = params[key]
+
+      form.appendChild(hiddenField)
+    }
+  }
+
+  document.body.appendChild(form)
+  form.submit()
+}
+
+function addFilterToSubmit() {
+  var form = document.querySelector('form')
+
+  form.onsubmit = function (e) {
+    e.preventDefault()
+    var formData = new FormData(form)
+    var revisedData = getFormDataPacked(formData)
+    return post(form.action, revisedData)
+  }
+}

data/app/controllers/concerns/works_cited/params.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module WorksCited
+  # Concern to allow strong parameters when using accepts_nested_attributes_for to edit Cookbook Uses
+  module Params
+    extend ActiveSupport::Concern
+
+    included do
+      # rescue_from Pundit::NotAuthorizedError, with: :authorization_error
+      # include Pundit
+    end
+
+    def works_cited_params
+      {
+        works_cited_citations_attributes: [
+          :id, :citation_type, :title, :container_title, :publisher, :city, :edition, :volume,
+          :number, :series, :year, :record, :media, :url, :pages, :published_at, :online_database, :doi,
+          :accessed_at, :_destroy,
+          { works_cited_contributors_attributes: %i[id contributor_role first middle last suffix handle _destroy] }
+        ]
+      }
+    end
+  end
+end

data/app/controllers/works_cited/citations_controller.rb

@@ -16,11 +16,10 @@ module WorksCited
 
     # GET /citations/preview
     def preview
-      @citation = Citation.new(citation_params)
-      authorize! :preview, @citation
-
-      @contributors = contributors_from_params
-      render :preview, layout: false
+      authorize! :preview, Citation
+      locals = { citation: citation_for_preview, contributors: contributors_for_preview }
+      html = render_to_string('preview', formats: [:html], layout: false, locals: locals)
+      render json: { html: html }
     end
 
     # GET /citations/new
@@ -55,9 +54,15 @@ module WorksCited
 
     private
 
-    def contributors_from_params
+    def citation_for_preview
+      return Citation.new(preview_params) unless params[:id].present?
+
+      Citation.find(params[:id]).assign_attributes(preview_params)
+    end
+
+    def contributors_for_preview
       contributors_array = []
-      raw_contributors = citation_params[:works_cited_contributors_attributes]
+      raw_contributors = preview_params[:works_cited_contributors_attributes]
       raw_contributors&.each do |_index, contributor|
         destroy = contributor.delete(:_destroy)
         next if destroy == '1'
@@ -92,12 +97,8 @@ module WorksCited
 
     def model_option_group(items, model)
       model_name = model.model_name
-      OpenStruct.new(
-        {
-          record_type: model_name,
-          records: items.map { |item| item_option(item, model_name) }
-        }
-      )
+      records = items.map { |item| item_option(item, model_name) }
+      OpenStruct.new({ record_type: model_name, records: records })
     end
 
     def set_records
@@ -109,6 +110,18 @@ module WorksCited
       end.compact
     end
 
+    # Only allow a list of trusted parameters through.
+    def preview_params
+      json = JSON.parse(request.raw_post)
+      json_params = ActionController::Parameters.new(json)
+      json_params.require(:citation).permit(
+        :id, :citation_type, :title, :container_title, :publisher, :city, :edition, :volume,
+        :number, :series, :year, :record, :media, :url, :pages, :published_at, :online_database, :doi,
+        :accessed_at,
+        works_cited_contributors_attributes: %i[id contributor_role first middle last suffix handle _destroy]
+      )
+    end
+
     # Only allow a list of trusted parameters through.
     def citation_params
       params.require(:citation).permit(

data/app/helpers/works_cited/application_helper.rb

@@ -18,11 +18,15 @@ module WorksCited
       render path, citation: citation, contributors: contributors
     end
 
-    def works_cited_citation_fields(form, citation_type)
+    def works_cited_type_fields(form, citation_type)
       path = partial_path_or_default('fields', citation_type)
       render path, f: form, citation_type: citation_type
     end
 
+    def works_cited_citations_fields(form)
+      render 'works_cited/citations/fields', form: form
+    end
+
     def list_names(names)
       first = first_contributor(names)
       first_contributor_name = first&.full_name(first.name_order)
@@ -36,6 +40,47 @@ module WorksCited
       end
     end
 
+    # This method creates a link with `data-id` `data-fields` attributes. These attributes are
+    # used to create new instances of the nested fields through Javascript.
+    def works_cited_link_to_add_fields(name, form, association, partial = nil)
+      # Takes an object (@person) and creates a new instance of its associated model (:addresses)
+      # To better understand, run the following in your terminal:
+      # rails c --sandbox
+      # @person = Person.new
+      # new_object = @person.send(:addresses).klass.new
+      new_object = form.object.send(association).klass.new
+
+      # Saves the unique ID of the object into a variable.
+      # This is needed to ensure the key of the associated array is unique. This is makes
+      # parsing the content in the `data-fields` attribute easier through Javascript.
+      # We could use another method to achive this.
+      id = new_object.object_id
+
+      # https://api.rubyonrails.org/ fields_for(record_name, record_object = nil, fields_options = {}, &block)
+      # record_name = :addresses
+      # record_object = new_object
+      # fields_options = { child_index: id }
+      # child_index` is used to ensure the key of the associated array is unique, and that it
+      # matched the value in the `data-id` attribute.
+      # `person[addresses_attributes][child_index_value][_destroy]`
+      fields = form.simple_fields_for(association, new_object, child_index: id) do |builder|
+        # `association.to_s.singularize + "_fields"` ends up evaluating to `address_fields`
+        # The render function will then look for `views/people/_address_fields.html.erb`
+        # The render function also needs to be passed the value of 'builder', because
+        # `views/people/_address_fields.html.erb` needs this to render the form tags.
+        partial ||= "#{association.to_s.singularize}_fields"
+        render(partial, record: new_object, f: builder)
+      end
+
+      # This renders a simple link, but passes information into `data` attributes.
+      # This info can be named anything we want, but in this case we chose `data-id:` and `data-fields:`.
+      # The `id:` is from `new_object.object_id`.
+      # The `fields:` are rendered from the `fields` blocks.
+      # We use `gsub("\n", "")` to remove anywhite space from the rendered partial.
+      # The `id:` value needs to match the value used in `child_index: id`.
+      link_to(name, '#', class: 'add_fields button button-action', data: { id: id, fields: fields.gsub("\n", '') })
+    end
+
     private
 
     def partial_path_or_default(purpose, citation_type)

data/app/models/works_cited/citation.rb

@@ -17,20 +17,24 @@ module WorksCited
 
     # Relationships
     belongs_to :record, polymorphic: true
-    has_many :works_cited_contributors, -> { order(:last, :first, :middle, :suffix, :handle) }, inverse_of: :works_cited_citation, class_name: 'WorksCited::Contributor',
-             foreign_key: :works_cited_citation_id
-    has_many :works_cited_authors, -> { authors.order(:last, :first, :middle, :suffix, :handle) }, inverse_of: :works_cited_citation, class_name: 'WorksCited::Contributor',
-             foreign_key: :works_cited_citation_id
-    accepts_nested_attributes_for :works_cited_contributors, allow_destroy: true
+    has_many :works_cited_contributors, lambda {
+                                          order(:last, :first, :middle, :suffix, :handle)
+                                        }, inverse_of: :works_cited_citation, class_name: 'WorksCited::Contributor',
+                                           foreign_key: :works_cited_citation_id, dependent: :destroy
+    has_many :works_cited_authors, lambda {
+                                     authors.order(:last, :first, :middle, :suffix, :handle)
+                                   }, inverse_of: :works_cited_citation, class_name: 'WorksCited::Contributor',
+                                      foreign_key: :works_cited_citation_id
+    accepts_nested_attributes_for :works_cited_contributors, reject_if: :all_blank, allow_destroy: true
 
     # Scopes
     scope :ordered_by_author, (lambda do
       joins(:works_cited_authors)
         .order('MIN(works_cited_contributors.last) ASC, '\
-          'MIN(works_cited_contributors.first) ASC, '\
-          'MIN(works_cited_contributors.middle) ASC, '\
-          'MIN(works_cited_contributors.suffix) ASC, '\
-          'MIN(works_cited_contributors.handle) ASC')
+               'MIN(works_cited_contributors.first) ASC, '\
+               'MIN(works_cited_contributors.middle) ASC, '\
+               'MIN(works_cited_contributors.suffix) ASC, '\
+               'MIN(works_cited_contributors.handle) ASC')
         .group(:id)
     end)
 
@@ -44,17 +48,31 @@ module WorksCited
     end
 
     # Instance Methods
+    def works_cited_contributors_attributes=(raw_contributors)
+      array = []
+      raw_contributors&.each do |_index, contributor|
+        destroy = contributor.delete(:_destroy)
+        if destroy == '1'
+          Contributor.find(contributor[:id]).destroy if contributor[:id]
+          next
+        end
+
+        array << contributor
+      end
+      super array
+    end
+
     def record=(value)
       unless value.is_a? String
         super(value)
         return
       end
 
-      model_name, id = value.split(':')
-      return unless model_name.present? && id.present?
+      model_name, my_id = value.split(':')
+      return unless model_name.present? && my_id.present?
 
       model = model_name.constantize
-      super model.find(id)
+      super model.find(my_id)
     end
 
     def periodical?
@@ -76,27 +94,5 @@ module WorksCited
     def email?
       citation_type == 'email'
     end
-
-    if defined?(RailsAdmin)
-      rails_admin do
-        visible false
-        edit do
-          field :citation_type, :enum do
-            enum do
-              WorksCited.configuration.valid_citation_types
-            end
-          end
-          field :works_cited_contributors do
-            label 'Contributors'
-          end
-          include_all_fields
-          field :media
-          field :record do
-            # Can't remove this using :inverse_of because it's polymorphic
-            visible false
-          end
-        end
-      end
-    end
   end
 end

data/app/models/works_cited/contributor.rb

@@ -57,28 +57,25 @@ module WorksCited
 
     def name_parts
       parts = []
-      parts << first_name_or_initial
-      parts << middle_initial
-      parts << if suffix.present? && last.present?
-                 "#{last}, #{suffix}"
-               else
-                 suffix.presence || last.presence
-               end
+      parts << first_name_or_initial.presence
+      parts << middle_initial.presence
+      parts << [last.presence, suffix.presence].compact.join(', ').presence
       parts
     end
 
     def name_parts_reversed
       parts = []
-      parts << "#{last}," if last.present?
-      parts << first_name_or_initial
-      parts << if suffix.present? && middle_initial.present?
-                 "#{middle_initial}, #{suffix}"
-               else
-                 suffix.presence || middle_initial
-               end
+      parts << [last.presence, name_other_parts.presence].compact.join(', ').presence
       parts
     end
 
+    def name_other_parts
+      other_parts = []
+      other_parts << first_name_or_initial.presence
+      other_parts << [middle_initial.presence, suffix.presence].compact.join(', ').presence
+      other_parts.compact.join(' ').presence
+    end
+
     def first_name_or_initial
       return nil unless first.present?
 
@@ -90,19 +87,5 @@ module WorksCited
 
       middle[0, 1]&.upcase
     end
-
-    if defined?(RailsAdmin)
-      rails_admin do
-        visible false
-        edit do
-          field :contributor_role, :enum do
-            enum do
-              WorksCited.configuration.valid_contributor_roles
-            end
-          end
-          include_all_fields
-        end
-      end
-    end
   end
 end

data/app/views/works_cited/citation_types/fields/_book.html.haml

@@ -1,4 +1,4 @@
-= f.input :url
+= f.input :url, as: :string
 = f.input :title
 = f.input :edition, hint: 'like "1st ed."'
 = f.input :volume, hint: 'like "vol. 1"'

data/app/views/works_cited/citation_types/fields/_electronic.html.haml

@@ -10,5 +10,5 @@
 = f.input :pages, hint: 'like "pp. 150-53"'
 = f.input :online_database
 = f.input :doi, label: 'Digital Object Identifier'
-= f.input :url
+= f.input :url, as: :string
 = f.input :accessed_at, as: :string, hint: 'Will be parsed and formatted', input_html: { value: f.object.accessed_at&.mla_date }

data/app/views/works_cited/citation_types/fields/_email.html.haml

@@ -10,4 +10,4 @@
 = f.input :pages, hint: 'like "pp. 150-53"'
 = f.input :online_database
 = f.input :doi, label: 'Digital Object Identifier'
-= f.input :url
+= f.input :url, as: :string

data/app/views/works_cited/citation_types/fields/_interview.html.haml

@@ -9,4 +9,4 @@
 = f.input :series
 = f.input :published_at, as: :string, hint: 'Will be parsed and formatted', input_html: { value: f.object.published_at&.mla_date }
 = f.input :pages, hint: 'like "pp. 150-53"'
-= f.input :url
+= f.input :url, as: :string

data/app/views/works_cited/citation_types/fields/_periodical.html.haml

@@ -7,4 +7,4 @@
 = f.input :year
 = f.input :published_at, as: :string, hint: 'Will be parsed and formatted', input_html: { value: f.object.published_at&.mla_date }
 = f.input :pages, hint: 'like "pp. 150-53"'
-= f.input :url
+= f.input :url, as: :string

data/app/views/works_cited/citation_types/fields/_tweet.html.haml

@@ -1,4 +1,4 @@
 = f.input :title, label: 'Tweet text'
 = f.input :publisher, default: 'Twitter'
 = f.input :published_at, label: 'Time posted', as: :string, hint: 'Will be parsed and formatted', input_html: { value: f.object.published_at&.mla_datetime }
-= f.input :url
+= f.input :url, as: :string