View Issue Details

This bug affects 1 person(s).
 10
IDProjectCategoryView StatusLast Update
06719Bug reportsSurvey takingpublic2012-10-19 16:21
Reporterklaster Assigned Toc_schmitz  
PriorityhighSeveritypartial_block 
Status closedResolutionfixed 
Product Version2.00+ 
Fixed in Version2.00+ 
Summary06719: Language parameter is incorrect in survey url
Description

multilanguage survey url is incorrect:

http://www-myserver.com/survey/index.php/survey/index/sid/112131/token/rd97y3j9tsguxvz/langcode/nl

should be

http://www-myserver.com/survey/index.php/survey/index/sid/112131/token/rd97y3j9tsguxvz/lang/nl

TagsNo tags attached.
Bug heat10
Complete LimeSurvey version number (& build)121017
I will donate to the project if issue is resolvedNo
Browser
Database type & versionmysql 5.5
Server OS (if known)centos 6.3
Webserver software & version (if known)apache 2.2.22
PHP Version5.3.16

Relationships

has duplicate 06725 closedc_schmitz langcode portion in survey invite URL does not set default language correctly 

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2012-10-17 14:42

administrator   ~21404

And where do you see that?

klaster

klaster

2012-10-17 14:45

reporter   ~21405

this url is sent to survey participants in the e-mail message

c_schmitz

c_schmitz

2012-10-17 16:51

administrator   ~21409

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

klaster

klaster

2012-10-17 19:20

reporter   ~21411

After applying the patch, urls are still incorrect (langcode vs lang)

c_schmitz

c_schmitz

2012-10-17 21:07

administrator   ~21413

Last edited: 2012-10-17 21:07

So, are we talking about invitations, reminders or both? It really helps if you are more specific.

klaster

klaster

2012-10-17 21:13

reporter   ~21414

sorry for being vague previously - it is present in both invitations and reminders

klaster

klaster

2012-10-18 10:04

reporter   ~21421

/survey/application/controllers/admin/tokens.php (line 1400):

delete:
$fieldsarray["{SURVEYURL}"] = $this->getController() ->createAbsoluteUrl("/survey/index/sid/{$iSurveyId}/token/{$emrow['token']}/langcode/" . trim($emrow['language']) . "/");

and replace with:
$fieldsarray["{SURVEYURL}"] = $this->getController() ->createAbsoluteUrl("/survey/index/sid/{$iSurveyId}/token/{$emrow['token']}/lang/" . trim($emrow['language']) . "/");

fixes the url in both invitation letter and in reminder.

c_schmitz

c_schmitz

2012-10-18 11:04

administrator   ~21426

Thank you very much!

c_schmitz

c_schmitz

2012-10-18 11:04

administrator   ~21427

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

c_schmitz

c_schmitz

2012-10-19 16:21

administrator   ~21490

New version 2.00+ Build 121019 released. Please update.

Related Changesets

LimeSurvey: master d9c6820a

2012-10-17 09:50

c_schmitz


Details Diff
Fixed issue 06719: Language parameter is incorrect in survey url Affected Issues
06719
mod - application/helpers/admin/token_helper.php Diff File

LimeSurvey: master d86778de

2012-10-18 04:04

c_schmitz


Details Diff
Fixed issue 06719: Language parameter is incorrect in survey url Affected Issues
06719
mod - application/controllers/admin/tokens.php Diff File

Issue History

Date Modified Username Field Change
2012-10-17 13:52 klaster New Issue
2012-10-17 14:42 c_schmitz Note Added: 21404
2012-10-17 14:42 c_schmitz Assigned To => c_schmitz
2012-10-17 14:42 c_schmitz Status new => feedback
2012-10-17 14:45 klaster Note Added: 21405
2012-10-17 14:45 klaster Status feedback => assigned
2012-10-17 16:50 c_schmitz Status assigned => resolved
2012-10-17 16:50 c_schmitz Fixed in Version => 2.00+
2012-10-17 16:50 c_schmitz Resolution open => fixed
2012-10-17 16:51 c_schmitz Changeset attached => LimeSurvey master d9c6820a
2012-10-17 16:51 c_schmitz Note Added: 21409
2012-10-17 19:20 klaster Note Added: 21411
2012-10-17 19:20 klaster Status resolved => feedback
2012-10-17 19:20 klaster Resolution fixed => reopened
2012-10-17 21:07 c_schmitz Note Added: 21413
2012-10-17 21:07 c_schmitz Note Edited: 21413
2012-10-17 21:13 klaster Note Added: 21414
2012-10-17 21:13 klaster Status feedback => assigned
2012-10-18 10:04 klaster Note Added: 21421
2012-10-18 11:04 c_schmitz Note Added: 21426
2012-10-18 11:04 c_schmitz Status assigned => resolved
2012-10-18 11:04 c_schmitz Resolution reopened => fixed
2012-10-18 11:04 c_schmitz Changeset attached => LimeSurvey master d86778de
2012-10-18 11:04 c_schmitz Note Added: 21427
2012-10-18 11:05 c_schmitz Relationship added has duplicate 06725
2012-10-19 16:21 c_schmitz Note Added: 21490
2012-10-19 16:21 c_schmitz Status resolved => closed