416 postgresql support #1071
Closed
416 postgresql support #1071
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Conflicts: CHANGELOG README.fr.md README.md app/Controllers/feedController.php app/Controllers/indexController.php app/i18n/en.php app/i18n/fr.php app/views/helpers/view/normal_view.phtml app/views/stats/index.phtml app/views/stats/repartition.phtml constants.php p/scripts/main.js
Add categoryDAO creator in factory, as CAtegoryDAOpgSQL is needed. use new constructor from factory for categoryDAO. Added app/SQL/install.sql.pgsql.php for database creation with postgresql.
Merge branch 'master' into dev
…lls to it.
Removal of FeedDAOpgSQL, merged into FeedAO
backticks (`) replaced with double quotes (") for better compatibility between DBMS.
ModelPdo modified so taht mysql is launched with ANSI_QUOTE (http://dev.mysql.com/doc/refman/5.0/en/sql-mode.html#sqlmode_ansi_quotes), to support double quotes instead of backticks.
Rewrite calls to boolean fields as, depending on the DBMS, we can/need to use 0 or 1, boolean values or 't' or 'f'. Now it is treated by pdo.
…backticks replacement, remove them.
Alkarex
added a commit
to Alkarex/FreshRSS
that referenced
this pull request
Aug 2, 2016
FreshRSS#416 Based on @Damstre work FreshRSS#1071 Not tested
Merged
|
New version of this pull request in #1195 |
|
Continued in #1195 |
javerous
pushed a commit
to javerous/FreshRSS
that referenced
this pull request
Jan 20, 2020
FreshRSS#416 Based on @Damstre work FreshRSS#1071 Not tested
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add support for postgresql dbms.
Mysql and sqlite support not tested.
The column names were changed to be all in lower case. So it might not work with an existing installation. Renaming the columns should work.
Installation not tested either.