Merge pull request #94 from Dacid99/main

Fixed bug in nextcloud auth

app/src/main/kotlin/ac/mdiq/podcini/net/sync/nextcloud/NextcloudLoginFlow.kt

@@ -22,6 +22,7 @@ class NextcloudLoginFlow(private val httpClient: OkHttpClient, private val rawHo
     private val hostname = HostnameParser(rawHostUrl)
     private var token: String? = null
     private var endpoint: String? = null
+    private var isWaitingForBrowser:Boolean = false
 
     fun saveInstanceState(): ArrayList<String?> {
         val state = ArrayList<String?>()
@@ -51,7 +52,7 @@ class NextcloudLoginFlow(private val httpClient: OkHttpClient, private val rawHo
                 withContext(Dispatchers.Main) {
                     val browserIntent = Intent(Intent.ACTION_VIEW, Uri.parse(result))
                     context.startActivity(browserIntent)
-                    poll()
+                    isWaitingForBrowser = true
                 }
             } catch (e: Throwable) {
                 Log.e(TAG, Log.getStackTraceString(e))
@@ -79,6 +80,13 @@ class NextcloudLoginFlow(private val httpClient: OkHttpClient, private val rawHo
         throw RuntimeException("Maximum retries exceeded")
     }
 
+    fun onResume(){ //trigger poll only when returning from the browser
+        if (token != null && isWaitingForBrowser){
+            poll()
+            isWaitingForBrowser = false
+        }
+    }
+
     private fun poll() {
         CoroutineScope(Dispatchers.IO).launch {
             try {

app/src/main/kotlin/ac/mdiq/podcini/preferences/fragments/SynchronizationPreferencesFragment.kt

@@ -285,6 +285,8 @@ class SynchronizationPreferencesFragment : PreferenceFragmentCompat() {
         }
         override fun onResume() {
             super.onResume()
+            nextcloudLoginFlow?.onResume()
+
             if (shouldDismiss) dismiss()
         }
         override fun onNextcloudAuthenticated(server: String, username: String, password: String) {