Merge pull request #13197 from gcalero/your_last_location

Add "Your Last Location" to Android Home

android/app/src/main/cpp/native.cpp

@@ -22,6 +22,7 @@
 #include <AddressManager.h>
 #include "AndroidHelper.h"
 #include <udt/PacketHeaders.h>
+#include <SettingHandle.h>
 
 QAndroidJniObject __interfaceActivity;
 QAndroidJniObject __loginCompletedListener;
@@ -172,7 +173,7 @@ JNIEXPORT void Java_io_highfidelity_hifiinterface_InterfaceActivity_nativeOnDest
 
 JNIEXPORT void Java_io_highfidelity_hifiinterface_InterfaceActivity_nativeGotoUrl(JNIEnv* env, jobject obj, jstring url) {
     QAndroidJniObject jniUrl("java/lang/String", "(Ljava/lang/String;)V", url);
-    DependencyManager::get<AddressManager>()->handleLookupString(jniUrl.toString());
+    DependencyManager::get<AddressManager>()->loadSettings(jniUrl.toString());
 }
 
 JNIEXPORT void Java_io_highfidelity_hifiinterface_InterfaceActivity_nativeOnPause(JNIEnv* env, jobject obj) {
@@ -210,6 +211,12 @@ JNIEXPORT jstring JNICALL Java_io_highfidelity_hifiinterface_HifiUtils_protocolV
     return env->NewStringUTF(protocolVersionsSignatureBase64().toLatin1().data());
 }
 
+JNIEXPORT jstring JNICALL Java_io_highfidelity_hifiinterface_fragment_HomeFragment_nativeGetLastLocation(JNIEnv *env, jobject instance) {
+    Setting::Handle<QUrl> currentAddressHandle(QStringList() << "AddressManager" << "address", QString());
+    QUrl lastLocation = currentAddressHandle.get();
+    return env->NewStringUTF(lastLocation.toString().toLatin1().data());
+}
+
 JNIEXPORT void JNICALL
 Java_io_highfidelity_hifiinterface_fragment_LoginFragment_nativeLogin(JNIEnv *env, jobject instance,
                                                             jstring username_, jstring password_,

android/app/src/main/java/io/highfidelity/hifiinterface/fragment/HomeFragment.java

@@ -33,6 +33,8 @@ public class HomeFragment extends Fragment {
 
     private OnHomeInteractionListener mListener;
 
+    public native String nativeGetLastLocation();
+
     public HomeFragment() {
         // Required empty public constructor
     }
@@ -60,7 +62,7 @@ public class HomeFragment extends Fragment {
         int numberOfColumns = 1;
         GridLayoutManager gridLayoutMgr = new GridLayoutManager(getContext(), numberOfColumns);
         mDomainsView.setLayoutManager(gridLayoutMgr);
-        mDomainAdapter = new DomainAdapter(getContext(), HifiUtils.getInstance().protocolVersionSignature());
+        mDomainAdapter = new DomainAdapter(getContext(), HifiUtils.getInstance().protocolVersionSignature(), nativeGetLastLocation());
         mDomainAdapter.setClickListener((view, position, domain) -> {
             new Handler(getActivity().getMainLooper()).postDelayed(() -> mListener.onSelectedDomain(domain.url), 400); // a delay so the ripple effect can be seen
         });

android/app/src/main/java/io/highfidelity/hifiinterface/view/DomainAdapter.java

@@ -12,13 +12,6 @@ import android.widget.TextView;
 
 import com.squareup.picasso.Picasso;
 
-import org.json.JSONArray;
-import org.json.JSONException;
-import org.json.JSONObject;
-
-import java.io.IOException;
-import java.io.InputStream;
-import java.util.ArrayList;
 import java.util.List;
 
 import io.highfidelity.hifiinterface.R;
@@ -31,20 +24,23 @@ import io.highfidelity.hifiinterface.provider.UserStoryDomainProvider;
 public class DomainAdapter extends RecyclerView.Adapter<DomainAdapter.ViewHolder> {
 
     private static final String TAG = "HiFi Interface";
+    private static final String DEFAULT_THUMBNAIL_PLACE = "android.resource://io.highfidelity.hifiinterface/" + R.drawable.domain_placeholder;
     private Context mContext;
     private LayoutInflater mInflater;
     private ItemClickListener mClickListener;
     private String mProtocol;
+    private String mLastLocation;
     private UserStoryDomainProvider domainProvider;
     private AdapterListener mAdapterListener;
 
     // references to our domains
     private Domain[] mDomains = {};
 
-    public DomainAdapter(Context c, String protocol) {
+    public DomainAdapter(Context c, String protocol, String lastLocation) {
         mContext = c;
         this.mInflater = LayoutInflater.from(mContext);
         mProtocol = protocol;
+        mLastLocation = lastLocation;
         domainProvider = new UserStoryDomainProvider(mProtocol);
         loadDomains("");
     }
@@ -57,6 +53,29 @@ public class DomainAdapter extends RecyclerView.Adapter<DomainAdapter.ViewHolder
         domainProvider.retrieve(filterText, new DomainProvider.DomainCallback() {
             @Override
             public void retrieveOk(List<Domain> domain) {
+                if (filterText.length() == 0) {
+                    Domain lastVisitedDomain = new Domain(mContext.getString(R.string.your_last_location), mLastLocation, DEFAULT_THUMBNAIL_PLACE);
+                    if (!mLastLocation.isEmpty() && mLastLocation.contains("://")) {
+                        int startIndex = mLastLocation.indexOf("://");
+                        int endIndex = mLastLocation.indexOf("/", startIndex + 3);
+                        String toSearch = mLastLocation.substring(0, endIndex + 1).toLowerCase();
+                        for (Domain d : domain) {
+                            if (d.url.toLowerCase().startsWith(toSearch)) {
+                                lastVisitedDomain.thumbnail = d.thumbnail;
+                            }
+                        }
+                    }
+                    domain.add(0, lastVisitedDomain);
+                }
+
+                for (Domain d : domain) {
+                    // we override the default picture added in the server by an android specific version
+                    if (d.thumbnail != null &&
+                            d.thumbnail.endsWith("assets/places/thumbnail-default-place-e5a3f33e773ab699495774990a562f9f7693dc48ef90d8be6985c645a0280f75.png")) {
+                        d.thumbnail = DEFAULT_THUMBNAIL_PLACE;
+                    }
+                }
+
                 mDomains = new Domain[domain.size()];
                 mDomains = domain.toArray(mDomains);
                 notifyDataSetChanged();

android/app/src/main/res/values/strings.xml

@@ -20,5 +20,6 @@
     <string name="search_loading">Loading places…</string>
     <string name="search_no_results">No places exist with that name</string>
     <string name="privacyPolicy">Privacy Policy</string>
+    <string name="your_last_location">Your Last Location</string>
 
 </resources>

interface/src/Application.cpp

@@ -3312,19 +3312,18 @@ void Application::handleSandboxStatus(QNetworkReply* reply) {
 
     // If this is a first run we short-circuit the address passed in
     if (firstRun.get()) {
-#if defined(Q_OS_ANDROID)
-        qCDebug(interfaceapp) << "First run... going to" << qPrintable(addressLookupString.isEmpty() ? QString("default location") : addressLookupString);
-        DependencyManager::get<AddressManager>()->loadSettings(addressLookupString);
-#else
+#if !defined(Q_OS_ANDROID)
         DependencyManager::get<AddressManager>()->goToEntry();
         sentTo = SENT_TO_ENTRY;
 #endif
         firstRun.set(false);
 
     } else {
+#if !defined(Q_OS_ANDROID)
         qCDebug(interfaceapp) << "Not first run... going to" << qPrintable(addressLookupString.isEmpty() ? QString("previous location") : addressLookupString);
         DependencyManager::get<AddressManager>()->loadSettings(addressLookupString);
         sentTo = SENT_TO_PREVIOUS_LOCATION;
+#endif
     }
 
     UserActivityLogger::getInstance().logAction("startup_sent_to", {