From ead329812e2692edbc78174fee5553e19d4e837a Mon Sep 17 00:00:00 2001
From: Fabio Teixeira <fabio1079@gmail.com>
Date: Thu, 16 Jan 2014 09:42:33 -0200
Subject: [PATCH] check_all_manage_fields:  - When the field "check/uncheck all" is checked or unchecked, all fields below are checked or unchecked  - Corrected onclick events on active/required/signup fields  - Replaced javascript logic from person/community/enterprise to a single javascript file  - Cucumber tests for check/uncheck all fields

---
 app/views/features/_manage_community_fields.rhtml  | 30 +++++++++++++++++++++++-------
 app/views/features/_manage_enterprise_fields.rhtml | 30 +++++++++++++++++++++++-------
 app/views/features/_manage_person_fields.rhtml     | 31 ++++++++++++++++++++++++-------
 app/views/features/manage_fields.rhtml             |  2 ++
 features/manage_fields.feature                     | 67 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 public/javascripts/manage-fields.js                | 83 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 6 files changed, 222 insertions(+), 21 deletions(-)
 create mode 100644 features/manage_fields.feature
 create mode 100644 public/javascripts/manage-fields.js

diff --git a/app/views/features/_manage_community_fields.rhtml b/app/views/features/_manage_community_fields.rhtml
index f5972be..491dffe 100644
--- a/app/views/features/_manage_community_fields.rhtml
+++ b/app/views/features/_manage_community_fields.rhtml
@@ -9,21 +9,37 @@
     <th><%= _('Required') %></th>
     <th><%= _('Display on creation?') %></th>
   </tr>
+
+  <tr style="border-bottom:solid 2px #000">
+    <td>
+      <%= _("Check/Uncheck All")%>
+    </td>
+    <td>
+      <input type="checkbox" id="community_active" />
+    </td>
+    <td>
+      <input type="checkbox" id="community_required" />
+    </td>
+    <td>
+      <input type="checkbox" id="community_signup" />
+    </td>
+  </tr>
+
   <% @community_fields.each do |field| %>
   <tr>
     <td><label for="community_fields[<%= field %>][active]"><%= _(field.humanize) %></label></td>
 
     <td>
       <%= hidden_field_tag  "community_fields[#{field}][active]", false %>
-      <%= check_box_tag "community_fields[#{field}][active]", true, environment.custom_community_field(field, 'active'), :onclick => "$('community_fields[#{field}][required]').disabled=$('community_fields[#{field}][signup]').disabled=!this.checked;" %>
+      <%= check_box_tag "community_fields[#{field}][active]", true, environment.custom_community_field(field, 'active'), :onclick => "active_action(this, 'community_fields[#{field}][required]', 'community_fields[#{field}][signup]')" %>
     </td>
     <td>
       <%= hidden_field_tag  "community_fields[#{field}][required]", false %>
-      <%= check_box_tag "community_fields[#{field}][required]", true, environment.custom_community_field(field, 'required'), :onclick => "if(this.checked) $('community_fields[#{field}][signup]').checked = true;" %>
+      <%= check_box_tag "community_fields[#{field}][required]", true, environment.custom_community_field(field, 'required'), :onclick => "required_action('community_fields[#{field}][active]','community_fields[#{field}][required]', 'community_fields[#{field}][signup]')" %>
     </td>
     <td>
       <%= hidden_field_tag  "community_fields[#{field}][signup]", false %>
-      <%= check_box_tag "community_fields[#{field}][signup]", true, environment.custom_community_field(field, 'signup'), :onclick => "if(!this.checked) $('community_fields[#{field}][required]').checked = false;" %>
+      <%= check_box_tag "community_fields[#{field}][signup]", true, environment.custom_community_field(field, 'signup'), :onclick => "signup_action('community_fields[#{field}][active]','community_fields[#{field}][required]', 'community_fields[#{field}][signup]')" %>
     </td>
 
   </tr>
@@ -31,18 +47,18 @@
 </table>
 
 <script type='text/javascript'>
-  var trs = $$('#community_fields_conf tr');
+  var trs = jQuery('#community_fields_conf tr');
   var tr, td2;
-  for ( var i=0; tr=trs[i]; i++ ) {
+  for ( var i=2; tr=trs[i]; i++ ) {
     if ( td2 = tr.getElementsByTagName('td')[1] ) {
-      td2.getElementsByTagName('input')[0].onclick();
+      td2.getElementsByTagName('input')[1].onclick();
     }
   }
 </script>
 
 <div>
   <% button_bar do %>
-    <%= submit_button('save', _('Save changes')) %>
+    <%= submit_button('save', _('Save changes'), :id=>"save_community_fields") %>
     <%= button :back, _('Back to admin panel'), :controller => 'admin_panel', :action => 'index' %>
   <% end %>
 </div>
diff --git a/app/views/features/_manage_enterprise_fields.rhtml b/app/views/features/_manage_enterprise_fields.rhtml
index 6c4b731..0f3195f 100644
--- a/app/views/features/_manage_enterprise_fields.rhtml
+++ b/app/views/features/_manage_enterprise_fields.rhtml
@@ -9,21 +9,37 @@
     <th><%= _('Required') %></th>
     <th><%= _('Display on registration?') %></th>
   </tr>
+
+  <tr style="border-bottom:solid 2px #000">
+    <td>
+      <%= _("Check/Uncheck All")%>
+    </td>
+    <td>
+      <input type="checkbox" id="enterprise_active" />
+    </td>
+    <td>
+      <input type="checkbox" id="enterprise_required" />
+    </td>
+    <td>
+      <input type="checkbox" id="enterprise_signup" />
+    </td>
+  </tr>
+
   <% @enterprise_fields.each do |field| %>
   <tr>
 
     <td><label for="enterprise_fields[<%= field %>][active]"><%= _(field.humanize) %></label></td>
     <td>
       <%= hidden_field_tag  "enterprise_fields[#{field}][active]", false %>
-      <%= check_box_tag "enterprise_fields[#{field}][active]", true, environment.custom_enterprise_field(field, 'active'), :onclick => "$('enterprise_fields[#{field}][required]').disabled=$('enterprise_fields[#{field}][signup]').disabled=!this.checked;" %>
+      <%= check_box_tag "enterprise_fields[#{field}][active]", true, environment.custom_enterprise_field(field, 'active'), :onclick => "active_action(this, 'enterprise_fields[#{field}][required]', 'enterprise_fields[#{field}][signup]')" %>
     </td>
     <td>
       <%= hidden_field_tag  "enterprise_fields[#{field}][required]", false %>
-      <%= check_box_tag "enterprise_fields[#{field}][required]", true, environment.custom_enterprise_field(field, 'required'), :onclick => "if(this.checked) $('enterprise_fields[#{field}][signup]').checked = true;" %>
+      <%= check_box_tag "enterprise_fields[#{field}][required]", true, environment.custom_enterprise_field(field, 'required'), :onclick => "required_action('enterprise_fields[#{field}][active]','enterprise_fields[#{field}][required]', 'enterprise_fields[#{field}][signup]')" %>
     </td>
     <td>
       <%= hidden_field_tag  "enterprise_fields[#{field}][signup]", false %>
-      <%= check_box_tag "enterprise_fields[#{field}][signup]", true, environment.custom_enterprise_field(field, 'signup'), :onclick => "if(!this.checked) $('enterprise_fields[#{field}][required]').checked = false;" %>
+      <%= check_box_tag "enterprise_fields[#{field}][signup]", true, environment.custom_enterprise_field(field, 'signup'), :onclick => "signup_action('enterprise_fields[#{field}][active]','enterprise_fields[#{field}][required]', 'enterprise_fields[#{field}][signup]')" %>
     </td>
 
   </tr>
@@ -31,18 +47,18 @@
 </table>
 
 <script type='text/javascript'>
-  var trs = $$('#enterprise_fields_conf tr');
+  var trs = jQuery('#enterprise_fields_conf tr');
   var tr, td2;
-  for ( var i=0; tr=trs[i]; i++ ) {
+  for ( var i=2; tr=trs[i]; i++ ) {
     if ( td2 = tr.getElementsByTagName('td')[1] ) {
-      td2.getElementsByTagName('input')[0].onclick();
+      td2.getElementsByTagName('input')[1].onclick();
     }
   }
 </script>
 
 <div>
   <% button_bar do %>
-    <%= submit_button('save', _('Save changes')) %>
+    <%= submit_button('save', _('Save changes'), :id=>"save_enterprise_fields") %>
     <%= button :back, _('Back to admin panel'), :controller => 'admin_panel', :action => 'index' %>
   <% end %>
 </div>
diff --git a/app/views/features/_manage_person_fields.rhtml b/app/views/features/_manage_person_fields.rhtml
index 39f879b..2f5e31d 100644
--- a/app/views/features/_manage_person_fields.rhtml
+++ b/app/views/features/_manage_person_fields.rhtml
@@ -9,31 +9,48 @@
     <th><%= _('Required') %></th>
     <th><%= _('Display on signup?') %></th>
   </tr>
+
+  <tr style="border-bottom:solid 2px #000">
+    <td>
+      <%= _("Check/Uncheck All")%>
+    </td>
+    <td>
+      <input type="checkbox" id="person_active" />
+    </td>
+    <td>
+      <input type="checkbox" id="person_required" />
+    </td>
+    <td>
+      <input type="checkbox" id="person_signup" />
+    </td>
+  </tr>
+
   <% @person_fields.each do |field| %>
   <tr>
     <td><label for="person_fields[<%= field %>][active]"><%= _(field.humanize) %></label></td>
     <td>
       <%= hidden_field_tag  "person_fields[#{field}][active]", false %>
-      <%= check_box_tag "person_fields[#{field}][active]", true, environment.custom_person_field(field, 'active'), :onclick => "$('person_fields[#{field}][required]').disabled=$('person_fields[#{field}][signup]').disabled=!this.checked;" %>
+      <%= check_box_tag "person_fields[#{field}][active]", true, environment.custom_person_field(field, 'active'), :onclick => "active_action(this, 'person_fields[#{field}][required]', 'person_fields[#{field}][signup]')" %>
     </td>
     <td>
       <%= hidden_field_tag  "person_fields[#{field}][required]", false %>
-      <%= check_box_tag "person_fields[#{field}][required]", true, environment.custom_person_field(field, 'required'), :onclick => "if(this.checked) $('person_fields[#{field}][signup]').checked = true;" %>
+      <%= check_box_tag "person_fields[#{field}][required]", true, environment.custom_person_field(field, 'required'), :onclick => "required_action('person_fields[#{field}][active]','person_fields[#{field}][required]', 'person_fields[#{field}][signup]')" %>
     </td>
     <td>
       <%= hidden_field_tag  "person_fields[#{field}][signup]", false %>
-      <%= check_box_tag "person_fields[#{field}][signup]", true, environment.custom_person_field(field, 'signup'), :onclick => "if(!this.checked) $('person_fields[#{field}][required]').checked = false;" %>
+      <%= check_box_tag "person_fields[#{field}][signup]", true, environment.custom_person_field(field, 'signup'), :onclick => "signup_action('person_fields[#{field}][active]','person_fields[#{field}][required]', 'person_fields[#{field}][signup]')" %>
     </td>
   </tr>
   <% end %>
 </table>
 
 <script type='text/javascript'>// <!--
-  var trs = $$('#person_fields_conf tr');
+  var trs = jQuery('#person_fields_conf tr');
+
   var tr, td2;
-  for ( var i=0; tr=trs[i]; i++ ) {
+  for ( var i=2; tr=trs[i]; i++ ) {
     if ( td2 = tr.getElementsByTagName('td')[1] ) {
-      td2.getElementsByTagName('input')[0].onclick();
+      td2.getElementsByTagName('input')[1].onclick();
     }
   }
   // -->
@@ -41,7 +58,7 @@
 
 <div>
   <% button_bar do %>
-    <%= submit_button('save', _('Save changes')) %>
+    <%= submit_button('save', _('Save changes'), :id=>"save_person_fields") %>
     <%= button :back, _('Back to admin panel'), :controller => 'admin_panel', :action => 'index' %>
   <% end %>
 </div>
diff --git a/app/views/features/manage_fields.rhtml b/app/views/features/manage_fields.rhtml
index e6db213..5a66c7b 100644
--- a/app/views/features/manage_fields.rhtml
+++ b/app/views/features/manage_fields.rhtml
@@ -1,3 +1,5 @@
+<%= javascript_include_tag "manage-fields.js" %>
+
 <%= render :partial => 'manage_person_fields' %>
 
 <% if !environment.enabled?('disable_asset_enterprises') %>
diff --git a/features/manage_fields.feature b/features/manage_fields.feature
new file mode 100644
index 0000000..7d69546
--- /dev/null
+++ b/features/manage_fields.feature
@@ -0,0 +1,67 @@
+Feature: check all manage fields
+  As an administrator
+  I want to check and uncheck all person, enterprise and community's fields
+
+  Background:
+    Given the following users
+      | login      | name        |
+      | mariasilva | Maria Silva |
+    And the following enterprises
+      | identifier   | owner      | name         | contact_email               | contact_phone  | enabled |
+      | paper-street | mariasilva | Paper Street | marial.silva@workerbees.org | (288) 555-0153 | true    |
+    And the following community
+      | identifier  | name         |
+      | mycommunity | My Community |
+    And "Maria Silva" is admin of "My Community"
+    And I am logged in as admin
+    And I go to /admin/features/manage_fields
+
+  @selenium
+  Scenario: check all active person fields
+    Given I check "person_active"
+    And I press "save_person_fields"
+    And I follow "Control panel"
+    When I follow "Edit Profile"
+    Then I should see "Custom area of study"
+
+  @selenium
+  Scenario: check all active enterprise fields
+    Given I check "enterprise_active"
+    And I press "save_enterprise_fields"
+    When I go to /myprofile/paper-street/profile_editor/edit
+    Then I should see "Historic and current context"
+
+  @selenium
+  Scenario: check all active community fields
+    Given I check "community_active"
+    And I press "save_community_fields"
+    When I go to /myprofile/mycommunity/profile_editor/edit
+    Then I should see "Economic activity"
+
+  @selenium
+  Scenario: uncheck Check/Uncheck All active person field
+    Given I check "person_active"
+    And I press "save_person_fields"
+    And I uncheck "person_active"
+    And I press "save_person_fields"
+    And I follow "Control panel"
+    When I follow "Edit Profile"
+    Then I should not see "Custom area of study"
+
+  @selenium
+  Scenario: uncheck Check/Uncheck All active community field
+    Given I check "community_active"
+    And I press "save_community_fields"
+    And I uncheck "community_active"
+    And I press "save_community_fields"
+    When I go to /myprofile/mycommunity/profile_editor/edit
+    Then I should not see "Economic activity"
+
+  @selenium
+  Scenario: uncheck Check/Uncheck All active enterprise field
+    Given I check "enterprise_active"
+    And I press "save_enterprise_fields"
+    And I uncheck "enterprise_active"
+    And I press "save_enterprise_fields"
+    When I go to /myprofile/paper-street/profile_editor/edit
+    Then I should not see "Historic and current context"
\ No newline at end of file
diff --git a/public/javascripts/manage-fields.js b/public/javascripts/manage-fields.js
new file mode 100644
index 0000000..95e254b
--- /dev/null
+++ b/public/javascripts/manage-fields.js
@@ -0,0 +1,83 @@
+function update_active(name_active, name_required, name_signup) {
+  var required = jQuery("input[name='" + name_required + "']")[1]
+  var signup = jQuery("input[name='" + name_signup + "']")[1]
+  var active = jQuery("input[name='" + name_active + "']")[1]
+
+  if(required.checked || signup.checked)
+    active.checked = true
+}
+
+function active_action(obj_active, name_required, name_signup) {
+  var required = jQuery("input[name='" + name_required + "']")[0]
+  var signup = jQuery("input[name='" + name_signup + "']")[0]
+
+  required.disabled = signup.disabled = !obj_active.checked
+}
+
+function required_action(name_active, name_required, name_signup) {
+  var obj_required = jQuery("input[name='" + name_required + "']")[1]
+
+  if(obj_required.checked) {
+    jQuery("input[name='" + name_signup + "']")[0].checked = true
+    jQuery("input[name='" + name_signup + "']")[1].checked = true
+  }
+
+  update_active(name_active, name_required, name_signup)
+}
+
+function signup_action(name_active, name_required, name_signup) {
+  var obj_signup = jQuery("input[name='" + name_signup + "']")[1]
+
+  if(!obj_signup.checked) {
+    jQuery("input[name='" + name_required + "']")[0].checked = false
+    jQuery("input[name='" + name_required + "']")[1].checked = false
+  }
+
+  update_active(name_active, name_required, name_signup)
+}
+
+
+jQuery(document).ready(function(){
+  function check_fields(check, table_id, start) {
+    var checkboxes = jQuery("#" + table_id + " tbody tr td input[type='checkbox']")
+    for (var i = start; i < checkboxes.length; i+=3) {
+      checkboxes[i].checked = check
+    }
+  }
+
+  function verify_checked(fields_id){
+    var checkboxes = jQuery("#" + fields_id + "_fields_conf tbody tr td input[type='checkbox']")
+    for (var i = 2; i >= 0; i--) {
+      var allchecked = true
+      for (var j = i+3; j < checkboxes.length; j+=3) {
+        if(!checkboxes[j].checked) {
+          allchecked = false
+          break
+        }
+      }
+
+      var checkbox = jQuery(checkboxes[i+3]).attr("id").split("_")
+      jQuery("#" + checkbox.first() + "_" + checkbox.last()).attr("checked", allchecked)
+    }
+  }
+
+  function check_all(fields_id) {
+    jQuery("#" + fields_id + "_active").click(function (){check_fields(this.checked, fields_id + "_fields_conf", 0)})
+    jQuery("#" + fields_id + "_required").click(function (){check_fields(this.checked, fields_id + "_fields_conf", 1)})
+    jQuery("#" + fields_id +"_signup").click(function (){check_fields(this.checked, fields_id + "_fields_conf", 2)})
+    verify_checked(fields_id)
+  }
+
+  check_all("person")
+  check_all("enterprise")
+  check_all("community")
+
+  jQuery("input[type='checkbox']").click(function (){
+    var checkbox = jQuery(this).attr("id").split("_")
+    verify_checked(checkbox.first())
+
+    if(this.checked == false) {
+      jQuery("#" + checkbox.first() + "_" + checkbox.last()).attr("checked", false)
+    }
+  })
+})
--
libgit2 0.21.2