Add FIXME comment about a meaningless merge strategy

1 files changed, 2 insertions, 0 deletions

diff --git a/main/src/cgeo/geocaching/cgCache.java b/main/src/cgeo/geocaching/cgCache.java
index 7fbbf0c..0f5315e 100644
--- a/main/src/cgeo/geocaching/cgCache.java
+++ b/main/src/cgeo/geocaching/cgCache.java
@@ -273,6 +273,8 @@ public class cgCache implements ICache, IWaypoint {
         if (StringUtils.isBlank(description)) {
             description = other.description;
         }
+        // FIXME: this makes no sense to favor this over the other. 0 should not be a special case here as it is
+        // in the range of acceptable values. This is probably the case at other places (rating, votes, etc.) too.
         if (favoritePoints == 0) {
             favoritePoints = other.favoritePoints;
         }