Changeset 13726

Timestamp:

05/13/08 10:11:14 (2 months ago)

Author:

bill

Message:

Fix detection of whether there's more data to fetch or not. Fixes #6690.
Patch from Jared (IBM, CCLA).

!strict

Files:

• dijit/trunk/form/ComboBox.js (modified) (4 diffs)

dijit/trunk/form/ComboBox.js

@@ -320 +320 @@
                                 this._autoCompleteText(zerothvalue);
                         }
+                        dataObject._maxOptions = this._maxOptions;
                         this._popupWidget.createOptions(
                                 results, 
@@ -526 +527 @@
                                         },
                                         query: query,
+                                        onBegin: dojo.hitch(this, "_setMaxOptions"),
                                         onComplete: dojo.hitch(this, "_openResultList"), 
                                         onError: function(errText){
@@ -545 +547 @@
                                 this._nextSearch = this._popupWidget.onPage = dojo.hitch(this, nextSearch, dataObject);
                         }, query, this), this.searchDelay);
+                },
+
+                _setMaxOptions: function(size, request){
+                         this._maxOptions = size;
                 },
 
@@ -768 +774 @@
                         }, this);
                         // display "Next . . ." button
-                        this.nextButton.style.display = (dataObject.count == results.length) ? "" : "none";
-                        dojo.attr(this.nextButton,"id", this.id + "_next")
+                        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.nextButton.style.display = displayMore ? "" : "none";
+                        dojo.attr(this.nextButton,"id", this.id + "_next");
                 },