The update 6104 makes non-ANSI SQL queries, misses a closing quote in serialized string, and does not return an array. Also, I changed "boarder" to "border" in CSS style name.

CommentFileSizeAuthor
wysiwyg_imageupload.install.patch1.38 KBtuffnatty

Comments

eugenmayer’s picture

eugenmayer commented

Good catch. Thanks for the patch.

Fixed in BETA6

eugenmayer’s picture

eugenmayer commented
Status: Needs review » Fixed
eugenmayer’s picture

eugenmayer commented
Status: Fixed » Closed (fixed)
tuffnatty’s picture

tuffnatty commented
Status: Closed (fixed) » Active

You didn't apply my patch exactly, and thus the closing quote on serialized string is still missing, also the patch fixes coding style per Drupal standards (SQL keywords case, but this is a minor issue).

tuffnatty’s picture

tuffnatty commented
Version: 6.x-1.0-beta5 » 6.x-1.0-beta7
eugenmayer’s picture

eugenmayer commented
Status: Active » Fixed
eugenmayer’s picture

eugenmayer commented
Status: Fixed » Closed (fixed)