From 0f28169faf68b19b1218bf3c6e32c52b42017909 Mon Sep 17 00:00:00 2001
From: Bananeweizen <bananeweizen@gmx.de>
Date: Sat, 21 Feb 2015 11:43:42 +0100
Subject: new: menu item to open geo checker

---
 main/res/menu/cache_options.xml                   |  8 +++++-
 main/res/values/changelog_master.xml              |  1 +
 main/res/values/strings.xml                       |  1 +
 main/src/cgeo/geocaching/CacheDetailActivity.java |  5 ++++
 main/src/cgeo/geocaching/utils/CheckerUtils.java  | 34 +++++++++++++++++++++++
 5 files changed, 48 insertions(+), 1 deletion(-)
 create mode 100644 main/src/cgeo/geocaching/utils/CheckerUtils.java

(limited to 'main')

diff --git a/main/res/menu/cache_options.xml b/main/res/menu/cache_options.xml
index 8b40765..1032d99 100644
--- a/main/res/menu/cache_options.xml
+++ b/main/res/menu/cache_options.xml
@@ -83,5 +83,11 @@
         app:actionProviderClass="android.support.v7.widget.ShareActionProvider"
         app:showAsAction="ifRoom">
     </item>
-
+    <item
+        android:id="@+id/menu_checker"
+        android:title="@string/cache_menu_checker"
+        android:visible="false"
+        app:showAsAction="ifRoom">
+    </item>
+    
 </menu>
\ No newline at end of file
diff --git a/main/res/values/changelog_master.xml b/main/res/values/changelog_master.xml
index 51cf8f9..f861f3f 100644
--- a/main/res/values/changelog_master.xml
+++ b/main/res/values/changelog_master.xml
@@ -14,6 +14,7 @@
     · New: Allow export of waypoints without coords to Locus\n
     · New: Confirmation on backup/restore\n
     · New: Links in personal notes can be clicked\n
+    · New: Menu in cache details to open geo checker\n
     · Fix: Improve detection pattern for event start time\n
     · Fix: Don\'t allow logs in the future\n
     · Fix: Allow shortcut creation for All caches list\n
diff --git a/main/res/values/strings.xml b/main/res/values/strings.xml
index 3990ff6..be8cdf3 100644
--- a/main/res/values/strings.xml
+++ b/main/res/values/strings.xml
@@ -731,6 +731,7 @@
     <string name="cache_menu_pebble">Pebble</string>
     <string name="cache_menu_android_wear">Android Wear</string>
     <string name="cache_menu_vote">Vote</string>
+    <string name="cache_menu_checker">Open geo checker</string>
     <string name="cache_status">Status</string>
     <string name="cache_status_offline_log">Saved Log</string>
     <string name="cache_status_found">Found</string>
diff --git a/main/src/cgeo/geocaching/CacheDetailActivity.java b/main/src/cgeo/geocaching/CacheDetailActivity.java
index 93f2d31..14b6b81 100644
--- a/main/src/cgeo/geocaching/CacheDetailActivity.java
+++ b/main/src/cgeo/geocaching/CacheDetailActivity.java
@@ -44,6 +44,7 @@ import cgeo.geocaching.ui.OwnerActionsClickListener;
 import cgeo.geocaching.ui.WeakReferenceHandler;
 import cgeo.geocaching.ui.dialog.Dialogs;
 import cgeo.geocaching.ui.logs.CacheLogsViewCreator;
+import cgeo.geocaching.utils.CheckerUtils;
 import cgeo.geocaching.utils.CryptUtils;
 import cgeo.geocaching.utils.Formatter;
 import cgeo.geocaching.utils.ImageUtils;
@@ -498,6 +499,7 @@ public class CacheDetailActivity extends AbstractViewPagerActivity<CacheDetailAc
         menu.findItem(R.id.menu_delete).setVisible(cache != null && cache.isOffline());
         menu.findItem(R.id.menu_refresh).setVisible(cache != null && cache.isOffline());
         menu.findItem(R.id.menu_gcvote).setVisible(cache != null && GCVote.isVotingPossible(cache));
+        menu.findItem(R.id.menu_checker).setVisible(cache != null && StringUtils.isNotEmpty(CheckerUtils.getCheckerUrl(cache)));
         return super.onPrepareOptionsMenu(menu);
     }
 
@@ -522,6 +524,9 @@ public class CacheDetailActivity extends AbstractViewPagerActivity<CacheDetailAc
             case R.id.menu_gcvote:
                 showVoteDialog();
                 return true;
+            case R.id.menu_checker:
+                startActivity(new Intent(Intent.ACTION_VIEW, Uri.parse(CheckerUtils.getCheckerUrl(cache))));
+                return true;
             default:
                 if (NavigationAppFactory.onMenuItemSelected(item, this, cache)) {
                     return true;
diff --git a/main/src/cgeo/geocaching/utils/CheckerUtils.java b/main/src/cgeo/geocaching/utils/CheckerUtils.java
new file mode 100644
index 0000000..956ebbf
--- /dev/null
+++ b/main/src/cgeo/geocaching/utils/CheckerUtils.java
@@ -0,0 +1,34 @@
+package cgeo.geocaching.utils;
+
+import cgeo.geocaching.Geocache;
+
+import org.apache.commons.lang3.StringUtils;
+import org.eclipse.jdt.annotation.NonNull;
+import org.eclipse.jdt.annotation.Nullable;
+
+import android.util.Patterns;
+
+import java.util.regex.Matcher;
+
+public final class CheckerUtils {
+    private static final String[] CHECKERS = new String[] { "geocheck.org", "geochecker.com", "certitudes.org" };
+
+    private CheckerUtils() {
+        // utility class
+    }
+
+    @Nullable
+    public static String getCheckerUrl(@NonNull final Geocache cache) {
+        final String description = cache.getDescription();
+        final Matcher matcher = Patterns.WEB_URL.matcher(description);
+        while (matcher.find()) {
+            final String url = matcher.group();
+            for (final String checker : CHECKERS) {
+                if (StringUtils.containsIgnoreCase(url, checker)) {
+                    return url;
+                }
+            }
+        }
+        return null;
+    }
+}
-- 
cgit v1.1