Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Indonesian translation #787

Closed
wants to merge 1 commit into from
Closed

Update Indonesian translation #787

wants to merge 1 commit into from

Conversation

triyanwn
Copy link
Contributor

Current progress is 50% translated.

code-gnucash-org pushed a commit that referenced this pull request Sep 17, 2020
Copy link
Member

@fellen fellen left a comment

Choose a reason for hiding this comment

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

Nice work, thanks!

"POT-Creation-Date: 2020-09-02 10:44+0700\n"
"PO-Revision-Date: 2020-09-05 11:36+0700\n"
"POT-Creation-Date: 2020-09-15 20:18+0700\n"
"PO-Revision-Date: 2020-09-17 20:40+0700\n"
"Last-Translator: Triyan W. Nugroho <[email protected]>\n"
"Language-Team: Indonesian <[email protected]>\n"
Copy link
Member

Choose a reason for hiding this comment

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

As you are no registered member of https://translationproject.org/team/id.html you should change the line to
"Language-Team: \n"

I assume msginit had prepopulated it for you.
As explained in https://wiki.gnucash.org/wiki/Translation#translationproject.org ff. we get some translations from that project and they use that entry.

Copy link
Member

Choose a reason for hiding this comment

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

I have fixed it in commit 5de41ab. Other changes there:
For some unknown reason in the glossary PROJECT was not replaced.
Our release manager has to inform the TP manager that you send your changes directly to us and not their id team. Until that is done, it belongs in NEW_LINGUAS in CMakeLists.txt

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I will check it for future work.

Comment on lines +31102 to +31104
#~ msgid "Change a Business Association"
#~ msgstr "Ubah Asosiasi Bisnis"

Copy link
Member

Choose a reason for hiding this comment

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

You can remove the section starting here, see https://wiki.gnucash.org/wiki/Translation#Orphaned_Records.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks again for the tip!

@fellen
Copy link
Member

fellen commented Sep 17, 2020

Because I have rewritten the commit message, it became commit b823431.
Merged!

@fellen fellen closed this Sep 17, 2020
@fellen
Copy link
Member

fellen commented Oct 13, 2020

@triyanwn By running I18nspector I got the following error message:
E: id.po: unusual-plural-forms 'nplurals=2; plural=n != 1;' => 'nplurals=1; plural=0;'

That would mean, by example in English I would write:

He has 1 book
He has 2 books.

You see the difference?
But in indonesian:

Dia memiliki 1 buku.
Dia memiliki 2 buku.

you can use the same form. If that is correct you should

  1. adjust that line in the header,

  2. drop everywhere the 2. occurence of msgstring

@triyanwn
Copy link
Contributor Author

For some sentences there are no difference. But for example for these sentences:

The book has been read.
The books have been read.

I would write in Indonesia:

Buku itu sudah dibaca.
Buku-buku itu sudah dibaca.

According to GNU Gettext Manual, Bahasa Indonesia should use nplurals=2; plural=n != 1;

@fellen
Copy link
Member

fellen commented Oct 14, 2020

Thanks, then there is an error in I18nspector's database.

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