refactor: extract common part from if on filter field

Signed-off-by: Pinto Spindler, Stephan <25225092+s-spindler@users.noreply.github.com>
This commit is contained in:
Pinto Spindler, Stephan 2022-07-15 22:51:33 +02:00
parent 325be1728e
commit 4f5a8fe3f1

View file

@ -97,8 +97,6 @@ final class ProjectQueryManager extends QueryManager implements IQueryManager {
}
params.put("name", filterString);
params.put("tag", tag);
preprocessACLs(query, queryFilter, params, false);
result = execute(query, params);
} else {
if (excludeInactive) {
queryFilter = "(name.toLowerCase().matches(:name) && (active == true || active == null))";
@ -106,16 +104,14 @@ final class ProjectQueryManager extends QueryManager implements IQueryManager {
queryFilter = "(name.toLowerCase().matches(:name))";
}
params.put("name", filterString);
preprocessACLs(query, queryFilter, params, false);
result = execute(query, params);
}
} else {
if (excludeInactive) {
queryFilter = " (active == true || active == null) ";
}
preprocessACLs(query, queryFilter, params, false);
result = execute(query, params);
}
preprocessACLs(query, queryFilter, params, false);
result = execute(query, params);
if (includeMetrics) {
// Populate each Project object in the paginated result with transitive related
// data to minimize the number of round trips a client needs to make, process, and render.