GCHQDeveloper314
GCHQDeveloper314
**Describe the bug** The code to check for if nested named operations are allowed exists in [AddNamedOperationHandler.java](https://github.com/gchq/Gaffer/blob/6213159e6ee112d31a67612a59be970bbc72bada/core/store/src/main/java/uk/gov/gchq/gaffer/store/operation/handler/named/AddNamedOperationHandler.java#L46) and [checks if a new named operation to add contains further named operations](https://github.com/gchq/Gaffer/blob/6213159e6ee112d31a67612a59be970bbc72bada/core/store/src/main/java/uk/gov/gchq/gaffer/store/operation/handler/named/AddNamedOperationHandler.java#L120)....
Prior to #3147, `FederatedStoreCache.java` was inheriting from `Cache.java` solely to enforce some generics. It had various overriding methods which were unused but were left in place in that PR. Either...
In [ChainedIterable.java](https://github.com/gchq/koryphe/blob/36c63c3e24145e04405d7620524b23cf9b5b5849/core/src/main/java/uk/gov/gchq/koryphe/iterable/ChainedIterable.java#L39-L41) there's a varargs constructor to take any number of Iterables to create a single Iterable: https://github.com/gchq/koryphe/blob/36c63c3e24145e04405d7620524b23cf9b5b5849/core/src/main/java/uk/gov/gchq/koryphe/iterable/ChainedIterable.java#L39 The [ChainedIterator.java](https://github.com/gchq/koryphe/blob/36c63c3e24145e04405d7620524b23cf9b5b5849/core/src/main/java/uk/gov/gchq/koryphe/iterable/ChainedIterator.java#L34) class is very similar but is missing this kind of...
An issue was raised in gaffer-doc to improve the documentation for the `InDateRangeDual` predicate [(#450)](https://github.com/gchq/gaffer-doc/issues/450). These improvements should be reflected in the Javadoc as well. Also, an issue with line...
Koryphe has a dependency on Guava, but the Guava methods used are simple and most functionality can be achieved with Java 8 built in methods. Having a dependency on Guava...
Currently the JavaDoc is quite basic for these classes in Koryphe. In particular the handling of null and empty values for functions and predicates is not explained sufficiently well. For...
# Related Issue - Resolve #308
The Javadoc for `ToList` states that it "takes an Object and converts it to a list. If the object is an array or iterable the items will be added to...
**Describe the issue** In `SystemProperty.java` the fields [`SERVICES_PACKAGE_PREFIX`](https://github.com/gchq/Gaffer/blob/6b7e3d24961f495c526f0a1ecd6ed766dcb3cf7c/rest-api/common-rest/src/main/java/uk/gov/gchq/gaffer/rest/SystemProperty.java#L43) and [`SERVICES_PACKAGE_PREFIX_DEFAULT`](https://github.com/gchq/Gaffer/blob/6b7e3d24961f495c526f0a1ecd6ed766dcb3cf7c/rest-api/common-rest/src/main/java/uk/gov/gchq/gaffer/rest/SystemProperty.java#L69) are declared but are not used anywhere in the Gaffer codebase. It's possible they were used at the time they...
**Describe the improvement you'd like** Testing for [`NamedOperationResolver`](https://github.com/gchq/Gaffer/blob/6b7e3d24961f495c526f0a1ecd6ed766dcb3cf7c/core/graph/src/main/java/uk/gov/gchq/gaffer/graph/hook/NamedOperationResolver.java#L44) should cover configuration of a named operation depth using JSON. **Why do you want this improvement?** Although not yet documented, if admins...