if devel mem-usage or theme developer is active, ajax wont work due to broken json.

I could fix the mem output issue with $GLOBALS['devel_shutdown'] = FALSE; - anyway you cant do this for the theme developer. There will be a warning now, if you have the theme developer active.

Will be released in rc5

CommentFileSizeAuthor
#3 wysiwyg_imageupload.form_.inc_.patch549 bytespfrenssen

Comments

eugenmayer’s picture

eugenmayer commented
Status: Fixed » Closed (fixed)

released in rc5

pfrenssen’s picture

pfrenssen
Location Sofia
commented

This is partially fixed in rc5.

the /ajax/wysiwyg_imgupl/render_wysiwyg/ returns the correct data now (without devel debug information).

The debug data however remains in the POST request that is sent beforehand:

request:
http://www.localhost.dev/wysiwyg_imageupload/edit/15?alignment=imgupl_floating_none&form_build_id=form-276cc0b171cbfd8bbc74dcbe653c56e9&form_id=wysiwyg_imageupload_edit_form&form_token=38f635cc602228127c712960a6109038&imagecache=grote-foto&style=0&submitimagedetails=JSinsert&title=20000_1600x1200-wallpaper-cb1273512264.jpg

response:
{ "status": true, "data": { "iid": "15" } }<div class="dev-timer"> Page execution time was 126.03 ms. </div><div class="dev-memory-usage"><h3>Memory usage:</h3> Memory used at: devel_init()=<strong>1.64</strong> MB, devel_shutdown()=<strong>17.54</strong> MB.</div>

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Version: 6.x-2.0-rc4 » 6.x-2.0-rc5
Status: Closed (fixed) » Active
StatusFileSize
new wysiwyg_imageupload.form_.inc_.patch549 bytes

patch in attachment

eugenmayer’s picture

eugenmayer commented
Priority: Critical » Normal
Status: Active » Patch (to be ported)

I will take that one in in the hopefuly stable release or the next RC. Thanks for the contribution and feedback!

eugenmayer’s picture

eugenmayer commented
Status: Patch (to be ported) » Fixed

aplied to git

eugenmayer’s picture

eugenmayer commented
Status: Fixed » Closed (fixed)

released in rc6