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
Comment #1
TravisCarden CreditAttribution: TravisCarden commentedWanna make the whole database issue go away? #1571642: Re-implement on Checklist API? ;-)
Comment #2
TravisCarden CreditAttribution: TravisCarden commentedThank you, @RealJohn. This is fixed in the new 4.x release. Please try it out and give your feedback in the queue!
Comment #3
TravisCarden CreditAttribution: TravisCarden commentedComment #4.0
(not verified) CreditAttribution: commentedtypo corrected