fe 2.3.1 → 2.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e4595077d46b768d5c9ce3252aba5a8e19ea53e9d16c2c80231117b1e5e42978
-  data.tar.gz: 4d8da1609cfb02cf31939937cc9715b34e6bd0b1718ba8abd18fe7b0861b07b9
+  metadata.gz: 63c54a49ce10c97d77e21066af68ae2e4d138f57e1d78789a0b304f2ea26d771
+  data.tar.gz: 860211697773fc5643d50e85617065cd49697725ac2479fb644d321eac620c91
 SHA512:
-  metadata.gz: a70cc0c35ccdce9c5829f6a2c9942a2c5888ff4e054ab9f82e4b40a0d86dfc0f306888d57740aa1b7ae6f6aec1b7a991b318af91c67eaec80191273e3e6d62a9
-  data.tar.gz: 5d96fe5b57c0fa56493e2cf811114ffcad519bdf101936ddd27bfa12381aafe819901216b60361d651e3faa50a8e3cfb8823c184eeaf2e009fc7d103f0ceeeaa
+  metadata.gz: 20f8c1697604eb9bdf58be607a9c81e86ee4dd0ca9e12e740113c173cbe1402a6371dfa9861ef6861c46b1d9fc6e9ae9f48c069dea829a130ac92d4e32f0e685
+  data.tar.gz: 498cec05620ede4dbee334badd0c2e3810c7d3719816a8a3b7c986238badc9deba878ae0a75ef0a84385330bd17a0a40a2c5cae404e5b32e6a0d688ccaf30591

data/app/assets/javascripts/fe/fe.public.nojquery.js.erb

@@ -583,6 +583,12 @@ $(document).on('ready turbo:load', function () {
   });
   <% end %>
 
+  // Keep in step with Fe::ReferenceSheet#email=, which strips the value on the server.
+  let validateEmail = $.validator.methods.email;
+  $.validator.methods.email = function(value, element, param) {
+    return validateEmail.call(this, $.trim(value), element, param);
+  };
+
   $(document).on('click', '.save_button', function() {
     fe.debugLog('[fe concurrency] Save button clicked');
     fe.pageHandler.savePage($(this).closest('.answer-page'), true);

data/app/models/fe/reference_sheet.rb

@@ -114,6 +114,10 @@ module Fe
 
     def email_sent?() !self.email_sent_at.nil? end
 
+    def email=(value)
+      super(value.is_a?(String) ? value.gsub(/\A[[:space:]]+|[[:space:]]+\z/, '') : value)
+    end
+
     # send email invite
     def send_invite(host)
       return if self.email.blank?

data/lib/fe/version.rb

@@ -1,3 +1,3 @@
 module Fe
-  VERSION = "2.3.1"
+  VERSION = "2.3.2"
 end

data/spec/controllers/fe/answer_pages_controller_spec.rb

@@ -76,6 +76,19 @@ describe Fe::AnswerPagesController, type: :controller do
       expect(reference_sheet.reload.email).to eq('email@reference.com')
     end
 
+    it 'strips whitespace around the reference email' do
+      reference_sheet = create(:reference_sheet, applicant_answer_sheet_id: answer_sheet.id, email: 'initial@ref.com')
+
+      put :update, params: {
+        answers: { "#{element.id}" => 'answer here' },
+        reference: { "#{reference_sheet.id}" => { email: ' email@reference.com ' } },
+        id: page.id,
+        answer_sheet_id: answer_sheet.id
+      }, xhr: true
+
+      expect(reference_sheet.reload.email).to eq('email@reference.com')
+    end
+
     it 'should store a reference sheet answer' do
       # create a normal applicant sheet to make sure the answer isn't saved to that
       # ref