From 30c7daa9cb05019d6c43b55c50e0107b84ce9ded Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Mon, 28 Feb 2022 20:26:56 +0530
Subject: [PATCH 1/7] image storing & retrieving (Internal Storage)

---
 .idea/gradle.xml                              |   3 +-
 .idea/misc.xml                                |   9 +-
 .idea/runConfigurations.xml                   |  10 --
 .idea/vcs.xml                                 |   6 +
 .../plcoding/androidstorage/MainActivity.kt   | 103 +++++++++++++++++-
 5 files changed, 117 insertions(+), 14 deletions(-)
 delete mode 100644 .idea/runConfigurations.xml
 create mode 100644 .idea/vcs.xml

diff --git a/.idea/gradle.xml b/.idea/gradle.xml
index 440480e..526b4c2 100644
--- a/.idea/gradle.xml
+++ b/.idea/gradle.xml
@@ -1,9 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
+  <component name="GradleMigrationSettings" migrationVersion="1" />
   <component name="GradleSettings">
     <option name="linkedExternalProjectsSettings">
       <GradleProjectSettings>
-        <option name="testRunner" value="PLATFORM" />
+        <option name="testRunner" value="GRADLE" />
         <option name="distributionType" value="DEFAULT_WRAPPED" />
         <option name="externalProjectPath" value="$PROJECT_DIR$" />
         <option name="modules">
diff --git a/.idea/misc.xml b/.idea/misc.xml
index 860da66..3b963d8 100644
--- a/.idea/misc.xml
+++ b/.idea/misc.xml
@@ -1,6 +1,13 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
-  <component name="ProjectRootManager" version="2" languageLevel="JDK_11" default="true" project-jdk-name="1.8" project-jdk-type="JavaSDK">
+  <component name="DesignSurface">
+    <option name="filePathToZoomLevelMap">
+      <map>
+        <entry key="..\:/Android Projects/AndroidStorage/app/src/main/res/layout/item_photo.xml" value="0.19947916666666668" />
+      </map>
+    </option>
+  </component>
+  <component name="ProjectRootManager" version="2" languageLevel="JDK_11" default="true" project-jdk-name="11" project-jdk-type="JavaSDK">
     <output url="file://$PROJECT_DIR$/build/classes" />
   </component>
   <component name="ProjectType">
diff --git a/.idea/runConfigurations.xml b/.idea/runConfigurations.xml
deleted file mode 100644
index 797acea..0000000
--- a/.idea/runConfigurations.xml
+++ /dev/null
@@ -1,10 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="RunConfigurationProducerService">
-    <option name="ignoredProducers">
-      <set>
-        <option value="com.android.tools.idea.compose.preview.runconfiguration.ComposePreviewRunConfigurationProducer" />
-      </set>
-    </option>
-  </component>
-</project>
\ No newline at end of file
diff --git a/.idea/vcs.xml b/.idea/vcs.xml
new file mode 100644
index 0000000..35eb1dd
--- /dev/null
+++ b/.idea/vcs.xml
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project version="4">
+  <component name="VcsDirectoryMappings">
+    <mapping directory="" vcs="Git" />
+  </component>
+</project>
\ No newline at end of file
diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index dbc263b..300e1ca 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -1,12 +1,111 @@
 package com.plcoding.androidstorage
 
-import androidx.appcompat.app.AppCompatActivity
+import android.graphics.Bitmap
+import android.graphics.BitmapFactory
 import android.os.Bundle
+import android.util.Log
+import android.widget.Toast
+import androidx.activity.result.contract.ActivityResultContracts
+import androidx.activity.result.launch
+import androidx.appcompat.app.AppCompatActivity
+import androidx.lifecycle.lifecycleScope
+import androidx.recyclerview.widget.RecyclerView
+import androidx.recyclerview.widget.StaggeredGridLayoutManager
+import com.plcoding.androidstorage.databinding.ActivityMainBinding
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
+import java.io.IOException
+import java.util.*
 
 class MainActivity : AppCompatActivity() {
 
+    private lateinit var mBinding: ActivityMainBinding
+    private lateinit var mInternalStoragePhotoAdapter: InternalStoragePhotoAdapter
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
-        setContentView(R.layout.activity_main)
+        mBinding = ActivityMainBinding.inflate(layoutInflater)
+        setContentView(mBinding.root)
+
+        val takePicture = registerForActivityResult(ActivityResultContracts.TakePicturePreview()) { bitmap ->
+            bitmap?.let {
+                val isPrivate = mBinding.switchPrivate.isChecked
+                if (isPrivate) {
+                    val isSaveSuccess = storeBitmapIntoInternalStorage(bitmap)
+                    if (isSaveSuccess) {
+                        loadDataIntoInternalStorageRecyclerView()
+                        Toast.makeText(this, "Picture saved successfully", Toast.LENGTH_SHORT)
+                            .show()
+                    } else
+                        Toast.makeText(this, "Failed to save picture", Toast.LENGTH_SHORT)
+                            .show()
+                }
+            }
+        }
+
+        mInternalStoragePhotoAdapter = InternalStoragePhotoAdapter {
+            val isDeleteSuccess = deleteFile(it.name)
+            if (isDeleteSuccess) {
+                loadDataIntoInternalStorageRecyclerView()
+                Toast.makeText(this, "Picture delete successfully", Toast.LENGTH_SHORT)
+                    .show()
+            } else
+                Toast.makeText(this, "Failed to delete picture", Toast.LENGTH_SHORT)
+                    .show()
+        }
+
+        mBinding.btnTakePhoto.setOnClickListener {
+            takePicture.launch()
+        }
+
+        setUpInternalStorageRecyclerView()
+
+        loadDataIntoInternalStorageRecyclerView()
+    }
+
+    private fun setUpInternalStorageRecyclerView() = mBinding.rvPrivatePhotos.apply {
+        adapter = mInternalStoragePhotoAdapter
+        layoutManager = StaggeredGridLayoutManager(3,RecyclerView.VERTICAL)
+    }
+
+    private suspend fun getAllInternalImages(): List<InternalStoragePhoto> {
+        return withContext(Dispatchers.IO) {
+            val files = filesDir.listFiles() // get the root folder files.
+            files?.filter { file ->
+                file != null && file.canRead() && file.name.endsWith(".jpg")
+            }?.map { file ->
+                val bitmap =
+                    BitmapFactory.decodeByteArray(file.readBytes(), 0, file.readBytes().size)
+                InternalStoragePhoto(file.name, bitmap)
+            } ?: listOf()
+        }
+    }
+
+    private fun loadDataIntoInternalStorageRecyclerView() {
+        lifecycleScope.launch {
+            val pictures = getAllInternalImages()
+            mInternalStoragePhotoAdapter.submitList(pictures)
+        }
+    }
+
+    private fun storeBitmapIntoInternalStorage(bitmap: Bitmap): Boolean {
+        return try {
+            val fileName = "${UUID.randomUUID()}.jpg" // random file name.
+            openFileOutput(fileName, MODE_PRIVATE).use { outputStream ->
+                val isSaved = bitmap.compress(Bitmap.CompressFormat.JPEG, 95, outputStream)
+                if (!isSaved)
+                    throw IOException("Failed to saved bitmap into internal storage.")
+            }
+            true
+        } catch (e: IOException) {
+            e.printStackTrace()
+            Log.e(TAG, "storeBitmapIntoInternalStorage: ${e.message}")
+            false
+        }
+    }
+
+    companion object {
+        private const val TAG = "MainActivity"
     }
 }
\ No newline at end of file

From b9d7ec8ca03e298900d112fe52c0126f262d7a50 Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Tue, 1 Mar 2022 15:51:29 +0530
Subject: [PATCH 2/7] loading files from external storage

---
 app/src/main/AndroidManifest.xml              |   6 +
 .../plcoding/androidstorage/MainActivity.kt   | 320 +++++++++++++++---
 .../plcoding/androidstorage/StorageUtil.kt    |  13 +
 3 files changed, 297 insertions(+), 42 deletions(-)
 create mode 100644 app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 53e8047..66f5f23 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -2,6 +2,12 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="com.plcoding.androidstorage">
 
+    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
+    <!--Write storage permission required only for the devices running on API level 28 & below-->
+    <uses-permission
+        android:name="android.permission.WRITE_EXTERNAL_STORAGE"
+        android:maxSdkVersion="28" />
+
     <application
         android:allowBackup="true"
         android:icon="@mipmap/ic_launcher"
diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index 300e1ca..efbc04e 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -1,13 +1,22 @@
 package com.plcoding.androidstorage
 
+import android.Manifest
+import android.content.ContentUris
+import android.content.ContentValues
+import android.content.pm.PackageManager
+import android.database.ContentObserver
 import android.graphics.Bitmap
 import android.graphics.BitmapFactory
+import android.os.Build
 import android.os.Bundle
+import android.provider.MediaStore
 import android.util.Log
 import android.widget.Toast
+import androidx.activity.result.ActivityResultLauncher
 import androidx.activity.result.contract.ActivityResultContracts
 import androidx.activity.result.launch
 import androidx.appcompat.app.AppCompatActivity
+import androidx.core.content.ContextCompat
 import androidx.lifecycle.lifecycleScope
 import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.StaggeredGridLayoutManager
@@ -21,39 +30,90 @@ import java.util.*
 class MainActivity : AppCompatActivity() {
 
     private lateinit var mBinding: ActivityMainBinding
+
     private lateinit var mInternalStoragePhotoAdapter: InternalStoragePhotoAdapter
 
+    private lateinit var mSharedPhotoAdapter: SharedPhotoAdapter
+
+    private lateinit var mPermissionLauncher: ActivityResultLauncher<Array<String>>
+
+    private lateinit var mContentObserver: ContentObserver
+
+    private var readPermissionGranted = false
+
+    private var writePermissionGranted = false
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
+
         mBinding = ActivityMainBinding.inflate(layoutInflater)
+
         setContentView(mBinding.root)
 
-        val takePicture = registerForActivityResult(ActivityResultContracts.TakePicturePreview()) { bitmap ->
-            bitmap?.let {
-                val isPrivate = mBinding.switchPrivate.isChecked
-                if (isPrivate) {
-                    val isSaveSuccess = storeBitmapIntoInternalStorage(bitmap)
-                    if (isSaveSuccess) {
-                        loadDataIntoInternalStorageRecyclerView()
-                        Toast.makeText(this, "Picture saved successfully", Toast.LENGTH_SHORT)
-                            .show()
+        mInternalStoragePhotoAdapter = InternalStoragePhotoAdapter {
+            lifecycleScope.launch {
+                val isDeleteSuccess = deletePhotoFromInternalStorage(it.name)
+                if (isDeleteSuccess) {
+                    loadDataIntoInternalStorageAdapter()
+                    Toast.makeText(this@MainActivity, "Picture delete successfully", Toast.LENGTH_SHORT)
+                        .show()
+                } else
+                    Toast.makeText(this@MainActivity, "Failed to delete picture", Toast.LENGTH_SHORT)
+                        .show() }
+        }
+
+        mSharedPhotoAdapter = SharedPhotoAdapter {
+            // todo : delete image
+        }
+
+        setUpSharedStorageRecyclerView()
+
+        initContentObserver()
+
+        mPermissionLauncher =
+            registerForActivityResult(ActivityResultContracts.RequestMultiplePermissions()) { permissions ->
+                lifecycleScope.launch {
+                    readPermissionGranted = permissions[Manifest.permission.READ_EXTERNAL_STORAGE]
+                        ?: readPermissionGranted
+                    writePermissionGranted = permissions[Manifest.permission.WRITE_EXTERNAL_STORAGE]
+                        ?: writePermissionGranted
+
+                    if (readPermissionGranted) {
+                        loadDataIntoSharedStorageAdapter()
                     } else
-                        Toast.makeText(this, "Failed to save picture", Toast.LENGTH_SHORT)
-                            .show()
+                        Toast.makeText(
+                            this@MainActivity,
+                            "Permission required to load images",
+                            Toast.LENGTH_SHORT
+                        ).show()
                 }
             }
-        }
 
-        mInternalStoragePhotoAdapter = InternalStoragePhotoAdapter {
-            val isDeleteSuccess = deleteFile(it.name)
-            if (isDeleteSuccess) {
-                loadDataIntoInternalStorageRecyclerView()
-                Toast.makeText(this, "Picture delete successfully", Toast.LENGTH_SHORT)
-                    .show()
-            } else
-                Toast.makeText(this, "Failed to delete picture", Toast.LENGTH_SHORT)
-                    .show()
-        }
+        requestPermissions()
+
+        val takePicture =
+            registerForActivityResult(ActivityResultContracts.TakePicturePreview()) { bitmap ->
+               lifecycleScope.launch{
+                   bitmap?.let {
+                       val isPrivate = mBinding.switchPrivate.isChecked
+
+                       val isSaveSuccessfully = when {
+                           isPrivate -> saveBitmapIntoInternalStorage(bitmap)
+                           writePermissionGranted -> saveBitmapIntoExternalStorage(bitmap)
+                           else -> false
+                       }
+
+                       if (isSaveSuccessfully) {
+                           loadDataIntoInternalStorageAdapter()
+                           Toast.makeText(this@MainActivity, "Picture saved successfully", Toast.LENGTH_SHORT)
+                               .show()
+                       } else
+                           Toast.makeText(this@MainActivity, "Failed to save picture", Toast.LENGTH_SHORT)
+                               .show()
+
+                   }
+               }
+            }
 
         mBinding.btnTakePhoto.setOnClickListener {
             takePicture.launch()
@@ -61,15 +121,113 @@ class MainActivity : AppCompatActivity() {
 
         setUpInternalStorageRecyclerView()
 
-        loadDataIntoInternalStorageRecyclerView()
+        loadDataIntoInternalStorageAdapter()
+
+//        loadDataIntoSharedStorageAdapter()
+    }
+
+    private fun initContentObserver() {
+        mContentObserver =  object : ContentObserver(null) {
+            override fun onChange(selfChange: Boolean) {
+                if(readPermissionGranted){
+                    loadDataIntoSharedStorageAdapter()
+                }
+            }
+        }
+
+        // register to content observer for listening any changes made at image collection on shared storage.
+        contentResolver.registerContentObserver(MediaStore.Images.Media.EXTERNAL_CONTENT_URI,true,mContentObserver)
+    }
+
+    /**
+     * set up recyclerview
+     */
+    private fun setUpSharedStorageRecyclerView() = mBinding.rvPublicPhotos.apply {
+        adapter = mSharedPhotoAdapter
+        layoutManager = StaggeredGridLayoutManager(3, RecyclerView.VERTICAL)
     }
 
     private fun setUpInternalStorageRecyclerView() = mBinding.rvPrivatePhotos.apply {
         adapter = mInternalStoragePhotoAdapter
-        layoutManager = StaggeredGridLayoutManager(3,RecyclerView.VERTICAL)
+        layoutManager = StaggeredGridLayoutManager(3, RecyclerView.VERTICAL)
     }
 
-    private suspend fun getAllInternalImages(): List<InternalStoragePhoto> {
+    /**
+     * loads data into adapter
+     */
+    private fun loadDataIntoInternalStorageAdapter() {
+        lifecycleScope.launch {
+            val pictures = getPrivateStorageImages()
+            mInternalStoragePhotoAdapter.submitList(pictures)
+        }
+    }
+
+    private fun loadDataIntoSharedStorageAdapter() {
+        lifecycleScope.launch {
+            val pictures = getSharedStorageImages()
+            mSharedPhotoAdapter.submitList(pictures)
+        }
+    }
+
+    /**
+     * save bitmap into storage
+     */
+    private suspend fun saveBitmapIntoInternalStorage(bitmap: Bitmap): Boolean {
+        return withContext(Dispatchers.IO) {
+            try {
+                val fileName = "${UUID.randomUUID()}.jpg" // random file name.
+                openFileOutput(fileName, MODE_PRIVATE).use { outputStream ->
+                    val isSaved = bitmap.compress(Bitmap.CompressFormat.JPEG, 95, outputStream)
+                    if (!isSaved)
+                        throw IOException("Failed to saved bitmap into internal storage.")
+                }
+                true
+            } catch (e: IOException) {
+                e.printStackTrace()
+                Log.e(TAG, "storeBitmapIntoInternalStorage: ${e.message}")
+                false
+            }
+        }
+    }
+
+    private suspend fun saveBitmapIntoExternalStorage(bitmap: Bitmap): Boolean {
+        return withContext(Dispatchers.IO) {
+            try {
+                val imageCollection = isSDK29AndUp {
+                    MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL_PRIMARY) // content uri for sdk level 29 or higher
+                }
+                    ?: MediaStore.Images.Media.EXTERNAL_CONTENT_URI // content uri for sdk level lower than 29.
+
+                val displayName = UUID.randomUUID().toString()
+
+                val contentValues = ContentValues().apply {
+                    put(MediaStore.Images.Media.DISPLAY_NAME, "$displayName.jpg")
+                    put(MediaStore.Images.Media.WIDTH, bitmap.width)
+                    put(MediaStore.Images.Media.HEIGHT, bitmap.height)
+                    put(MediaStore.Images.Media.MIME_TYPE, "image/jpeg")
+                }
+
+                contentResolver.insert(imageCollection, contentValues)?.also { uri ->
+                    contentResolver.openOutputStream(uri).use {
+                        if (!bitmap.compress(Bitmap.CompressFormat.JPEG, 99, it)) {
+                            throw IOException("Couldn't save the bitmap in shared storage")
+                        }
+                    }
+                } ?: throw IOException("Couldn't create media store entry")
+
+                true
+            } catch (e: IOException) {
+                e.printStackTrace()
+                Log.e(TAG, "saveBitmapIntoExternalStorage: ${e.message}")
+                false
+            }
+        }
+    }
+
+    /**
+     * Gets image from the storage
+     */
+    private suspend fun getPrivateStorageImages(): List<InternalStoragePhoto> {
         return withContext(Dispatchers.IO) {
             val files = filesDir.listFiles() // get the root folder files.
             files?.filter { file ->
@@ -82,30 +240,108 @@ class MainActivity : AppCompatActivity() {
         }
     }
 
-    private fun loadDataIntoInternalStorageRecyclerView() {
-        lifecycleScope.launch {
-            val pictures = getAllInternalImages()
-            mInternalStoragePhotoAdapter.submitList(pictures)
+    private suspend fun getSharedStorageImages(): List<SharedStoragePhoto> {
+        return withContext(Dispatchers.IO) {
+
+            val collection = isSDK29AndUp {
+                MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL)
+            } ?: MediaStore.Images.Media.EXTERNAL_CONTENT_URI
+
+            val projection = arrayOf(
+                MediaStore.Images.Media._ID,
+                MediaStore.Images.Media.DISPLAY_NAME,
+                MediaStore.Images.Media.WIDTH,
+                MediaStore.Images.Media.HEIGHT,
+            )
+
+            val sharedPhotosList = mutableListOf<SharedStoragePhoto>()
+
+            contentResolver.query(
+                collection,
+                projection,
+                null,
+                null,
+                "${MediaStore.Images.Media.DISPLAY_NAME} ASC"
+            )?.use { cursor ->
+                val idColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media._ID)
+                val displayNameColumnIndex =
+                    cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
+                val widthColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
+                val heightColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
+
+                while (cursor.moveToNext()) {
+                    val id = cursor.getLong(idColumnIndex)
+                    val displayName = cursor.getString(displayNameColumnIndex)
+                    val width = cursor.getInt(widthColumnIndex)
+                    val height = cursor.getInt(heightColumnIndex)
+                    val imageUri = ContentUris.withAppendedId(MediaStore.Images.Media.EXTERNAL_CONTENT_URI, id)
+
+                    sharedPhotosList.add(
+                        SharedStoragePhoto(
+                            id,
+                            displayName,
+                            width,
+                            height,
+                            imageUri
+                        )
+                    )
+                }
+                sharedPhotosList.toList()
+            } ?: listOf()
         }
     }
 
-    private fun storeBitmapIntoInternalStorage(bitmap: Bitmap): Boolean {
-        return try {
-            val fileName = "${UUID.randomUUID()}.jpg" // random file name.
-            openFileOutput(fileName, MODE_PRIVATE).use { outputStream ->
-                val isSaved = bitmap.compress(Bitmap.CompressFormat.JPEG, 95, outputStream)
-                if (!isSaved)
-                    throw IOException("Failed to saved bitmap into internal storage.")
+
+    /**
+     * Delete image
+     */
+    private suspend fun deletePhotoFromInternalStorage(filename: String): Boolean {
+        return withContext(Dispatchers.IO) {
+            try {
+                deleteFile(filename)
+            } catch (e: Exception) {
+                e.printStackTrace()
+                false
             }
-            true
-        } catch (e: IOException) {
-            e.printStackTrace()
-            Log.e(TAG, "storeBitmapIntoInternalStorage: ${e.message}")
-            false
         }
     }
 
+    private fun requestPermissions() {
+        val permissions = mutableListOf<String>()
+        val hasReadPermission = ContextCompat.checkSelfPermission(
+            this,
+            Manifest.permission.READ_EXTERNAL_STORAGE
+        ) == PackageManager.PERMISSION_GRANTED
+        val hasWritePermission = ContextCompat.checkSelfPermission(
+            this,
+            Manifest.permission.WRITE_EXTERNAL_STORAGE
+        ) == PackageManager.PERMISSION_GRANTED
+        val minSdk29 = Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q
+
+        readPermissionGranted = hasReadPermission
+
+        writePermissionGranted = hasWritePermission || minSdk29
+
+        if (!readPermissionGranted) {
+            permissions.add(Manifest.permission.READ_EXTERNAL_STORAGE)
+        }
+
+        if (!writePermissionGranted) {
+            permissions.add(Manifest.permission.WRITE_EXTERNAL_STORAGE)
+        }
+
+        if (permissions.isNotEmpty()) {
+            mPermissionLauncher.launch(permissions.toTypedArray())
+        }
+    }
+
+
+    override fun onDestroy() {
+        super.onDestroy()
+        contentResolver.unregisterContentObserver(mContentObserver)
+    }
+
     companion object {
         private const val TAG = "MainActivity"
     }
-}
\ No newline at end of file
+}
diff --git a/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt b/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt
new file mode 100644
index 0000000..4319a77
--- /dev/null
+++ b/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt
@@ -0,0 +1,13 @@
+package com.plcoding.androidstorage
+
+import android.os.Build
+
+/**
+ * Created By Dhruv Limbachiya on 01-03-2022 11:46 AM.
+ */
+
+inline fun <T> isSDK29AndUp(onSdk29: () -> T): T? {
+    return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
+        onSdk29()
+    } else null
+}
\ No newline at end of file

From 18e9a85c22bc8ba849b2bd1ca543e02d06202f8e Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Tue, 1 Mar 2022 17:02:30 +0530
Subject: [PATCH 3/7] loading files from external storage

---
 .../plcoding/androidstorage/MainActivity.kt   | 177 ++++++++++--------
 1 file changed, 103 insertions(+), 74 deletions(-)

diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index efbc04e..e61bd19 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -9,6 +9,8 @@ import android.graphics.Bitmap
 import android.graphics.BitmapFactory
 import android.os.Build
 import android.os.Bundle
+import android.os.Handler
+import android.os.Looper
 import android.provider.MediaStore
 import android.util.Log
 import android.widget.Toast
@@ -55,11 +57,19 @@ class MainActivity : AppCompatActivity() {
                 val isDeleteSuccess = deletePhotoFromInternalStorage(it.name)
                 if (isDeleteSuccess) {
                     loadDataIntoInternalStorageAdapter()
-                    Toast.makeText(this@MainActivity, "Picture delete successfully", Toast.LENGTH_SHORT)
+                    Toast.makeText(
+                        this@MainActivity,
+                        "Picture delete successfully",
+                        Toast.LENGTH_SHORT
+                    )
                         .show()
                 } else
-                    Toast.makeText(this@MainActivity, "Failed to delete picture", Toast.LENGTH_SHORT)
-                        .show() }
+                    Toast.makeText(
+                        this@MainActivity,
+                        "Failed to delete picture",
+                        Toast.LENGTH_SHORT
+                    ).show()
+            }
         }
 
         mSharedPhotoAdapter = SharedPhotoAdapter {
@@ -93,26 +103,37 @@ class MainActivity : AppCompatActivity() {
 
         val takePicture =
             registerForActivityResult(ActivityResultContracts.TakePicturePreview()) { bitmap ->
-               lifecycleScope.launch{
-                   bitmap?.let {
-                       val isPrivate = mBinding.switchPrivate.isChecked
-
-                       val isSaveSuccessfully = when {
-                           isPrivate -> saveBitmapIntoInternalStorage(bitmap)
-                           writePermissionGranted -> saveBitmapIntoExternalStorage(bitmap)
-                           else -> false
-                       }
-
-                       if (isSaveSuccessfully) {
-                           loadDataIntoInternalStorageAdapter()
-                           Toast.makeText(this@MainActivity, "Picture saved successfully", Toast.LENGTH_SHORT)
-                               .show()
-                       } else
-                           Toast.makeText(this@MainActivity, "Failed to save picture", Toast.LENGTH_SHORT)
-                               .show()
-
-                   }
-               }
+                lifecycleScope.launch {
+                    bitmap?.let {
+                        val isPrivate = mBinding.switchPrivate.isChecked
+
+                        val isSaveSuccessfully = when {
+                            isPrivate -> saveBitmapIntoInternalStorage(bitmap)
+                            writePermissionGranted -> saveBitmapIntoExternalStorage(bitmap)
+                            else -> false
+                        }
+
+                        if (isSaveSuccessfully) {
+                            Toast.makeText(
+                                this@MainActivity,
+                                "Picture saved successfully",
+                                Toast.LENGTH_SHORT
+                            ).show()
+                        } else
+                            Toast.makeText(
+                                this@MainActivity,
+                                "Failed to save picture",
+                                Toast.LENGTH_SHORT
+                            ).show()
+
+
+                        if (isPrivate) {
+                            loadDataIntoInternalStorageAdapter()
+                        } else {
+                            loadDataIntoSharedStorageAdapter()
+                        }
+                    }
+                }
             }
 
         mBinding.btnTakePhoto.setOnClickListener {
@@ -127,16 +148,20 @@ class MainActivity : AppCompatActivity() {
     }
 
     private fun initContentObserver() {
-        mContentObserver =  object : ContentObserver(null) {
+        mContentObserver = object : ContentObserver(null) {
             override fun onChange(selfChange: Boolean) {
-                if(readPermissionGranted){
+                if (selfChange && readPermissionGranted) {
                     loadDataIntoSharedStorageAdapter()
                 }
             }
         }
 
         // register to content observer for listening any changes made at image collection on shared storage.
-        contentResolver.registerContentObserver(MediaStore.Images.Media.EXTERNAL_CONTENT_URI,true,mContentObserver)
+        contentResolver.registerContentObserver(
+            MediaStore.Images.Media.EXTERNAL_CONTENT_URI,
+            true,
+            mContentObserver
+        )
     }
 
     /**
@@ -164,7 +189,9 @@ class MainActivity : AppCompatActivity() {
 
     private fun loadDataIntoSharedStorageAdapter() {
         lifecycleScope.launch {
-            val pictures = getSharedStorageImages()
+            val pictures = withContext(Dispatchers.IO) {
+                getSharedStorageImages()
+            }
             mSharedPhotoAdapter.submitList(pictures)
         }
     }
@@ -240,55 +267,57 @@ class MainActivity : AppCompatActivity() {
         }
     }
 
-    private suspend fun getSharedStorageImages(): List<SharedStoragePhoto> {
-        return withContext(Dispatchers.IO) {
-
-            val collection = isSDK29AndUp {
-                MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL)
-            } ?: MediaStore.Images.Media.EXTERNAL_CONTENT_URI
-
-            val projection = arrayOf(
-                MediaStore.Images.Media._ID,
-                MediaStore.Images.Media.DISPLAY_NAME,
-                MediaStore.Images.Media.WIDTH,
-                MediaStore.Images.Media.HEIGHT,
-            )
-
-            val sharedPhotosList = mutableListOf<SharedStoragePhoto>()
-
-            contentResolver.query(
-                collection,
-                projection,
-                null,
-                null,
-                "${MediaStore.Images.Media.DISPLAY_NAME} ASC"
-            )?.use { cursor ->
-                val idColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media._ID)
-                val displayNameColumnIndex =
-                    cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
-                val widthColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
-                val heightColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
-
-                while (cursor.moveToNext()) {
-                    val id = cursor.getLong(idColumnIndex)
-                    val displayName = cursor.getString(displayNameColumnIndex)
-                    val width = cursor.getInt(widthColumnIndex)
-                    val height = cursor.getInt(heightColumnIndex)
-                    val imageUri = ContentUris.withAppendedId(MediaStore.Images.Media.EXTERNAL_CONTENT_URI, id)
-
-                    sharedPhotosList.add(
-                        SharedStoragePhoto(
-                            id,
-                            displayName,
-                            width,
-                            height,
-                            imageUri
-                        )
+    private fun getSharedStorageImages(): List<SharedStoragePhoto> {
+        val collection = isSDK29AndUp {
+            MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL)
+        } ?: MediaStore.Images.Media.EXTERNAL_CONTENT_URI
+
+        val projection = arrayOf(
+            MediaStore.Images.Media._ID,
+            MediaStore.Images.Media.DISPLAY_NAME,
+            MediaStore.Images.Media.WIDTH,
+            MediaStore.Images.Media.HEIGHT,
+        )
+
+        val sharedPhotosList = mutableListOf<SharedStoragePhoto>()
+
+        contentResolver.query(
+            collection,
+            projection,
+            null,
+            null,
+            "${MediaStore.Images.Media.DISPLAY_NAME} ASC"
+        )?.use { cursor ->
+            val idColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media._ID)
+            val displayNameColumnIndex =
+                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
+            val widthColumnIndex =
+                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
+            val heightColumnIndex =
+                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
+
+            while (cursor.moveToNext()) {
+                val id = cursor.getLong(idColumnIndex)
+                val displayName = cursor.getString(displayNameColumnIndex)
+                val width = cursor.getInt(widthColumnIndex)
+                val height = cursor.getInt(heightColumnIndex)
+                val imageUri = ContentUris.withAppendedId(
+                    MediaStore.Images.Media.EXTERNAL_CONTENT_URI,
+                    id
+                )
+
+                sharedPhotosList.add(
+                    SharedStoragePhoto(
+                        id,
+                        displayName,
+                        width,
+                        height,
+                        imageUri
                     )
-                }
-                sharedPhotosList.toList()
-            } ?: listOf()
+                )
+            }
         }
+        return sharedPhotosList.toList()
     }
 
 

From a8d0636197b9394722f70c9746c33f0b7a43973c Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Tue, 1 Mar 2022 18:19:43 +0530
Subject: [PATCH 4/7] solve image loading bug

---
 app/build.gradle                              |  2 +
 .../plcoding/androidstorage/MainActivity.kt   | 31 +++++++------
 .../androidstorage/SharedPhotoAdapter.kt      | 43 ++++++++++++-------
 3 files changed, 44 insertions(+), 32 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index 8cee05c..f08cced 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -55,4 +55,6 @@ dependencies {
     // Coroutine Lifecycle Scopes
     implementation "androidx.lifecycle:lifecycle-viewmodel-ktx:2.3.1"
     implementation "androidx.lifecycle:lifecycle-runtime-ktx:2.3.1"
+
+    implementation 'com.github.bumptech.glide:glide:4.13.0'
 }
\ No newline at end of file
diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index e61bd19..99802ab 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -23,9 +23,7 @@ import androidx.lifecycle.lifecycleScope
 import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.StaggeredGridLayoutManager
 import com.plcoding.androidstorage.databinding.ActivityMainBinding
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.launch
-import kotlinx.coroutines.withContext
+import kotlinx.coroutines.*
 import java.io.IOException
 import java.util.*
 
@@ -76,10 +74,6 @@ class MainActivity : AppCompatActivity() {
             // todo : delete image
         }
 
-        setUpSharedStorageRecyclerView()
-
-        initContentObserver()
-
         mPermissionLauncher =
             registerForActivityResult(ActivityResultContracts.RequestMultiplePermissions()) { permissions ->
                 lifecycleScope.launch {
@@ -140,11 +134,16 @@ class MainActivity : AppCompatActivity() {
             takePicture.launch()
         }
 
+
+        initContentObserver()
+
         setUpInternalStorageRecyclerView()
 
+        setUpSharedStorageRecyclerView()
+
         loadDataIntoInternalStorageAdapter()
 
-//        loadDataIntoSharedStorageAdapter()
+        loadDataIntoSharedStorageAdapter()
     }
 
     private fun initContentObserver() {
@@ -289,14 +288,13 @@ class MainActivity : AppCompatActivity() {
             "${MediaStore.Images.Media.DISPLAY_NAME} ASC"
         )?.use { cursor ->
             val idColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media._ID)
-            val displayNameColumnIndex =
-                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
-            val widthColumnIndex =
-                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
-            val heightColumnIndex =
-                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
-
-            while (cursor.moveToNext()) {
+            val displayNameColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
+            val widthColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
+            val heightColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
+
+            var count = 20
+
+            while (cursor.moveToNext() && count-- >= 0) {
                 val id = cursor.getLong(idColumnIndex)
                 val displayName = cursor.getString(displayNameColumnIndex)
                 val width = cursor.getInt(widthColumnIndex)
@@ -315,6 +313,7 @@ class MainActivity : AppCompatActivity() {
                         imageUri
                     )
                 )
+                Log.i(TAG, "getSharedStorageImages: $displayName")
             }
         }
         return sharedPhotosList.toList()
diff --git a/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt b/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
index 4ee3793..1e0a672 100644
--- a/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
@@ -6,20 +6,31 @@ import androidx.constraintlayout.widget.ConstraintSet
 import androidx.recyclerview.widget.DiffUtil
 import androidx.recyclerview.widget.ListAdapter
 import androidx.recyclerview.widget.RecyclerView
+import com.bumptech.glide.Glide
 import com.plcoding.androidstorage.databinding.ItemPhotoBinding
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
+import java.io.File
 
 class SharedPhotoAdapter(
     private val onPhotoClick: (SharedStoragePhoto) -> Unit
 ) : ListAdapter<SharedStoragePhoto, SharedPhotoAdapter.PhotoViewHolder>(Companion) {
 
-    inner class PhotoViewHolder(val binding: ItemPhotoBinding): RecyclerView.ViewHolder(binding.root)
+    inner class PhotoViewHolder(val binding: ItemPhotoBinding) :
+        RecyclerView.ViewHolder(binding.root)
 
     companion object : DiffUtil.ItemCallback<SharedStoragePhoto>() {
-        override fun areItemsTheSame(oldItem: SharedStoragePhoto, newItem: SharedStoragePhoto): Boolean {
+        override fun areItemsTheSame(
+            oldItem: SharedStoragePhoto,
+            newItem: SharedStoragePhoto
+        ): Boolean {
             return oldItem.id == newItem.id
         }
 
-        override fun areContentsTheSame(oldItem: SharedStoragePhoto, newItem: SharedStoragePhoto): Boolean {
+        override fun areContentsTheSame(
+            oldItem: SharedStoragePhoto,
+            newItem: SharedStoragePhoto
+        ): Boolean {
             return oldItem == newItem
         }
     }
@@ -36,20 +47,20 @@ class SharedPhotoAdapter(
 
     override fun onBindViewHolder(holder: PhotoViewHolder, position: Int) {
         val photo = currentList[position]
-        holder.binding.apply {
-            ivPhoto.setImageURI(photo.contentUri)
-
-            val aspectRatio = photo.width.toFloat() / photo.height.toFloat()
-            ConstraintSet().apply {
-                clone(root)
-                setDimensionRatio(ivPhoto.id, aspectRatio.toString())
-                applyTo(root)
-            }
+            holder.binding.apply {
+                ivPhoto.setImageURI(photo.contentUri)
+
+                val aspectRatio = photo.width.toFloat() / photo.height.toFloat()
+                ConstraintSet().apply {
+                    clone(root)
+                    setDimensionRatio(ivPhoto.id, aspectRatio.toString())
+                    applyTo(root)
+                }
 
-            ivPhoto.setOnLongClickListener {
-                onPhotoClick(photo)
-                true
+                ivPhoto.setOnLongClickListener {
+                    onPhotoClick(photo)
+                    true
+                }
             }
-        }
     }
 }
\ No newline at end of file

From 141966c5ebb4351e800521813e94d9f66239411f Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Wed, 2 Mar 2022 11:23:54 +0530
Subject: [PATCH 5/7] Deleting image in Scope Storage

---
 .../plcoding/androidstorage/MainActivity.kt   | 101 +++++++++++++++---
 .../plcoding/androidstorage/StorageUtil.kt    |   8 +-
 2 files changed, 92 insertions(+), 17 deletions(-)

diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index 99802ab..1f61200 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -1,20 +1,21 @@
 package com.plcoding.androidstorage
 
 import android.Manifest
+import android.app.RecoverableSecurityException
 import android.content.ContentUris
 import android.content.ContentValues
 import android.content.pm.PackageManager
 import android.database.ContentObserver
 import android.graphics.Bitmap
 import android.graphics.BitmapFactory
+import android.net.Uri
 import android.os.Build
 import android.os.Bundle
-import android.os.Handler
-import android.os.Looper
 import android.provider.MediaStore
 import android.util.Log
 import android.widget.Toast
 import androidx.activity.result.ActivityResultLauncher
+import androidx.activity.result.IntentSenderRequest
 import androidx.activity.result.contract.ActivityResultContracts
 import androidx.activity.result.launch
 import androidx.appcompat.app.AppCompatActivity
@@ -23,7 +24,9 @@ import androidx.lifecycle.lifecycleScope
 import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.StaggeredGridLayoutManager
 import com.plcoding.androidstorage.databinding.ActivityMainBinding
-import kotlinx.coroutines.*
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 import java.io.IOException
 import java.util.*
 
@@ -37,12 +40,16 @@ class MainActivity : AppCompatActivity() {
 
     private lateinit var mPermissionLauncher: ActivityResultLauncher<Array<String>>
 
+    private lateinit var mIntentSenderLauncher: ActivityResultLauncher<IntentSenderRequest>
+
     private lateinit var mContentObserver: ContentObserver
 
     private var readPermissionGranted = false
 
     private var writePermissionGranted = false
 
+    private var mDeleteContentUri: Uri? = null
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
 
@@ -52,7 +59,7 @@ class MainActivity : AppCompatActivity() {
 
         mInternalStoragePhotoAdapter = InternalStoragePhotoAdapter {
             lifecycleScope.launch {
-                val isDeleteSuccess = deletePhotoFromInternalStorage(it.name)
+                val isDeleteSuccess = deleteImageFromInternalStorage(it.name)
                 if (isDeleteSuccess) {
                     loadDataIntoInternalStorageAdapter()
                     Toast.makeText(
@@ -71,16 +78,45 @@ class MainActivity : AppCompatActivity() {
         }
 
         mSharedPhotoAdapter = SharedPhotoAdapter {
-            // todo : delete image
+            lifecycleScope.launch {
+                deleteImageFromSharedStorage(it.contentUri)
+                mDeleteContentUri = it.contentUri
+            }
         }
 
+        mIntentSenderLauncher =
+            registerForActivityResult(ActivityResultContracts.StartIntentSenderForResult()) { result ->
+                if (result.resultCode == RESULT_OK) {
+                    if (isSDK29()) {
+                        lifecycleScope.launch {
+                            deleteImageFromSharedStorage(mDeleteContentUri ?: return@launch)
+                        }
+                    }
+                    Toast.makeText(
+                        this@MainActivity,
+                        "Image deleted successfully",
+                        Toast.LENGTH_SHORT
+                    ).show()
+                } else {
+                    Toast.makeText(
+                        this@MainActivity,
+                        "Failed to delete image",
+                        Toast.LENGTH_SHORT
+                    ).show()
+                }
+                loadDataIntoSharedStorageAdapter()
+            }
+
         mPermissionLauncher =
-            registerForActivityResult(ActivityResultContracts.RequestMultiplePermissions()) { permissions ->
+            registerForActivityResult(ActivityResultContracts.RequestMultiplePermissions())
+            { permissions ->
                 lifecycleScope.launch {
-                    readPermissionGranted = permissions[Manifest.permission.READ_EXTERNAL_STORAGE]
-                        ?: readPermissionGranted
-                    writePermissionGranted = permissions[Manifest.permission.WRITE_EXTERNAL_STORAGE]
-                        ?: writePermissionGranted
+                    readPermissionGranted =
+                        permissions[Manifest.permission.READ_EXTERNAL_STORAGE]
+                            ?: readPermissionGranted
+                    writePermissionGranted =
+                        permissions[Manifest.permission.WRITE_EXTERNAL_STORAGE]
+                            ?: writePermissionGranted
 
                     if (readPermissionGranted) {
                         loadDataIntoSharedStorageAdapter()
@@ -134,7 +170,6 @@ class MainActivity : AppCompatActivity() {
             takePicture.launch()
         }
 
-
         initContentObserver()
 
         setUpInternalStorageRecyclerView()
@@ -219,7 +254,7 @@ class MainActivity : AppCompatActivity() {
     private suspend fun saveBitmapIntoExternalStorage(bitmap: Bitmap): Boolean {
         return withContext(Dispatchers.IO) {
             try {
-                val imageCollection = isSDK29AndUp {
+                val imageCollection = isSDK30AndUp {
                     MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL_PRIMARY) // content uri for sdk level 29 or higher
                 }
                     ?: MediaStore.Images.Media.EXTERNAL_CONTENT_URI // content uri for sdk level lower than 29.
@@ -267,7 +302,7 @@ class MainActivity : AppCompatActivity() {
     }
 
     private fun getSharedStorageImages(): List<SharedStoragePhoto> {
-        val collection = isSDK29AndUp {
+        val collection = isSDK30AndUp {
             MediaStore.Images.Media.getContentUri(MediaStore.VOLUME_EXTERNAL)
         } ?: MediaStore.Images.Media.EXTERNAL_CONTENT_URI
 
@@ -288,7 +323,8 @@ class MainActivity : AppCompatActivity() {
             "${MediaStore.Images.Media.DISPLAY_NAME} ASC"
         )?.use { cursor ->
             val idColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media._ID)
-            val displayNameColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
+            val displayNameColumnIndex =
+                cursor.getColumnIndexOrThrow(MediaStore.Images.Media.DISPLAY_NAME)
             val widthColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
             val heightColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
 
@@ -323,7 +359,7 @@ class MainActivity : AppCompatActivity() {
     /**
      * Delete image
      */
-    private suspend fun deletePhotoFromInternalStorage(filename: String): Boolean {
+    private suspend fun deleteImageFromInternalStorage(filename: String): Boolean {
         return withContext(Dispatchers.IO) {
             try {
                 deleteFile(filename)
@@ -334,6 +370,41 @@ class MainActivity : AppCompatActivity() {
         }
     }
 
+    private suspend fun deleteImageFromSharedStorage(contentUri: Uri) {
+        return withContext(Dispatchers.IO) {
+            try {
+                // case 1 : delete image on api level 29 below
+                contentResolver.delete(contentUri, null, null)
+            } catch (e: SecurityException) {
+                // case 2 : delete image on api level 29 and above
+                val intentSender = when {
+                    isSDK29() -> {
+                        val recoverableSecurityException =
+                            e as? RecoverableSecurityException ?: throw RuntimeException(
+                                e.message,
+                                e
+                            )
+                        recoverableSecurityException.userAction.actionIntent.intentSender
+                    }
+
+                    isSDK30AndUp() -> {
+                        MediaStore.createDeleteRequest(
+                            contentResolver,
+                            listOf(contentUri)
+                        ).intentSender
+                    }
+
+                    else -> null
+                }
+
+                intentSender?.let {
+                    mIntentSenderLauncher.launch(IntentSenderRequest.Builder(it).build())
+                }
+                Log.e(TAG, "deleteImageFromSharedStorage: ${e.message}")
+            }
+        }
+    }
+
     private fun requestPermissions() {
         val permissions = mutableListOf<String>()
         val hasReadPermission = ContextCompat.checkSelfPermission(
diff --git a/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt b/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt
index 4319a77..dbdf378 100644
--- a/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/StorageUtil.kt
@@ -6,8 +6,12 @@ import android.os.Build
  * Created By Dhruv Limbachiya on 01-03-2022 11:46 AM.
  */
 
-inline fun <T> isSDK29AndUp(onSdk29: () -> T): T? {
+inline fun <T> isSDK30AndUp(onSdk29: () -> T): T? {
     return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
         onSdk29()
     } else null
-}
\ No newline at end of file
+}
+
+fun isSDK30AndUp() = Build.VERSION.SDK_INT >= Build.VERSION_CODES.R
+
+fun isSDK29() = Build.VERSION.SDK_INT == Build.VERSION_CODES.Q
\ No newline at end of file

From 6eae0a745258a930b276dab0c40b8be3fa243e6e Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Fri, 4 Mar 2022 13:32:50 +0530
Subject: [PATCH 6/7] modify SharedPhotoAdapter

---
 .idea/misc.xml                                |  3 +
 app/build.gradle                              |  3 +
 .../plcoding/androidstorage/BaseViewHolder.kt | 38 ++++++++
 .../plcoding/androidstorage/MainActivity.kt   |  1 +
 .../androidstorage/SharedPhotoAdapter.kt      | 96 +++++++++++--------
 app/src/main/res/layout/activity_main.xml     | 91 ++++++++++--------
 app/src/main/res/layout/item_loading.xml      | 16 ++++
 7 files changed, 166 insertions(+), 82 deletions(-)
 create mode 100644 app/src/main/java/com/plcoding/androidstorage/BaseViewHolder.kt
 create mode 100644 app/src/main/res/layout/item_loading.xml

diff --git a/.idea/misc.xml b/.idea/misc.xml
index 3b963d8..e4a884f 100644
--- a/.idea/misc.xml
+++ b/.idea/misc.xml
@@ -3,6 +3,9 @@
   <component name="DesignSurface">
     <option name="filePathToZoomLevelMap">
       <map>
+        <entry key="..\:/Android Projects/AndroidStorage/app/src/main/res/drawable-v24/ic_baseline_photo_camera_24.xml" value="0.1435" />
+        <entry key="..\:/Android Projects/AndroidStorage/app/src/main/res/layout/activity_main.xml" value="0.19947916666666668" />
+        <entry key="..\:/Android Projects/AndroidStorage/app/src/main/res/layout/item_loading.xml" value="0.19947916666666668" />
         <entry key="..\:/Android Projects/AndroidStorage/app/src/main/res/layout/item_photo.xml" value="0.19947916666666668" />
       </map>
     </option>
diff --git a/app/build.gradle b/app/build.gradle
index f08cced..32272fc 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -57,4 +57,7 @@ dependencies {
     implementation "androidx.lifecycle:lifecycle-runtime-ktx:2.3.1"
 
     implementation 'com.github.bumptech.glide:glide:4.13.0'
+
+    // swipe refresh layout
+    implementation "androidx.swiperefreshlayout:swiperefreshlayout:1.1.0"
 }
\ No newline at end of file
diff --git a/app/src/main/java/com/plcoding/androidstorage/BaseViewHolder.kt b/app/src/main/java/com/plcoding/androidstorage/BaseViewHolder.kt
new file mode 100644
index 0000000..bd3f5ee
--- /dev/null
+++ b/app/src/main/java/com/plcoding/androidstorage/BaseViewHolder.kt
@@ -0,0 +1,38 @@
+package com.plcoding.androidstorage
+
+import android.util.Log
+import androidx.constraintlayout.widget.ConstraintSet
+import androidx.recyclerview.widget.RecyclerView
+import androidx.viewbinding.ViewBinding
+import com.plcoding.androidstorage.databinding.ItemLoadingBinding
+import com.plcoding.androidstorage.databinding.ItemPhotoBinding
+
+/**
+ * Created By Dhruv Limbachiya on 04-03-2022 12:05 PM.
+ */
+sealed class BaseViewHolder(binding:ViewBinding) : RecyclerView.ViewHolder(binding.root) {
+
+    class ImageViewHolder(private val binding: ItemPhotoBinding) : BaseViewHolder(binding) {
+        fun bind (item: SharedStoragePhoto,onPhotoClick: (SharedStoragePhoto) -> Unit) {
+
+            binding.ivPhoto.setImageURI(item.contentUri)
+
+//            val aspectRatio = binding.ivPhoto.width.toFloat() / binding.ivPhoto.height.toFloat()
+//
+//            ConstraintSet().apply {
+//                clone(binding.root)
+//                setDimensionRatio(binding.ivPhoto.id, aspectRatio.toString())
+//                applyTo(binding.root)
+//            }
+
+            binding.ivPhoto.setOnLongClickListener {
+                onPhotoClick(item)
+                true
+            }
+
+            Log.i("BaseViewHolder", "bind: ")
+        }
+    }
+
+    class ProgressViewHolder(binding:ItemLoadingBinding) : BaseViewHolder(binding) {}
+}
\ No newline at end of file
diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index 1f61200..e57a967 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -21,6 +21,7 @@ import androidx.activity.result.launch
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.content.ContextCompat
 import androidx.lifecycle.lifecycleScope
+import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.StaggeredGridLayoutManager
 import com.plcoding.androidstorage.databinding.ActivityMainBinding
diff --git a/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt b/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
index 1e0a672..a92f5df 100644
--- a/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
@@ -1,66 +1,82 @@
 package com.plcoding.androidstorage
 
+import android.util.Log
 import android.view.LayoutInflater
 import android.view.ViewGroup
 import androidx.constraintlayout.widget.ConstraintSet
 import androidx.recyclerview.widget.DiffUtil
 import androidx.recyclerview.widget.ListAdapter
 import androidx.recyclerview.widget.RecyclerView
-import com.bumptech.glide.Glide
+import com.plcoding.androidstorage.databinding.ItemLoadingBinding
 import com.plcoding.androidstorage.databinding.ItemPhotoBinding
-import kotlinx.coroutines.GlobalScope
-import kotlinx.coroutines.launch
-import java.io.File
+import java.lang.IllegalArgumentException
 
 class SharedPhotoAdapter(
     private val onPhotoClick: (SharedStoragePhoto) -> Unit
-) : ListAdapter<SharedStoragePhoto, SharedPhotoAdapter.PhotoViewHolder>(Companion) {
+) : ListAdapter<SharedStoragePhoto, BaseViewHolder>(sharePhotoDiffUtil) {
 
-    inner class PhotoViewHolder(val binding: ItemPhotoBinding) :
-        RecyclerView.ViewHolder(binding.root)
+    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): BaseViewHolder {
+        return when (viewType) {
+            R.layout.item_photo -> {
+                BaseViewHolder.ImageViewHolder(
+                    ItemPhotoBinding.inflate(
+                        LayoutInflater.from(parent.context),
+                        parent,
+                        false
+                    )
+                )
+            }
 
-    companion object : DiffUtil.ItemCallback<SharedStoragePhoto>() {
-        override fun areItemsTheSame(
-            oldItem: SharedStoragePhoto,
-            newItem: SharedStoragePhoto
-        ): Boolean {
-            return oldItem.id == newItem.id
+            R.layout.item_loading -> {
+                BaseViewHolder.ProgressViewHolder(
+                    ItemLoadingBinding.inflate(
+                        LayoutInflater.from(
+                            parent.context
+                        ), parent, false
+                    )
+                )
+            }
+            else -> throw IllegalArgumentException("Invalid ViewHolder type provided")
         }
+    }
 
-        override fun areContentsTheSame(
-            oldItem: SharedStoragePhoto,
-            newItem: SharedStoragePhoto
-        ): Boolean {
-            return oldItem == newItem
+    override fun onBindViewHolder(holder: BaseViewHolder, position: Int) {
+        val item = currentList[position]
+        when(holder) {
+            is BaseViewHolder.ImageViewHolder -> holder.bind(item,onPhotoClick)
+            is BaseViewHolder.ProgressViewHolder -> { /* No operation */
+                Log.i("SharedPhotoAdapter", "onBindViewHolder: ProgressViewholder")
+            }
         }
     }
 
-    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): PhotoViewHolder {
-        return PhotoViewHolder(
-            ItemPhotoBinding.inflate(
-                LayoutInflater.from(parent.context),
-                parent,
-                false
-            )
-        )
+    override fun getItemCount(): Int {
+        return currentList.size
     }
 
-    override fun onBindViewHolder(holder: PhotoViewHolder, position: Int) {
-        val photo = currentList[position]
-            holder.binding.apply {
-                ivPhoto.setImageURI(photo.contentUri)
+    override fun getItemViewType(position: Int): Int {
+        return if(currentList[position] == null) {
+            R.layout.item_loading  // if current element is last element then show loading view
+        } else {
+            R.layout.item_photo // if current element is not last element then continue showing images.
+        }
+    }
 
-                val aspectRatio = photo.width.toFloat() / photo.height.toFloat()
-                ConstraintSet().apply {
-                    clone(root)
-                    setDimensionRatio(ivPhoto.id, aspectRatio.toString())
-                    applyTo(root)
-                }
+    companion object {
+        val sharePhotoDiffUtil = object : DiffUtil.ItemCallback<SharedStoragePhoto>() {
+            override fun areItemsTheSame(
+                oldItem: SharedStoragePhoto,
+                newItem: SharedStoragePhoto
+            ): Boolean {
+                return oldItem.id == newItem.id
+            }
 
-                ivPhoto.setOnLongClickListener {
-                    onPhotoClick(photo)
-                    true
-                }
+            override fun areContentsTheSame(
+                oldItem: SharedStoragePhoto,
+                newItem: SharedStoragePhoto
+            ): Boolean {
+                return oldItem == newItem
             }
+        }
     }
 }
\ No newline at end of file
diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index 9ac2ec8..8cd0cda 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -19,54 +19,61 @@
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent">
 
-        <LinearLayout
+        <androidx.swiperefreshlayout.widget.SwipeRefreshLayout
             android:layout_width="match_parent"
-            android:layout_height="match_parent"
-            android:orientation="vertical">
+            android:layout_height="match_parent">
 
-            <TextView
-                android:id="@+id/tvPrivatePhotos"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:layout_marginBottom="8dp"
-                android:text="Your private photos"
-                android:textColor="@color/black"
-                android:textSize="24sp"
-                android:textStyle="bold" />
-
-            <androidx.recyclerview.widget.RecyclerView
-                android:id="@+id/rvPrivatePhotos"
+            <LinearLayout
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layout_marginBottom="16dp"
-                app:layout_constraintBottom_toTopOf="@+id/btnTakePhoto"
-                app:layout_constraintEnd_toEndOf="parent"
-                app:layout_constraintHorizontal_bias="0.5"
-                app:layout_constraintStart_toStartOf="parent"
-                app:layout_constraintTop_toTopOf="parent" />
+                android:layout_height="match_parent"
+                android:orientation="vertical">
 
-            <TextView
-                android:id="@+id/tvAllPhotos"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:layout_marginBottom="8dp"
-                android:text="Shared photos"
-                android:textColor="@color/black"
-                android:textSize="24sp"
-                android:textStyle="bold" />
+                <TextView
+                    android:id="@+id/tvPrivatePhotos"
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:layout_marginBottom="8dp"
+                    android:text="Your private photos"
+                    android:textColor="@color/black"
+                    android:textSize="24sp"
+                    android:textStyle="bold" />
 
-            <androidx.recyclerview.widget.RecyclerView
-                android:id="@+id/rvPublicPhotos"
-                android:layout_width="match_parent"
-                android:layout_height="match_parent"
-                android:layout_marginBottom="16dp"
-                app:layout_constraintBottom_toTopOf="@+id/btnTakePhoto"
-                app:layout_constraintEnd_toEndOf="parent"
-                app:layout_constraintHorizontal_bias="0.5"
-                app:layout_constraintStart_toStartOf="parent"
-                app:layout_constraintTop_toTopOf="parent" />
+                <androidx.recyclerview.widget.RecyclerView
+                    android:id="@+id/rvPrivatePhotos"
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:layout_marginBottom="16dp"
+                    app:layout_constraintBottom_toTopOf="@+id/btnTakePhoto"
+                    app:layout_constraintEnd_toEndOf="parent"
+                    app:layout_constraintHorizontal_bias="0.5"
+                    app:layout_constraintStart_toStartOf="parent"
+                    app:layout_constraintTop_toTopOf="parent" />
+
+                <TextView
+                    android:id="@+id/tvAllPhotos"
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:layout_marginBottom="8dp"
+                    android:text="Shared photos"
+                    android:textColor="@color/black"
+                    android:textSize="24sp"
+                    android:textStyle="bold" />
+
+                <androidx.recyclerview.widget.RecyclerView
+                    android:id="@+id/rvPublicPhotos"
+                    android:layout_width="match_parent"
+                    android:layout_height="match_parent"
+                    android:layout_marginBottom="16dp"
+                    app:layout_constraintBottom_toTopOf="@+id/btnTakePhoto"
+                    app:layout_constraintEnd_toEndOf="parent"
+                    app:layout_constraintHorizontal_bias="0.5"
+                    app:layout_constraintStart_toStartOf="parent"
+                    tools:listitem="@layout/item_photo"
+                    app:layout_constraintTop_toTopOf="parent" />
+
+            </LinearLayout>
 
-        </LinearLayout>
+        </androidx.swiperefreshlayout.widget.SwipeRefreshLayout>
 
     </androidx.core.widget.NestedScrollView>
 
diff --git a/app/src/main/res/layout/item_loading.xml b/app/src/main/res/layout/item_loading.xml
new file mode 100644
index 0000000..0d3471c
--- /dev/null
+++ b/app/src/main/res/layout/item_loading.xml
@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    xmlns:app="http://schemas.android.com/apk/res-auto">
+
+    <ProgressBar
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:visibility="visible"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintTop_toTopOf="parent"
+        app:layout_constraintBottom_toBottomOf="parent"/>
+
+</androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file

From 39040fab6465e86a6bed730e1cd44d771917ff81 Mon Sep 17 00:00:00 2001
From: Dhruv Limbachiya <dhruv.mail4work@gmail.com>
Date: Fri, 4 Mar 2022 16:01:17 +0530
Subject: [PATCH 7/7] added pagination & swipe refresh feature

---
 .../plcoding/androidstorage/MainActivity.kt   | 70 ++++++++++++++++---
 .../androidstorage/SharedPhotoAdapter.kt      |  6 +-
 .../pagination/PaginationScrollListener.kt    | 40 +++++++++++
 app/src/main/res/layout/activity_main.xml     |  1 +
 4 files changed, 104 insertions(+), 13 deletions(-)
 create mode 100644 app/src/main/java/com/plcoding/androidstorage/pagination/PaginationScrollListener.kt

diff --git a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
index e57a967..58a9ebd 100644
--- a/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/MainActivity.kt
@@ -21,10 +21,14 @@ import androidx.activity.result.launch
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.content.ContextCompat
 import androidx.lifecycle.lifecycleScope
+import androidx.recyclerview.widget.GridLayoutManager
 import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.StaggeredGridLayoutManager
 import com.plcoding.androidstorage.databinding.ActivityMainBinding
+import com.plcoding.androidstorage.pagination.PaginationScrollListener
+import com.plcoding.androidstorage.pagination.PaginationScrollListener.Companion.PAGE_SIZE
+import com.plcoding.androidstorage.pagination.PaginationScrollListener.Companion.PAGE_START
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
@@ -51,6 +55,19 @@ class MainActivity : AppCompatActivity() {
 
     private var mDeleteContentUri: Uri? = null
 
+    // pagination vars
+    private var currentPage = PAGE_START
+
+    private var currentItemCount = currentPage * PAGE_SIZE
+
+    private var totalPage = 40
+
+    private var isLastPage = false
+
+    private var isLoading = false
+
+    private var sharedPhotosList = mutableListOf<SharedStoragePhoto>()
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
 
@@ -171,6 +188,8 @@ class MainActivity : AppCompatActivity() {
             takePicture.launch()
         }
 
+        setupSwipeRefreshListener()
+
         initContentObserver()
 
         setUpInternalStorageRecyclerView()
@@ -182,6 +201,18 @@ class MainActivity : AppCompatActivity() {
         loadDataIntoSharedStorageAdapter()
     }
 
+    private fun setupSwipeRefreshListener() {
+        mBinding.swipeRefreshLayout.setOnRefreshListener {
+            currentPage = PAGE_START
+            currentItemCount = currentPage
+            isLastPage = false
+            isLoading = false
+            sharedPhotosList = emptyList<SharedStoragePhoto>().toMutableList()
+            mBinding.swipeRefreshLayout.isRefreshing = false
+            loadDataIntoSharedStorageAdapter()
+        }
+    }
+
     private fun initContentObserver() {
         mContentObserver = object : ContentObserver(null) {
             override fun onChange(selfChange: Boolean) {
@@ -203,8 +234,23 @@ class MainActivity : AppCompatActivity() {
      * set up recyclerview
      */
     private fun setUpSharedStorageRecyclerView() = mBinding.rvPublicPhotos.apply {
+        val layoutManager = GridLayoutManager(this@MainActivity, 3)
+
         adapter = mSharedPhotoAdapter
-        layoutManager = StaggeredGridLayoutManager(3, RecyclerView.VERTICAL)
+
+        this.layoutManager = layoutManager
+
+        this.addOnScrollListener(object :
+            PaginationScrollListener(layoutManager) {
+            override fun loadMoreItems() {
+                currentItemCount = currentPage++ * PAGE_SIZE
+                loadDataIntoSharedStorageAdapter()
+            }
+
+            override fun isLoading() = isLoading
+
+            override fun isLastPage() = isLastPage
+        })
     }
 
     private fun setUpInternalStorageRecyclerView() = mBinding.rvPrivatePhotos.apply {
@@ -224,10 +270,18 @@ class MainActivity : AppCompatActivity() {
 
     private fun loadDataIntoSharedStorageAdapter() {
         lifecycleScope.launch {
-            val pictures = withContext(Dispatchers.IO) {
-                getSharedStorageImages()
+            isLoading = true
+            if (currentPage > totalPage) {
+                isLastPage = true
+            } else {
+                val pictures = withContext(Dispatchers.IO) {
+                    getSharedStorageImages()
+                }
+                mSharedPhotoAdapter.submitList(pictures)
+                Log.i(TAG, "loadDataIntoSharedStorageAdapter: ${pictures.size}")
             }
-            mSharedPhotoAdapter.submitList(pictures)
+
+            isLoading = false
         }
     }
 
@@ -314,14 +368,12 @@ class MainActivity : AppCompatActivity() {
             MediaStore.Images.Media.HEIGHT,
         )
 
-        val sharedPhotosList = mutableListOf<SharedStoragePhoto>()
-
         contentResolver.query(
             collection,
             projection,
             null,
             null,
-            "${MediaStore.Images.Media.DISPLAY_NAME} ASC"
+            "${MediaStore.Images.Media.DISPLAY_NAME} ASC limit $PAGE_SIZE offset $currentItemCount"
         )?.use { cursor ->
             val idColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media._ID)
             val displayNameColumnIndex =
@@ -329,9 +381,9 @@ class MainActivity : AppCompatActivity() {
             val widthColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.WIDTH)
             val heightColumnIndex = cursor.getColumnIndexOrThrow(MediaStore.Images.Media.HEIGHT)
 
-            var count = 20
+            var count = currentItemCount
 
-            while (cursor.moveToNext() && count-- >= 0) {
+            while (cursor.moveToNext() ) {
                 val id = cursor.getLong(idColumnIndex)
                 val displayName = cursor.getString(displayNameColumnIndex)
                 val width = cursor.getInt(widthColumnIndex)
diff --git a/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt b/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
index a92f5df..0c11903 100644
--- a/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
+++ b/app/src/main/java/com/plcoding/androidstorage/SharedPhotoAdapter.kt
@@ -44,9 +44,7 @@ class SharedPhotoAdapter(
         val item = currentList[position]
         when(holder) {
             is BaseViewHolder.ImageViewHolder -> holder.bind(item,onPhotoClick)
-            is BaseViewHolder.ProgressViewHolder -> { /* No operation */
-                Log.i("SharedPhotoAdapter", "onBindViewHolder: ProgressViewholder")
-            }
+            is BaseViewHolder.ProgressViewHolder -> { /* No operation */ }
         }
     }
 
@@ -55,7 +53,7 @@ class SharedPhotoAdapter(
     }
 
     override fun getItemViewType(position: Int): Int {
-        return if(currentList[position] == null) {
+        return if(position == currentList.size - 1) {
             R.layout.item_loading  // if current element is last element then show loading view
         } else {
             R.layout.item_photo // if current element is not last element then continue showing images.
diff --git a/app/src/main/java/com/plcoding/androidstorage/pagination/PaginationScrollListener.kt b/app/src/main/java/com/plcoding/androidstorage/pagination/PaginationScrollListener.kt
new file mode 100644
index 0000000..80a37d1
--- /dev/null
+++ b/app/src/main/java/com/plcoding/androidstorage/pagination/PaginationScrollListener.kt
@@ -0,0 +1,40 @@
+package com.plcoding.androidstorage.pagination
+
+import android.util.Log
+import androidx.recyclerview.widget.GridLayoutManager
+import androidx.recyclerview.widget.RecyclerView
+
+/**
+ * Created By Dhruv Limbachiya on 04-03-2022 01:52 PM.
+ */
+abstract class PaginationScrollListener(private val layoutManager: RecyclerView.LayoutManager) :
+    RecyclerView.OnScrollListener() {
+
+    override fun onScrolled(recyclerView: RecyclerView, dx: Int, dy: Int) {
+        val visibleItemCount = layoutManager.childCount
+        val totalItemCount = layoutManager.itemCount
+        val firstVisibleItemPosition =
+            (layoutManager as GridLayoutManager).findFirstVisibleItemPosition()
+
+        if (!isLoading() && !isLastPage()) {
+            if ((visibleItemCount + firstVisibleItemPosition) >= totalItemCount && firstVisibleItemPosition >= 0 && totalItemCount >= PAGE_SIZE) {
+                loadMoreItems()
+            }
+        }
+    }
+
+    protected abstract fun loadMoreItems()
+
+    abstract fun isLoading(): Boolean
+
+    abstract fun isLastPage(): Boolean
+
+    companion object {
+        const val PAGE_START = 1
+
+        /**
+         * Set scrolling threshold here (for now i'm assuming 10 item in one page)
+         */
+        const val PAGE_SIZE = 10
+    }
+}
\ No newline at end of file
diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index 8cd0cda..eefc45b 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -20,6 +20,7 @@
         app:layout_constraintTop_toTopOf="parent">
 
         <androidx.swiperefreshlayout.widget.SwipeRefreshLayout
+            android:id="@+id/swipe_refresh_layout"
             android:layout_width="match_parent"
             android:layout_height="match_parent">