danhper / danhper/python-i18n

I18nFileLoadError for locales ‘am’ and ‘ml’ due to Misidentification in load_directory

Aperta
#43 1 commento 2 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Python
Stelle
246
Fork
46
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

### Body
Hello,

When the user's locale is set to 'am' or 'ml', the code in the `load_directory` function of `resource_loader.py` incorrectly assumes that a corresponding localization file exists. This leads to an `i18n.loaders.loader.I18nFileLoadError`. The error message displayed is:

```plaintext
I18nFileLoadError: error getting data from resources/translations/de.yaml: ml not defined
```

This issue arises due to improper file existence checking. The locales 'ml' and 'am' are mistakenly identified as existing because they are substrings of the ".yaml" file extension. This results in the erroneous assumption that the relevant localization files are present.

#### Suggested Fix
A more robust method of checking for the existence of localization files should be implemented to prevent substring matches within file extensions.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia in resource_loader.py, nella funzione load_directory, e riproduci il problema con le locale 'am' e 'ml'. Esamina come vengono controllati i nomi dei file di localizzazione, quindi verifica che questi nomi di locale non vengano identificati come sottostringhe dell’estensione .yaml e che l’erroneo I18nFileLoadError non si verifichi più.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
python
Ambito
internationalization
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.