Merge pull request #54 from MihaiCristianCondrea/codex/refactor-homerepository-and-dependencies

Refactor Home repository to remove Android dependencies

Author: MihaiCristianCondrea

app/src/main/java/com/d4rk/androidtutorials/java/data/repository/DefaultHomeRepository.java

@@ -1,10 +1,8 @@
 package com.d4rk.androidtutorials.java.data.repository;
 
-import android.content.Intent;
-
 import com.d4rk.androidtutorials.java.data.source.HomeLocalDataSource;
 import com.d4rk.androidtutorials.java.data.source.HomeRemoteDataSource;
-import com.d4rk.androidtutorials.java.data.source.HomeRemoteDataSource.PromotedAppsCallback;
+import com.d4rk.androidtutorials.java.data.repository.HomeRepository.PromotedAppsCallback;
 
 /**
  * Default implementation of {@link HomeRepository} combining local and remote sources.
@@ -21,13 +19,13 @@ public DefaultHomeRepository(HomeRemoteDataSource remoteDataSource,
     }
 
     @Override
-    public Intent getPlayStoreIntent() {
-        return localDataSource.getPlayStoreIntent();
+    public String getPlayStoreUrl() {
+        return localDataSource.getPlayStoreUrl();
     }
 
     @Override
-    public Intent getAppPlayStoreIntent(String packageName) {
-        return localDataSource.getAppPlayStoreIntent(packageName);
+    public String getAppPlayStoreUrl(String packageName) {
+        return localDataSource.getAppPlayStoreUrl(packageName);
     }
 
     @Override
@@ -37,6 +35,6 @@ public String getDailyTip() {
 
     @Override
     public void fetchPromotedApps(PromotedAppsCallback callback) {
-        remoteDataSource.fetchPromotedApps(callback);
+        remoteDataSource.fetchPromotedApps(callback::onResult);
     }
 }

app/src/main/java/com/d4rk/androidtutorials/java/data/repository/HomeRepository.java

@@ -1,20 +1,22 @@
 package com.d4rk.androidtutorials.java.data.repository;
 
-import android.content.Intent;
-
 import com.d4rk.androidtutorials.java.data.model.PromotedApp;
-import com.d4rk.androidtutorials.java.data.source.HomeRemoteDataSource.PromotedAppsCallback;
+import java.util.List;
 
 /**
  * Abstraction over home data operations.
  */
 public interface HomeRepository {
 
-    Intent getPlayStoreIntent();
+    String getPlayStoreUrl();
 
-    Intent getAppPlayStoreIntent(String packageName);
+    String getAppPlayStoreUrl(String packageName);
 
     String getDailyTip();
 
     void fetchPromotedApps(PromotedAppsCallback callback);
+
+    interface PromotedAppsCallback {
+        void onResult(List<PromotedApp> apps);
+    }
 }

app/src/main/java/com/d4rk/androidtutorials/java/data/source/DefaultHomeLocalDataSource.java

@@ -1,8 +1,6 @@
 package com.d4rk.androidtutorials.java.data.source;
 
 import android.content.Context;
-import android.content.Intent;
-import android.net.Uri;
 
 import com.d4rk.androidtutorials.java.R;
 
@@ -18,15 +16,13 @@ public DefaultHomeLocalDataSource(Context context) {
     }
 
     @Override
-    public Intent getPlayStoreIntent() {
-        String playStoreUrl = "https://play.google.com/store/apps/details?id=com.d4rk.androidtutorials";
-        return buildPlayStoreIntent(playStoreUrl);
+    public String getPlayStoreUrl() {
+        return "https://play.google.com/store/apps/details?id=com.d4rk.androidtutorials";
     }
 
     @Override
-    public Intent getAppPlayStoreIntent(String packageName) {
-        String url = "https://play.google.com/store/apps/details?id=" + packageName;
-        return buildPlayStoreIntent(url);
+    public String getAppPlayStoreUrl(String packageName) {
+        return "https://play.google.com/store/apps/details?id=" + packageName;
     }
 
     @Override
@@ -37,12 +33,4 @@ public String getDailyTip() {
         return tips[index];
     }
 
-    private Intent buildPlayStoreIntent(String url) {
-        Intent playStoreIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
-        playStoreIntent.setPackage("com.android.vending");
-        if (playStoreIntent.resolveActivity(context.getPackageManager()) != null) {
-            return playStoreIntent;
-        }
-        return new Intent(Intent.ACTION_VIEW, Uri.parse(url));
-    }
 }

app/src/main/java/com/d4rk/androidtutorials/java/data/source/HomeLocalDataSource.java

@@ -1,15 +1,13 @@
 package com.d4rk.androidtutorials.java.data.source;
 
-import android.content.Intent;
-
 /**
  * Local data access for the home feature.
  */
 public interface HomeLocalDataSource {
 
-    Intent getPlayStoreIntent();
+    String getPlayStoreUrl();
 
-    Intent getAppPlayStoreIntent(String packageName);
+    String getAppPlayStoreUrl(String packageName);
 
     String getDailyTip();
 }

app/src/main/java/com/d4rk/androidtutorials/java/ui/screens/home/HomeViewModel.java

@@ -2,6 +2,7 @@
 
 import android.app.Application;
 import android.content.Intent;
+import android.net.Uri;
 
 import androidx.annotation.NonNull;
 import androidx.lifecycle.AndroidViewModel;
@@ -90,7 +91,7 @@ public LiveData<String> getDailyTip() {
      * The HomeFragment can startActivity(...) on it.
      */
     public Intent getOpenPlayStoreIntent() {
-        return homeRepository.getPlayStoreIntent();
+        return buildPlayStoreIntent(homeRepository.getPlayStoreUrl());
     }
 
     /**
@@ -104,6 +105,15 @@ public LiveData<List<PromotedApp>> getPromotedApps() {
      * Builds an intent to open the Google Play listing for the provided package.
      */
     public Intent getPromotedAppIntent(String packageName) {
-        return homeRepository.getAppPlayStoreIntent(packageName);
+        return buildPlayStoreIntent(homeRepository.getAppPlayStoreUrl(packageName));
+    }
+
+    private Intent buildPlayStoreIntent(String url) {
+        Intent playStoreIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
+        playStoreIntent.setPackage("com.android.vending");
+        if (playStoreIntent.resolveActivity(getApplication().getPackageManager()) != null) {
+            return playStoreIntent;
+        }
+        return new Intent(Intent.ACTION_VIEW, Uri.parse(url));
     }
 }