Fixed issue where pagination total number of pages was based on the full location set total instead of the storeLimit setting

Author: bjorn2404

dist/assets/js/plugins/storeLocator/jquery.storelocator.js

@@ -1,4 +1,4 @@
-/*! jQuery Google Maps Store Locator - v2.6.2 - 2016-07-16
+/*! jQuery Google Maps Store Locator - v2.6.2 - 2016-07-19
 * http://www.bjornblog.com/web/jquery-store-locator-plugin
 * Copyright (c) 2016 Bjorn Holine; Licensed MIT */
 
@@ -869,9 +869,16 @@
 		paginationSetup: function (currentPage) {
 			this.writeDebug('paginationSetup',arguments);
 			var pagesOutput = '';
-			var totalPages = locationset.length / this.settings.locationsPerPage;
+			var totalPages;
 			var $paginationList = $('.bh-sl-pagination-container .bh-sl-pagination');
 
+			// Total pages
+			if ( this.settings.storeLimit === -1 || locationset.length < this.settings.storeLimit ) {
+				totalPages = locationset.length / this.settings.locationsPerPage;
+			} else {
+				totalPages = this.settings.storeLimit / this.settings.locationsPerPage;
+			}
+
 			// Current page check
 			if (typeof currentPage === 'undefined') {
 				currentPage = 0;
@@ -2377,8 +2384,8 @@
 			});
 
 			// Add the list li background colors - this wil be dropped in a future version in favor of CSS
-			$('.' + _this.settings.locationList + ' ul li:even').css('background', _this.settings.listColor1);
-			$('.' + _this.settings.locationList + ' ul li:odd').css('background', _this.settings.listColor2);
+			$('.' + _this.settings.locationList + ' ul > li:even').css('background', _this.settings.listColor1);
+			$('.' + _this.settings.locationList + ' ul > li:odd').css('background', _this.settings.listColor2);
 
 			// Visible markers list
 			if(_this.settings.visibleMarkersList === true) {

dist/assets/js/plugins/storeLocator/jquery.storelocator.min.js

@@ -36,8 +36,10 @@ filtering.
 ## Changelog
 
 ### Version 2.6.2
+
 * Added callbackMapSet callback that fires after the map has been set up.
-* Fixed issue where locations without attributes could get those attribute values from prior locations.
+* Fixed issue where locations without attributes set could get the attribute values from prior locations.
+* Fixed issue where pagination total number of pages was based on the full location set total instead of the storeLimit setting.
 * Removed form markup from initial query string example index file as it's not needed until the submission page.
 
 ### Version 2.6.1

readme.md

@@ -866,9 +866,16 @@
 		paginationSetup: function (currentPage) {
 			this.writeDebug('paginationSetup',arguments);
 			var pagesOutput = '';
-			var totalPages = locationset.length / this.settings.locationsPerPage;
+			var totalPages;
 			var $paginationList = $('.bh-sl-pagination-container .bh-sl-pagination');
 
+			// Total pages
+			if ( this.settings.storeLimit === -1 || locationset.length < this.settings.storeLimit ) {
+				totalPages = locationset.length / this.settings.locationsPerPage;
+			} else {
+				totalPages = this.settings.storeLimit / this.settings.locationsPerPage;
+			}
+
 			// Current page check
 			if (typeof currentPage === 'undefined') {
 				currentPage = 0;
@@ -2374,8 +2381,8 @@
 			});
 
 			// Add the list li background colors - this wil be dropped in a future version in favor of CSS
-			$('.' + _this.settings.locationList + ' ul li:even').css('background', _this.settings.listColor1);
-			$('.' + _this.settings.locationList + ' ul li:odd').css('background', _this.settings.listColor2);
+			$('.' + _this.settings.locationList + ' ul > li:even').css('background', _this.settings.listColor1);
+			$('.' + _this.settings.locationList + ' ul > li:odd').css('background', _this.settings.listColor2);
 
 			// Visible markers list
 			if(_this.settings.visibleMarkersList === true) {