View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 10
IDProjectCategoryView StatusDate SubmittedLast Update
18929Bug reportsSurvey editingpublic2023-06-28 15:512023-07-17 10:07
Reporterc_schmitz Assigned Totibor.pacalat  
PriorityhighSeverityblock 
Status closedResolutionfixed 
Product Version6.1.x 
Summary18929: Cannot edit data notification entries
Description

If data notification entries are set to 'Inherit', I cannot deactivate inheritance to provide my own value in text fields.

Steps To Reproduce

Steps to reproduce

Install LimeSurvey in a subfolder on the web server root (for example /limesurvey ).
Create a new survey. Go to tab 'Notification & data' and try to deactivate inheritance for the field 'Send basic admin notification email to:'

Expected result

Text field should be enabled, and I should be able to enter my own value.

Actual result

Text field stays disabled. I cannot enter a new value.

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 & version.
Server OS (if known).
Webserver software & version (if known).
PHP Version.

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2023-06-28 15:54

administrator   ~75899

The path to related JS files are wrong if you installed LimeSurvey in a subfolder, see image. (I installed LimeSurvey in localhost/limesurvey)

Screenshot 2023-06-28 155243.png (9,390 bytes)   
Screenshot 2023-06-28 155243.png (9,390 bytes)   
gabrieljenik

gabrieljenik

2023-06-29 16:30

manager   ~75911

Master: https://github.com/LimeSurvey/LimeSurvey/pull/3265
gabrieljenik

gabrieljenik

2023-07-03 15:19

manager   ~75921

v5: https://github.com/LimeSurvey/LimeSurvey/pull/3266
tibor.pacalat

tibor.pacalat

2023-07-10 14:45

administrator   ~76007

@gabrieljenik I don't see any issue in master and 5.x. Don't know if I did something wrong with the setup.

I installed the app in subdirectory of public (.../public/subdir)

The screenshot is from master.

Screenshot 2023-07-10 at 14.31.49.png (115,211 bytes)
gabrieljenik

gabrieljenik

2023-07-10 14:48

manager   ~76008

So you couldn't reproduce after the patch of before the patch?
tibor.pacalat

tibor.pacalat

2023-07-10 15:02

administrator   ~76009

I tested on master and 5.x branch first, to reproduce the issue, not the branch with the fix.
gabrieljenik

gabrieljenik

2023-07-10 15:06

manager   ~76011

Have you checked the js console log?
tibor.pacalat

tibor.pacalat

2023-07-10 15:14

administrator   ~76013

The bug is "Cannot edit data notification entries", which I could not reproduce. Didn't see anything in the console log, but I didn't pay too much attention to it, after I couldn't reproduce the issue.
gabrieljenik

gabrieljenik

2023-07-11 14:26

manager   ~76040

Sorry. On master, the error was on 2 screens: Failed Emails and Notification Settings.
On v5 it was only on FailedEmails screen.
tibor.pacalat

tibor.pacalat

2023-07-11 16:05

administrator   ~76046

Tested and merged.
guest

guest

2023-07-11 16:28

viewer   ~76048

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

guest

2023-07-11 16:28

viewer   ~76050

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

LimeBot

2023-07-17 10:07

administrator   ~76103

Fixed in Release 6.1.8+230717

Related Changesets

LimeSurvey: master 6d4934a1

2023-07-11 18:04

Gabriel Jenik

Committer: GitHub


Details Diff		 Fixed issue 18929: Cannot edit data notification entries (#3265)

Co-authored-by: lapiudevgit <devgit@lapiu.biz>		 Affected Issues
18929
mod - application/controllers/FailedEmailController.php Diff File
mod - application/views/layouts/adminmenu.php Diff File
mod - application/views/surveyAdministration/editLocalSettings_main_view.php Diff File

LimeSurvey: 5.x 152a933d

2023-07-11 18:05

Gabriel Jenik

Committer: GitHub


Details Diff		 Fixed issue 18929: Cannot edit data notification entries (#3266)

Co-authored-by: lapiudevgit <devgit@lapiu.biz>		 Affected Issues
18929
mod - application/controllers/FailedEmailController.php Diff File

Issue History

Date Modified Username Field Change
2023-06-28 15:51 c_schmitz New Issue
2023-06-28 15:51 c_schmitz Assigned To => gabrieljenik
2023-06-28 15:51 c_schmitz Status new => assigned
2023-06-28 15:54 c_schmitz Note Added: 75899
2023-06-28 15:54 c_schmitz File Added: Screenshot 2023-06-28 155243.png
2023-06-28 15:54 c_schmitz Bug heat 0 => 2
2023-06-28 15:54 c_schmitz Priority none => high
2023-06-28 16:04 c_schmitz Steps to Reproduce Updated
2023-06-28 16:04 c_schmitz Steps to Reproduce Updated
2023-06-29 16:30 gabrieljenik Assigned To gabrieljenik => DenisChenu
2023-06-29 16:30 gabrieljenik Status assigned => ready for code review
2023-06-29 16:30 gabrieljenik Note Added: 75911
2023-06-29 16:30 gabrieljenik Bug heat 2 => 4
2023-07-03 15:19 gabrieljenik Note Added: 75921
2023-07-04 17:04 DenisChenu Assigned To DenisChenu => tibor.pacalat
2023-07-04 17:04 DenisChenu Status ready for code review => ready for testing
2023-07-10 14:45 tibor.pacalat Note Added: 76007
2023-07-10 14:45 tibor.pacalat File Added: Screenshot 2023-07-10 at 14.31.49.png
2023-07-10 14:45 tibor.pacalat Bug heat 4 => 6
2023-07-10 14:48 gabrieljenik Note Added: 76008
2023-07-10 15:02 tibor.pacalat Note Added: 76009
2023-07-10 15:06 gabrieljenik Note Added: 76011
2023-07-10 15:14 tibor.pacalat Note Added: 76013
2023-07-11 14:26 gabrieljenik Note Added: 76040
2023-07-11 16:05 tibor.pacalat Status ready for testing => resolved
2023-07-11 16:05 tibor.pacalat Resolution open => fixed
2023-07-11 16:05 tibor.pacalat Note Added: 76046
2023-07-11 16:28 Changeset attached => LimeSurvey master 6d4934a1
2023-07-11 16:28 guest Note Added: 76048
2023-07-11 16:28 guest Bug heat 6 => 8
2023-07-11 16:28 Changeset attached => LimeSurvey 5.x 152a933d
2023-07-11 16:28 guest Note Added: 76050
2023-07-17 10:07 LimeBot Note Added: 76103
2023-07-17 10:07 LimeBot Status resolved => closed
2023-07-17 10:07 LimeBot Bug heat 8 => 10