View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
19302 | Bug reports | Survey taking | public | 2023-12-07 11:30 | 2024-04-22 11:50 |
Reporter | bismark | Assigned To | tibor.pacalat | ||
Priority | none | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 5.6.x | ||||
Summary | 19302: 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
| ||||
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) | ||||
Tags | No tags attached. | ||||
Bug heat | 14 | ||||
Complete LimeSurvey version number (& build) | 6.X | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | MySQL | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | 8.0 | ||||
The sample plugin do redirect itself … |
|
@DenisChenu nevertheless, this is a LimeSurvey Bug |
|
@DenisChenu, we faced this issue after upding from LS3.x to latest 5.x. Can you take this one? |
|
@bismark can not do the PR ? |
|
@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. |
|
Both, You have a test method for Tibor ? |
|
@denisChenu, please review and pull the request if everything is fine. |
|
Need master version too :) |
|
Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35831 |
|
Fixed in Release 5.6.49+231212 |
|
@bismark : can you do it for 6.X/master too please ? |
|
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 :-) |
|
@tibor.pacalat, can your team please merge this into LS 6.x as well? |
|
@tibor.pacalat: Since this alos affects Limesurvey 6.x, can you please make sure the fix gets added there as well?! |
|
@gabrieljenik or @DenisChenu, can you take care of this, to port this fix to 6.x? Whenever is good for you. Thank you |
|
I will |
|
Master PR: https://github.com/LimeSurvey/LimeSurvey/pull/3822 |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36275 |
|
Fixed in Release 6.5.4+240422 |
|
LimeSurvey: 5.x 23795ae7 2023-12-11 14:03 Committer: GitHub Details Diff |
Fixed issue 19302: Wrong redirect when directLogin is true (03660) |
Affected Issues 19302 |
|
mod - application/controllers/RegisterController.php | Diff File | ||
LimeSurvey: master 60d827a6 2024-04-19 13:57 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19302: Wrong redirect when directLogin is true (#3822) Co-authored-by: Marko Bischof <marko.bischof@gmail.com> |
Affected Issues 19302 |
|
mod - application/controllers/RegisterController.php | Diff File |
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 |
2024-04-09 17:11 | Mazi | Note Added: 79896 | |
2024-04-09 17:58 | tibor.pacalat | Note Added: 79897 | |
2024-04-09 17:58 | tibor.pacalat | Bug heat | 10 => 12 |
2024-04-11 15:06 | gabrieljenik | Note Added: 79903 | |
2024-04-11 15:06 | gabrieljenik | Bug heat | 12 => 14 |
2024-04-11 15:06 | gabrieljenik | Assigned To | tibor.pacalat => gabrieljenik |
2024-04-18 23:24 | gabrieljenik | Status | assigned => ready for code review |
2024-04-18 23:24 | gabrieljenik | Note Added: 79951 | |
2024-04-18 23:24 | gabrieljenik | Assigned To | gabrieljenik => DenisChenu |
2024-04-19 08:20 | DenisChenu | Assigned To | DenisChenu => tibor.pacalat |
2024-04-19 08:20 | DenisChenu | Status | ready for code review => ready for testing |
2024-04-19 11:57 | tibor.pacalat | Status | ready for testing => resolved |
2024-04-19 11:57 | Changeset attached | => LimeSurvey master 60d827a6 | |
2024-04-19 11:57 | guest | Note Added: 79954 | |
2024-04-22 11:50 | LimeBot | Note Added: 79958 | |
2024-04-22 11:50 | LimeBot | Status | resolved => closed |
2024-04-22 11:50 | LimeBot | Resolution | reopened => fixed |