#2 SUBSTRING_FOR_DATE breaks mariadb

Closed
rodneys_mission wants to merge 0 commits from deleted into fox/master

Removed SQL function and added 'substr' to php. Fixes MariaDB.

Removed SQL function and added 'substr' to php. Fixes MariaDB.
rodneys_mission commented 4 months ago
Poster

Note: I did not test against postgres b/c I do not have an install to do so. If the postgres PDO returns dates like 'YYYY-mm-dd HH:MM:SS.....'; then it should be ok for both db's.

Note: I did not test against postgres b/c I do not have an install to do so. If the postgres PDO returns dates like 'YYYY-mm-dd HH:MM:SS.....'; then it should be ok for both db's.
rodneys_mission commented 4 months ago
Poster

Never mind, I see other code where SUBSTRING_FOR_DATE is outside the quotes. I will update to be the same.

Never mind, I see other code where SUBSTRING_FOR_DATE is outside the quotes. I will update to be the same.
fox commented 4 months ago
Owner

postgres is the reason this exists, because of type checking you can't just use substring on a date object there, mysql doesn't care

postgres is the reason this exists, because of type checking you can't just use substring on a date object there, mysql doesn't care
Please reopen this pull request to perform merge operation.
Sign in to join this conversation.
No Label
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.