[david-allison]

Note

Assisted-by: Claude Fable 5 - implementation
Assisted-by: GPT-6.1-Sol xhigh - review and fixups

Purpose / Description

Split out from a larger PR to land #13574

Fixes

• Part of Don't force full (F-Droid/GitHub) releases users to grant the All files access permission #13574

Approach

This is effectively a non-functional change:

The collection path will now be unset until the permissions screen if the directory is inaccessible (a public folder using a full build), it now gets set when MANAGE_EXTERNAL_STORAGE is granted.

How Has This Been Tested?

This area of the code is already HEAVILY tested

Checklist

• You have a descriptive commit message with a short title (first line, max 50 chars).
• You have commented your code, particularly in hard-to-understand areas
• You have performed a self-review of your own code
• UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
• UI Changes: You have tested your change using the Google Accessibility Scanner

[criticalAY]

Reviewed manually, looks clean!

[mikehardy]

Assisted-by: grok-4.7-medium

Thank you — deferring the public path until All files access is granted keeps the play and legacy paths unchanged and matches the policy tests.

Git: checked out the PR branch; HEAD matched the tip; rebased three commits onto ankidroid/main without conflicts (no push).

Local validation: lint (lintAll, ktlintCheck, lint-rules:test), unit (jacocoUnitTestReport), and release emulator tests (TEST_RELEASE_BUILD=true via run_emulator_tests.py on emulator-5556 / API35_GAPI; emulator green on retry after a configuration-cache pack error).

Reviewed the startup storage deferral: on full builds UserChoosesOnPermissionScreen leaves PREF_COLLECTION_PATH unset until MANAGE_EXTERNAL_STORAGE is granted, so storageDecision stays undecided and IntentHandler no longer treats an unreadable public folder as decided; play builds stay on app-private storage and Android 10 and below stay on the legacy public path. DeckPicker / permissions flow still persist public storage only after the grant, with coverage in StoragePolicyTest and SetupStorageTest.

some acceptable findings, no action required, just leaving for posterity: UserChoosesOnPermissionScreen KDoc mentions app-private skip work not in this PR; PR body “non-functional” vs user-visible intent launch behavior; commit subjects sit just under the 50-character checklist line.