Improve RemoteAttachmentDownloader handling of network errors,

Fix text colors on AddByIdentifierScreen Adding arxiv.org to list of cleartextTrafficPermitted websites. Upping versionCode to 60
zotero · May 9, 2024 · 400ad1a · 400ad1a
1 parent fb08915
commit 400ad1a
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 6 deletions.
diff --git a/app/src/main/java/org/zotero/android/attachmentdownloader/RemoteAttachmentDownloader.kt b/app/src/main/java/org/zotero/android/attachmentdownloader/RemoteAttachmentDownloader.kt
@@ -185,6 +185,20 @@ class RemoteAttachmentDownloader @Inject constructor(
                     )
                 }
             }
+            is CustomResult.GeneralError.NetworkError -> {
+                this.errors[download] = Exception(result.stringResponse)
+                Timber.e(
+                    result.stringResponse,
+                    "RemoteAttachmentDownloader: failed to download attachment ${download.key}, ${download.libraryId}"
+                )
+                attachmentDownloaderEventStream.emitAsync(
+                    Update(
+                        download = download,
+                        kind = Update.Kind.failed
+                    )
+                )
+            }
+
             is CustomResult.GeneralSuccess -> {
                 Timber.i("RemoteAttachmentDownloader: finished downloading ${download.key}")
 

diff --git a/...c/main/java/org/zotero/android/uicomponents/addbyidentifier/IdentifierLookupController.kt b/...c/main/java/org/zotero/android/uicomponents/addbyidentifier/IdentifierLookupController.kt
@@ -634,12 +634,14 @@ class IdentifierLookupController @Inject constructor(
         remoteFileDownloader.stop()
         val lookupData = this.lookupData
         cleanupLookupIfNeeded(force = true) {
-            this.observable.emitAsync(
-                Update(
-                    kind = Update.Kind.finishedAllLookups,
-                    lookupData = emptyList()
+            if (this::observable.isInitialized) {
+                this.observable.emitAsync(
+                    Update(
+                        kind = Update.Kind.finishedAllLookups,
+                        lookupData = emptyList()
+                    )
                 )
-            )
+            }
         }
         val storedItemResponses = lookupData.values.mapNotNull {
             when (it.state) {

diff --git a/app/src/main/java/org/zotero/android/uicomponents/addbyidentifier/ui/AddByIdentifierTable.kt b/app/src/main/java/org/zotero/android/uicomponents/addbyidentifier/ui/AddByIdentifierTable.kt
@@ -97,6 +97,7 @@ internal fun LookupItemRow(
                     title,
                     HtmlCompat.FROM_HTML_MODE_LEGACY
                 ).toString(),
+                color = CustomTheme.colors.primaryContent,
                 style = CustomTheme.typography.newBody,
             )
             NewDivider(modifier = Modifier.padding(top = 8.dp))
@@ -163,6 +164,7 @@ internal fun LookupAttachmentRow(
                     title,
                     HtmlCompat.FROM_HTML_MODE_LEGACY
                 ).toString(),
+                color = CustomTheme.colors.primaryContent,
                 style = CustomTheme.typography.newBody,
             )
             NewDivider(modifier = Modifier.padding(top = 8.dp))
@@ -187,6 +189,7 @@ internal fun LookupIdentifierRow(
                 modifier = Modifier.weight(1f),
                 text = title,
                 maxLines = 1,
+                color = CustomTheme.colors.primaryContent,
                 overflow = TextOverflow.Ellipsis,
                 style = CustomTheme.typography.newBody,
             )

diff --git a/app/src/main/res/xml/network_security_config.xml b/app/src/main/res/xml/network_security_config.xml
@@ -1,5 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <network-security-config>
+    <domain-config cleartextTrafficPermitted="true">
+        <domain includeSubdomains="true">arxiv.org</domain>
+    </domain-config>
     <base-config>
         <trust-anchors>
             <certificates src="system" />

diff --git a/buildSrc/src/main/kotlin/BuildConfig.kt b/buildSrc/src/main/kotlin/BuildConfig.kt
@@ -4,7 +4,7 @@ object BuildConfig {
     const val compileSdkVersion = 34
     const val targetSdk = 33
 
-    val versionCode = 59 // Must be updated on every build
+    val versionCode = 60 // Must be updated on every build
     val version = Version(
         major = 1,
         minor = 0,