View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
08515 | Bug reports | Survey taking | public | 2014-01-07 13:19 | 2014-01-09 23:31 |
Reporter | tpartner | Assigned To | DenisChenu | ||
Priority | normal | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.05+ | ||||
Target Version | 2.05+ | Fixed in Version | 2.05+ | ||
Summary | 08515: 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 | ||||
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? | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 769eca5f993cb7ba0a896fc36f03 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | FF | ||||
Database type & version | N/A | ||||
Server OS (if known) | N/A | ||||
Webserver software & version (if known) | N/A | ||||
PHP Version | N/A | ||||
Denis, your fix to survey_runtime.js on 02/01/2014 seems to have broken this :) |
|
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 .... |
|
It's not JS ... i send 'lang' now, but same think ... |
|
@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 |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13650 |
|
Updated to use existing form (if exist) 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 |
|
Hi Denis, The language now switches correctly but, since you are submitting the "register" form, the CAPTCHA warning appears after the switch. |
|
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. |
|
Got it, add to test with preview to see if it's OK. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13652 |
|
Tony : 2nd fix : working in preview/token/register and in survey even if {LANGUAGECHANGER} is not at the good place. |
|
You got it :) |
|
JS needed for change language in token and register. |
|
2.05+ build 140109 released |
|
LimeSurvey: master 5d88fbd6 2014-01-07 16:40 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 17:53 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 |
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 |