Add fallback to synchronous fetch when onCreate() fails

Co-authored-by: adbenitez <24558636+adbenitez@users.noreply.github.com>

.github/workflows/release.yml

@@ -59,6 +59,8 @@ jobs:
           rm build/outputs/apk/foss/release/*universal*
           ./gradlew assembleGplayRelease
           mv build/outputs/apk/gplay/release/*universal* build/outputs/apk/foss/release/ArcaneChat-gplay.apk
+          mv build/outputs/mapping/fossRelease/mapping.txt build/outputs/mapping/fossRelease/mapping-foss.txt
+          mv build/outputs/mapping/gplayRelease/mapping.txt build/outputs/mapping/fossRelease/mapping-gplay.txt
 
       - name: Release on GitHub
         uses: softprops/action-gh-release@v1
@@ -67,7 +69,9 @@ jobs:
           body: '[<img src="store/get-it-on-gplay.png" alt="Get it on Google Play" height="48">](https://play.google.com/store/apps/details?id=com.github.arcanechat) [<img src="store/get-it-on-fdroid.png" alt="Get it on F-Droid" height="48">](https://f-droid.org/packages/chat.delta.lite) [<img src="store/get-it-on-github.png" alt="Get it on GitHub" height="48">](https://github.com/ArcaneChat/android/releases/latest/download/ArcaneChat-gplay.apk)'
           prerelease: ${{ contains(github.event.ref, '-beta') }}
           fail_on_unmatched_files: true
-          files: build/outputs/apk/foss/release/*.apk
+          files: |
+            build/outputs/apk/foss/release/*.apk
+            build/outputs/mapping/fossRelease/mapping-*.txt
 
       - name: Release on ZapStore
         run: |

src/main/java/org/thoughtcrime/securesms/components/InputPanel.java

@@ -202,7 +202,8 @@ public class InputPanel extends ConstraintLayout
 
   public void setSubjectVisible(boolean visible) {
     subjectText.setVisibility(visible ? View.VISIBLE : View.GONE);
-    emojiToggle.setVisibility(!visible ? View.VISIBLE : View.GONE);
+    // don't make it visible if visible is false to avoid showing it while recording audio and an event triggers setSubjectVisible(false)
+    if (visible) emojiToggle.setVisibility(View.GONE);
   }
 
   public String getSubject() {

src/main/java/org/thoughtcrime/securesms/providers/PersistentBlobProvider.java

@@ -122,26 +122,8 @@ public class PersistentBlobProvider {
   }
 
   public Uri createForExternal(@NonNull Context context, @NonNull String mimeType) throws IOException, IllegalStateException, NullPointerException {
-    String filename = System.currentTimeMillis() + "." + getExtensionFromMimeType(mimeType);
-    
-    // Try external cache first
-    try {
-      File externalDir = getExternalDir(context);
-      File target = new File(externalDir, filename);
-      return FileProviderUtil.getUriFor(context, target);
-    } catch (IllegalArgumentException e) {
-      // FileProvider doesn't support the external cache path (e.g., on removable SD card).
-      // Note: getExternalDir() already falls back to internal cache when external cache is null,
-      // but when external cache exists on a removable SD card, FileProvider may reject it.
-      // In that case, we explicitly use internal cache which FileProvider always supports.
-      Log.w(TAG, "FileProvider doesn't support external cache path, falling back to internal cache", e);
-      File internalDir = context.getCacheDir();
-      if (internalDir == null) {
-        throw new IOException("no cache directory available");
-      }
-      File target = new File(internalDir, filename);
-      return FileProviderUtil.getUriFor(context, target);
-    }
+    File target = new File(getExternalDir(context), System.currentTimeMillis() + "." + getExtensionFromMimeType(mimeType));
+    return FileProviderUtil.getUriFor(context, target);
   }
 
   public boolean delete(@NonNull Context context, @NonNull Uri uri) {

src/main/java/org/thoughtcrime/securesms/service/FetchForegroundService.java

@@ -88,7 +88,27 @@ public final class FetchForegroundService extends Service {
       .setSmallIcon(R.drawable.notification_permanent)
       .build();
 
-    startForeground(NotificationCenter.ID_FETCH, notification);
+    try {
+      startForeground(NotificationCenter.ID_FETCH, notification);
+    } catch (Exception e) {
+      Log.w(TAG, "Failed to start foreground service, falling back to synchronous fetch", e);
+      synchronized (SERVICE_LOCK) {
+        service = null;
+      }
+      stopSelf();
+      // Fallback to synchronous fetching when foreground service fails
+      fetchingSynchronously = true;
+      if (ApplicationContext.getDcAccounts().backgroundFetch(10)) {
+        synchronized (STOP_NOTIFIER) {
+          while (fetchingSynchronously) {
+            try {
+              STOP_NOTIFIER.wait();
+            } catch (InterruptedException ex) {}
+          }
+        }
+      }
+      return;
+    }
 
     Util.runOnAnyBackgroundThread(() -> {
       Log.i(TAG, "Starting fetch");

src/main/res/xml/file_provider_paths.xml

@@ -1,11 +1,13 @@
 <?xml version="1.0" encoding="utf-8"?>
 <paths>
-    <cache-path name="cache" path="." />
     <external-cache-path name="external_cache" path="." />
     <external-files-path name="external_files" path="." />
 
+    <!-- this is needed for access to the cache dir in SD card -->
+    <root-path name="external_root" path="/storage/" />
+
     <external-path name="external_pictures" path="Pictures"/>
     <external-path name="external_video" path="Movies"/>
     <external-path name="external_audio" path="Music"/>
     <external-path name="external_download" path="Download"/>
-</paths>
+</paths>