Ticket #193 (assigned defect)

Opened 16 years ago

Last modified 14 years ago

refactoring vocabularies for i18n

Reported by: bdelbosc Owned by: fguillaume
Priority: P2 Milestone: CPS 3.5.7
Component: CPSSchemas Version: unspecified
Severity: normal Keywords: vocabularies

Description (last modified by fguillaume) (diff)

Refactoring for i18n

  • instead of storing 3 strings: key, value and msgid we should store 2 strings: key, value and a boolean is_i18n
  • the get accessor should i18n the value if is_i18n
  • the getDocumentVocabularies should looks like this: the addVocabularies should eat it easily
vocabularies = {
    'theme_voc': [ ('key': 'society',
                    'value': 'cpsdoc_theme_voc_society',
                    'is_i18n': 1),
                   ('key': 'news',
                    'value': 'News',
                   ('key': 'techno',
                    'value': 'Technology'),

Change History

comment:1 Changed 16 years ago by fguillaume

  • Status changed from new to assigned

Ok, but I prefer to have the is_i18n field global to the vocabulary and not specific to one entry. Having it entry by entry doesn't make much sense to me.

But you want i18n done by the get accessor directly? Not by the display method, in the ZPT, as we usually do?

comment:2 Changed 16 years ago by bdelbosc

  • Severity changed from normal to enhancement

comment:3 Changed 16 years ago by disabled-hc@…

  • Summary changed from refactoring for i18n to refactoring vocabularies for i18n

Was: refactoring for i18n

comment:4 Changed 16 years ago by disabled-hc@…

I'm in the mood for doing it. Is it still on schedule?

comment:5 Changed 15 years ago by bdelbosc

  • Priority changed from P2 to P1

this is a requirement if we want to be i18n when using layout

comment:6 Changed 15 years ago by edegoute

I have made a first step of i18n for vocabularies. Just test it for validation.

comment:7 Changed 15 years ago by fguillaume

  • Cc edegoute@… added

What's the status of this bug? Encolpe, what is your refactoring? CVS diff info ?

comment:8 Changed 14 years ago by fguillaume

  • Priority changed from P1 to P2
  • Severity changed from enhancement to normal
  • Description modified (diff)
  • Milestone changed from unspecified to CPS 3.5.0

Note that by the time refactor implement this, we'll have get rid of dict-based config and will use CMFSetup.

comment:9 Changed 14 years ago by fguillaume

  • Description modified (diff)
Note: See TracTickets for help on using tickets.