Needs review
Project:
SWFUpload
Version:
6.x-2.0-beta8
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Feb 2011 at 00:25 UTC
Updated:
22 Apr 2013 at 16:08 UTC
I'm not sure of exact details of what caused my White Screen Of Death... but saw that
- something was making a POST request to "www.mysite.com/swfupload,/swfupload"
- all settings in the JS swfupload_settings object were 'arrays of duplicate strings'... instead of simple strings.
- drupal_add_js() is called twice with the same setting, sometimes.
(In my case: when drupal_rebuild_form() is called, somehow, when I preview a node.)
Patch attached. Fixed my WSOD.
| Comment | File | Size | Author |
|---|---|---|---|
| swfupload.check_add_js.patch | 5.71 KB | roderik |
Comments
Comment #1
roderikBy the way,
the patch looks horrible, but is a simple addition of 4 lines on top, and '$added[$element['#id']] = TRUE; }' at the bottom. The rest is just indenting.
Comment #2
sansui commentedPatch applied cleanly. I was not having the issues you described, but the patch does not seem to have negatively impacted anything either.
Maybe we can get someone else to test this too and get another confirmation?
Comment #3
Trexor commentedI was getting a flash of the preview then a white screen, I applied the patch and my preview is working perfectly, Thank You Roderik.
Comment #4
Trexor commentedSorry I spoke too soon, things looked OK after the patch but adding more than one image on the preview page, all the file names are the same as the last one added but when I save this it’s ok again, this would confuse most users so I’ve hidden the preview button for now with the Preview module.
Comment #5
roderikI have seen that behavior but it does not have anything to do with this patch. See #875706-28: Rearrange images does not work in Opera & Chrome