l10n: There is no text string in Transifex

[Valdnet]

The following text string is missing from Transifex:

calendar/src/components/Editor/Repeat/RepeatFirstLastSelect.vue

Line 65 in 9697327

label: this.$t('calendar', 'second'),

calendar/src/filters/recurrenceRuleFormat.js

Line 208 in 2d5f39b

return t('calendar', 'second')

This is because a similar text string is found in this:

calendar/src/components/Editor/Alarm/AlarmTimeUnitSelect.vue

Line 74 in ecac7cc

label: this.$n('calendar', 'second', 'seconds', this.count),

which is in the plural.

@ChristophWurst Do you have any idea for that?

[tcitworld]

I think we already had the issue once.

[rakekniven]

Still the same for v3.0.1 on NC v22.2.3
Browser: Firefox v94
OS: macOS Big Sur 11.6.1

[st3iny]

So I dove into gettext and found the root of this problem. Every msgid inside a PO file has to be unique. In calendar there is an ambiguity between "second" singular and "second/seconds" plural. Both instances are assigned to the same msgid "second" which is not allowed.

However, more modern versions of gettext are able to handle message contexts but neither our javascript library (@nextcloud/l10n) not our extraction tool (translationtool.phar) are able to handle those at the moment.

Ref http://pology.nedohodnik.net/doc/user/en_US/ch-poformat.html -> scroll to 2.2.2. Disambiguating Contexts

The only quick fix I can come up with right now is to change the wording. See also the warning message that translationtool.phar emits:

xgettext: Warnung: msgid 'second' is used without plural and with plural.
                   /srv/http/dev_apps/calendar/coverage/src/components/Editor/Repeat/RepeatFirstLastSelect.vue.html:324: Here is the occurrence without plural.
                   /srv/http/dev_apps/calendar/translationfiles/templates/calendar.pot:645: Here is the occurrence with plural.
                   Workaround: If the msgid is a sentence, change the wording of the sentence; otherwise, use contexts for disambiguation.

[rakekniven]

Thank you for digging into it.
Explanation is clear.

Unfortunately I have no good idea for another wording.

[rakekniven]

Another issue referencing to this one.

nextcloud/server#48840

[rakekniven]

@st3iny Who can tackle this one again?

[SebastianKrupinski]

@rakekniven we are probably going to need another class for positions "this.$t('calendar-poistion', 'second')"

Let me check in to this.