paul
f053f42b6d
Mirror to GitHub / mirror (push) Successful in 19s
Test and Lint / backend-test (push) Successful in 1m8s
Test and Lint / frontend-test (push) Successful in 2m11s
Version and Release / version-bump (push) Successful in 34s
Version and Release / trigger-drone (push) Successful in 3s
continuous-integration/drone/push Build is passing
Critical fixes for database compatibility issues:
INSERT operations:
- Fix all INSERT queries to use .returning('id')
- Handle both PostgreSQL (returns objects) and SQLite (returns IDs)
- Fixed in: events.js, adminArchives.js, adminEvents-enhanced.js, create-test-event.js
Date operations:
- Replace SQLite-specific db.raw("datetime('now', '+30 days')")
- Use JavaScript Date objects for cross-database compatibility
- Fixed in: adminArchives.js
Database utilities:
- Add dbCompat.js utility for handling database differences
- Provides consistent API for inserts, dates, booleans, and DB-specific operations
- Centralized database compatibility logic
Migration:
- Add migration 023 documenting PostgreSQL compatibility requirements
- Ensures future developers are aware of compatibility needs
This resolves all 'not iterable' errors and ensures the application
works correctly with both PostgreSQL (production) and SQLite (development).
22 lines
709 B
JavaScript
22 lines
709 B
JavaScript
/**
|
|
* Ensure PostgreSQL compatibility for all insert operations
|
|
* This migration doesn't change the schema but ensures all tables
|
|
* are compatible with .returning() syntax
|
|
*/
|
|
|
|
exports.up = async function(knex) {
|
|
// This migration is informational only
|
|
// All insert operations should use .returning('id') going forward
|
|
|
|
console.log('PostgreSQL compatibility check:');
|
|
console.log('- All INSERT operations should use .returning("id")');
|
|
console.log('- All date operations should use ISO strings');
|
|
console.log('- Boolean values are handled automatically by Knex');
|
|
|
|
return Promise.resolve();
|
|
};
|
|
|
|
exports.down = async function(knex) {
|
|
// No rollback needed
|
|
return Promise.resolve();
|
|
}; |