Anonymous Login
2016-09-27 08:54 CEST

View Issue Details Jump to Notes ] Related Changesets ]
IDProjectCategoryView StatusLast Update
08515Bug reports[All Projects] Survey takingpublic2014-01-09 23:31
Reportertpartner 
Assigned ToDenisChenu 
PrioritynormalSeveritymajor 
StatusclosedResolutionfixed 
Product Version2.05+ 
Target Version2.05+Fixed in Version2.05+ 
Summary08515: Language switcher does not work in token registration form
DescriptionLanguage switcher does not work in token registration form
Steps To ReproduceUpload attached survey
Test survey and try to switch language in the registration screen
Additional InformationThis was broken in 1.92, fixed in 2.0 and is now broken again. Can anyone shed some light on why the functionality of the language switcher select was changed?
Complete LimeSurvey version number (& build)769eca5f993cb7ba0a896fc36f03
I will donate to the project if issue is resolvedNo
BrowserFF
Database & DB-VersionN/A
Operating System (Server)N/A
Webserver software & versionN/A
PHP VersionN/A
Attached Files

-Relationships
+Relationships

-Notes

~27729

tpartner (developer)

Denis, your fix to survey_runtime.js on 02/01/2014 seems to have broken this :)

~27730

DenisChenu (developer)

Last edited: 2014-01-07 14:52

View 2 revisions

OK, i try to fix another one with old template ...

I revert or search for a beter way

Token registration form use form#limesurvey too ....

~27733

DenisChenu (developer)

It's not JS ... i send 'lang' now, but same think ...

~27734

DenisChenu (developer)

@tpartner : Can anyone shed some light on why the functionality of the language switcher select was changed?

Answer is here : http://bugs.limesurvey.org/view.php?id=7967

~27735

DenisChenu (developer)

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13650

~27736

DenisChenu (developer)

Updated to use existing form (if exist)
Maybe it's better to use allways $_GET for token and register ?
But then need removing existing lang param in URL.

What do you think Tony : better in $_GET or in $_POST ?

Before it's only by url : no get or post, but http://bugs.limesurvey.org/view.php?id=7967

~27737

tpartner (developer)

Hi Denis,

The language now switches correctly but, since you are submitting the "register" form, the CAPTCHA warning appears after the switch.

~27738

DenisChenu (developer)

Last edited: 2014-01-07 17:24

View 2 revisions

Yes ...

Surely better in $_GET then. But if you have already a /lang/fr in URL : getParam don't get the good param.

But we can have lang-fr or /lang/fr or &lang=fr or ?lang=fr

Did you have an idea to remove it in JS ?

PS: another alternative is to add a data-submit="UrlOfSurveyController" in the select or in the button.
Then we can add directly it to this url.

~27739

DenisChenu (developer)

Got it, add to test with preview to see if it's OK.

~27740

DenisChenu (developer)

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13652

~27741

DenisChenu (developer)

Tony : 2nd fix : working in preview/token/register and in survey even if {LANGUAGECHANGER} is not at the good place.

~27742

tpartner (developer)

You got it :)

~27745

DenisChenu (developer)

JS needed for change language in token and register.

~27804

c_schmitz (administrator)

2.05+ build 140109 released
+Notes

+Related Changesets

-Issue History
Date Modified Username Field Change
2014-01-07 13:19 tpartner New Issue
2014-01-07 13:19 tpartner File Added: limesurvey_survey_926833.lss
2014-01-07 14:00 tpartner Note Added: 27729
2014-01-07 14:00 tpartner Assigned To => DenisChenu
2014-01-07 14:00 tpartner Status new => assigned
2014-01-07 14:31 DenisChenu Note Added: 27730
2014-01-07 14:52 DenisChenu Note Edited: 27730 View Revisions
2014-01-07 15:22 DenisChenu Note Added: 27733
2014-01-07 16:36 DenisChenu Note Added: 27734
2014-01-07 16:40 DenisChenu Changeset attached => LimeSurvey master 5d88fbd6
2014-01-07 16:40 DenisChenu Note Added: 27735
2014-01-07 16:40 DenisChenu Resolution open => fixed
2014-01-07 16:46 DenisChenu Note Added: 27736
2014-01-07 17:12 tpartner Note Added: 27737
2014-01-07 17:23 DenisChenu Note Added: 27738
2014-01-07 17:24 DenisChenu Note Edited: 27738 View Revisions
2014-01-07 17:39 DenisChenu Note Added: 27739
2014-01-07 17:53 DenisChenu Changeset attached => LimeSurvey master 80c3d693
2014-01-07 17:53 DenisChenu Note Added: 27740
2014-01-07 17:54 DenisChenu Note Added: 27741
2014-01-07 18:12 tpartner Note Added: 27742
2014-01-07 19:25 DenisChenu Note Added: 27745
2014-01-07 19:25 DenisChenu Status assigned => resolved
2014-01-07 19:25 DenisChenu Fixed in Version => 2.05+
2014-01-09 23:31 c_schmitz Note Added: 27804
2014-01-09 23:31 c_schmitz Status resolved => closed
+Issue History