[HAP][v8] - My Files Preview Bug

[nickbro]

Hi all,

 

Just found a bug in the My Files v8 where student's couldn't upload an acceptable file. The fix is a simple one:

 

Open ~/Myfiles/default.aspx in notepad

 

Go to line 455

 

and change it to

 

else if ("<%=AcceptedExtensions %>".toLowerCase().indexOf(this.File.name.substr(this.File.name.lastIndexOf('.')).toLowerCase()) == -1 && "<%=DropZoneAccepted %>" != "") {

[mattgrimley]

hmm, my line 455 is just a "}" close bracket..

if i replace that line, then it doesn't load properly at all.. - it just loads all of the forms with no formatting or content.. i'll post more if needed, but im presuming my line 455 isnt the same as yours...

 

It looks like this (before patching):

453 uploads.pop(this);

454 return false;

455 }

456 if(this.File.size > <%=maxRequestLength%>) {

[nickbro]

Ah, dame, I'm on the v8 ALPHA Trunk, it's prob line 450 in that case.

 

It's just a change from <%=DropZoneAccepted%> to <%=AcceptedExtensions%>

[mattgrimley]

ahh cheers will try it out tomorrow!

 

I moved the v8 pre-release over to be my main site after some extensive beta testing. I'll let you know if anything crops up - though it'll be easy enough to revert back to 7.10 as the booking and helpdesk config files remain the same format

 

I must say i really do love it! Grand job!

[mattgrimley]

It's line 451

 

Change:

else if ("<%=DropZoneAccepted %>".toLowerCase().indexOf(this.File.name.substr(this.File.name.lastIndexOf('.')).toLowerCase()) == -1 && "<%=DropZoneAccepted %>" != "") {

 

To:

else if ("<%=[b][color="#FF0000"]AcceptedExtensions[/color][/b] %>".toLowerCase().indexOf(this.File.name.substr(this.File.name.lastIndexOf('.')).toLowerCase()) == -1 && "<%=DropZoneAccepted %>" != "") {

[calvin_il]

cool, that fixed the problem in v7.10 as well...

thanks!

[s1ndr0me]

Thanks, this fixed it for me :-)