Last modified: 2014-03-19 15:50:11 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T56958, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54958 - Variable GRAMMAR for Slovene language: mestnik, orodnik cases include prepositions
Variable GRAMMAR for Slovene language: mestnik, orodnik cases include preposi...
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Internationalization (Other open bugs)
1.22.0
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-04 08:04 UTC by Dejan Papež
Modified: 2014-03-19 15:50 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Removed hard-coded prepositions from grammar function (1.41 KB, patch)
2014-03-19 15:37 UTC, Domen
Details
Removed prepositions from Slovene grammar forms (28.44 KB, patch)
2014-03-19 15:44 UTC, Domen
Details

Description Dejan Papež 2013-10-04 08:04:29 UTC
Hi, the GRAMMAR variable for Slovene language, the cases 'mestnik' and 'orodnik', currently includes the prepositions (o and z). These depend on the context and should therefore be left out of the default pattern.

Mestnik:
* o Wikipediji > Wikipediji

Orodnik:
* z Wikipedijo > Wikipedijo

Etc.

Thank you.
Comment 1 Siebrand Mazeland 2013-10-04 09:18:39 UTC
Please have a look at https://git.wikimedia.org/raw/mediawiki%2Fcore.git/HEAD/languages%2Fclasses%2FLanguageSl.php

It seems like the current implementation is correct?
Comment 2 Dejan Papež 2013-10-04 14:23:18 UTC
It doesn't seem:

Locativ: "word = 'o ' . $word" - this "o" is redundant
Instrumental: "word = 'z ' . $word;"  - this "z" is redundant
Comment 3 Domen 2014-03-19 15:37:00 UTC
Created attachment 14859 [details]
Removed hard-coded prepositions from grammar function

Patched file https://git.wikimedia.org/raw/mediawiki%2Fcore.git/HEAD/languages%2Fclasses%2FLanguageSl.php.
Comment 4 Domen 2014-03-19 15:44:24 UTC
Created attachment 14860 [details]
Removed prepositions from Slovene grammar forms

Patched file http://git.wikimedia.org/blob/mediawiki%2Fextensions%2FWikimediaMessages.git/HEAD/WikimediaGrammarForms.php.

(I also saw that 'rmy' language uses Slovene grammar forms for dative, locative, ablative and instrumental, which is probably a copy-paste error.)
Comment 5 Domen 2014-03-19 15:46:38 UTC
I have uploaded two patches that fix this bug.
Comment 6 Domen 2014-03-19 15:50:11 UTC
I just saw that my patched LanguageSl::convertGrammar() is same as Language::convertGrammar(), so file languages/classes/LanguageSl.php can be completely removed.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links