Verified Commit 1af89d40 authored by Lukas Wiest's avatar Lukas Wiest 🚂
Browse files

fix(systems/github): Filter: set query parameter, instead of adding it mulitple times

ref 4f845899
parent 4027818e
...@@ -50,7 +50,7 @@ public class GithubFilter extends Filter<GithubTicket, GithubFilter> ...@@ -50,7 +50,7 @@ public class GithubFilter extends Filter<GithubTicket, GithubFilter>
// set default values before checking user filters, // set default values before checking user filters,
// to match default behaviour // to match default behaviour
urlBuilder.addQueryParameter("state", "all"); urlBuilder.setQueryParameter("state", "all");
for (Map.Entry<String, Object> mapEntry : setFilters.entrySet()) for (Map.Entry<String, Object> mapEntry : setFilters.entrySet())
{ {
...@@ -101,7 +101,7 @@ public class GithubFilter extends Filter<GithubTicket, GithubFilter> ...@@ -101,7 +101,7 @@ public class GithubFilter extends Filter<GithubTicket, GithubFilter>
urlBuilder.addQueryParameter("per_page", String.valueOf(v)); urlBuilder.addQueryParameter("per_page", String.valueOf(v));
} else if (f.equals(FilterNames.OPEN.name())) } else if (f.equals(FilterNames.OPEN.name()))
{ {
urlBuilder.addQueryParameter("state", ((boolean) v) ? "open" : "closed"); urlBuilder.setQueryParameter("state", ((boolean) v) ? "open" : "closed");
} else if (f.equals(FilterNames.TITLE_CONTAINS.name())) } else if (f.equals(FilterNames.TITLE_CONTAINS.name()))
{ {
logger.log(Level.FINE, "title contain check only possible after request |" + logger.log(Level.FINE, "title contain check only possible after request |" +
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment