File provider fix

iamSahdeep · Jun 9, 2019 · ac0bcf8 · ac0bcf8
1 parent 987659d
commit ac0bcf8
Show file tree

Hide file tree

Showing 7 changed files with 33 additions and 8 deletions.
diff --git a/.idea/caches/build_file_checksums.ser b/.idea/caches/build_file_checksums.ser
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -70,6 +70,17 @@
             </intent-filter>
         </activity>
         <activity android:name=".Activities.PlayerView" />
+
+        <provider
+            android:name="androidx.core.content.FileProvider"
+            android:authorities="com.sahdeepsingh.Bop"
+            android:exported="false"
+            android:grantUriPermissions="true">
+            <meta-data
+                android:name="android.support.FILE_PROVIDER_PATHS"
+                android:resource="@xml/file_provider_paths" />
+        </provider>
+
     </application>
 
 </manifest>
diff --git a/app/src/main/java/com/sahdeepsingh/Bop/BopUtils/ExtraUtils.java b/app/src/main/java/com/sahdeepsingh/Bop/BopUtils/ExtraUtils.java
@@ -21,6 +21,7 @@
 import com.sahdeepsingh.Bop.fragments.SongDetailsFragment;
 import com.sahdeepsingh.Bop.playerMain.Main;
 
+import java.io.File;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.LinkedHashMap;
@@ -31,6 +32,7 @@
 import androidx.appcompat.app.AppCompatActivity;
 import androidx.browser.customtabs.CustomTabsIntent;
 import androidx.core.content.ContextCompat;
+import androidx.core.content.FileProvider;
 import androidx.palette.graphics.Palette;
 
 public class ExtraUtils {
@@ -127,11 +129,13 @@ public static int getThemeColor(Context context, int colorPrimary, int dkgray) {
     }
 
     public static void shareSong(Context context, Song song) {
-        Intent share = new Intent(Intent.ACTION_SEND);
-        share.setType("audio/*");
-        share.putExtra(Intent.EXTRA_STREAM, Uri.parse("content:///" + song.getFilePath()));
-        context.startActivity(Intent.createChooser(share, "Share Sound File"));
-        context.getContentResolver().notifyChange(Uri.parse("content://media"), null);
+
+        Intent shareIntent = new Intent(Intent.ACTION_SEND);
+        shareIntent.setType("audio/*");
+        Uri fileUri = FileProvider.getUriForFile(context, "com.sahdeepsingh.Bop", new File(song.getFilePath()));
+        shareIntent.addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION);
+        shareIntent.putExtra(Intent.EXTRA_STREAM, fileUri);
+        context.startActivity(Intent.createChooser(shareIntent, "Share File"));
     }
 
     public static void showSongDetails(Context context, Long id) {

diff --git a/app/src/main/java/com/sahdeepsingh/Bop/fragments/SongDetailsFragment.java b/app/src/main/java/com/sahdeepsingh/Bop/fragments/SongDetailsFragment.java
@@ -1,6 +1,8 @@
 package com.sahdeepsingh.Bop.fragments;
 
 
+import android.content.Intent;
+import android.net.Uri;
 import android.os.Bundle;
 import android.text.format.DateUtils;
 import android.view.LayoutInflater;
@@ -15,8 +17,11 @@
 import com.sahdeepsingh.Bop.SongData.Song;
 import com.squareup.picasso.Picasso;
 
+import androidx.core.content.FileProvider;
 import androidx.fragment.app.Fragment;
 
+import java.io.File;
+
 public class SongDetailsFragment extends Fragment {
 
     private static final String Songparam = "param1";
@@ -102,6 +107,7 @@ private void addtoPlaylist(View v) {
     }
 
     private void shareSong(View v) {
+        ExtraUtils.shareSong(v.getContext(),song);
 
     }
 

diff --git a/app/src/main/res/xml/file_provider_paths.xml b/app/src/main/res/xml/file_provider_paths.xml
@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
+<paths xmlns:android="http://schemas.android.com/apk/res/android">
+    <external-path name="Share Audio" path="audio/" />
+</paths>
diff --git a/build.gradle b/build.gradle
@@ -7,7 +7,7 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:3.3.1'
+        classpath 'com.android.tools.build:gradle:3.4.1'
 
 
         // NOTE: Do not place your application dependencies here; they belong

diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties
@@ -1,6 +1,6 @@
-#Mon Mar 11 22:43:13 IST 2019
+#Thu May 30 18:51:28 IST 2019
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.1-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-5.1.1-all.zip