aboutsummaryrefslogtreecommitdiffstats
path: root/web/js/map-OpenLayers.js
diff options
context:
space:
mode:
authorStruan Donald <struan@exo.org.uk>2018-06-15 12:33:47 +0100
committerMatthew Somerville <matthew-github@dracos.co.uk>2018-07-03 11:55:53 +0100
commit04e49afc9a7c10405464c62b0cfd6f6c55b286ac (patch)
treeec6b4d950a201c87a26c4aae6a1a4fe6bed45b5b /web/js/map-OpenLayers.js
parentdf2e6c6fd66c46ac0b00a38c5aa00d056bbde4d1 (diff)
fix pushState pagination
Using `$('.pagination')` everywhere in javascript was causing issues because that returns two elements. This was especially true when triggering events as it was triggering two events so the data was being loaded twice. This was also resulting in resetting the page to the initial page so clicking the back button would always return to the first page of results. This also sets `use_page` when clicking the back button otherwise it also loads the initial page.
Diffstat (limited to 'web/js/map-OpenLayers.js')
-rw-r--r--web/js/map-OpenLayers.js10
1 files changed, 5 insertions, 5 deletions
diff --git a/web/js/map-OpenLayers.js b/web/js/map-OpenLayers.js
index 645e5114e..281940aca 100644
--- a/web/js/map-OpenLayers.js
+++ b/web/js/map-OpenLayers.js
@@ -387,7 +387,7 @@ $.extend(fixmystreet.utils, {
}
var qs = fixmystreet.utils.parse_query_string();
- var page = $('.pagination').data('page');
+ var page = $('.pagination:first').data('page');
if (page > 1) {
qs.p = page;
} else {
@@ -643,11 +643,11 @@ $.extend(fixmystreet.utils, {
$('.js-pagination').on('change.filters', categories_or_status_changed);
$('.js-pagination').on('click', 'a', function(e) {
e.preventDefault();
- var page = $('.pagination').data('page');
+ var page = $('.pagination:first').data('page');
if ($(this).hasClass('next')) {
- $('.pagination').data('page', page + 1);
+ $('.pagination:first').data('page', page + 1);
} else {
- $('.pagination').data('page', page - 1);
+ $('.pagination:first').data('page', page - 1);
}
fixmystreet.markers.protocol.use_page = true;
$(this).trigger('change');
@@ -916,7 +916,7 @@ OpenLayers.Protocol.FixMyStreet = OpenLayers.Class(OpenLayers.Protocol.HTTP, {
});
var page;
if (this.use_page) {
- page = $('.pagination').data('page');
+ page = $('.pagination:first').data('page');
this.use_page = false;
} else if (this.initial_page) {
page = 1;