From 8c55fbbde9927cf01d7f7d2236bb6fb89eb2c67c Mon Sep 17 00:00:00 2001
From: Alex Ning <chineseperson5@gmail.com>
Date: Fri, 12 Jul 2019 17:24:02 +0800
Subject: Fixed issues of saving images and videos on Android Q devices.

---
 .../infinityforreddit/ViewImageActivity.java       | 60 ++++++++++++++++++----
 .../infinityforreddit/ViewVideoActivity.java       | 14 ++++-
 2 files changed, 64 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/ml/docilealligator/infinityforreddit/ViewImageActivity.java b/app/src/main/java/ml/docilealligator/infinityforreddit/ViewImageActivity.java
index 1dcba0c1..73b3148b 100644
--- a/app/src/main/java/ml/docilealligator/infinityforreddit/ViewImageActivity.java
+++ b/app/src/main/java/ml/docilealligator/infinityforreddit/ViewImageActivity.java
@@ -5,16 +5,21 @@ import android.animation.Animator;
 import android.animation.ArgbEvaluator;
 import android.animation.ValueAnimator;
 import android.annotation.SuppressLint;
+import android.content.ContentResolver;
+import android.content.ContentValues;
 import android.content.Intent;
 import android.content.pm.PackageManager;
 import android.graphics.Bitmap;
 import android.graphics.PorterDuff;
 import android.graphics.drawable.ColorDrawable;
 import android.graphics.drawable.Drawable;
+import android.net.Uri;
 import android.os.AsyncTask;
 import android.os.Build;
 import android.os.Bundle;
 import android.os.Environment;
+import android.os.ParcelFileDescriptor;
+import android.provider.MediaStore;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.MotionEvent;
@@ -437,17 +442,54 @@ public class ViewImageActivity extends AppCompatActivity {
                                         }
                                     }
 
-                                    File file = new File(path + "/Infinity/", mImageFileName + ".jpg");
-                                    int postfix = 1;
-                                    while(file.exists()) {
-                                        file = new File(path + "/Infinity/", mImageFileName + "-" + postfix + ".jpg");
-                                    }
-                                    OutputStream outputStream = new FileOutputStream(file);
+                                    //Android Q support
+                                    if(android.os.Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
+                                        ContentValues values = new ContentValues();
+                                        values.put(MediaStore.Images.Media.DISPLAY_NAME, mImageFileName + ".jpg");
+                                        values.put(MediaStore.Images.Media.MIME_TYPE, "image/jpeg");
+                                        values.put(MediaStore.Images.Media.IS_PENDING, 1);
+
+                                        ContentResolver resolver = getContentResolver();
+                                        Uri collection = MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL_PRIMARY);
+                                        Uri item = resolver.insert(collection, values);
+
+                                        if(item == null) {
+                                            saveSuccess = false;
+                                            return null;
+                                        }
+
+                                        try (ParcelFileDescriptor pfd = resolver.openFileDescriptor(item, "w", null)) {
+                                            if(pfd == null) {
+                                                saveSuccess = false;
+                                                return null;
+                                            }
 
-                                    resource.compress(Bitmap.CompressFormat.JPEG, 100, outputStream);
+                                            FileOutputStream outputStream = new FileOutputStream(pfd.getFileDescriptor());
+                                            resource.compress(Bitmap.CompressFormat.JPEG, 100, outputStream);
 
-                                    outputStream.flush();
-                                    outputStream.close();
+                                            outputStream.flush();
+                                            outputStream.close();
+                                        } catch (IOException e) {
+                                            saveSuccess = false;
+                                            e.printStackTrace();
+                                        }
+
+                                        values.clear();
+                                        values.put(MediaStore.Images.Media.IS_PENDING, 0);
+                                        resolver.update(item, values, null, null);
+                                    } else {
+                                        File file = new File(path + "/Infinity/", mImageFileName + ".jpg");
+                                        int postfix = 1;
+                                        while(file.exists()) {
+                                            file = new File(path + "/Infinity/", mImageFileName + "-" + (postfix++) + ".jpg");
+                                        }
+                                        OutputStream outputStream = new FileOutputStream(file);
+
+                                        resource.compress(Bitmap.CompressFormat.JPEG, 100, outputStream);
+
+                                        outputStream.flush();
+                                        outputStream.close();
+                                    }
                                 } catch (IOException e) {
                                     saveSuccess = false;
                                     e.printStackTrace();
diff --git a/app/src/main/java/ml/docilealligator/infinityforreddit/ViewVideoActivity.java b/app/src/main/java/ml/docilealligator/infinityforreddit/ViewVideoActivity.java
index 976ff8c5..5d8a3e07 100644
--- a/app/src/main/java/ml/docilealligator/infinityforreddit/ViewVideoActivity.java
+++ b/app/src/main/java/ml/docilealligator/infinityforreddit/ViewVideoActivity.java
@@ -385,9 +385,21 @@ public class ViewVideoActivity extends AppCompatActivity {
 
         request.allowScanningByMediaScanner();
         request.setNotificationVisibility(DownloadManager.Request.VISIBILITY_VISIBLE_NOTIFY_COMPLETED);
-        request.setDestinationInExternalPublicDir(Environment.DIRECTORY_PICTURES + "/Infinity/", mGifOrVideoFileName);
+
+        //Android Q support
+        if(android.os.Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
+            request.setDestinationInExternalPublicDir(Environment.DIRECTORY_DOWNLOADS, mGifOrVideoFileName);
+        } else {
+            request.setDestinationInExternalPublicDir(Environment.DIRECTORY_PICTURES + "/Infinity/", mGifOrVideoFileName);
+        }
 
         DownloadManager manager = (DownloadManager) getSystemService(Context.DOWNLOAD_SERVICE);
+
+        if(manager == null) {
+            Toast.makeText(this, R.string.download_failed, Toast.LENGTH_SHORT).show();
+            return;
+        }
+
         manager.enqueue(request);
     }
 
-- 
cgit v1.2.3