Handling null Uri when selecting file & Prevent double large file transfer

app/src/main/AndroidManifest.xml

@@ -28,9 +28,7 @@
             </intent-filter>
         </receiver>
 
-        <activity
-            android:name=".ChatActivity"
-            android:windowSoftInputMode="adjustResize"/>
+        <activity android:name=".ChatActivity"/>
         <activity android:name=".MainActivity">
             <intent-filter android:label="@string/share_label">
                 <action android:name="android.intent.action.SEND"/>

app/src/main/java/sushi/hardcore/aira/ChatActivity.kt

@@ -23,6 +23,7 @@ import sushi.hardcore.aira.databinding.DialogFingerprintsBinding
 import sushi.hardcore.aira.databinding.DialogInfoBinding
 import sushi.hardcore.aira.utils.FileUtils
 import sushi.hardcore.aira.utils.StringUtils
+import java.io.FileNotFoundException
 import java.util.*
 
 class ChatActivity : AppCompatActivity() {
@@ -39,15 +40,19 @@ class ChatActivity : AppCompatActivity() {
         if (::airaService.isInitialized && uri != null) {
             contentResolver.query(uri, null, null, null, null)?.let { cursor ->
                 if (cursor.moveToFirst()) {
-                    contentResolver.openInputStream(uri)?.let { inputStream ->
-                        val fileName = cursor.getString(cursor.getColumnIndex(OpenableColumns.DISPLAY_NAME))
-                        val fileSize = cursor.getLong(cursor.getColumnIndex(OpenableColumns.SIZE))
-                        airaService.sendFileTo(sessionId, fileName, fileSize, inputStream)?.let { msg ->
-                            chatAdapter.newMessage(ChatItem(true, msg))
-                        }
-                        if (airaService.contacts.contains(sessionId)) {
-                            lastLoadedMessageOffset += 1
+                    try {
+                        contentResolver.openInputStream(uri)?.let { inputStream ->
+                            val fileName = cursor.getString(cursor.getColumnIndex(OpenableColumns.DISPLAY_NAME))
+                            val fileSize = cursor.getLong(cursor.getColumnIndex(OpenableColumns.SIZE))
+                            airaService.sendFileTo(sessionId, fileName, fileSize, inputStream)?.let { msg ->
+                                chatAdapter.newMessage(ChatItem(true, msg))
+                            }
+                            if (airaService.contacts.contains(sessionId)) {
+                                lastLoadedMessageOffset += 1
+                            }
                         }
+                    } catch (e: FileNotFoundException) {
+                        Toast.makeText(this, e.localizedMessage, Toast.LENGTH_SHORT).show()
                     }
                 }
                 cursor.close()
@@ -70,7 +75,9 @@ class ChatActivity : AppCompatActivity() {
                 chatAdapter = ChatAdapter(this@ChatActivity, ::onClickSaveFile)
                 binding.recyclerChat.apply {
                     adapter = chatAdapter
-                    layoutManager = LinearLayoutManager(this@ChatActivity, LinearLayoutManager.VERTICAL, false)
+                    layoutManager = LinearLayoutManager(this@ChatActivity, LinearLayoutManager.VERTICAL, false).apply {
+                        stackFromEnd = true
+                    }
                     addOnScrollListener(object : RecyclerView.OnScrollListener() {
                         fun loadMsgsIfNeeded(recyclerView: RecyclerView) {
                             if (!recyclerView.canScrollVertically(-1) && ::airaService.isInitialized) {

app/src/main/java/sushi/hardcore/aira/MainActivity.kt

@@ -10,6 +10,7 @@ import android.os.IBinder
 import android.provider.OpenableColumns
 import android.view.Menu
 import android.view.MenuItem
+import android.view.View
 import android.widget.AbsListView
 import android.widget.AdapterView
 import android.widget.Toast
@@ -30,7 +31,7 @@ class MainActivity : AppCompatActivity() {
     private lateinit var binding: ActivityMainBinding
     private lateinit var airaService: AIRAService
     private lateinit var onlineSessionAdapter: SessionAdapter
-    private lateinit var offlineSessionAdapter: SessionAdapter
+    private var offlineSessionAdapter: SessionAdapter? = null
     private val onSessionsItemClick = AdapterView.OnItemClickListener { adapter, _, position, _ ->
         launchChatActivity(adapter.getItemAtPosition(position) as Session)
     }
@@ -55,7 +56,7 @@ class MainActivity : AppCompatActivity() {
             runOnUiThread {
                 onlineSessionAdapter.remove(sessionId)?.let { session ->
                     if (session.isContact) {
-                        offlineSessionAdapter.add(session)
+                        offlineSessionAdapter?.add(session)
                     }
                 }
             }
@@ -100,15 +101,20 @@ class MainActivity : AppCompatActivity() {
                 }
             setOnScrollListener(onSessionsScrollListener)
         }
-        offlineSessionAdapter = SessionAdapter(this)
-        binding.offlineSessions.apply {
-            adapter = offlineSessionAdapter
-            onItemClickListener = if (openedToShareFile) {
+        if (openedToShareFile) {
+            binding.offlineSessions.visibility = View.GONE
+            binding.textOfflineSessions.visibility = View.GONE
+        } else {
+            offlineSessionAdapter = SessionAdapter(this)
+            binding.offlineSessions.apply {
+                adapter = offlineSessionAdapter
+                onItemClickListener = if (openedToShareFile) {
                     onSessionsItemClickSendFile
                 } else {
                     onSessionsItemClick
                 }
-            setOnScrollListener(onSessionsScrollListener)
+                setOnScrollListener(onSessionsScrollListener)
+            }
         }
         Intent(this, AIRAService::class.java).also { serviceIntent ->
             bindService(serviceIntent, object : ServiceConnection {
@@ -206,7 +212,7 @@ class MainActivity : AppCompatActivity() {
                 airaService.isAppInBackground = false
                 airaService.uiCallbacks = uiCallbacks //restoring callbacks
                 onlineSessionAdapter.reset()
-                offlineSessionAdapter.reset()
+                offlineSessionAdapter?.reset()
                 loadContacts()
                 loadSessions()
                 title = airaService.identityName
@@ -217,8 +223,10 @@ class MainActivity : AppCompatActivity() {
     }
 
     private fun loadContacts() {
-        for ((sessionId, contact) in airaService.contacts) {
-            offlineSessionAdapter.add(Session(sessionId, true, contact.verified, contact.seen, null, contact.name))
+        if (offlineSessionAdapter != null) {
+            for ((sessionId, contact) in airaService.contacts) {
+                offlineSessionAdapter!!.add(Session(sessionId, true, contact.verified, contact.seen, null, contact.name))
+            }
         }
     }
 
@@ -235,7 +243,7 @@ class MainActivity : AppCompatActivity() {
             onlineSessionAdapter.add(Session(sessionId, false, false, seen, ip, airaService.savedNames[sessionId]))
         } else {
             onlineSessionAdapter.add(Session(sessionId, true, contact.verified, seen, ip, contact.name))
-            offlineSessionAdapter.remove(sessionId)
+            offlineSessionAdapter?.remove(sessionId)
         }
     }
 

app/src/main/java/sushi/hardcore/aira/background_service/AIRAService.kt

@@ -8,6 +8,7 @@ import android.net.nsd.NsdServiceInfo
 import android.os.*
 import android.os.Process.THREAD_PRIORITY_BACKGROUND
 import android.util.Log
+import android.widget.Toast
 import androidx.annotation.RequiresApi
 import androidx.core.app.NotificationCompat
 import androidx.core.app.NotificationManagerCompat
@@ -16,7 +17,6 @@ import sushi.hardcore.aira.*
 import java.io.IOException
 import java.io.InputStream
 import java.net.*
-import java.nio.ByteBuffer
 import java.nio.channels.*
 
 class AIRAService : Service() {
@@ -143,10 +143,14 @@ class AIRAService : Service() {
                 return msg
             }
         } else {
-            val fileTransfer = SendFileTransfer(fileName, fileSize, inputStream)
-            sendFileTransfers[sessionId] = fileTransfer
-            createFileTransferNotification(sessionId, fileTransfer)
-            sendTo(sessionId, Protocol.askLargeFile(fileSize, fileName))
+            if (sendFileTransfers[sessionId] == null && receiveFileTransfers[sessionId] == null) {
+                val fileTransfer = SendFileTransfer(fileName, fileSize, inputStream)
+                sendFileTransfers[sessionId] = fileTransfer
+                createFileTransferNotification(sessionId, fileTransfer)
+                sendTo(sessionId, Protocol.askLargeFile(fileSize, fileName))
+            } else {
+                Toast.makeText(this, R.string.file_transfer_already_in_progress, Toast.LENGTH_SHORT).show()
+            }
         }
         return null
     }

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

@@ -71,4 +71,5 @@
     <string name="remove_contact">Remove contact</string>
     <string name="details">Details</string>
     <string name="your_addresses">Your IP addresses:</string>
+    <string name="file_transfer_already_in_progress">Another file transfer is already in progress</string>
 </resources>