[Fix] Do not save empty tags during updates

jawg · May 2, 2018 · 8b755b0 · 8b755b0
1 parent aeee414
commit 8b755b0
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/java/io/jawg/osmcontributor/model/entities/Poi.java b/src/main/java/io/jawg/osmcontributor/model/entities/Poi.java
@@ -34,6 +34,7 @@
 import java.util.Map;
 import java.util.Set;
 
+import io.jawg.osmcontributor.utils.StringUtils;
 import io.jawg.osmcontributor.utils.core.MapElement;
 import timber.log.Timber;
 
@@ -425,8 +426,8 @@ public void applyChanges(Map<String, String> tagsMap) {
 
         // Add the new tags to the Poi
         for (Map.Entry<String, String> tagToAdd : tagsMap.entrySet()) {
-            //The default level is zero, we don't send this tag if it's the default value
-            if (!("level".equals(tagToAdd.getKey()) && "0".equals(tagToAdd.getValue()))) {
+            //The default level is zero, we don't send this tag if it's the default value. We also do not want to send an empty tag.
+            if (!("level".equals(tagToAdd.getKey()) && "0".equals(tagToAdd.getValue())) && !StringUtils.isEmpty(tagToAdd.getValue())) {
                 PoiTag tempPoiTag = new PoiTag();
                 tempPoiTag.setKey(tagToAdd.getKey());
                 tempPoiTag.setValue(tagToAdd.getValue());