View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 10
IDProjectCategoryView StatusDate SubmittedLast Update
14159Bug reports_ Unknownpublic2018-10-22 09:372019-04-30 09:12
ReporterDeusdeorum Assigned ToDenisChenu  
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version3.15.x 
Target Version3.15.x 
Summary14159: Cannot delete response
Description

If you have a survey with answers, and go to responses and try to delete one answer while clicking the red garbage can it wont be deleted, you will have to mark the response and click on the "Selected responses" and delete it that way.

Steps To Reproduce

i) choose a survey with answers
ii) go to response table
iii) click the red garbage can and try deleting it

it wont be deleted

this is tested in "get" url formats, not in "path"

TagsNo tags attached.
Attached Files
delete_response.png (80,453 bytes)   
delete_response.png (80,453 bytes)   
Bug heat10
Complete LimeSurvey version number (& build)3.15.1
I will donate to the project if issue is resolvedNo
BrowserTested in Chrome and Safari
Database type & versionmysqlnd 5.0.12-dev (LS db 355)
Server OS (if known)Linux stretch 4.9.0-4-amd64
Webserver software & version (if known)nginx/1.10.3
PHP Version7.0.27-0+deb9u1

Relationships

Relationship GraphDependency Graph
related to 14222 closedDenisChenu When deleting a single response : all page is reloaded and current filter lost 
related to 14219 closedDenisChenu Unable to quick delete reponse with urlFormat to get 

Users monitoring this issue

There are no users monitoring this issue.

Activities

dominikvitt

dominikvitt

2018-10-23 17:52

developer   ~49418

Unable to reproduce this issue both locally and on hosting server using Firefox and Chrome, response record got deleted every time.
Deusdeorum

Deusdeorum

2018-10-24 09:05

reporter   ~49419

@dominikvitt That's very odd. I just reproduced this error on demo.limesurvey.org as well.. what version are you using? I had another person confirming it on another installation as well
DenisChenu

DenisChenu

2018-10-24 09:39

developer   ~49420

I can confirm issue with last master on browse response

Currently : no action seems done after click on Yes on confirm box. excepot just reloading current page.

Capture du 2018-10-24 09-38-45.png (7,627 bytes)   
Capture du 2018-10-24 09-38-45.png (7,627 bytes)   

user153797

2018-10-24 10:34

  ~49421

Last edited: 2018-10-24 10:54

Offensive commit: bf4e3132 (maybe)
dominikvitt

dominikvitt

2018-11-09 18:21

developer   ~49599

I was testing with "path" url format, so it worked then.
It doesn't work for "get" url format.
DenisChenu

DenisChenu

2018-11-09 18:37

developer   ~49600

@dominikvitt : https://github.com/LimeSurvey/LimeSurvey/pull/1166
We MUST use Yii CButton and not do our work …

Current pull request fix this issue (and REALLY) fix it; And i start fixing security issue for token.
DenisChenu

DenisChenu

2018-11-11 19:08

developer   ~49605

https://github.com/LimeSurvey/LimeSurvey/pull/1166
DenisChenu

DenisChenu

2018-11-12 15:14

developer   ~49612

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

DenisChenu

2018-11-12 15:15

developer   ~49613

Quick fix fixing convertGETtoPOST function
Deusdeorum

Deusdeorum

2018-11-13 09:24

reporter   ~49620

Thank you Denis!
DenisChenu

DenisChenu

2018-11-13 11:11

developer   ~49621

Need more update … https://github.com/LimeSurvey/LimeSurvey/pull/1166
c_schmitz

c_schmitz

2019-04-30 09:12

administrator   ~51655

Fixed in Version 3.1.7.3

Related Changesets

LimeSurvey: master 7cadc79d

2018-11-12 16:14

DenisChenu


Details Diff		 Fixed issue 14159: Cannot delete response urlParam to get
Dev: quick fixed before moving to CButtonColumn		 Affected Issues
14159
mod - application/helpers/common_helper.php Diff File
mod - application/models/SurveyDynamic.php Diff File

Issue History

Date Modified Username Field Change
2018-10-22 09:37 Deusdeorum New Issue
2018-10-22 09:37 Deusdeorum File Added: delete_response.png
2018-10-23 17:35 dominikvitt Assigned To => dominikvitt
2018-10-23 17:35 dominikvitt Status new => assigned
2018-10-23 17:52 dominikvitt Status assigned => resolved
2018-10-23 17:52 dominikvitt Resolution open => unable to reproduce
2018-10-23 17:52 dominikvitt Note Added: 49418
2018-10-24 09:05 Deusdeorum Note Added: 49419
2018-10-24 09:39 DenisChenu File Added: Capture du 2018-10-24 09-38-45.png
2018-10-24 09:39 DenisChenu Note Added: 49420
2018-10-24 10:29 dominikvitt Status resolved => feedback
2018-10-24 10:29 dominikvitt Resolution unable to reproduce => reopened
2018-10-24 10:34 user153797 Note Added: 49421
2018-10-24 10:54 ollehar Note Edited: 49421
2018-10-26 11:38 Deusdeorum Steps to Reproduce Updated
2018-10-26 11:38 Deusdeorum Status feedback => confirmed
2018-11-09 18:21 dominikvitt Note Added: 49599
2018-11-09 18:34 DenisChenu Relationship added related to 14222
2018-11-09 18:35 DenisChenu Relationship added related to 14219
2018-11-09 18:37 DenisChenu Note Added: 49600
2018-11-09 19:24 DenisChenu Assigned To dominikvitt => DenisChenu
2018-11-11 19:08 DenisChenu Note Added: 49605
2018-11-12 15:14 DenisChenu Changeset attached => LimeSurvey master 7cadc79d
2018-11-12 15:14 DenisChenu Note Added: 49612
2018-11-12 15:15 DenisChenu Status confirmed => resolved
2018-11-12 15:15 DenisChenu Resolution reopened => fixed
2018-11-12 15:15 DenisChenu Note Added: 49613
2018-11-13 09:24 Deusdeorum Note Added: 49620
2018-11-13 11:11 DenisChenu Note Added: 49621
2019-04-30 09:12 c_schmitz Note Added: 51655
2019-04-30 09:12 c_schmitz Status resolved => closed