Staging release

app/assets/javascripts/application.js.coffee

@@ -951,6 +951,7 @@ jQuery ->
         height: 200
         wordcount:
           maxWordCount: $(this).data('word-max')
+        versionCheck: false
 
       CKEDITOR.on 'instanceCreated', (event) ->
         editor = event.editor

app/assets/javascripts/frontend/custom_questions/support_letters.js.coffee

@@ -2,38 +2,42 @@ window.SupportLetters =
   init: ->
     $('.js-support-letter-attachment').each (idx, el) ->
       SupportLetters.fileupload_init(el)
-    SupportLetters.save_collection_init()
+
+    $(document).on 'change', '.js-trigger-autosave', debounce(SupportLetters.submit, 1000)
 
   new_item_init: (el) ->
     SupportLetters.clean_up_system_tags(el)
     SupportLetters.enable_item_fields_and_controls(el)
-    SupportLetters.fileupload_init(el.find(".js-support-letter-attachment"))
+    SupportLetters.fileupload_init(el.find('.js-support-letter-attachment'))
     el.find('input,textarea,select').first().focus()
 
   fileupload_init: (el) ->
     $el = $(el)
-    parent = $el.closest(".govuk-form-group")
+    parent = $el.closest('.govuk-form-group')
 
     upload_done = (e, data) ->
       SupportLetters.clean_up_system_tags(parent)
 
       if data.result['original_filename']
         filename = data.result['original_filename']
       else
-        filename = "File uploaded"
-      file_title = $("<p class='govuk-body support-letter-attachment-filename'>" + filename + "</p>")
-      hidden_input = $("<input class='js-support-letter-attachment-id' type='hidden' name='#{$el.attr("name")}' value='#{data.result['id']}' />")
-
-      parent.find(".govuk-error-message").html("")
-      parent.find(".govuk-error-message").closest(".govuk-form-group").removeClass("govuk-form-group--error")
-      parent.append(file_title)
-      parent.append(hidden_input)
+        filename = 'File uploaded'
+
+      parent.find('.govuk-error-message').html('')
+      parent.find('.govuk-error-message').closest('.govuk-form-group').removeClass('govuk-form-group--error')
+
+      label = $('<p class="govuk-body support-letter-attachment-filename">' + filename + '</p>')
+      hiddenInput = $("<input class='js-support-letter-attachment-id' type='hidden' name='#{$el.attr("name")}' value='#{data.result['id']}' />")
+
+      parent.append(label)
+      parent.append(hiddenInput)
       SupportLetters.autosave()
+      SupportLetters.submit(e)
 
     failed = (error_message) ->
       SupportLetters.clean_up_system_tags(parent)
-      parent.find(".govuk-error-message").html(error_message)
-      parent.closest(".govuk-form-group").addClass("govuk-form-group--error")
+      parent.find('.govuk-error-message').html(error_message)
+      parent.closest('.govuk-form-group').addClass('govuk-form-group--error')
 
     success_or_error = (e, data) ->
       errors = data.result.errors
@@ -44,108 +48,30 @@ window.SupportLetters =
         upload_done(e, data)
 
     $el.fileupload(
-      url: $el.closest(".list-add").data('attachments-url') + ".json"
+      url: $el.closest('.list-add').data('attachments-url') + '.json'
       forceIframeTransport: true
       dataType: 'json'
       formData: [
-        { name: "authenticity_token", value: $("meta[name='csrf-token']").attr("content") }
+        { name: 'authenticity_token', value: $('meta[name="csrf-token"]').attr('content') }
       ]
       always: success_or_error
     )
 
   clean_up_system_tags: (parent) ->
-    parent.find("input[type='hidden']").remove()
-    parent.find(".support-letter-attachment-filename").remove()
+    parent.find('input[type="hidden"]').remove()
+    parent.find('.support-letter-attachment-filename').remove()
 
   enable_item_fields_and_controls: (parent) ->
-    parent.find(".js-save-collection").removeClass("govuk-!-display-none")
-    parent.find(".visible-read-only").hide()
-    fields = parent.find("input")
-    fields.removeClass("read-only")
-    parent.find(".govuk-error-message").html("")
-    form_name_prefix = parent.find(".js-system-tag").data("new-hidden-input-name")
-    letter_id_hidden_input = $("<input class='js-support-entry-id'>").prop('type', 'hidden').
-                                                                       prop('name', form_name_prefix)
-    parent.append(letter_id_hidden_input)
-
-  disable_item_fields_and_controls: (parent) ->
-    parent.find(".js-save-collection").addClass("govuk-!-display-none")
-    parent.find(".visible-read-only").show()
-    fields = parent.find("input")
-    fields.addClass("read-only")
-
-  save_collection_init: () ->
-    $(document).on 'click', '.js-save-collection', (e) ->
-      e.preventDefault()
-      e.stopPropagation()
-
-      button = $(this)
-      parent = $(this).closest("li")
-
-      if !button.hasClass("govuk-!-display-none")
-        save_url = button.data 'save-collection-url'
-
-        first_name = parent.find(".js-support-letter-first-name").val()
-        last_name = parent.find(".js-support-letter-last-name").val()
-        email = parent.find(".js-support-letter-email").val()
-        relationship_to_nominee = parent.find(".js-support-letter-relationship-to-nominee").val()
-        attachment_id = parent.find(".js-support-letter-attachment-id").val()
-
-        data = {
-          "support_letter": {
-            "first_name": first_name,
-            "last_name": last_name,
-            "relationship_to_nominee": relationship_to_nominee
-          }
-        }
-
-        if attachment_id
-          data["support_letter"]["attachment"] = attachment_id
-
-        if email
-          data["support_letter"]["email"] = email
-
-        if save_url
-          $.ajax
-            url: save_url
-            type: 'post'
-            data: data
-            dataType: 'json'
-            success: (response) ->
-              parent.find(".js-support-entry-id").prop('value', response)
-              parent.find(".govuk-error-message").html("")
-              parent.removeClass("govuk-form-group--error")
-              parent.addClass("read-only")
-              parent.addClass("js-support-letter-received")
-              parent.closest('li').find("input[type='text']").each ->
-                show_el = $(this).closest(".govuk-form-group").find(".visible-read-only")
-                show_el.text($(this).val())
-              SupportLetters.disable_item_fields_and_controls(parent)
-              window.FormValidation.validateStep()
-              SupportLetters.autosave()
-              SupportLetters.showRemoveLink(parent, data, response)
-
-              return
-            error: (response) ->
-              parent.find(".govuk-error-message").html("")
-              parent.removeClass("govuk-form-group--error")
-              error_message = response.responseText
-              $.each $.parseJSON(response.responseText), (question_key, error_message) ->
-                key_selector = ".js-support-letter-" + question_key.replace(/_/g, "-")
-                field_error_container = parent.find(key_selector).
-                                              closest(".govuk-form-group").
-                                              find(".govuk-error-message")
-                field_error_container.html(error_message[0])
-                field_error_container.closest(".govuk-form-group").addClass("govuk-form-group--error")
-              button.removeClass("govuk-visually-hidden")
-
-              return
+    parent.find('.govuk-error-message').html('')
+    prefixed = parent.find('.js-system-tag').data('new-hidden-input-name')
+    hiddenInput = $('<input class="js-support-entry-id">').prop('type', 'hidden').prop('name', prefixed)
+    parent.append(hiddenInput)
 
   autosave: () ->
     url = $('form.qae-form').data('autosave-url')
     if url
       # Setting current_step_id to form as we updating only current section form_data (not whole form)
-      $("#current_step_id").val($(".js-step-condition.step-current").attr("data-step"))
+      $('#current_step_id').val($('.js-step-condition.step-current').attr('data-step'))
 
       form_data = $('form.qae-form').serialize()
       $.ajax({
@@ -155,8 +81,71 @@ window.SupportLetters =
         dataType: 'json'
       })
 
-  showRemoveLink: (parent, data, response) ->
-    removeLink = $(".remove-supporter", parent)
-    removeLink.data("url", "/users/form_answers/#{response['form_answer_id']}/support_letters/#{response['id']}")
-    removeLink.attr("aria-label", "Delete support letter from #{data['support_letter']['first_name']} #{data['support_letter']['last_name']}")
-    removeLink.removeClass("govuk-!-display-none")
+  submit: (e) ->
+    e.preventDefault()
+    e.stopPropagation()
+
+    target = $(e.target)
+    parent = target.closest("li")
+
+    data = {'support_letter': {}}
+    data['support_letter']['first_name'] = parent.find('.js-support-letter-first-name').val()
+    data['support_letter']['last_name'] = parent.find('.js-support-letter-last-name').val()
+    data['support_letter']['relationship_to_nominee'] = parent.find('.js-support-letter-relationship-to-nominee').val()
+
+    attachmentId = parent.find('.js-support-letter-attachment-id').val()
+    if attachmentId
+      data['support_letter']['attachment'] = attachmentId
+
+    email = parent.find('.js-support-letter-email').val()
+    if email
+      data['support_letter']['email'] = email
+
+    createUrl = parent.data('create-url')
+    updateUrl = parent.data('update-url')
+    persistUrl = updateUrl || createUrl
+
+    type = if !!updateUrl 
+      'put'
+    else
+      'post'
+
+    if persistUrl
+      $.ajax
+        url: persistUrl
+        type: type
+        data: data
+        dataType: 'json'
+        success: (response) ->
+          parent.find('.js-support-entry-id').prop('value', response['id'])
+          parent.find('.govuk-error-message').html('')
+          parent.removeClass('govuk-form-group--error')
+          parent.addClass('js-support-letter-received')
+          parent.attr('data-update-url', response['update_url'])
+          window.FormValidation.validateStep()
+          SupportLetters.autosave()
+
+          return
+        error: (response) ->
+          parent.find('.govuk-error-message').html('')
+          parent.removeClass('govuk-form-group--error')
+          msg = response.responseText
+          $.each $.parseJSON(response.responseText), (key, msg) ->
+            selector = '.js-support-letter-' + key.replace(/_/g, '-')
+
+            errorContainer = parent.find(selector).closest('.govuk-form-group').find('.govuk-error-message')
+            errorContainer.html(msg[0])
+            errorContainer.closest('.govuk-form-group').addClass('govuk-form-group--error')
+
+          return
+
+debounce = (fn, wait = 500) ->
+  last = (new Date) - wait
+  ->
+    now = new Date
+
+    # Return if we haven't waited long enough
+    return if wait > (now - last)
+
+    fn.apply null, arguments
+    last = now