View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
19082Bug reportsFile managerpublic2023-11-16 21:13
Reportertibor.pacalat Assigned To 
PriorityhighSeveritypartial_block 
Status confirmedResolutionopen 
Product Version6.3.0-dev 
Summary19082: File manager for Email attachments stops working after first upload
Description

File manager for Email attachments stops working after first upload

Steps To Reproduce

Steps to reproduce

  1. Create new survey
  2. Go to Email Templates
  3. Upload and add an attachment
  4. Save and try to upload another one
  5. Go to survey list and try again to add an attachment

There is a video of the issue attached.

Expected result

I should be able to add an attachment.

Actual result

I can add an attachment, but afterwards I can not upload another one or add another one.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)6.3.0-dev
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

tibor.pacalat

tibor.pacalat

2023-09-13 14:10

administrator   ~76983

ollehar

ollehar

2023-09-13 14:16

administrator   ~76984

Can you please click F12 in the browser and give us a screenshot of any errors shown in the JavaScript console? Thank you.

ollehar

ollehar

2023-09-13 14:16

administrator   ~76985

(That's a template snippet, but F12 usually gives us some good hint. :) )

tibor.pacalat

tibor.pacalat

2023-09-13 14:43

administrator   ~76987

I don't see anything in the console or network requests that is failing.

gabrieljenik

gabrieljenik

2023-09-14 14:00

manager   ~77033

I was able to reproduce it in the following way:
1) Open the email templates screen of a survey with attachments, with or without errors.
2) Click on an attachment to open KCFinder, and close it
3) Press the Back button of the browser

Considering that it only happens after you have opened the KCFinder and apparently it hangs loading a gif, it must be something related to a loader that must be set the first time you open the kcfinder and it must remain set.

tibor.pacalat

tibor.pacalat

2023-09-14 15:06

administrator   ~77038

@gabrieljenik can you take this and make it a priority? It seems pretty bad to me.

gabrieljenik

gabrieljenik

2023-09-14 15:34

manager   ~77040

Well, for me, pretty bad would be not allowing people to take surveys :)
This is not good, obviously, but many things are still working even with this bug :)

gabrieljenik

gabrieljenik

2023-10-23 18:05

manager   ~77886

This is very hard to reproduce.
Couldn't reproduce the ticket problem exactly as described, so we focused a bit on the problem "going back" situation mentioned in the comments.

I could only reproduce the crash itself in Vivaldi (it stays in pending loading the gif and I have to kill the process no matter what). In Chrome it hangs a little but you can close the tab.

The first thing I see, which doesn't seem right to me, is that when you close the modal the history of the browser is modified, so when you press Back it doesn't go to the previous page, but instead tries to load back what was in the iframe. But that's not the problem.

Until the commit prior to 16aa81f2810abbcba24a22ae09b5d85b8b8b44fc it worked that way, but it didn't hang, and when I pressed back again it worked OK. In 16aa81f2810abbcba24a22ae09b5d85b8b8b44fc we updated the KCFinder jquery and from there the problem started.

So @tibor.pacalat, could you please try to reproduce it on this commit 16aa81f2810abbcba24a22ae09b5d85b8b8b44fc?

Thanks

tibor.pacalat

tibor.pacalat

2023-10-24 16:38

administrator   ~77922

I could not reproduce this on this commit, however this commit is 5.x and I have reported issue on 6.x where I can still consistently reproduce it.

gabrieljenik

gabrieljenik

2023-10-24 16:47

manager   ~77926

Which browser do you use?

tibor.pacalat

tibor.pacalat

2023-10-24 16:52

administrator   ~77930

I have all browsers installed, but I tested this on Chrome Version 117.

gabrieljenik

gabrieljenik

2023-11-16 21:13

manager   ~78505

@tibor.pacalat
Are you sure you don't have any warnings on the console?
Of any kind?

We were not able to reproduce it.

Something that happened in one of our tests was beacuse of a cookie issue that made the JS to fail.
Maybe that could be happening on your end.

Our end is not the same: We are using Linux + Chrome or Windows + Chrome.
I guess the best is to have this doen by someone who has Mac.

I will deassign it.

Issue History

Date Modified Username Field Change
2023-09-13 13:59 tibor.pacalat New Issue
2023-09-13 14:10 tibor.pacalat Note Added: 76983
2023-09-13 14:10 tibor.pacalat File Added: Screen-2023-09-13-135002.mp4
2023-09-13 14:10 tibor.pacalat Bug heat 0 => 2
2023-09-13 14:16 ollehar Note Added: 76984
2023-09-13 14:16 ollehar Bug heat 2 => 4
2023-09-13 14:16 ollehar Note Added: 76985
2023-09-13 14:43 tibor.pacalat Note Added: 76987
2023-09-13 15:22 ollehar Priority none => high
2023-09-14 14:00 gabrieljenik Note Added: 77033
2023-09-14 14:00 gabrieljenik Bug heat 4 => 6
2023-09-14 14:00 gabrieljenik Status new => confirmed
2023-09-14 14:01 gabrieljenik Severity block => partial_block
2023-09-14 15:06 tibor.pacalat Note Added: 77038
2023-09-14 15:34 gabrieljenik Note Added: 77040
2023-10-19 13:47 gabrieljenik Assigned To => gabrieljenik
2023-10-19 13:47 gabrieljenik Status confirmed => assigned
2023-10-23 18:05 gabrieljenik Note Added: 77886
2023-10-23 18:07 gabrieljenik Status assigned => feedback
2023-10-24 16:38 tibor.pacalat Note Added: 77922
2023-10-24 16:38 tibor.pacalat Status feedback => assigned
2023-10-24 16:47 gabrieljenik Note Added: 77926
2023-10-24 16:52 tibor.pacalat Note Added: 77930
2023-11-16 21:13 gabrieljenik Note Added: 78505
2023-11-16 21:13 gabrieljenik Assigned To gabrieljenik =>
2023-11-16 21:13 gabrieljenik Status assigned => confirmed