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

Inflect Russian date of printing #1200

Closed
opoudjis opened this issue Aug 5, 2024 · 7 comments
Closed

Inflect Russian date of printing #1200

opoudjis opened this issue Aug 5, 2024 · 7 comments
Assignees
Labels

Comments

@opoudjis
Copy link
Contributor

opoudjis commented Aug 5, 2024

From #1089:

While this is an edge case, the Russian ordinal needs inflection:

date_printing: Дата % печати

the % needs to be in the genitive. Also the French instance's ordinal needs to be in the feminine.

I've got that implemented somewhere, just need to look it up; hopefully twitter-cldr-rb knows about inflected ordinals.

@opoudjis opoudjis added the bug label Aug 5, 2024
@opoudjis opoudjis self-assigned this Aug 5, 2024
@opoudjis
Copy link
Contributor Author

opoudjis commented Aug 8, 2024

So @Intelligent2013 I am going to use my i18n to inflect the Russian and French ordinals; that means that I will populate it instead of you.

I see that this is driven by <xsl:variable name="number_printing" select="normalize-space(/iso:iso-standard/iso:metanorma-extension/iso:presentation-metadata[iso:printing-date][1]/iso:printing-date)"/> and

:presentation-metadata-printing-date: second,1965-12-01

My change would allow the user to just specify a number:

:presentation-metadata-printing-date: 2,1965-12-01

And in English, French, and Russian, that will generate second, deuxième (feminine), and второво (fem.sg.gen). Without this, you will need to tell the user to type :presentation-metadata-printing-date: второво,1965-12-01 for Russian.

I will give you a draft to test out before committing this.

@Intelligent2013
Copy link
Contributor

@opoudjis ok, agree. But one comment. Instead of второво, should be второй.

@opoudjis
Copy link
Contributor Author

opoudjis commented Aug 8, 2024

Wait, it's a genitive, right? Дата второво печати?

The information will appear in /bibdata/ext/date-printing

@opoudjis
Copy link
Contributor Author

opoudjis commented Aug 8, 2024

Oh! That IS the feminine genitive!

@Intelligent2013
Copy link
Contributor

Wait, it's a genitive, right? Дата второво печати?

Yes, genitive. In Russian there isn't word 'второво'. There is 'второго' (numeral for masculine noun). For feminine noun - второй.

@opoudjis
Copy link
Contributor Author

opoudjis commented Aug 8, 2024

@Intelligent2013 It's late, so I was confused enough to spell второго phonetically lol, and I was further confused enough not to realise that on Wiktionary, the feminine was the third column, not the second in the declension table. :-)

Please try out with this file:

a.presentation.xml.zip

Intelligent2013 added a commit to metanorma/mn-native-pdf that referenced this issue Aug 8, 2024
@Intelligent2013
Copy link
Contributor

@opoudjis everything is ok:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

2 participants