Développement #54147
traduction messages select2, le retour
0%
Description
Il y a eu #47855 pour pointer le manque de trad, fermé avec un patch de mon cru qui ne marche pas du tout, étonnant que ni moi ni personne n'ait testé tout de même.
Là je m'aperçois que les messages sont en fait déjà censés être traduits, ça a été ajouté il y a longtemps par #7335.
Il faut donc comprendre pourquoi ça ne marche plus (et revert le commit inutile de #47855).
Files
Associated revisions
History
Updated by Frédéric Péters over 3 years ago
Je ne sais pas ce que fait django-select2 etc. Côté w.c.s. où select2 est utilisé directement, dans l'appel les chaines traduites sont passées en options,
var options = { minimumInputLength: 1, formatResult: function(result) { return result.text; }, language: { errorLoading: function() { return WCS_I18N.s2_errorloading; }, noResults: function () { return WCS_I18N.s2_nomatches; }, inputTooShort: function (input, min) { return WCS_I18N.s2_tooshort; }, loadingMore: function () { return WCS_I18N.s2_loadmore; }, searching: function () { return WCS_I18N.s2_searching; } } };
Updated by Valentin Deniaud over 3 years ago
- File 0001-misc-restore-select2-translations-54147.patch 0001-misc-restore-select2-translations-54147.patch added
- Tracker changed from Bug to Développement
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Voilà ça marche, plein de problèmes imbriqués, le js pas inclu puis les settings mal appliqués puis des changements dans le nommage.
Updated by Benjamin Dauvergne over 3 years ago
Chez moi ça ne marche pas, jsi18n n'est pas chargé en front, par contre ça marche en backoffice jsi18n étant chargé par ce bout de code sur toutes les pages :
class MediaMixin(object, metaclass=MediaMixinBase): '''Expose needed CSS and JS files as a media object''' class Media: js = ( xstatic('jquery.js', 'jquery.min.js'), reverse_lazy('a2-manager-javascript-catalog'), <- ICI xstatic('jquery-ui.js', 'jquery-ui.min.js'), 'js/gadjo.js', 'jquery/js/jquery.form.js', 'admin/js/urlify.js', 'authentic2/js/purl.js', 'authentic2/manager/js/manager.js', ) css = {'all': ('authentic2/manager/css/style.css',)} def get_context_data(self, **kwargs): kwargs['media'] = self.media ctx = super(MediaMixin, self).get_context_data(**kwargs) if 'form' in ctx: ctx['media'] += ctx['form'].media return ctx
Updated by Valentin Deniaud over 3 years ago
- File 0001-misc-restore-select2-translations-54147.patch 0001-misc-restore-select2-translations-54147.patch added
- Status changed from En cours to Solution proposée
Avec chargement de jsi18n en front.
Updated by Benjamin Dauvergne over 3 years ago
- Status changed from Solution proposée to Solution validée
On pourrait mettre directement le reverse_lazy de la vue jsi18n dans Select2Mixin non ?
Updated by Valentin Deniaud over 3 years ago
Benjamin Dauvergne a écrit :
On pourrait mettre directement le reverse_lazy de la vue jsi18n dans Select2Mixin non ?
À la place de le mettre dans AddressAutocompleteInput ou dans MediaMixin ? Parce que pour le premier il n'y a pas d'héritage de Select2Mixin, pas chaud d'en ajouter un pour ça.
Updated by Valentin Deniaud over 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 92a98868720da6d537d605a75e92e9640d5190aa Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu May 20 16:23:38 2021 +0200 misc: restore select2 translations (#54147) Also revert previous fix attempt of commit 0676783c.
Updated by Frédéric Péters over 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: restore select2 translations (#54147)
Also revert previous fix attempt of commit 0676783c.