View Issue Details

This bug affects 1 person(s).
 10
IDProjectCategoryView StatusLast Update
19302Bug reportsSurvey takingpublic2024-01-11 14:05
Reporterbismark Assigned Totibor.pacalat  
PrioritynoneSeveritypartial_block 
Status assignedResolutionreopened 
Product Version5.6.x 
Summary19302: directLogin = true with wrong redirect
Description

https://manual.limesurvey.org/BeforeRegister

Line 137 `$redirectUrl = Yii::app()->getController()->createUrl('/survey/', array('sid' => $iSurveyId,'token' => $oToken->token, 'lang' => $sLanguage));

must be

$redirectUrl = Yii::app()->getController()->createUrl('/survey/index', array('sid' => $iSurveyId,'token' => $oToken->token, 'lang' => $sLanguage));
`

Steps To Reproduce

Steps to reproduce

(Replace this text with detailed step-by-step instructions on how to reproduce the issue)

Expected result

(Write here what you expected to happen)

Actual result

(Write here what happened instead)

TagsNo tags attached.
Bug heat10
Complete LimeSurvey version number (& build)6.X
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMySQL
Server OS (if known)
Webserver software & version (if known)
PHP Version8.0

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2023-12-07 11:32

developer   ~78909

The sample plugin do redirect itself …
https://gitlab.com/SondagesPro/SurveyAccess/registerQuick/-/blob/master/registerQuick.php?ref_type=heads#L391

bismark

bismark

2023-12-07 11:40

reporter   ~78910

@DenisChenu nevertheless, this is a LimeSurvey Bug

Mazi

Mazi

2023-12-07 13:38

updater   ~78912

Last edited: 2023-12-07 14:05

@DenisChenu, we faced this issue after upding from LS3.x to latest 5.x. Can you take this one?

DenisChenu

DenisChenu

2023-12-07 15:37

developer   ~78914

@bismark can not do the PR ?

Mazi

Mazi

2023-12-07 15:57

updater   ~78916

@DenisChenu, if you consider the fix valid and no unit test or the like is needed for fixing this, we can send a pull request. Please let me know.
Do we need this for 5.x and 6.x?

DenisChenu

DenisChenu

2023-12-07 16:00

developer   ~78917

Both,
And i review.

You have a test method for Tibor ?
A micro plugin ?

bismark

bismark

2023-12-11 09:42

reporter   ~78931

https://github.com/LimeSurvey/LimeSurvey/pull/3660

Mazi

Mazi

2023-12-11 09:53

updater   ~78932

@denisChenu, please review and pull the request if everything is fine.

DenisChenu

DenisChenu

2023-12-11 09:57

developer   ~78933

Need master version too :)

guest

guest

2023-12-11 13:15

viewer   ~78938

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

LimeBot

LimeBot

2023-12-11 13:18

administrator   ~78939

Fixed in Release 5.6.49+231212

DenisChenu

DenisChenu

2023-12-11 14:23

developer   ~78940

@bismark : can you do it for 6.X/master too please ?

Mazi

Mazi

2023-12-11 15:49

updater   ~78941

We would appreciate if you could do that or someone from the Limesurvey development team. We already pay Limesurvey partner fees, we cannot also do the bug fixing or fixes for different releases :-)
Should be a piece of cake for this little issue.

Mazi

Mazi

2024-01-11 14:05

updater   ~79150

@tibor.pacalat, can your team please merge this into LS 6.x as well?

Related Changesets

LimeSurvey: 5.x 23795ae7

2023-12-11 13:03:39

bismark


Committer: GitHub Details Diff
Fixed issue 19302: Wrong redirect when directLogin is true (03660) Affected Issues
19302
mod - application/controllers/RegisterController.php Diff File

Issue History

Date Modified Username Field Change
2023-12-07 11:30 bismark New Issue
2023-12-07 11:32 DenisChenu Note Added: 78909
2023-12-07 11:32 DenisChenu Bug heat 0 => 2
2023-12-07 11:40 bismark Note Added: 78910
2023-12-07 11:40 bismark Bug heat 2 => 4
2023-12-07 11:40 guest Bug heat 4 => 2
2023-12-07 11:40 bismark Bug heat 2 => 4
2023-12-07 13:38 Mazi Note Added: 78912
2023-12-07 13:38 Mazi Bug heat 4 => 6
2023-12-07 14:05 Mazi Note Edited: 78912
2023-12-07 15:37 DenisChenu Note Added: 78914
2023-12-07 15:57 Mazi Note Added: 78916
2023-12-07 16:00 DenisChenu Note Added: 78917
2023-12-11 09:42 bismark Note Added: 78931
2023-12-11 09:52 Mazi Assigned To => DenisChenu
2023-12-11 09:52 Mazi Status new => assigned
2023-12-11 09:53 Mazi Note Added: 78932
2023-12-11 09:57 DenisChenu Assigned To DenisChenu => tibor.pacalat
2023-12-11 09:57 DenisChenu Status assigned => ready for testing
2023-12-11 09:57 DenisChenu Note Added: 78933
2023-12-11 13:15 bismark Changeset attached => LimeSurvey 5.x 23795ae7
2023-12-11 13:15 guest Note Added: 78938
2023-12-11 13:15 guest Bug heat 6 => 8
2023-12-11 13:18 LimeBot Note Added: 78939
2023-12-11 13:18 LimeBot Status ready for testing => closed
2023-12-11 13:18 LimeBot Resolution open => fixed
2023-12-11 13:18 LimeBot Bug heat 8 => 10
2023-12-11 14:23 DenisChenu Assigned To tibor.pacalat =>
2023-12-11 14:23 DenisChenu Status closed => feedback
2023-12-11 14:23 DenisChenu Resolution fixed => reopened
2023-12-11 14:23 DenisChenu Note Added: 78940
2023-12-11 14:23 DenisChenu Status feedback => confirmed
2023-12-11 14:23 DenisChenu Complete LimeSurvey version number (& build) 5.6.36 => 6.X
2023-12-11 15:49 Mazi Note Added: 78941
2024-01-11 14:05 Mazi Note Added: 79150
2024-01-11 14:05 Mazi Assigned To => tibor.pacalat
2024-01-11 14:05 Mazi Status confirmed => assigned