Skip to content

Commit

Permalink
Merge pull request #5 from stevie-mayhew/bugfix/reportadmin-params
Browse files Browse the repository at this point in the history
BUGFIX: pass the correct param values through to reports
  • Loading branch information
wilr committed Mar 12, 2014

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
2 parents 2303ce3 + f1024db commit 193a2ab
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion javascript/ReportAdmin.js
Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@
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
if(params) url = $.path.addSearchParams(url, params);
if(params) url = $.path.addSearchParams(url, $.param(params));
$('.cms-container').loadPanel(url);
return false;
}

0 comments on commit 193a2ab

Please sign in to comment.