From e50b3992f552fc5c845faaf9d86e1eb2e21eed86 Mon Sep 17 00:00:00 2001
From: Sascha Grossenbacher <saschagros@gmail.com>
Date: Wed, 6 Jul 2011 12:41:38 +0200
Subject: [PATCH] Issue #1054322 by Berdir: Improved language for vertical tabs and summaries.

---
 userpoints_nodeaccess.js     |   21 ++++++++++++++++++---
 userpoints_nodeaccess.module |   11 +++++++++--
 2 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/userpoints_nodeaccess.js b/userpoints_nodeaccess.js
index 08d2293..78df0a4 100644
--- a/userpoints_nodeaccess.js
+++ b/userpoints_nodeaccess.js
@@ -24,14 +24,29 @@
               '@points': $('#edit-userpoints-nodeaccess-points-price', context).val(),
               '@category': $('select#edit-userpoints-nodeaccess-points-category :selected', context).text()
             };
-            return Drupal.t('Userpoints node access enabled. @points in category @category required.', params);
+            return Drupal.t('Enabled, @points in category @category required.', params);
           }
           else {
-            return Drupal.t('Userpoints node access enabled. No points required by default.');
+            return Drupal.t('Enabled, no points required by default.');
           }
         }
         else {
-          return Drupal.t('Userpoints node access not enabled.');
+          return Drupal.t('Not enabled.');
+        }
+      });
+    }
+  };
+  Drupal.behaviors.userpointsNodeAccessSettingsFieldsetSummaries = {
+    attach: function (context) {
+      $('fieldset.userpoints-nodeaccess-settings-form', context).drupalSetSummary(function (context) {
+        if ($('#edit-userpoints-nodeaccess-enabled').is(':checked')) {
+          var params = {
+            '@category': $('select#edit-userpoints-nodeaccess-points-category :selected', context).text()
+          };
+          return Drupal.t('Enabled by default.');
+        }
+        else {
+          return Drupal.t('Not enabled by default.');
         }
       });
     }
diff --git a/userpoints_nodeaccess.module b/userpoints_nodeaccess.module
index 9077761..06a9f65 100644
--- a/userpoints_nodeaccess.module
+++ b/userpoints_nodeaccess.module
@@ -57,6 +57,13 @@ function userpoints_nodeaccess_userpoints($op, $points = NULL) {
       '#collapsed' => TRUE,
       '#collapsible' => TRUE,
       '#group' => 'settings_additional',
+      '#weight' => 30,
+      '#attributes' => array('class' => array('userpoints-nodeaccess-settings-form')),
+      '#attached' => array(
+        'js' => array(
+          'userpoints_nodeaccess' => drupal_get_path('module', 'userpoints_nodeaccess') . '/userpoints_nodeaccess.js',
+        ),
+      ),
     );
 
     $form['userpoints_nodeaccess']['userpoints_nodeaccess_enabled'] = array(
@@ -86,7 +93,7 @@ function userpoints_nodeaccess_form_alter(&$form, $form_state, $form_id) {
   if (!empty($form['#node_edit_form']) && variable_get('userpoints_nodeaccess_enabled_' . $form['#node']->type, variable_get('userpoints_nodeaccess_enabled', TRUE))) {
     $form['userpoints_nodeaccess'] = array(
       '#type' => 'fieldset',
-      '#title' => t('Content access by !points', userpoints_translation()),
+      '#title' => t('!Points for content access', userpoints_translation()),
       '#collapsible' => TRUE,
       '#collapsed' => FALSE,
       // Check if user either has permission for all nodes or for own nodes and
@@ -120,7 +127,7 @@ function userpoints_nodeaccess_form_alter(&$form, $form_state, $form_id) {
 function userpoints_nodeaccess_form_node_type_form_alter(&$form, &$form_state) {
   $form['userpoints_nodeaccess'] = array(
     '#type' => 'fieldset',
-    '#title' => t('Content access by !points', userpoints_translation()),
+    '#title' => t('!Points for content access', userpoints_translation()),
     '#collapsible' => TRUE,
     '#collapsed' => FALSE,
     '#access' => user_access('administer userpoints'),
-- 
1.7.4.1