Ticket #6690: dijit.form.ComboBox_20080507.patch

form/ComboBox.js

@@ -297 +297 @@
                                 // if they are just previewing the options available.
                                 this._autoCompleteText(zerothvalue);
                         }
+                        dataObject._maxOptions = this._maxOptions;
                         this._popupWidget.createOptions(
                                 results, 
                                 dataObject, 
@@ -502 +503 @@
                                                 deep: true
                                         },
                                         query: query,
+                                        onBegin: dojo.hitch(this, "_setMaxOptions"),
                                         onComplete: dojo.hitch(this, "_openResultList"), 
                                         onError: function(errText){
                                                 console.error('dijit.form.ComboBox: ' + errText);
@@ -523 +525 @@
                         }, query, this), this.searchDelay);
                 },
 
+                _setMaxOptions: function(size, request){
+                         this._maxOptions = size;
+                },
+
                 _getValueField:function(){
                         return this.searchAttr;
                 },
@@ -701 +707 @@
                         //this._dataObject=dataObject;
                         //this._dataObject.onComplete=dojo.hitch(comboBox, comboBox._openResultList);
                         // display "Previous . . ." button
+                        var displayMore = false;
+                        //Try to determine if we should show 'more'...
+                        if(dataObject._maxOptions && dataObject._maxOptions != -1){
+                                if((dataObject.start + dataObject.count) < dataObject._maxOptions){
+                                        displayMore = true;
+                                }else if((dataObject.start + dataObject.length) > (dataObject._maxOptions - 1)){
+                                        //Weird return from a datastore, where a start + count > maxOptions
+                                        //implies maxOptions isn't really valid and we have to go into faking it.
+                                        //And more or less assume more if count == results.length
+                                        if(dataObject.count == results.length){
+                                                displayMore = true;
+                                        }
+                                }
+                        }else if(dataObject.count == results.length){
+                                //Don't know the size, so we do the best we can based off count alone.
+                                //So, if we have an exact match to count, assume more.
+                                displayMore = true;
+                        }
+
                         this.previousButton.style.display = (dataObject.start == 0) ? "none" : "";
                         dojo.attr(this.previousButton, "id", this.id + "_prev");
                         // create options using _createOption function defined by parent
@@ -714 +739 @@
                                 this.domNode.insertBefore(menuitem, this.nextButton);
                         }, this);
                         // display "Next . . ." button
-                        this.nextButton.style.display = (dataObject.count == results.length) ? "" : "none";
-                        dojo.attr(this.nextButton,"id", this.id + "_next")
+                        this.nextButton.style.display = displayMore ? "" : "none";
+                        dojo.attr(this.nextButton,"id", this.id + "_next");
                 },
 
                 clearResultList: function(){
@@ -1010 +1035 @@
 
                 var start = args.start || 0,
                         end = ("count" in args && args.count != Infinity) ? (start + args.count) : items.length ;
+                args.onBegin(items.length, args);
                 args.onComplete(items.slice(start, end), args);
                 return args; // Object
                 // TODO: I don't need to return the length?