fix #4245: IllegalStateException with NFC

Setting the Ndef record creator in onCreate() is the recommended way to
do it. A null can also be returned if no message should be created.

1 files changed, 8 insertions, 10 deletions

diff --git a/main/src/cgeo/geocaching/CacheDetailActivity.java b/main/src/cgeo/geocaching/CacheDetailActivity.java
index cbc3840..bebb63a 100644
--- a/main/src/cgeo/geocaching/CacheDetailActivity.java
+++ b/main/src/cgeo/geocaching/CacheDetailActivity.java
@@ -305,6 +305,14 @@ public class CacheDetailActivity extends AbstractViewPagerActivity<CacheDetailAc
         });
 
         locationUpdater = new CacheDetailsGeoDirHandler(this);
+
+        // If we have a newer Android device setup Android Beam for easy cache sharing
+        initializeAndroidBeam(new ActivitySharingInterface() {
+            @Override
+            public String getUri() {
+                return cache != null ? cache.getCgeoUrl() : null;
+            }
+        });
     }
 
     @Override
@@ -583,16 +591,6 @@ public class CacheDetailActivity extends AbstractViewPagerActivity<CacheDetailAc
 
         updateTitleBar(cache.getGeocode(), cache.getName(), cache.getType());
 
-        // if we have a newer Android device setup Android Beam for easy cache sharing
-        initializeAndroidBeam(
-                new ActivitySharingInterface() {
-                    @Override
-                    public String getUri() {
-                        return cache.getCgeoUrl();
-                    }
-                }
-        );
-
         // reset imagesList so Images view page will be redrawn
         imagesList = null;
         reinitializeViewPager();