Fix: Save all option overrides attachment files

[mohitsatr]

Fixes #10965

Description

Makes use of OpenDocumentTree Contract for onSaveAllAttachments option to let users choose a destination folder for all the attachments. Previously, we were forced to click 'save' for each attachment file.

Screen Shots

recording_20260710_122921.mp4

AI Disclosure

Select one of the following (mandatory)

• This contribution does not include any changes created or assisted by AI.
• This contribution includes changes assisted by AI.
• This contribution includes changes created by AI.

Contribution Checklist

• I have read and affirm that my contribution adheres to Mozilla’s Community Participation Guidelines
• This contribution is in Kotlin where possible
• This contribution does not use merge commits
• This contribution adheres to the existing codestyle (run gradlew spotlessCheck to check and gradlew spotlessApply to format your source code; will be checked by CI).
• This contribution does not break existing unit tests (run gradlew testDebugUnitTest; will be checked by CI).
• This contribution includes tests for any new functionality, and maintains tests for any updated functionality.
• This contribution adheres to our Engineering process (RFC/Technical Design/ADR)
• This PR has a descriptive title and body that accurately outlines all changes made, and contains a reference to any issues that it fixes (e.g. Closes #XXX or Fixes #XXX).

[github-actions]

✅ Validation Passed: All report and feature-flag labels are correctly set.

[dani-zilla]

This class is one we're planning on deprecating. I understand this was a kotlin conversion of a Java class, but if we're introducing new code (and a small change at that) we should use the newer class. It's net.thunderbird.core.logging.Logger. You can inject an instance of it.

We have more details of it here: https://github.com/thunderbird/thunderbird-android/blob/49e1eab7e2df55ac83ec02e6e3f8c370ac4843c1/core/logging/README.md

[rafaeltonholo]

Although we are aiming to migrate all Java to Kotlin, the conversion should not be done in bug fixes like this one. Having a Kotlin conversion in the same changeset as a bug fix makes it harder to review and to revert in case of necessity. Please, revert the AttachmentController Kotlin conversion and apply the fix on the Java side.

[mohitsatr]

@rafaeltonholo
I'm not familiar with AyncTasks.
Would it be okay if I send a separate PR for AttachmentController.java -> .kt conversion first, and then continue on this PR?

[rafaeltonholo]

@rafaeltonholo I'm not familiar with AyncTasks. Would it be okay if I send a separate PR for AttachmentController.java -> .kt conversion first, and then continue on this PR?

Yes, please! Here is the GitHub issue for the conversion: #11096

[rafaeltonholo]

Hi @mohitsatr, now that the AttachmentController is already converted to Kotlin, would you please rebase your changes and apply the fix?

Thanks