Iss #62 - Code smell fixes, testing harness update, minor derecations

A bunch of various code smell fixes, as well as adding some deprecation flags for content that shouldn't be used or should be replaced like the SortableField annotation and helpers.

The testing harness was updated to use a more fluent chained calling structure rather than the custom methods for each that were restrictive to testing.

core/src/main/java/org/eclipsefoundation/core/request/OptionalPathFilter.java

@@ -36,7 +36,7 @@ public class OptionalPathFilter implements ContainerRequestFilter {
         Method m = ((PostMatchContainerRequestContext) requestContext).getResourceMethod().getMethod();
         OptionalPath opt = m.getAnnotation(OptionalPath.class);
         if (opt != null) {
-            if (!optionalResourcesEnabled.get()) {
+            if (Boolean.FALSE.equals(optionalResourcesEnabled.get())) {
                 LOGGER.trace("Request to '{}' rejected as optional resources are not enabled",
                         requestContext.getUriInfo().getAbsolutePath());
                 // abort with 404 as we should hide that this exists
@@ -44,7 +44,7 @@ public class OptionalPathFilter implements ContainerRequestFilter {
             }
             // get the specified config value fresh and check that it is enabled, or enabled by default if missing
             Optional<Boolean> configValue = ConfigProvider.getConfig().getOptionalValue(opt.value(), Boolean.class);
-            if (configValue.isPresent() && configValue.get()) {
+            if (configValue.isPresent() && Boolean.TRUE.equals(configValue.get())) {
                 LOGGER.trace("Request to '{}' enabled by config, allowing call",
                         requestContext.getUriInfo().getAbsolutePath());
             } else if (configValue.isEmpty() && opt.enabledByDefault()) {