Merge pull request #501 from ocadotechnology/438_autocomplete-picker-…

…event

fix(autocomplete-picker): Update change event to emit on undefined value

docs/content/docs/autocomplete-picker/api/AutocompletePicker/events/change.um

@@ -1,4 +1,10 @@
 @event change [String]
+  @bugfix nextReleaseVersion
+    @issue 438
+    @description
+      Updated the AutocompletePicker to update the value when setting the value
+      to @code[undefined]
+
   @description
     The event called whenever the input text is updated by the auto
     complete object. The data is the value of the input field.

modules/autocomplete-picker/main/index.coffee

@@ -21,15 +21,16 @@ setPickerValue = (picker, results, cause) ->
   _.valueText.clear()
   if results.length
     _.current = results[0]
-    picker.emit('change', {
-      cause: cause
-      value: results[0]
-    })
     _.renderer _.valueText.node(), results[0]
   else
     _.current = undefined
     _.valueText.text(_.options.chooseValueText)
 
+  picker.emit('change', {
+    cause: cause
+    value: _.current
+  })
+
 class AutocompletePicker extends hx.EventEmitter
   constructor: (selector, items, options = {}) ->
     super()

modules/autocomplete-picker/test/spec.coffee

@@ -502,6 +502,18 @@ describe 'autocomplete-picker', ->
       showstart.should.have.been.called.once()
 
 
+    it 'should emit the "change" event when the value is changed to undefined', ->
+      testClosedAutocomplete trivialItems, { value: 'a' }, (ap) ->
+        value = chai.spy()
+        ap.on 'change', value
+        ap.value(undefined)
+        value.should.have.been.called.with({
+          cause: 'api',
+          value: undefined
+        })
+        value.should.have.been.called.once()
+
+
     it 'should emit the "change" event when the value is changed', ->
       testClosedAutocomplete trivialItems, undefined, (ap) ->
         value = chai.spy()