Skip to content

Commit

Permalink
don't merge resourceActions for nested queries, allow recursive engin…
Browse files Browse the repository at this point in the history
…e to handle this for us
  • Loading branch information
andrewazores committed Aug 2, 2022
1 parent 7feded2 commit d32a3aa
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,7 @@ class EnvironmentNodesFetcher extends AbstractPermissionedDataFetcher<List<Envir

@Override
public Set<ResourceAction> resourceActions() {
EnumSet<ResourceAction> actions = EnumSet.of(ResourceAction.READ_TARGET);
actions.addAll(rootNodeFetcher.resourceActions());
return actions;
return EnumSet.of(ResourceAction.READ_TARGET);
}

@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,7 @@ class NodeFetcher extends AbstractPermissionedDataFetcher<AbstractNode> {

@Override
public Set<ResourceAction> resourceActions() {
EnumSet<ResourceAction> actions = EnumSet.of(ResourceAction.READ_TARGET);
actions.addAll(rootNodeFetcher.resourceActions());
return actions;
return EnumSet.of(ResourceAction.READ_TARGET);
}

@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,10 +73,7 @@ class TargetNodesFetcher extends AbstractPermissionedDataFetcher<List<TargetNode

@Override
public Set<ResourceAction> resourceActions() {
EnumSet<ResourceAction> actions = EnumSet.of(ResourceAction.READ_TARGET);
actions.addAll(rootNodeFetcher.resourceActions());
actions.addAll(recurseFetcher.resourceActions());
return actions;
return EnumSet.of(ResourceAction.READ_TARGET);
}

@Override
Expand Down

0 comments on commit d32a3aa

Please sign in to comment.