View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
08515Bug reportsSurvey takingpublic2014-01-09 23:31
Reportertpartner Assigned ToDenisChenu  
PrioritynormalSeveritypartial_block 
Status closedResolutionfixed 
Product Version2.05+ 
Target Version2.05+Fixed in Version2.05+ 
Summary08515: Language switcher does not work in token registration form
Description

Language switcher does not work in token registration form

Steps To Reproduce

Upload attached survey
Test survey and try to switch language in the registration screen

Additional Information

This 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?

TagsNo tags attached.
Attached Files
Bug heat6
Complete LimeSurvey version number (& build)769eca5f993cb7ba0a896fc36f03
I will donate to the project if issue is resolvedNo
BrowserFF
Database type & versionN/A
Server OS (if known)N/A
Webserver software & version (if known)N/A
PHP VersionN/A

Users monitoring this issue

There are no users monitoring this issue.

Activities

tpartner

tpartner

2014-01-07 14:00

partner   ~27729

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

DenisChenu

DenisChenu

2014-01-07 14:31

developer   ~27730

Last edited: 2014-01-07 14:52

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

DenisChenu

DenisChenu

2014-01-07 15:22

developer   ~27733

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

DenisChenu

DenisChenu

2014-01-07 16:36

developer   ~27734

@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

DenisChenu

DenisChenu

2014-01-07 16:40

developer   ~27735

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

DenisChenu

DenisChenu

2014-01-07 16:46

developer   ~27736

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

tpartner

tpartner

2014-01-07 17:12

partner   ~27737

Hi Denis,

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

DenisChenu

DenisChenu

2014-01-07 17:23

developer   ~27738

Last edited: 2014-01-07 17:24

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.

DenisChenu

DenisChenu

2014-01-07 17:39

developer   ~27739

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

DenisChenu

DenisChenu

2014-01-07 17:53

developer   ~27740

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

DenisChenu

DenisChenu

2014-01-07 17:54

developer   ~27741

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

tpartner

tpartner

2014-01-07 18:12

partner   ~27742

You got it :)

DenisChenu

DenisChenu

2014-01-07 19:25

developer   ~27745

JS needed for change language in token and register.

c_schmitz

c_schmitz

2014-01-09 23:31

administrator   ~27804

2.05+ build 140109 released

Related Changesets

LimeSurvey: master 5d88fbd6

2014-01-07 15:40:15

DenisChenu

Details Diff
Fixed issue 08515: Language switcher does not work in token registration form
Dev: lang search for all existing form
Dev: alternative if form don't exist : remove /lang-XX, /lang/XX and lang=XX from url and do it in GET param
Dev: Remove some hardly coded SQL in RegisterController
Dev: remove some uneended code in RegisterController
Affected Issues
08515
mod - application/controllers/RegisterController.php Diff File
mod - application/controllers/survey/index.php Diff File
mod - scripts/survey_runtime.js Diff File

LimeSurvey: master 80c3d693

2014-01-07 16:53:01

DenisChenu

Details Diff
Dev: refix 08515: Language switcher does not work in token registration form
Dev: use GET if we are no in form, except if #limesurvey for exist or we are not in preview mode
Affected Issues
08515
mod - application/helpers/frontend_helper.php Diff File
mod - scripts/survey_runtime.js Diff File

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
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
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