From 6bfcb338d4c9b93164bba02b94c4bafbef553fdb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Mart=C3=AD?= <mvdan@mvdan.cc>
Date: Mon, 22 Jul 2013 11:33:21 +0200
Subject: [PATCH] Use the new /app/app.id uri scheme

Also, differentiate between market://details and http://f-droid.org/app
better.
---
 AndroidManifest.xml                   |  3 ++-
 src/org/fdroid/fdroid/AppDetails.java | 17 +++++++++++------
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index 7674fc99c..e42668d5e 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -73,7 +73,8 @@
                 <category android:name="android.intent.category.DEFAULT" />
                 <category android:name="android.intent.category.BROWSABLE" />
 
-                <data android:scheme="http" android:host="f-droid.org" android:path="/app" />
+                <data android:scheme="http" android:host="f-droid.org"
+                    android:pathPrefix="/app" />
             </intent-filter>
             <meta-data
                 android:name="android.support.PARENT_ACTIVITY"
diff --git a/src/org/fdroid/fdroid/AppDetails.java b/src/org/fdroid/fdroid/AppDetails.java
index b5e421c2a..84c6fe298 100644
--- a/src/org/fdroid/fdroid/AppDetails.java
+++ b/src/org/fdroid/fdroid/AppDetails.java
@@ -193,13 +193,18 @@ public class AppDetails extends ListActivity {
         appid = "";
         Uri data = getIntent().getData();
         if (data != null) {
-            if (data.isHierarchical())
-                // fdroid://details?id=app.id
-                // market://details?id=app.id
-                appid = data.getQueryParameter("id");
-            else
+            if (data.isHierarchical()) {
+                if (data.getHost().equals("details")) {
+                    // market://details?id=app.id
+                    appid = data.getQueryParameter("id");
+                } else {
+                    // http://f-droid.org/app/app.id
+                    appid = data.getLastPathSegment();
+                }
+            } else {
                 // fdroid.app:app.id (old scheme)
                 appid = data.getEncodedSchemeSpecificPart();
+            }
             Log.d("FDroid", "AppDetails launched from link, for '" + appid
                     + "'");
         } else if (!i.hasExtra("appid")) {
@@ -771,7 +776,7 @@ public class AppDetails extends ListActivity {
         shareIntent.setType("text/plain");
 
         shareIntent.putExtra(android.content.Intent.EXTRA_SUBJECT, app.name);
-        shareIntent.putExtra(android.content.Intent.EXTRA_TEXT, "http://f-droid.org/app?id="+app.id);
+        shareIntent.putExtra(android.content.Intent.EXTRA_TEXT, "http://f-droid.org/app/"+app.id);
 
         startActivity(Intent.createChooser(shareIntent, getString(R.string.menu_share)));
     }