#54 locale "nl" not supported by collation plugin
Closed: wontfix None Opened 12 years ago by mkosek.

https://bugzilla.redhat.com/show_bug.cgi?id=697082

There seems to be some problem with nl and nl_BG - the collation plugin says
they are not supported.  We need to investigate this.

set default ticket origin to Community

Added initial screened field value.

"ldap/schema/slapd-collations.conf" has nl and nl-BE:
{{{
collation nl "" "" 1 3 2.16.840.1.113730.3.3.2.33.1 nl (Dutch)
collation nl BE "" 1 3 2.16.840.1.113730.3.3.2.34.1 nl-BE (Dutch Belgium)
}}}

Bug description: In the recent version of ICU, some locales do not
have its specific collator, but are included in the default (root)
locale. "nl", "en", and "fr" are in the class. ICU API ucol_open
takes the locale string and returns the collator with the status.
If the locale has no dedicated collator and the root collator is
picked up, status U_USING_DEFAULT_WARNING is returned, which is not
an error. But collation_indexer_create (collate.c) treats it as an
error and stops the collation.

Fix description: As ICU doc suggests, error checking for ucol_open
is replaced with "(U_SUCCESS(err)", by which the status U_USING_
DEFAULT_WARNING is correctly handled.

Reviewed by Rich (Thank you!!)

Pushed to master:
5862d6c..c8d1cf5 master -> master
commit c8d1cf5

Metadata Update from @nhosoi:
- Issue assigned to nhosoi
- Issue set to the milestone: 1.3.2 - 09/13 (September)

7 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/54

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: Fixed)

3 years ago

Login to comment on this ticket.

Metadata