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

Generic fix for ORACLE db #38

Open
ilbassa opened this issue Jul 18, 2017 · 1 comment
Open

Generic fix for ORACLE db #38

ilbassa opened this issue Jul 18, 2017 · 1 comment
Labels

Comments

@ilbassa
Copy link

ilbassa commented Jul 18, 2017

For more compatibility of the extension can tell you some fix needed for ORACLE database:

  • in the matching of "boolean" value we must use 1/0 because in ORACLE boolean doesn't exist. The migration tool convert an attribute set as $this->boolean() in Numer(1,0) #[36] (ref. issue 36)

  • I have ovverride the notify() function also because for ORACLE the Expression NOW() doesn't exist, so first I replaced it with SYSDATE() that is suited for ORACLE but I can suggest you to use "new Expression('CURRENT_TIMESTAMP')".

  • In the migration I also need to change "created_at" attribute from dateTime to date() because dateTime() is converted into TIMESTAMP and the models return a datetime like this "18-LUG-17 17:53:09,000000", it is possible to have TIMESTAMP with precision 0 in ORACLE that returns "18-LUG-17 17:53:09" but I have not figured out how to set precision 0 in the migration ($this->dateTime(0) doesn't work)

Glad if this can help improve compatibiliy of the extension.

@machour
Copy link
Owner

machour commented Jul 18, 2017 via email

@drsdre drsdre added the bug label Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants