#6449 Extended file name support to include characters from multiple languages, including Cyrillic and Han scripts

[christianrowlands]

Type of change

• [ ] Feature
• [X] Bugfix
• [ ] Technical
• [ ] Other :

Content

I fixed an issue when sharing files that had non-Latin characters would have the file name replaced with underscores. For example, here is a screenshot that shows up when "forwarding" a file that was send in Element Android.

Motivation and context

Here is a link to the issue: https://github.com/element-hq/element-android/issues/6449

Worthy of note is that I thought about a couple different approaches to fixing this problem. My first regex approach was to use the existing "inclusion" approach, and add Cyrillic and Han scripts. However, after realizing that it could get messy to add support for all the different scripts, I switched to an "exclusion" approach where I remove any known invalid characters.

For reference, here was the first approach

.replace("[^\\p{sc=Cyrillic}\\p{sc=Han}a-z A-Z0-9\\\\.\\-]".toRegex(), "_")

And version 2

.replace("[\\\\?%*:|\"<>\\s]".toRegex(), "_")

Tests

1. I sent a file containing Cyrillic characters in Element Web.
2. I viewed that message in Element Android
3. I clicked the share button for that file.
4. I verified that the file name in the share UI was not all underscores.

I also wrote unit tests to verify the new regex works as expected (see the code diff).

Tested devices

• [X] Physical
• [X] Emulator
• OS version(s): Android 15 and Android 5.1

Checklist

• [X] Changes has been tested on an Android device or Android emulator with API 21
• [-] UI change has been tested on both light and dark themes
• [X] Accessibility has been taken into account. See https://github.com/element-hq/element-android/blob/develop/CONTRIBUTING.md#accessibility
• [X] Pull request is based on the develop branch
• [X] Pull request includes a new file under ./changelog.d. See https://github.com/element-hq/element-android/blob/develop/CONTRIBUTING.md#changelog
• [X] Pull request includes screenshots or videos if containing UI changes
• [X] Pull request includes a sign off
• [X] You've made a self review of your PR
• [-] If you have modified the screen flow, or added new screens to the application, you have updated the test UiAllScreensSanityTest.allScreensTest()

Signed-off-by: Christian Rowlands <craxiomdev [at] gmail.com>

[christianrowlands]

@bmarty , I have another simple PR for you if you can take a look at it. If you have any objections to the new RegEx, I am happy to update it as necessary, or add more tests to verify different scenarios.

[CLAassistant]

All committers have signed the CLA.

[bmarty]

@christianrowlands can you handle the errors reported by the CI please? Let me know if you need some help.

[christianrowlands]

Thanks for reviewing!

I will look into the CI failures.

[christianrowlands]

@bmarty , give it another run and see if that resolves some or all of the issues.

[bmarty]

Can you run ./gradlew ktlintFormat first? I think it will remove some unused imports.

[christianrowlands]

Whoops, I thought I already removed that unused Timber import. I will run it again now.

[bmarty]

I think you need to either rebase your PR's branch, or merge develop into your branch so that the code can be compiled. We are now using Java 21.

[christianrowlands]

Ok, I merged develop into my branch. Hopefully we are good now 🤞