Skip to content

ADFA-5223: Fix Indonesian "resource" mistranslated as "sumber" (source) - #1711

Merged
hal-eisen-adfa merged 1 commit into
stagefrom
bugfix/ADFA-5223-indonesian-resource-terminology
Aug 21, 2026
Merged

ADFA-5223: Fix Indonesian "resource" mistranslated as "sumber" (source)#1711
hal-eisen-adfa merged 1 commit into
stagefrom
bugfix/ADFA-5223-indonesian-resource-terminology

Conversation

@hal-eisen-adfa

@hal-eisen-adfa hal-eisen-adfa commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Follow-up to #1703. Fixes ADFA-5223.

The Indonesian localization rendered the Android term "resource" as "sumber", which means source. The correct term is "sumber daya" — what Google's own Indonesian Android documentation uses. Sumber alone is only right for source in the source-code / open-source sense.

Six user-facing strings, 6 lines across 3 files. Five came in with #1703; new_xml_resource is pre-existing.

File Key Before After
app/…/values-in/strings.xml msg_strings_injection_failed Gagal memperbarui sumber string… sumber daya string…
msg_strings_injection_file_not_found …file sumber string proyek. …file sumber daya string proyek.
msg_strings_injection_invalid_xml Sumber string yang dihasilkan… Sumber daya string yang dihasilkan…
msg_strings_injection_parser_unsupported …memperbarui sumber string… …memperbarui sumber daya string…
apk-viewer-plugin/…/values-in/strings.xml section_resource_directories Direktori Sumber (%d) Direktori Sumber Daya (%d)
resources/…/values-in-rID/strings.xml new_xml_resource Sumber XML baru Sumber daya XML baru

Deliberately not changed

sumber is correct in these — they really do mean source:

  • idepref_java_diagnosticsEnabled_summary — "file sumber Java" (Java source files)
  • title_open_source_licenses / summary_open_source_licenses — "sumber terbuka" (open source)
  • view_source (markdown-preview-plugin) — "Sumber" (view source)

Verification

Each changed value was round-tripped back to English through both Gemini and Google Cloud Translate.

Before — both engines independently returned source:

section_resource_directories
  EN  : Resource Directories (%d)
  ID  : Direktori Sumber (%d)
  gem : Source Directories (%d)
  goog: Source Directory (%d)

After — both return resource:

section_resource_directories
  EN  : Resource Directories (%d)
  ID  : Direktori Sumber Daya (%d)
  gem : Resource Directories (%d)
  goog: Resource Directory (%d)

msg_strings_injection_failed
  EN  : Failed to update generated string resources.
  ID  : Gagal memperbarui sumber daya string yang dihasilkan.
  gem : Failed to update generated string resources.
  goog: Failed to update the generated string resource.

All six now back-translate as "resource"; three return the English source verbatim.

Also checked: %d / %1$s placeholders intact, all three files parse as well-formed XML, and the three legitimate sumber uses above are untouched.

No UI layout change — text-only, and the replacement is two words where there was one, so no new truncation risk beyond what the English already has. Not separately re-verified at 2x font scale for that reason.

The Indonesian localization rendered the Android term "resource" as
"sumber", which means source. The correct term is "sumber daya".

Six user-facing strings affected -- five from #1703, plus the
pre-existing new_xml_resource.

Left "sumber" alone where it correctly means source:
idepref_java_diagnosticsEnabled_summary (Java source files),
title_open_source_licenses / summary_open_source_licenses (open
source), and markdown-preview's view_source.

Verified by round-tripping each changed value back to English through
both Gemini and Google Cloud Translate; all six now return "resource".

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 750dbb32-b33f-4e05-b2e9-d0b0aabc6d1d

📥 Commits

Reviewing files that changed from the base of the PR and between 8f3a89d and 61d4970.

📒 Files selected for processing (3)
  • apk-viewer-plugin/src/main/res/values-in/strings.xml
  • app/src/main/res/values-in/strings.xml
  • resources/src/main/res/values-in-rID/strings.xml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough
  • Corrected six Indonesian Android translations from “sumber” to “sumber daya” for resource-related messages and labels.
  • Preserved valid “sumber” translations that refer to source files, open source, and viewing source.
  • Verified placeholders, XML structure, and translation back-translations.
  • Risk: Changes affect localization text only. No UI layout or application logic changes are included.

Walkthrough

The changes revise Indonesian translations in three Android resource files. Resource terminology now uses “sumber daya”. String keys, XML structure, and application logic remain unchanged.

Changes

Indonesian translation updates

Layer / File(s) Summary
Resource terminology translations
apk-viewer-plugin/src/main/res/values-in/strings.xml, app/src/main/res/values-in/strings.xml, resources/src/main/res/values-in-rID/strings.xml
Updated Indonesian labels and messages to use “sumber daya” terminology for resource directories, string resources, and XML resources.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 61d49

This is a narrowly scoped Indonesian translation correction with no actionable merge-blocking risk remaining after normal checks and review.

Suggested reviewers: elissa-appdevforall, mcikadu-dev

Poem

A rabbit checks each translated line,
“Sumber daya” now reads just fine.
Keys stay still, XML stays neat,
Three small files make terms complete.
Hop, hop—review is sweet!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the Indonesian resource translation fix and matches the main changes across the localization files.
Description check ✅ Passed The description accurately explains the six Indonesian translation corrections, retained source translations, and verification performed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix/ADFA-5223-indonesian-resource-terminology

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hal-eisen-adfa
hal-eisen-adfa merged commit 0ba730d into stage Aug 21, 2026
6 checks passed
@hal-eisen-adfa
hal-eisen-adfa deleted the bugfix/ADFA-5223-indonesian-resource-terminology branch August 21, 2026 01:44
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.

1 participant