Skip to content

Commit

Permalink
Avoid duplicating ReportAdmin search params (fixes #8046)
Browse files Browse the repository at this point in the history
  • Loading branch information
chillu committed Dec 4, 2012
1 parent f9a5601 commit 98e824b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion javascript/ReportAdmin.js
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@
$.entwine('ss', function($){ $.entwine('ss', function($){
$('.ReportAdmin .cms-edit-form').entwine({ $('.ReportAdmin .cms-edit-form').entwine({
onsubmit: function(e) { onsubmit: function(e) {
var url = document.location.href, params = this.find(':input[name^=filters]').serializeArray(); var url = $.path.parseUrl(document.location.href).hrefNoSearch,
params = this.find(':input[name^=filters]').serializeArray();
params = $.grep(params, function(param) {return (param.value);}); // filter out empty params = $.grep(params, function(param) {return (param.value);}); // filter out empty
if(params) url = $.path.addSearchParams(url, params); if(params) url = $.path.addSearchParams(url, params);
$('.cms-container').loadPanel(url); $('.cms-container').loadPanel(url);
Expand Down

0 comments on commit 98e824b

Please sign in to comment.