[GEOS-8034] Fix ysld color conversion master (geotools#1649)

* [GEOS-8034] Fixed ysld color conversion * Revert "[GEOS-8034] Fixed ysld color conversion" This reverts commit 8adae77. * fixed ysdl encoder to allow for Color objects * incorporate requested changes Added TODOs to determine where exceptions need to be changed
johngschulz · Nov 17, 2017 · cf344d6 · cf344d6
1 parent c9c2af5
commit cf344d6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/modules/unsupported/mbstyle/src/main/java/org/geotools/mbstyle/MBStyle.java b/modules/unsupported/mbstyle/src/main/java/org/geotools/mbstyle/MBStyle.java
@@ -450,7 +450,7 @@ public StyledLayerDescriptor transform() {
                 }
             }
         }
-
+        // TODO determine if the below exception needs to be changed here.
         if (sld.layers().isEmpty()) {
             throw new MBFormatException("No visibile layers");
         }

diff --git a/modules/unsupported/mbstyle/src/main/java/org/geotools/mbstyle/parse/MBObjectStops.java b/modules/unsupported/mbstyle/src/main/java/org/geotools/mbstyle/parse/MBObjectStops.java
@@ -193,7 +193,7 @@ public long getStop(MBLayer layer) {
 
     LayerStops containsStops(JSONObject jsonObject, LayerStops ls) {
         Boolean hasStops = false;
-
+        // TODO determine if a stop that needs the "property" key has it, and throw an appropriate exception if not.
         Set<?> keySet = jsonObject.keySet();
         Iterator<?> keys = keySet.iterator();
         while (keys.hasNext()) {