Fix encoding issues in Langviews and Redirect Views

MusikAnimal · Jan 9, 2017 · f753a27 · f753a27
1 parent fe39a56
commit f753a27
Show file tree

Hide file tree

Showing 9 changed files with 24,875 additions and 106 deletions.
diff --git a/javascripts/langviews/langviews.js b/javascripts/langviews/langviews.js
@@ -286,7 +286,7 @@ class LangViews extends mix(Pv).with(ChartHelpers, ListHelpers) {
       return history.replaceState(null, document.title, location.href.split('?')[0]);
     }
 
-    const escapedPageName = $(this.config.sourceInput).val().score().replace(/[&%?]/g, escape);
+    const escapedPageName = $(this.config.sourceInput).val().score().replace(/[&%?+]/g, encodeURIComponent);
 
     window.history.replaceState({}, document.title, `?${$.param(this.getParams())}&page=${escapedPageName}`);
 

diff --git a/javascripts/redirectviews/redirectviews.js b/javascripts/redirectviews/redirectviews.js
@@ -288,7 +288,7 @@ class RedirectViews extends mix(Pv).with(ChartHelpers, ListHelpers) {
       return history.replaceState(null, document.title, location.href.split('?')[0]);
     }
 
-    const escapedPageName = $(this.config.sourceInput).val().score().replace(/[&%?]/g, escape);
+    const escapedPageName = $(this.config.sourceInput).val().score().replace(/[&%?+]/g, encodeURIComponent);
 
     window.history.replaceState({}, document.title, `?${$.param(this.getParams())}&page=${escapedPageName}`);
 

diff --git a/public_html/application.js b/public_html/application.js
diff --git a/public_html/langviews/application.js b/public_html/langviews/application.js
diff --git a/public_html/massviews/application.js b/public_html/massviews/application.js
diff --git a/public_html/meta/application.js b/public_html/meta/application.js
diff --git a/public_html/redirectviews/application.js b/public_html/redirectviews/application.js
diff --git a/public_html/siteviews/application.js b/public_html/siteviews/application.js
diff --git a/public_html/topviews/application.js b/public_html/topviews/application.js