From 5c3cb11b5b7807cb836492df1e817bd9c47a274c Mon Sep 17 00:00:00 2001
From: Matt Todd <chiology@gmail.com>
Date: Sun, 21 Sep 2014 16:41:38 -0700
Subject: [PATCH 1/3] :nail_care: whitespace

---
 test/unit/test_updates.coffee | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/test/unit/test_updates.coffee b/test/unit/test_updates.coffee
index 4c653a0..c4e7975 100644
--- a/test/unit/test_updates.coffee
+++ b/test/unit/test_updates.coffee
@@ -28,7 +28,7 @@ module "TaskList updates",
       class: 'task-list-item-checkbox'
       disabled: true
       checked: false
-      
+
     @blockquote = $ '<blockquote>'
 
     @quotedList = $ '<ul>', class: 'task-list'
@@ -46,7 +46,7 @@ module "TaskList updates",
       class: 'task-list-item-checkbox'
       disabled: true
       checked: false
-      
+
     @innerBlockquote = $ '<blockquote>'
 
     @innerList = $ '<ul>', class: 'task-list'
@@ -232,7 +232,7 @@ module "TaskList updates",
 
     @blockquote.append @innerBlockquote
 
-    @container.append @blockquote 
+    @container.append @blockquote
 
     @orderedCompleteItem.append @orderedCompleteCheckbox
     @orderedList.append @orderedCompleteItem
@@ -241,9 +241,9 @@ module "TaskList updates",
     @orderedIncompleteItem.append @orderedIncompleteCheckbox
     @orderedList.append @orderedIncompleteItem
     @orderedIncompleteItem.expectedIndex = 9
-    
+
     @container.append @orderedList
-    
+
     @blockquote.append @field
 
     $('#qunit-fixture').append(@container)

From 0cee4bf2a151062b6bdf689a5570d6fe632cd745 Mon Sep 17 00:00:00 2001
From: Matt Todd <chiology@gmail.com>
Date: Sun, 21 Sep 2014 16:41:52 -0700
Subject: [PATCH 2/3] Add failing test for list prefix requirement

---
 test/unit/test_updates.coffee | 61 +++++++++++++++++++++++++++++++++++
 1 file changed, 61 insertions(+)

diff --git a/test/unit/test_updates.coffee b/test/unit/test_updates.coffee
index c4e7975..1f7c595 100644
--- a/test/unit/test_updates.coffee
+++ b/test/unit/test_updates.coffee
@@ -379,3 +379,64 @@ asyncTest "updates the source of an ordered list item, marking the complete item
 
   @orderedCompleteCheckbox.click()
 
+asyncTest "update ignores items that look like Task List items but lack list prefix", ->
+  expect 3
+
+  $('#qunit-fixture').empty()
+
+  container = $ '<div>', class: 'js-task-list-container'
+
+  list = $ '<ul>', class: 'task-list'
+
+  item1 = $ '<li>', class: 'task-list-item'
+  item1Checkbox = $ '<input>',
+    type: 'checkbox'
+    class: 'task-list-item-checkbox'
+    disabled: true
+    checked: false
+
+  item2 = $ '<li>', class: 'task-list-item'
+  item2Checkbox = $ '<input>',
+    type: 'checkbox'
+    class: 'task-list-item-checkbox'
+    disabled: true
+    checked: false
+
+  field = $ '<textarea>', class: 'js-task-list-field', text: """
+    [ ] one
+    [ ] two
+    - [ ] three
+    - [ ] four
+  """
+
+  changes = """
+    [ ] one
+    [ ] two
+    - [ ] three
+    - [x] four
+  """
+
+  item1.append item1Checkbox
+  list.append item1
+  item1.expectedIndex = 1
+
+  item2.append item2Checkbox
+  list.append item2
+  item2.expectedIndex = 2
+
+  container.append list
+  container.append field
+
+  $('#qunit-fixture').append(container)
+  container.taskList()
+
+  field.on 'tasklist:changed', (event, index, checked) =>
+    ok checked
+    equal index, item2.expectedIndex
+    equal field.val(), changes
+
+  setTimeout ->
+    start()
+  , 20
+
+  item2Checkbox.click()

From fc0b015584598b965a4dfc0c77fd076f7f3c12e4 Mon Sep 17 00:00:00 2001
From: Matt Todd <chiology@gmail.com>
Date: Sun, 21 Sep 2014 19:19:27 -0700
Subject: [PATCH 3/3] List prefix is not optional

---
 app/assets/javascripts/task_list.coffee | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/task_list.coffee b/app/assets/javascripts/task_list.coffee
index d7b76c0..8996cab 100644
--- a/app/assets/javascripts/task_list.coffee
+++ b/app/assets/javascripts/task_list.coffee
@@ -115,11 +115,11 @@ completePattern = ///
 # Useful when you need iterate over all items.
 itemPattern = ///
   ^
-  (?:                     # optional prefix, consisting of
+  (?:                     # prefix, consisting of
     \s*                   # optional leading whitespace
     (?:>\s*)*             # zero or more blockquotes
     (?:[-+*]|(?:\d+\.))   # list item indicator
-  )?
+  )
   \s*                     # optional whitespace prefix
   (                       # checkbox
     #{escapePattern(complete)}|