From 2660da872ef9a96e99a4306a6761438d50123285 Mon Sep 17 00:00:00 2001
From: Nitin <hashtagerrors@gmail.com>
Date: Sun, 11 Aug 2019 19:51:50 +0530
Subject: [PATCH] Fixed Duplication of query params bug

---
 CHANGELOG.md                         |  4 ++++
 composer.json                        |  2 +-
 src/templates/_render/_paginate.twig | 30 ++++++++++++++--------------
 3 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 9ea1ead..acec646 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/) and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## 1.0.2 - 2019-08-11
+### Added
+- Fixed Duplication of query params bug
+
 ## 1.0.1 - 2019-05-16
 ### Added
 - Fixed FirstText and LastText Bug
diff --git a/composer.json b/composer.json
index ebc2d29..f390a8d 100644
--- a/composer.json
+++ b/composer.json
@@ -2,7 +2,7 @@
     "name": "hashtagerrors/craft-pagination",
     "description": "Pagination plugin for Craft 3",
     "type": "craft-plugin",
-    "version": "1.0.1",
+    "version": "1.0.2",
     "keywords": [
         "craft",
         "cms",
diff --git a/src/templates/_render/_paginate.twig b/src/templates/_render/_paginate.twig
index 5ae5b6f..865929f 100644
--- a/src/templates/_render/_paginate.twig
+++ b/src/templates/_render/_paginate.twig
@@ -22,13 +22,13 @@
 
 {% set showPageInfo = options.showPageInfo ?? '' %}
 
-{% set appendUrl = craft.app.request.getQueryStringWithoutPath() ? '?'~craft.app.request.getQueryStringWithoutPath() : '' %}
+{# {% set appendUrl = craft.app.request.getQueryStringWithoutPath() ? '?'~craft.app.request.getQueryStringWithoutPath() : '' %} #}
 
 <ul class="{{ ulClass }}">
     {% if type == 'firstNext' %}
         {% if pageInfo.prevUrl %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.prevUrl }}{{ appendUrl }}">{{ prevText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.prevUrl }}">{{ prevText|raw }}</a>
             </li>
         {% endif %}
         {% if showPageInfo %}
@@ -36,73 +36,73 @@
         {% endif %}
         {% if pageInfo.nextUrl %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.nextUrl }}{{ appendUrl }}">{{ nextText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.nextUrl }}">{{ nextText|raw }}</a>
             </li>
         {% endif %}
     {% elseif type == 'ellipsis' %}
         {% if pageInfo.prevUrl %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.prevUrl }}{{ appendUrl }}">{{ prevText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.prevUrl }}">{{ prevText|raw }}</a>
             </li>
         {% endif %}
         {% if pageInfo.currentPage > pageRange + 1 %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.firstUrl }}{{ appendUrl }}">1</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.firstUrl }}">1</a>
             </li>
         {% endif %}
         {% if pageInfo.currentPage > pageRange + 2 %}
             <span class="pageEllipsis">...</span>
         {% endif %}
         {% for page, url in pageInfo.getPrevUrls(pageRange) %}
-            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}{{ appendUrl }}">{{ page }}</a></li>
+            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}">{{ page }}</a></li>
         {% endfor %}
         <li class="{{ liClass }} {{ activeClass }}">
             <a class="{{ aClass }}" href="javascript:void(0);">{{ pageInfo.currentPage }}</a>
         </li>
         {% for page, url in pageInfo.getNextUrls(pageRange) %}
-            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}{{ appendUrl }}">{{ page }}</a></li>
+            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}">{{ page }}</a></li>
         {% endfor %}
         {% if pageInfo.currentPage < pageInfo.totalPages-(pageRange + 1) %}
             <span class="pageEllipsis">...</span>
         {% endif %}
         {% if pageInfo.currentPage < pageInfo.totalPages-pageRange %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.lastUrl }}{{ appendUrl }}">{{ pageInfo.totalPages }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.lastUrl }}">{{ pageInfo.totalPages }}</a>
             </li>
         {% endif %}
         {% if pageInfo.nextUrl %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.nextUrl }}{{ appendUrl }}">{{ nextText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.nextUrl }}">{{ nextText|raw }}</a>
             </li>
         {% endif %}
     {% else %}
         {% if includeFirstLast %} 
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.firstUrl }}{{ appendUrl }}">{{ firstText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.firstUrl }}">{{ firstText|raw }}</a>
             </li>
         {% endif %}
         {% if includePrevNext and pageInfo.prevUrl %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.prevUrl }}{{ appendUrl }}">{{ prevText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.prevUrl }}">{{ prevText|raw }}</a>
             </li>
         {% endif %}
         {% for page, url in pageInfo.getPrevUrls(pageRange) %}
-            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}{{ appendUrl }}">{{ page }}</a></li>
+            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}">{{ page }}</a></li>
         {% endfor %}
         <li class="{{ liClass }} {{ activeClass }}">
             <a class="{{ aClass }}" href="javascript:void(0);">{{ pageInfo.currentPage }}</a>
         </li>
         {% for page, url in pageInfo.getNextUrls(pageRange) %}
-            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}{{ appendUrl }}">{{ page }}</a></li>
+            <li class="{{ liClass }}"><a class="{{ aClass }}" href="{{ url }}">{{ page }}</a></li>
         {% endfor %}
         {% if includePrevNext and pageInfo.nextUrl %}
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.nextUrl }}{{ appendUrl }}">{{ nextText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.nextUrl }}">{{ nextText|raw }}</a>
             </li>
         {% endif %}
         {% if includeFirstLast %} 
             <li class="{{ liClass }}">
-                <a class="{{ aClass }}" href="{{ pageInfo.lastUrl }}{{ appendUrl }}">{{ lastText|raw }}</a>
+                <a class="{{ aClass }}" href="{{ pageInfo.lastUrl }}">{{ lastText|raw }}</a>
             </li>
         {% endif %}
     {% endif %}