Skip to content

Conversation

@swakwork
Copy link
Collaborator

No description provided.

Comment on lines 167 to 180
if (isRunningOnManager) {
setOf(
ResourceGroup("values-ja", "strings.xml", "arrays.xml"),
).forEach {
replaceStringsInFile(
it,
replacements =
mapOf(
"X" to "Twitter",
"ポスト" to "ツイート",
),
)
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

String replacement for Japanese is only run when isRunningOnManager

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, setOf ~ forEach is no longer needed

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol I thought, isRunningOnManager is required here as well, will correct it, thanks

@kitadai31
Copy link
Contributor

It is reported that Bring back twitter patch fails if values-ja resources don't exist
#645 (comment)
This is occurs if user merged split apks without including Japanese locale split

Possible solutions:

  • Call replaceStringsInFile only if values-ja exists
  • Catch the exception

Alternatively, this pull request could be merged as-is, and fix in another PR

@swakwork
Copy link
Collaborator Author

swakwork commented Sep 30, 2025

It is reported that Bring back twitter patch fails if values-ja resources don't exist #645 (comment) This is occurs if user merged split apks without including Japanese locale split

Possible solutions:

  • Call replaceStringsInFile only if values-ja exists
  • Catch the exception

Alternatively, this pull request could be merged as-is, and fix in another PR

ya I have handled it here Utils.kt#L80

@swakwork swakwork merged commit 18be3ce into crimera:dev Oct 4, 2025
1 check failed
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.

2 participants