View Issue Details

This bug affects 1 person(s).
 20
IDProjectCategoryView StatusLast Update
06550Bug reportsSurvey takingpublic2012-10-05 15:26
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version1.92+ 
Fixed in Version2.00+ 
Summary06550: Translation of 'Other:' comment mandatory for survey taking (And text is untranslatable)
Description

In french, with:

  • List (radio) with Other and 'Other:' comment mandatory to Yes, the sentence:

Please also fill in the "other comment" field.

Are not translated.
BUT this sentence is translated in fr.mo files.

Steps To Reproduce

Import included survey,
Test the question:
Seen : Please also fill in the & quot;other comment& quot; field.

(before http://bugs.limesurvey.org/plugin.php?page=Source/view&id=9410, same problem but with ")

Additional Information

The sentence is, line 2567 of LimeExpressionManager

$qtips['other_comment_mandatory']=$this->gT('Please also fill in the "other comment" field.');
Think we have to use:
$qtips['other_comment_mandatory']=$this->gT('Please also fill in the other text input field.');

And give the opportunity to translate.

TagsNo tags attached.
Attached Files
Bug heat20
Complete LimeSurvey version number (& build)120822
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionnot relevant
Server OS (if known)debian/linux
Webserver software & version (if known)apache
PHP VersionPHP Version 5.3

Relationships

related to 06536 closedc_schmitz HTML editor converts special characters to html entities within equations 
has duplicate 06646 closedc_schmitz HTML entities shown when answering survey and not filling out mandatory questions (all languages) 
related to 06556 closedDenisChenu 'Other:' comment mandatory with list (and condition on that question) hides entire group? 

Users monitoring this issue

Bitbull

Activities

DenisChenu

DenisChenu

2012-09-10 16:08

developer   ~20653

Alternative:
$qtips['other_comment_mandatory']=$this->gT('Please also fill in the <q>other</q> text input field.');

But need to review if it's OK.

DenisChenu

DenisChenu

2012-09-11 19:49

developer   ~20656

Fix committed to Yii branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&amp;id=9478

DenisChenu

DenisChenu

2012-09-19 10:35

developer   ~20725

Don't work ...
Translation is OK but :

Veuillez indiquer un commentaire pour la réponse & amp; 08220;autre& amp; #8221;.

Denis

DenisChenu

DenisChenu

2012-09-19 10:45

developer   ~20726

Hello Cartsen,

It's seems easy to fix THIS one for one language. ( use utf8 caracter can be a good solution) but we can found this issue elsewhere in future.

Here i test with “ ” directly in language file ( http://translate.limesurvey.org/projects/limesurvey2/fr/default?filters[status]=either&amp;filters[original_id]=5620&amp;filters[translation_id]=268194 )

but there are different solution:

  • don't change code but put information in wiki/glotpress for HTML entity caracter (don't use it, or directly in utf8)
  • see the core code to accept html caracter in en string and translation string (decimal numeric character references, HTML’s character entity , HTML hexadecimal numeric etc ....)( http://www.dwheeler.com/essays/quotes-in-html.html)

Denis

c_schmitz

c_schmitz

2012-09-26 09:35

administrator   ~20821

This is weird, because this has not been a problem in the past. I recently found that the automatically created .mo files had some errors.
I changed the way they are created and regenerated all of them. Can you check if the original issue is still there (without your patch) ?

Bitbull

Bitbull

2012-09-26 09:55

reporter   ~20830

(please consider setting this issue related to http://bugs.limesurvey.org/view.php?id=6556)

c_schmitz

c_schmitz

2012-10-05 15:26

administrator   ~21072

New version 2.00+ build 121005 released.

Related Changesets

LimeSurvey: Yii 17109ccc

2012-09-11 10:45:39

DenisChenu

Details Diff
Dev: fixed issue 06550 : Translation of 'Other:' comment mandatory for survey taking is untranslatable
Dev : use Curling Quotes : tested with 192 translation and poedit: time to see with glotpress
Affected Issues
06550
mod - application/helpers/expressions/em_manager_helper.php Diff File

Issue History

Date Modified Username Field Change
2012-09-06 14:56 DenisChenu New Issue
2012-09-06 14:56 DenisChenu File Added: limesurvey_survey_63681.lss
2012-09-06 14:56 DenisChenu Description Updated
2012-09-06 14:56 DenisChenu Additional Information Updated
2012-09-10 16:08 DenisChenu Note Added: 20653
2012-09-11 19:49 DenisChenu Changeset attached => LimeSurvey Yii 17109ccc
2012-09-11 19:49 DenisChenu Note Added: 20656
2012-09-11 19:49 DenisChenu Assigned To => DenisChenu
2012-09-11 19:49 DenisChenu Resolution open => fixed
2012-09-14 12:01 c_schmitz Status new => resolved
2012-09-14 12:01 c_schmitz Fixed in Version => 2.00+
2012-09-14 12:11 Bitbull Issue Monitored: Bitbull
2012-09-19 10:35 DenisChenu Note Added: 20725
2012-09-19 10:35 DenisChenu Status resolved => feedback
2012-09-19 10:35 DenisChenu Resolution fixed => reopened
2012-09-19 10:36 DenisChenu Status feedback => confirmed
2012-09-19 10:37 DenisChenu Relationship added related to 06536
2012-09-19 10:38 DenisChenu Assigned To DenisChenu => c_schmitz
2012-09-19 10:38 DenisChenu Status confirmed => assigned
2012-09-19 10:45 DenisChenu Note Added: 20726
2012-09-26 09:35 c_schmitz Note Added: 20821
2012-09-26 09:35 c_schmitz Assigned To c_schmitz => DenisChenu
2012-09-26 09:35 c_schmitz Status assigned => feedback
2012-09-26 09:55 Bitbull Note Added: 20830
2012-09-28 08:17 DenisChenu Relationship added related to 06556
2012-10-04 09:04 DenisChenu Status feedback => resolved
2012-10-04 09:04 DenisChenu Resolution reopened => fixed
2012-10-05 09:12 DenisChenu Relationship added has duplicate 06646
2012-10-05 15:26 c_schmitz Note Added: 21072
2012-10-05 15:26 c_schmitz Status resolved => closed
2021-08-02 16:53 guest Bug heat 16 => 20