[ckan-changes] [okfn/ckan] 6a2793: [#2304] Use jQuery .on() and .off() for follow but...

• Previous message: [ckan-changes] [okfn/ckan] 289a36: Add additional bold variant of the Ubuntu font fam...
• Next message: [ckan-changes] [okfn/ckan] 42c16b: [#2304] Javascript code formatting tweaks
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

  Branch: refs/heads/feature-2304-follow
  Home:   https://github.com/okfn/ckan
  Commit: 6a279370998fe095acf089d58b390f63fc10b473
      https://github.com/okfn/ckan/commit/6a279370998fe095acf089d58b390f63fc10b473
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2012-04-30 (Mon, 30 Apr 2012)

  Changed paths:
    M ckan/public/scripts/application.js

  Log Message:
  -----------
  [#2304] Use jQuery .on() and .off() for follow button events


diff --git a/ckan/public/scripts/application.js b/ckan/public/scripts/application.js
index 109b073..742644c 100644
--- a/ckan/public/scripts/application.js
+++ b/ckan/public/scripts/application.js
@@ -1343,97 +1343,107 @@ CKAN.Utils = function($, my) {
     return count;
   };
 
+  function userFollowClicked() {
+    $.ajax({
+      contentType: 'application/json',
+      url: '/api/action/follower_create',
+      data: JSON.stringify({
+        object_id: this.attributes.userid.nodeValue,
+        object_type: 'user',
+      }),
+      dataType: 'json',
+      processData: false,
+      type: 'POST',
+      success: function(data) {
+        $('#user_follow_button').attr('state', 'unfollow');
+        my.setupUserFollowButton();
+      },
+    });
+    return false;
+  };
+
+  function userUnfollowClicked() {
+    $.ajax({
+      contentType: 'application/json',
+      url: '/api/action/follower_delete',
+      data: JSON.stringify({
+        id: this.attributes.userid.nodeValue,
+      }),
+      dataType: 'json',
+      processData: false,
+      type: 'POST',
+      success: function(data) {
+        $('#user_follow_button').attr('state', 'follow');
+        my.setupUserFollowButton();
+      },
+    });
+    return false;
+  };
+
+  function datasetFollowClicked() {
+    $.ajax({
+      contentType: 'application/json',
+      url: '/api/action/follower_create',
+      data: JSON.stringify({
+        object_id: this.attributes.package_id.nodeValue,
+        object_type: 'dataset',
+      }),
+      dataType: 'json',
+      processData: false,
+      type: 'POST',
+      success: function(data) {
+        $('#dataset_follow_button').attr('state', 'unfollow');
+        my.setupDatasetFollowButton();
+      },
+    });
+    return false;
+  };
+
+  function datasetUnfollowClicked() {
+    $.ajax({
+      contentType: 'application/json',
+      url: '/api/action/follower_delete',
+      data: JSON.stringify({
+        id: this.attributes.package_id.nodeValue,
+      }),
+      dataType: 'json',
+      processData: false,
+      type: 'POST',
+      success: function(data) {
+        $('#dataset_follow_button').attr('state', 'follow');
+        my.setupDatasetFollowButton();
+      },
+    });
+    return false;
+  };
+
   my.setupUserFollowButton = function() {
-    $('#user_follow_button').unbind();
     if ($('#user_follow_button').attr('state') == 'follow')
     {
+        $('#user_follow_button').off("click", userUnfollowClicked);
         $('#user_follow_button').html('Follow');
-        $('#user_follow_button').click(function(e) {
-            $.ajax({
-                contentType: 'application/json',
-                url: '/api/action/follower_create',
-                data: JSON.stringify({
-                    object_id: this.attributes.userid.nodeValue,
-                    object_type: 'user',
-                }),
-                dataType: 'json',
-                processData: false,
-                type: 'POST',
-                success: function(data) {
-                    $('#user_follow_button').attr('state', 'unfollow');
-                    my.setupUserFollowButton();
-                },
-            });
-            return false;
-        });
+        $('#user_follow_button').on("click", userFollowClicked);
     }
     else
     {
+        $('#user_follow_button').off("click", userFollowClicked);
         $('#user_follow_button').html('Unfollow');
-        $('#user_follow_button').click(function(e) {
-            $.ajax({
-                contentType: 'application/json',
-                url: '/api/action/follower_delete',
-                data: JSON.stringify({
-                    id: this.attributes.userid.nodeValue,
-                }),
-                dataType: 'json',
-                processData: false,
-                type: 'POST',
-                success: function(data) {
-                    $('#user_follow_button').attr('state', 'follow');
-                    my.setupUserFollowButton();
-                },
-            });
-            return false;
-        });
+        $('#user_follow_button').on("click", userUnfollowClicked);
     }
   };
 
   my.setupDatasetFollowButton = function() {
-    $('#dataset_follow_button').unbind();
     if ($('#dataset_follow_button').attr('state') == 'follow')
     {
+        $('#dataset_follow_button').off("click", datasetUnfollowClicked);
         $('#dataset_follow_button').html('Follow');
-        $('#dataset_follow_button').click(function(e) {
-            $.ajax({
-                contentType: 'application/json',
-                url: '/api/action/follower_create',
-                data: JSON.stringify({
-                    object_id: this.attributes.package_id.nodeValue,
-                    object_type: 'dataset',
-                }),
-                dataType: 'json',
-                processData: false,
-                type: 'POST',
-                success: function(data) {
-                    $('#dataset_follow_button').attr('state', 'unfollow');
-                    my.setupDatasetFollowButton();
-                },
-            });
-            return false;
-        });
+        $('#dataset_follow_button').on("click", datasetFollowClicked);
     }
     else
     {
+        $('#dataset_follow_button').off("click", datasetFollowClicked);
         $('#dataset_follow_button').html('Unfollow');
-        $('#dataset_follow_button').click(function(e) {
-            $.ajax({
-                contentType: 'application/json',
-                url: '/api/action/follower_delete',
-                data: JSON.stringify({
-                    id: this.attributes.package_id.nodeValue,
-                }),
-                dataType: 'json',
-                processData: false,
-                type: 'POST',
-                success: function(data) {
-                    $('#dataset_follow_button').attr('state', 'follow');
-                    my.setupDatasetFollowButton();
-                },
-            });
-            return false;
-        });
+        $('#dataset_follow_button').on("click", datasetUnfollowClicked);
     }
   };
 


================================================================

• Previous message: [ckan-changes] [okfn/ckan] 289a36: Add additional bold variant of the Ubuntu font fam...
• Next message: [ckan-changes] [okfn/ckan] 42c16b: [#2304] Javascript code formatting tweaks
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]