Bug 463976

Summary:	[kcm_users] Text context not suitable for translation
Product:	[Plasma] plasmashell	Reporter:	Emir SARI <emir_sari>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kde
Priority:	NOR
Version First Reported In:	master
Target Milestone:	1.0
Platform:	Other
OS:	Other
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/commit/1068f3e0dc2b0d97eb8fcf4fa690aad2738d5200	Version Fixed/Implemented In:
Sentry Crash Report:

Description Emir SARI 2023-01-07 20:12:09 UTC

SUMMARY

The following string is not possible to translate properly in Turkish:

"Please repeatedly %1 your %2 on the fingerprint sensor."

In the notes section, it is indicated that %1 is a verb (e.g. scan), and %2 is the name of a finger. For Turkish, due to the differences in verb conjugation, formal approach and vowel harmony, it is not possible to simply replace a place holder with a verb. Unlike English, it has to be conjugated due to the agglutinative nature of the Turkish language.

Let's go with the "scan" verb. The translation for "to scan" is "taramak", but I need to conjugate it as "tara-yın" to direct the action to the user. For another possible verb, the suffix might be "-yun", "-in", or something else because vowel harmony requires me to adopt a new suffix according to the verb root. So I am unable to blindly put a suffix for all.

For the finger name, let's say "index finger". It is "işaret parmağı", but this one also requires conjugation as well (işaret parmağınızı). The conjugation here does not violate the vowel harmony, but it is really not good practice to blindly put placeholders like this. I don't even know where the finger names are defined.

The ideal solution would be including all the possible action verbs and finger names and their all possible context exclusively in this file, so at least we can have an idea of how to modify them as needed.

Comment 1 Bug Janitor Service 2023-01-08 17:21:04 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2488

Comment 2 David Edmundson 2023-01-10 18:11:23 UTC

Git commit 1068f3e0dc2b0d97eb8fcf4fa690aad2738d5200 by David Edmundson, on behalf of Nicolas Fella.
Committed on 10/01/2023 at 16:40.
Pushed by davidedmundson into branch 'master'.

[kcms/users] Fix i18n for finger enrollment messages

We can't concatenate sentences like this, it's impossible to properly translate.

Instead we have to spell out all 2*10 combinations of scantype and finger

M  +50   -3    kcms/users/package/contents/ui/FingerprintDialog.qml
M  +3    -2    kcms/users/src/fingerprintmodel.cpp
M  +2    -2    kcms/users/src/fingerprintmodel.h
M  +16   -3    kcms/users/src/fprintdevice.cpp
M  +7    -1    kcms/users/src/fprintdevice.h
M  +1    -0    kcms/users/src/kcm.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/1068f3e0dc2b0d97eb8fcf4fa690aad2738d5200

Comment 3 Emir SARI 2023-01-11 16:36:29 UTC

Thank you very much!