The current riddler module for 7.x does not allow translations of questions/answers in a multi-language site.

The attached patch sends all questions/answers to the t() function, that allows 1:1 translations via the /admin/config/regional/translate/translate page. Of course the best approach would be to integrate i18n directly in the riddler form, but for now this simple way works for my humble needs and might work for others as well.

CommentFileSizeAuthor
riddler.translation.patch1.24 KBptaff
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jasom’s picture

jasom CreditAttribution: jasom commented

Nice works, thanks.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Issue summary: View changes

This patch works fine for me. I have tested it thoroughly on one 2-langauge site, and will test on more massively-multilingual sites soon. Thank you. Please merge this patch with the stable release.

zionduc’s picture

zionduc CreditAttribution: zionduc commented

It's not working for me.
I've added a new question/answer pair, cleared cache, but strings missing at /admin/config/regional/translate/translate page.
Is there any dependency to make this patch working?

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

@zionduc - are you searching for the exact case-sensitive string "Do you like SPAM?" for the translations of the default question? - it is working for me and I have tested this extensively.

However when I add a new question and answer, I do not find it anywhere in the translate system, and of course, it isn't in the variables list either. So, the patch is missing something as regards added question / answer pairs.

simon147’s picture

simon147 CreditAttribution: simon147 commented

Would it be possible to integrate this patch into the main file of this module?

bglogo’s picture

bglogo CreditAttribution: bglogo commented

The pactch works fine, many thanks. It's good users to know that Riddler questions should be generated again to become possible for translation.

simon147’s picture

simon147 CreditAttribution: simon147 commented

Installed this and - like others - can't find my specific questions under /admin/config/regional/translate/translate

jasom’s picture

jasom CreditAttribution: jasom commented

@simon147:

- load string in your localized version of the site
- refresh all strings (tab in /admin/config/regional/translate/)
- make sure you have enabled all necessary i18n submodules

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor as a volunteer and at Drupal Ukraine Community, GOLEMS GABB for GOLEMS GABB, Drupal Ukraine Community commented

Look good

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor as a volunteer and at Drupal Ukraine Community, GOLEMS GABB for GOLEMS GABB, Drupal Ukraine Community commented
Status: Needs review » Reviewed & tested by the community

  • alex_optim committed 440c11a on 7.x-1.x authored by ptaff 
    Issue #1853858 by ptaff, pifagor: Very basic patch for i18n
alex_optim’s picture

alex_optim
Location 🇺🇦 Rivne
CreditAttribution: alex_optim at GOLEMS GABB, Drupal Ukraine Community for GOLEMS GABB, Drupal Ukraine Community commented
Status: Reviewed & tested by the community » Patch (to be ported)
alex_optim’s picture

alex_optim
Location 🇺🇦 Rivne
CreditAttribution: alex_optim at GOLEMS GABB, Drupal Ukraine Community for GOLEMS GABB, Drupal Ukraine Community commented
Status: Patch (to be ported) » Fixed
alex_optim’s picture

alex_optim
Location 🇺🇦 Rivne
CreditAttribution: alex_optim at GOLEMS GABB, Drupal Ukraine Community for GOLEMS GABB, Drupal Ukraine Community commented
Status: Fixed » Closed (fixed)