Polishing.

Simplify filter predicate.

See #176.
This commit is contained in:
Mark Paluch
2021-03-17 10:44:50 +01:00
parent 272545aac8
commit a69fd8cb1f
2 changed files with 9 additions and 4 deletions

View File

@@ -14,6 +14,7 @@
<properties>
<spring-plugin.version>2.0.0.RELEASE</spring-plugin.version>
<lombok.version>1.18.18</lombok.version>
</properties>
<dependencies>

View File

@@ -142,9 +142,10 @@ class IssueTrackerCommands extends TimedCommand {
@CliOption(key = "module") String moduleName,
@CliOption(key = "filter-release-tickets") Boolean filterReleaseTickets) {
Predicate<Ticket> notResolved = it -> !it.isResolved();
return getTickets(iteration, moduleName,
it -> !it.isResolved() && ((filterReleaseTickets == null && !it.isReleaseTicket())
|| (filterReleaseTickets != null && filterReleaseTickets && !it.isReleaseTicket())));
notResolved.and(getFilterPredicate(filterReleaseTickets == null || filterReleaseTickets)));
}
@CliCommand("tracker all-tickets")
@@ -152,8 +153,11 @@ class IssueTrackerCommands extends TimedCommand {
@CliOption(key = "module") String moduleName,
@CliOption(key = "filter-release-tickets") Boolean filterReleaseTickets) {
return getTickets(iteration, moduleName, it -> (filterReleaseTickets == null && !it.isReleaseTicket())
|| (filterReleaseTickets != null && filterReleaseTickets && !it.isReleaseTicket()));
return getTickets(iteration, moduleName, getFilterPredicate(filterReleaseTickets == null || filterReleaseTickets));
}
private static Predicate<Ticket> getFilterPredicate(boolean filterReleaseTickets) {
return it -> filterReleaseTickets == !it.isReleaseTicket();
}
private String getTickets(TrainIteration iteration, String moduleName, Predicate<Ticket> ticketPredicate) {