Skip to content

Fix missing basmallah for gapless playback#3365

Merged
ahmedre merged 1 commit intomainfrom
missing_basmallah
Sep 20, 2025
Merged

Fix missing basmallah for gapless playback#3365
ahmedre merged 1 commit intomainfrom
missing_basmallah

Conversation

@ahmedre
Copy link
Copy Markdown
Contributor

@ahmedre ahmedre commented Sep 20, 2025

When playing from a gapless sura, the MediaPlayer code used to start at
the timestamp for ayah=0 (usually the basmallah for suras that aren't
Fatiha nor Tawba), falling back to ayah=1 otherwise. During the
migration to ExoPlayer, this was inadvertently changed to always start
at ayah=1, preventing the basmallah from playing.

When playing from a gapless sura, the MediaPlayer code used to start at
the timestamp for ayah=0 (usually the basmallah for suras that aren't
Fatiha nor Tawba), falling back to ayah=1 otherwise. During the
migration to ExoPlayer, this was inadvertently changed to always start
at ayah=1, preventing the basmallah from playing.
@github-actions
Copy link
Copy Markdown

OLD: app-madani-debug.apk (signature: V1, V2)
NEW: app-madani-debug.apk (signature: V1, V2)

          │          compressed           │         uncompressed          
          ├───────────┬───────────┬───────┼───────────┬───────────┬───────
 APK      │ old       │ new       │ diff  │ old       │ new       │ diff  
──────────┼───────────┼───────────┼───────┼───────────┼───────────┼───────
      dex │  25.6 MiB │  25.6 MiB │ -15 B │  78.2 MiB │  78.2 MiB │ -40 B 
     arsc │   2.1 MiB │   2.1 MiB │   0 B │   2.1 MiB │   2.1 MiB │   0 B 
 manifest │   6.1 KiB │   6.1 KiB │   0 B │  29.7 KiB │  29.7 KiB │   0 B 
      res │   1.8 MiB │   1.8 MiB │  +6 B │     2 MiB │     2 MiB │   0 B 
   native │  18.8 KiB │  18.8 KiB │   0 B │  36.5 KiB │  36.5 KiB │   0 B 
    asset │ 452.1 KiB │ 452.1 KiB │   0 B │ 808.2 KiB │ 808.2 KiB │   0 B 
    other │ 198.3 KiB │ 198.3 KiB │  +6 B │ 495.1 KiB │ 495.1 KiB │   0 B 
──────────┼───────────┼───────────┼───────┼───────────┼───────────┼───────
    total │  30.1 MiB │  30.1 MiB │  -3 B │  83.6 MiB │  83.6 MiB │ -40 B 

@ahmedre ahmedre added this pull request to the merge queue Sep 20, 2025
Merged via the queue into main with commit 77901f4 Sep 20, 2025
2 checks passed
@ahmedre ahmedre deleted the missing_basmallah branch September 20, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant