Project

General

Profile

Développement #54147

traduction messages select2, le retour

Added by Valentin Deniaud over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
20 May 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 92a98868 (diff)
Added by Valentin Deniaud over 3 years ago

misc: restore select2 translations (#54147)

Also revert previous fix attempt of commit 0676783c.

History

#1

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; }
        }
      };
#2

Updated by Valentin Deniaud over 3 years ago

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.

#3

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

#4

Updated by Benjamin Dauvergne over 3 years ago

  • Status changed from Solution proposée to En cours
#5

Updated by Valentin Deniaud over 3 years ago

Avec chargement de jsi18n en front.

#6

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 ?

#7

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.

#8

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.
#9

Updated by Frédéric Péters over 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF