New Route for getting all PermissionHolders with a specific Permission

[Carlos17Kopra]

Hello there.
I cant help myself, I added another Route :'D
This Routes gets all PermissionHolders with a specific Permission and Value (optional)
The Routes are:

/<Type>/has/{perm}

/<Type>/has/{perm}/{value}

What did I do:

First, I added the two Methods to the PermissionHolderController:

// GET /<type>/has/{permission}
void getHasPermSingle(Context ctx) throws Exception;

// GET <type>/has/{permission}/{value}
void getHasPermValue(Context ctx) throws Exception;

Here are my Implementations of the Methods
UserController

// GET /user/has/{permission}
@Override
public void getHasPermSingle(Context ctx) throws Exception {
    CompletableFuture<Set<UUID>> future = this.userManager.getUniqueUsers();
    String param = ctx.pathParam("perm");
    boolean value = true;
    ctx.future(future);
    this.hasPerm(ctx, future.get(), param, value);
}

// GET /user/has/{permission}/{value}
@Override
public void getHasPermValue(Context ctx) throws Exception {
    CompletableFuture<Set<UUID>> future = this.userManager.getUniqueUsers();
    String param = ctx.pathParam("perm");
    boolean value = Boolean.parseBoolean(ctx.pathParam("value"));;
    ctx.future(future);
    this.hasPerm(ctx, future.get(), param, value);
}

private void hasPerm(Context ctx, Set<UUID> uuids, String perm, boolean value) throws Exception {
    List<UUID> matchingUUIDs = new ArrayList<>();
    for(UUID uuid : uuids){
        CompletableFuture<User> loadFuture = loadUserCached(uuid);
        ctx.future(loadFuture);
        User user = loadFuture.get();
        if(user == null) continue;
        Collection<PermissionNode> permNodes = user.getNodes(NodeType.PERMISSION);
        if(permNodes.stream().anyMatch(node -> node.getPermission().equals(perm) && node.getValue() == value)){
            matchingUUIDs.add(uuid);
        }
    }
    ctx.json(matchingUUIDs);
}

GroupController

// GET /group/has/{perm}
@Override
public void getHasPermSingle(Context ctx) {
    String perm = ctx.pathParam("perm");
    this.hasPerm(ctx, perm, true);
}
// GET /group/has/{perm}/{value}
@Override
public void getHasPermValue(Context ctx) {
    String perm = ctx.pathParam("perm");
    boolean value = Boolean.parseBoolean(ctx.pathParam("value"));
    this.hasPerm(ctx, perm, value);
}

private void hasPerm(Context ctx, String perm, boolean value){
    CompletableFuture<List<String>> future = loadGroupsCached()
            .thenApply(groups -> groups.stream()
                    .filter(group -> {
                        Collection<PermissionNode> nodes = group.getNodes(NodeType.PERMISSION);
                        return nodes.stream().anyMatch(node -> node.getPermission().equals(perm) && node.getValue() == value);
                    })
                    .map(Group::getName)
                    .collect(Collectors.toList())
            );
    ctx.future(future);
}

Finally I added the Routes in the RestServer Class:

path("has", () -> {
    path("{perm}", () -> {
        get(controller::getHasPermSingle);
        path("{value}", () -> {
            get(controller::getHasPermValue);
        });
    });
});

This would be a cool Feature for the official API too and im open for questions or optimizations :)