Skip to content
New issue

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

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

Already on GitHub? # to your account

Postgres issue #14

Open
jamesbarwick opened this issue Feb 5, 2016 · 5 comments
Open

Postgres issue #14

jamesbarwick opened this issue Feb 5, 2016 · 5 comments
Assignees
Labels

Comments

@jamesbarwick
Copy link

Hey guys. Not everyone likes/uses mysql. So, being one of the few, I found this error:

[PDOException]
SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near "("
LINE 1: ...YING (50) NOT NULL DEFAULT '', "editable" BOOLEAN (1) NOT NU...

When trying to run migrate on a postgres database.

getting rid of the (1) should do the trick. Isn't needed on mysql either.

In the file 20150126111319_settings_initial.php there are two boolean fields.

I removed the 'limit' setting (limit => 1) on both fields and then I was able to run the migration.

@bobmulder
Copy link
Contributor

Hi @jamesbarwick,

Thank you very much for your message! We've struggled with this before on mysql.
Are you able to open up a Pull Request?

Thank you very much!

Greetz,

Bob

@bobmulder bobmulder added the bug label Feb 5, 2016
@bobmulder bobmulder self-assigned this Feb 5, 2016
@jamesbarwick
Copy link
Author

Sure. I will do so.

On 6 Feb 2016, at 12:28 AM, Bob Mulder notifications@github.com wrote:

Hi @jamesbarwick https://github.com/jamesbarwick,

Thank you very much for your message! We've struggled with this before on mysql.
Are you able to open up a Pull Request?

Thank you very much!

Greetz,

Bob


Reply to this email directly or view it on GitHub #14 (comment).

@bobmulder
Copy link
Contributor

Mention this issue number (#14) to refer to this issue

jamesbarwick pushed a commit to jamesbarwick/cakephp-settings that referenced this issue Feb 5, 2016
…t's not needed on boolean fields. And caused an error on Postgres...so...just remove it
@jamesbarwick
Copy link
Author

Hi. Sorry, I noticed the CI is failing. I hope that wasn’t me. All I can say is that I was able to run the migrate script during installation. I didn’t run the unit tests.

I don’t have all the DB’s and the phpunit environment setup…apologies for any inconvenience.

James

On 6 Feb 2016, at 12:31 AM, Bob Mulder notifications@github.com wrote:

Mention this issue number (#14 #14) to refer to this issue


Reply to this email directly or view it on GitHub #14 (comment).

@octisfall
Copy link

octisfall commented Mar 21, 2017

Guys, i'm also stuck with it. Seriosly, fix the problem on Postgres.

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

No branches or pull requests

3 participants