Expose $.fn.taskList instead

Author: mtodd

app/assets/javascripts/task_lists.coffee

@@ -179,8 +179,8 @@ enableTaskList = ($container) ->
       trigger 'tasklist:enabled'
 
 # Enables a collection of TaskList containers.
-$.fn.enableTaskList = ->
-  for container in this
+enableTaskLists = ($containers) ->
+  for container in $containers
     enableTaskList $(container)
 
 # Disable TaskList item changes.
@@ -192,10 +192,19 @@ disableTaskList = ($container) ->
     trigger 'tasklist:disabled'
 
 # Disables a collection of TaskList containers.
-$.fn.disableTaskList = ->
-  for container in this
+disableTaskLists = ($containers) ->
+  for container in $containers
     disableTaskList $(container)
 
+$.fn.taskList = (method) ->
+  $container = $(this).closest('.js-task-list-container')
+
+  methods =
+    enable: enableTaskLists
+    disable: disableTaskLists
+
+  methods[method || 'enable']($container)
+
 # When the page is updated, enable new TaskList containers.
 $.pageUpdate ->
-  $('.js-task-list-container.js-task-list-enable').enableTaskList()
+  $('.js-task-list-container.js-task-list-enable').taskList()

test/functional/test_task_lists_behavior.html

@@ -18,7 +18,7 @@
     $(document).on('tasklist:disabled', logEvent)
     $(document).on('tasklist:change', '.js-task-list-field', function(event){
       logEvent(event)
-      $(this).closest('.js-task-list-container').disableTaskList()
+      $(this).closest('.js-task-list-container').taskList('disable')
     })
     $(document).on('tasklist:changed', '.js-task-list-field', function(event){
       logEvent(event)
@@ -28,7 +28,7 @@
     $(document).on('ajaxStart', logEvent)
     $(document).on('ajaxSuccess', '.js-task-list-container', function(event){
       logEvent(event)
-      $(this).enableTaskList()
+      $(this).taskList()
     })
   </script>
 

test/unit/test_events.coffee

@@ -21,7 +21,7 @@ module "TaskList events",
     @container.append @field
 
     $('#qunit-fixture').append(@container)
-    @container.enableTaskList()
+    @container.taskList()
 
   teardown: ->
     $(document).off 'tasklist:enabled'
@@ -77,7 +77,7 @@ asyncTest "enables task list items when a .js-task-list-field is present", ->
   $(document).on 'tasklist:enabled', (event) ->
     ok true
 
-  @container.enableTaskList()
+  @container.taskList()
   setTimeout ->
     start()
   , 20
@@ -90,7 +90,7 @@ asyncTest "doesn't enable task list items when a .js-task-list-field is absent",
 
   @field.remove()
 
-  @container.enableTaskList()
+  @container.taskList()
   setTimeout ->
     start()
   , 20

test/unit/test_updates.coffee

@@ -47,7 +47,7 @@ module "TaskList updates",
     @container.append @field
 
     $('#qunit-fixture').append(@container)
-    @container.enableTaskList()
+    @container.taskList()
 
   teardown: ->
     $(document).off 'tasklist:changed'