refactor: convert AttachmentController.java to .kt

[mohitsatr]

Fixes #11096

[github-actions]

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

[mohitsatr]

@rafaeltonholo, how can I test this class? Could you point to anything similar in the codebase?

[rafaeltonholo]

@rafaeltonholo, how can I test this class? Could you point to anything similar in the codebase?

What do you mean by "how can I test this class"? Are you asking about building unit tests or testing it within the app?

If you want to test it in the app, you should verify that all the functionalities related to attachments in the MessageViewFragment are still working as intended.

While having a unit test for this class would be beneficial, during a conversion from Java to Kotlin, you are not required to introduce any new code. The focus of the task is solely on the conversion itself.

Please refer to our documentation "Java to Kotlin Conversion Guide" to understand the required steps for performing this task.

[rafaeltonholo]

Thanks for working on this conversion! There are a few things we need to address before merging this code.

Please check the Java to Kotlin Conversion Guide to verify what is needed during a Java to Kotlin conversion

[mohitsatr]

While having a unit test for this class would be beneficial, during a conversion from Java to Kotlin, you are not required to introduce any new code. The focus of the task is solely on the conversion itself.

Yes. I meant a unit test. I can send a separate PR for that after this.

[rafaeltonholo]

You just missed one hardcoded dispatcher. Everything else is good

[rafaeltonholo]

Use ioDispatcher instead.

Suggested change

	val intent = withContext(Dispatchers.IO) { getBestViewIntent() }
	val intent = withContext(ioDispatcher) { getBestViewIntent() }

[mohitsatr]

Ah my bad. Done

[rafaeltonholo]

LGTM! Thanks for this contribution!! We are currently on a soft freeze, but once it is lifted, we are good to merge this!