make selection machinery pluggable so that for instance we can use an ID based mechanism

Author: reiern70

wicketstuff-jquery-ui-parent/wicketstuff-jquery-ui/src/main/java/org/wicketstuff/jquery/ui/form/autocomplete/AbstractAutoCompleteTextField.java

@@ -16,6 +16,7 @@
  */
 package org.wicketstuff.jquery.ui.form.autocomplete;
 
+import java.util.Collections;
 import java.util.List;
 import java.util.Locale;
 import org.apache.wicket.ajax.AjaxRequestTarget;
@@ -328,7 +329,12 @@ protected void onSelected(AjaxRequestTarget target)
 	@Override
 	public JQueryBehavior newWidgetBehavior(String selector)
 	{
-		return new AutoCompleteBehavior(selector, this, getModel()) { // NOSONAR
+		return new AutoCompleteBehavior<T>(selector, this, new IModel<List<T>>() {
+            @Override
+            public List<T> getObject() {
+                return choices != null ? choices.getObject() : Collections.emptyList();
+            }
+        }) { // NOSONAR
 
 			private static final long serialVersionUID = 1L;
 

wicketstuff-jquery-ui-parent/wicketstuff-jquery-ui/src/main/java/org/wicketstuff/jquery/ui/form/autocomplete/AutoCompleteBehavior.java

@@ -43,7 +43,9 @@ public abstract class AutoCompleteBehavior<T> extends JQueryUIBehavior implement
 	/** event listener */
 	private final IAutoCompleteListener<T> listener;
 
-    private final IModel<List<T>> model;
+    /** the model producing values */
+
+    private final IModel<List<T>> valuesModel;
 
 	private JQueryAjaxBehavior onSelectAjaxBehavior = null;
 
@@ -53,9 +55,9 @@ public abstract class AutoCompleteBehavior<T> extends JQueryUIBehavior implement
 	 * @param selector the HTML selector (ie: "#myId")
 	 * @param listener the {@link IAutoCompleteListener}
 	 */
-	public AutoCompleteBehavior(String selector, IAutoCompleteListener<T> listener, IModel<List<T>> model)
+	public AutoCompleteBehavior(String selector, IAutoCompleteListener<T> listener, IModel<List<T>> valuesModel)
 	{
-		this(selector, new Options(), listener, model);
+		this(selector, new Options(), listener, valuesModel);
 	}
 
 	/**
@@ -65,12 +67,12 @@ public AutoCompleteBehavior(String selector, IAutoCompleteListener<T> listener,
 	 * @param options the {@link Options}
 	 * @param listener the {@link IAutoCompleteListener}
 	 */
-	public AutoCompleteBehavior(String selector, Options options, IAutoCompleteListener<T> listener, IModel<List<T>> model)
+	public AutoCompleteBehavior(String selector, Options options, IAutoCompleteListener<T> listener, IModel<List<T>> valuesModel)
 	{
 		super(selector, METHOD, options);
 
 		this.listener = Args.notNull(listener, "listener");
-        this.model = model;
+        this.valuesModel = valuesModel;
     }
 
 	// Methods //
@@ -116,7 +118,7 @@ public void onAjax(AjaxRequestTarget target, JQueryEvent event)
 	{
 		if (event instanceof SelectEvent selectEvent)
 		{
-			this.listener.onSelect(target, listener.getElementSelectionStrategy().findChoice(model.getObject(), selectEvent.getIdentifier()));
+			this.listener.onSelect(target, listener.getElementSelectionStrategy().findChoice(valuesModel.getObject(), selectEvent.getIdentifier()));
 		}
 	}