Cleaned up todos and some comments (#44)

Author: rzwiefel

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/metacard/MetacardApplication.java

@@ -231,7 +231,6 @@ public void init() {
             return util.getJson(validator.validateAttribute(attribute, value));
         });
 
-        // TODO (RCZ) - this could use some help
         get("/history/:id", (req, res) -> {
             String id = req.params(":id");
             List<Result> queryResponse = getMetacardHistory(id);

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/metacard/enumerations/ExperimentalEnumerationExtractor.java

@@ -31,9 +31,10 @@
 import ddf.catalog.data.impl.BasicTypes;
 import ddf.catalog.validation.AttributeValidatorRegistry;
 import ddf.catalog.validation.violation.ValidationViolation;
-import spark.Experimental;
 
-@Experimental // TODO (RCZ) - This is sparks experimental iface, could we use?
+/**
+ * This class is Experimental and subject to change
+ */
 public class ExperimentalEnumerationExtractor {
     private final AttributeValidatorRegistry attributeValidatorRegistry;
 

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/metacard/workspace/WorkspaceTransformer.java

@@ -194,7 +194,6 @@ public Metacard toMetacardFromXml(Serializable xml) {
                 }
             }
         } catch (Exception ex) {
-            // TODO (RCZ) - just throw?
             throw new RuntimeException(ex);
         }
 

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/searchui/standard/endpoints/MetacardEditEndpoint.java

@@ -196,7 +196,7 @@ public Response deleteAttribute(@Context HttpServletResponse response,
 
         if (metacardAttribute == null) {
             return Response.ok()
-                    .build(); // TODO (RCZ) - if it wasn't there is that an okay or bad?
+                    .build();
         }
 
 

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/util/EndpointUtil.java

@@ -187,7 +187,6 @@ public Map<String, Result> getMetacards(String attributeName, Collection<String>
         return results;
     }
 
-    // TODO (RCZ) - This method could do with some immutable/persistant objects
     @SuppressWarnings("unchecked")
     public Map<String, Object> getMetacardTypeMap() {
         Map<String, Object> resultTypes = new HashMap<>();
@@ -261,7 +260,6 @@ public Map<String, Object> transformToMap(List<Metacard> metacards) {
             typeMap.put("type-name", type);
             typeMap.put("type", getMetacardTypeMap().get(type));
 
-            // TODO (RCZ) - optimize this since might be hit a lot
             typeMap.put("ids",
                     metacards.stream()
                             .filter(mc -> type.equals(mc.getMetacardType()