View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
19251 | Bug reports | Conditions | public | 2023-11-18 18:22 | 2024-11-19 18:14 |
Reporter | marcgold | Assigned To | tibor.pacalat | ||
Priority | normal | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 6.3.x | ||||
Summary | 19251: Can't add Regex condition in survey | ||||
Description | Have tried to add a Regular Expression in Condition Designer - the data entry for the regex does not activate. | ||||
Steps To Reproduce | Steps to reproduceAdd conditions in Condition Designer. (Replace this text with detailed step-by-step instructions on how to reproduce the issue) Expected resultYou can enter data (Write here what you expected to happen) Actual result(Write here what happened instead) You cant | ||||
Tags | No tags attached. | ||||
Bug heat | 10 | ||||
Complete LimeSurvey version number (& build) | 6.4 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | any | ||||
Database type & version | Maria 10 | ||||
Server OS (if known) | Ubuntu | ||||
Webserver software & version (if known) | Nginx | ||||
PHP Version | 8.2 | ||||
Quick update on this, I was able to add the regex from the Quick Add conditions screen, However this screen looks a bit weird, with two input boxes at the bottom. Should only be one. |
|
https://github.com/LimeSurvey/LimeSurvey/pull/3657 Regex tab now works.. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35844 |
|
A fix has been committed to master branch, although the Regex tab now works we cannot actually add a regex to the question. When you do the regex entered is blank. If you then try and edit it to add it again, it stated that there is an error and the question has not been selected. |
|
Just circling back, Regex's still not able to save in either the main condition designer or the quick condition designer since the update to 6.4.1 |
|
Master: https://github.com/LimeSurvey/LimeSurvey/pull/3710 This second commit deals with some problems found on the QuickAdd Tab. |
|
I have applied the diff files, and can confirm that the Quick Add panel now adds the Regex and it saves. However the main Conditions pages still saves the regex's blank (ie no conditions). |
|
Tested adding and running a survey adding a regex via the Quick add panel. Works as expected. |
|
This is still an issue? I would add it to a different ticket. |
|
Hi Gabriel, This ticket is the one that deals with the primary condition designer and the regex functionality in that. So technically this the ticket dealing with this issue. But the original issue within the ticket remains an issue. |
|
Hi @marcgold! I am not sure why do you say I have tested this in 5.x and it works the same. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35999 |
|
Hi Tibor, The previous message was referring to Gabiel's comment on adding a new ticket. However this does not work for me on 6.4.3 (with your diff changes to the quick add in place). Here is a link to a Loom Video showing the issue live. https://www.loom.com/share/b2d072c6feec4727833c71ef49d9e093 |
|
I can see the regex on main conditions page with this fix. This will be resolved with the new release on Monday. |
|
THANK YOU!!! I will pull the diff file and test too. |
|
Again, thank you, confirmed working with the updated file edits :) |
|
Fixed in Release 6.4.4+240130 |
|
LimeSurvey: master 20004973 2023-12-12 18:27 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19251: Can't add Regex condition in survey (#3657) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 19251 |
|
mod - assets/scripts/admin/conditions.js | Diff File | ||
LimeSurvey: master f9bd362b 2024-01-29 14:57 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19251: Can't add Regex condition in survey (#3710) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 19251 |
|
mod - application/views/admin/conditions/includes/quickAddConditionForm.php | Diff File | ||
mod - assets/scripts/admin/conditions.js | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2023-11-18 18:22 | marcgold | New Issue | |
2023-11-20 12:47 | marcgold | Note Added: 78534 | |
2023-11-20 12:47 | marcgold | File Added: Screenshot 2023-11-20 at 11.29.40.png | |
2023-11-20 12:47 | marcgold | Bug heat | 0 => 2 |
2023-11-30 15:38 | gabrieljenik | Priority | none => normal |
2023-11-30 15:42 | gabrieljenik | Assigned To | => gabrieljenik |
2023-11-30 15:42 | gabrieljenik | Status | new => assigned |
2023-12-06 17:32 | gabrieljenik | Status | assigned => ready for code review |
2023-12-06 17:32 | gabrieljenik | Note Added: 78897 | |
2023-12-06 17:32 | gabrieljenik | Bug heat | 2 => 4 |
2023-12-06 17:32 | gabrieljenik | Assigned To | gabrieljenik => DenisChenu |
2023-12-12 16:57 | DenisChenu | Assigned To | DenisChenu => tibor.pacalat |
2023-12-12 16:57 | DenisChenu | Status | ready for code review => ready for testing |
2023-12-12 17:27 | tibor.pacalat | Status | ready for testing => resolved |
2023-12-12 17:27 | tibor.pacalat | Resolution | open => fixed |
2023-12-12 17:50 | Changeset attached | => LimeSurvey master 20004973 | |
2023-12-12 17:50 | guest | Note Added: 78961 | |
2023-12-12 17:50 | guest | Bug heat | 4 => 6 |
2023-12-27 14:55 | marcgold | Status | resolved => feedback |
2023-12-27 14:55 | marcgold | Resolution | fixed => reopened |
2023-12-27 14:55 | marcgold | Complete LimeSurvey version number (& build) | 6.3.5 => 6.4 |
2023-12-27 14:55 | marcgold | Note Added: 79072 | |
2024-01-12 13:03 | marcgold | Note Added: 79156 | |
2024-01-12 13:03 | marcgold | Status | feedback => assigned |
2024-01-23 09:34 | gabrieljenik | Status | assigned => ready for code review |
2024-01-23 09:34 | gabrieljenik | Note Added: 79286 | |
2024-01-23 14:24 | marcgold | Note Added: 79290 | |
2024-01-23 15:15 | marcgold | Note Added: 79293 | |
2024-01-23 23:55 | gabrieljenik | Note Added: 79303 | |
2024-01-29 13:31 | marcgold | Note Added: 79320 | |
2024-01-29 13:55 | tibor.pacalat | Note Added: 79321 | |
2024-01-29 13:55 | tibor.pacalat | File Added: Screenshot 2024-01-29 at 13.51.18.png | |
2024-01-29 13:55 | tibor.pacalat | Bug heat | 6 => 8 |
2024-01-29 13:56 | tibor.pacalat | Status | ready for code review => feedback |
2024-01-29 13:57 | Changeset attached | => LimeSurvey master f9bd362b | |
2024-01-29 13:57 | guest | Note Added: 79322 | |
2024-01-29 16:42 | marcgold | Note Added: 79329 | |
2024-01-29 16:42 | marcgold | Status | feedback => assigned |
2024-01-29 17:06 | tibor.pacalat | Note Added: 79333 | |
2024-01-29 17:06 | tibor.pacalat | File Added: Screenshot 2024-01-29 at 17.05.26.png | |
2024-01-29 17:07 | tibor.pacalat | Status | assigned => resolved |
2024-01-29 17:07 | tibor.pacalat | Resolution | reopened => fixed |
2024-01-29 17:17 | marcgold | Note Added: 79334 | |
2024-01-29 17:27 | marcgold | Note Added: 79336 | |
2024-01-30 10:18 | LimeBot | Note Added: 79344 | |
2024-01-30 10:20 | tibor.pacalat | Bug heat | 8 => 10 |
2024-11-19 18:14 | c_schmitz | Status | resolved => closed |