Bug 995873 - Don't use replaceState if it would result in an invalid URL

r=justdave, a=justdave

Bug 995873 - Don't use replaceState if it would result in an invalid URL
0b6c84fc · Simon Green · f5421b8a · 0b6c84fc
Commit 0b6c84fc authored Apr 15, 2014 by Simon Green
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 5 deletions

list.html.tmpl template/en/default/list/list.html.tmpl +7 -5

No files found.
--- a/template/en/default/list/list.html.tmpl
+++ b/template/en/default/list/list.html.tmpl
@@ -43,11 +43,13 @@
    [% new_param = cgi.canonicalise_query %]
  [% END %]

-  if (history && history.replaceState) {
-    history.replaceState(null, "[% unfiltered_title FILTER js %]",
-                         "buglist.cgi?[% new_param FILTER js %]");
-    document.title = "[% unfiltered_title FILTER js %]";
-  }
+  [% IF new_param.length + 12 < constants.CGI_URI_LIMIT %]
+    if (history && history.replaceState) {
+      history.replaceState(null, "[% unfiltered_title FILTER js %]",
+                           "buglist.cgi?[% new_param FILTER js %]");
+      document.title = "[% unfiltered_title FILTER js %]";
+    }
+  [% END %]
  [% javascript FILTER none %]
 [% END %]