Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDOStorageAdapter incorrect prepared statement? #7

Closed
darrentaytay opened this issue Jul 8, 2015 · 2 comments
Closed

PDOStorageAdapter incorrect prepared statement? #7

darrentaytay opened this issue Jul 8, 2015 · 2 comments

Comments

@darrentaytay
Copy link

Hi there,

I'm attempting to integrate RollOut into a project of mine, however, I'm having issues with the PDOStorageAdapter and I think I know what's happening but just wanted to confirm.

On line 14:

    const STMT_UPDATE = 'UPDATE :table SET settings = :value WHERE key = :key';

Should WHERE key = :key be WHERE name = :key to be consistent with the other prepared statements?

If so, I can submit a pull request as this resolved my issue.

Cheers,
Darren

@mattclements
Copy link
Contributor

I had the same error so have raised a Pull Request.

@richardfullmer
Copy link
Contributor

Fixed. Thanks @mattclements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants