Possible bug in app.js / async_upload_form() #4361

Closed
rcubetrac opened this Issue Oct 15, 2013 · 3 comments

Comments

Projects
None yet
1 participant

Reported by encomiast on 15 Oct 2013 19:50 UTC as Trac ticket #1489379

Hi Roundcube team,

From reading through the sources, I may have spotted an error on this line:
https://github.com/roundcube/roundcubemail/blob/master/program/js/app.js#L6570
I guess it should say

$('[+ frame_name + '"](name="')').bind('load', {ts:ts}, onload);

Otherwise, the iframe is not selected and binding won't take place.

Cheers,
Florian

Migrated-From: http://trac.roundcube.net/ticket/1489379

Comment by @alecpl on 16 Oct 2013 07:11 UTC

Fixed in ff993eb.

Status changed by @alecpl on 16 Oct 2013 07:11 UTC

new => closed

Milestone changed by @alecpl on 16 Oct 2013 07:11 UTC

later => 0.9.5

rcubetrac closed this Oct 16, 2013

rcubetrac added this to the 0.9.5 milestone Mar 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment