From d3d03a31776ed25d2fa38d069e72bd99c0914ed1 Mon Sep 17 00:00:00 2001
From: Simon Poole <simon@poole.ch>
Date: Fri, 26 Oct 2018 20:00:00 +0200
Subject: [PATCH] Move onclick handling of tou checkbox to ujs

---
 app/assets/javascripts/user.js | 5 +++++
 app/views/users/terms.html.erb | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/app/assets/javascripts/user.js b/app/assets/javascripts/user.js
index f898455b5..40ca866c5 100644
--- a/app/assets/javascripts/user.js
+++ b/app/assets/javascripts/user.js
@@ -124,4 +124,9 @@ $(document).ready(function () {
     $("#contributorTerms").html("<img src='" + OSM.SEARCHING + "' />");
     $("#contributorTerms").load(url);
   });
+
+  $("#read_tou").on("click", function () { 
+    var agreebtn = document.getElementById('agree');
+    agreebtn.disabled=!this.checked;
+  });
 });
diff --git a/app/views/users/terms.html.erb b/app/views/users/terms.html.erb
index 18b86dd12..7a6acbf59 100644
--- a/app/views/users/terms.html.erb
+++ b/app/views/users/terms.html.erb
@@ -47,7 +47,7 @@
   <p class="deemphasize"><%= t '.tou_explain' %></p>
   <div class="form-row">
     <label for="read_tou">
-      <%= check_box_tag 'read_tou',1,false,:onclick=>"agreebtn = document.getElementById('agree');agreebtn.disabled=!this.checked;"%>
+      <%= check_box_tag 'read_tou',1,false %>
       <%= t '.read_tou_html', :tou_link => link_to(t('layouts.tou'), "https://wiki.osmfoundation.org/wiki/Terms_of_Use", :target => :new) %>
     </label>
 
-- 
2.43.2