Hi Everyone,
this is my first issue report, so please don't be hard on me :)

Issue

So I've tried to install the newest beta of this module on D7. Steps:
1) install via url, success
2) Modules -> tick SEO Checklist on
3) Click save -> error: syntax error in seochecklist.install line 177. near 've'.

If I refresh the error page I'm redirected to the modules list with SEO checklist ticked on what suggests that I somehow managed to turn the module on. But opening the module's page I saw that menu items under 'Clean code' were missing which meant that the subsequent INSERTs had failed.

Reason

After a while I managed to figure out that the problem is with the escape of >>you\'ve<<.

/ from http://www.sqlite.org/lang_expr.html /
A string constant is formed by enclosing the string in single quotes ('). A single quote within the string can be encoded by putting two single quotes in a row - as in Pascal. C-style escapes using the backslash character are not supported because they are not standard SQL.

Workaround solution

I completely removed the module, replaced the occurrences of \' with '' in seochecklist.install and tried again. Everything went fine and I have all the items of the module checklist.

Permanent solution

SQLite-specific install script?
General escape sequence?
Change of the text?

Comments

TravisCarden’s picture

Wanna make the whole database issue go away? #1571642: Re-implement on Checklist API? ;-)

TravisCarden’s picture

Status: Active » Fixed

Thank you, @RealJohn. This is fixed in the new 4.x release. Please try it out and give your feedback in the queue!

TravisCarden’s picture

Version: 7.x-3.0-beta1 » 7.x-4.0-beta1

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

typo corrected