- Fix pageNumber bug causing index of -1 when pageCount == 0.

- Fix bug causing double refresh when setting but not changing pageNumber inside refresh()

Author: jehhynes

Griddly.NetCore/wwwroot/js/griddly.js

@@ -769,18 +769,18 @@
                 event.preventDefault();
             }, this));
 
-            $("input.pageNumber", this.$element).on("change", $.proxy(function (event)
-            {
+            $("input.pageNumber", this.$element).on("change", $.proxy(function (event) {
                 var value = parseInt($(event.target).val());
 
-                if (value < 1)
+                if (value < 1 || this.options.pageCount == 0)
                     value = 1;
                 else if (value > this.options.pageCount)
                     value = this.options.pageCount;
 
-                this.options.pageNumber = value - 1;
-
-                this.refresh();
+                if (this.options.pageNumber != value - 1) {
+                    this.options.pageNumber = value - 1;
+                    this.refresh();
+                }
             }, this));
 
             $("select.pageSize", this.$element).on("change", $.proxy(function (event)

Griddly/Scripts/griddly.js

@@ -773,14 +773,15 @@
             {
                 var value = parseInt($(event.target).val());
 
-                if (value < 1)
+                if (value < 1 || this.options.pageCount == 0)
                     value = 1;
                 else if (value > this.options.pageCount)
                     value = this.options.pageCount;
 
-                this.options.pageNumber = value - 1;
-
-                this.refresh();
+                if (this.options.pageNumber != value - 1) {
+                    this.options.pageNumber = value - 1;
+                    this.refresh();
+                }
             }, this));
 
             $("select.pageSize", this.$element).on("change", $.proxy(function (event)