The SQL syntax you've used in places for your INSERTs is MySQL-specific. Patch changes to the standard SQL INSERT syntax of INSERT INTO (field1, field2) VALUES (value1, value2)

Comments

redndahead’s picture

Status: Needs review » Needs work

I don't have postgreSQL to test, but I can verify that there are no adverse affects on mysql. I'll also say that you forgot a line in the navigate_favorites.module.patch file. The same change needs to be applied on line 149 also.

jaydub’s picture

StatusFileSize
new1.36 KB

re-rolled the patch

redndahead’s picture

Status: Needs work » Needs review

looks fine in mysql end.

chrisshattuck’s picture

Thanks, jaydub, I've applied the patch to dev and it will be added in BETA3.

chrisshattuck’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.