add dataValue option to populate with the item in addition to the value option

Author: rniemeyer

README.md

@@ -14,6 +14,8 @@ knockout-jqAutocomplete
 
 * **valueProp** - If specified, this is a property name to use to populate the value when a selection is made. If not specified, the actual item itself will be used as the value.
 
+* **dataValue** - If specified, this observable will be populated with the currently selected data item. This option will allow you to populate **value** with the **valueProp** and have access to the selected object at the same time through **dataValue**.
+
 * **template** - If specified, this is the name of a template to use when building each item in the menu choices. This allows full customization of the options shown to the user.
 
 * **options** - Anything passed in `options` will be included as options passed when initializing the `autocomplete` widget. See http://api.jqueryui.com/autocomplete/ for a list of the available options.

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "knockout-jqAutocomplete",
-  "version": "0.1.3",
+  "version": "0.2.0",
   "main": "build/knockout-jqAutocomplete.min.js",
   "ignore": [
     "examples",

build/knockout-jqAutocomplete.js

@@ -1,4 +1,4 @@
-// knockout-jqAutocomplete 0.1.3 | (c) 2013 Ryan Niemeyer |  http://www.opensource.org/licenses/mit-license
+// knockout-jqAutocomplete 0.2.0 | (c) 2013 Ryan Niemeyer |  http://www.opensource.org/licenses/mit-license
 ;(function(factory) {
     if (typeof define === "function" && define.amd) {
         // AMD anonymous module
@@ -43,6 +43,10 @@
             //handle updating the actual value
             config.select = function(event, ui) {
                 options.value(ui.item && ui.item.actual);
+
+                if (ko.isWriteableObservable(options.dataValue)) {
+                    options.dataValue(ui.item.data);
+                }
             };
 
             //user made a change without selecting a value from the list

build/knockout-jqAutocomplete.min.js

@@ -1,6 +1,6 @@
 {
   "name": "knockout-jqAutocomplete",
-  "version": "0.1.3",
+  "version": "0.2.0",
   "devDependencies": {
     "grunt": "~0.4.1",
     "grunt-contrib-uglify": "0.x.x",

package.json

@@ -785,6 +785,44 @@ describe("knockout-jqAutocomplete", function(){
             expect($input.text()).toEqual("one description");
         });
 
+        it("should respect the dataValue option on selection", function() {
+            var $listItems,
+                items = [
+                    {
+                        name: "one",
+                        description: "one description"
+                    },
+                    {
+                        name: "two",
+                        description: "two description"
+                    },
+                    {
+                        name: "three",
+                        description: "three description"
+                    }
+                ],
+                value = ko.observable(),
+                dataValue = ko.observable();
+
+            ko.applyBindingsToNode(input, {
+                jqAuto: {
+                    value: value,
+                    dataValue: dataValue,
+                    source: items,
+                    valueProp: "name"
+                }
+            });
+
+            $input.autocomplete("search", "one");
+
+            $listItems = $("ul.ui-autocomplete li");
+
+            $listItems.first("a").click();
+
+            expect(value()).toEqual("one");
+            expect(dataValue()).toEqual(items[0]);
+        });
+
         it("should initially set the input's value", function() {
             var items = [],
                 value = ko.observable("testing");

spec/knockout-jqAutocomplete.spec.js

@@ -42,6 +42,10 @@
             //handle updating the actual value
             config.select = function(event, ui) {
                 options.value(ui.item && ui.item.actual);
+
+                if (ko.isWriteableObservable(options.dataValue)) {
+                    options.dataValue(ui.item.data);
+                }
             };
 
             //user made a change without selecting a value from the list