Here's the list of errors. If I can get them fixed, I'll post workaround queries or possibly a diff to audio.install.
* warning: pg_query() [function.pg-query]: Query failed: ERROR: column "remote_size" of relation "d_audio" already exists in drupal/includes/database.pgsql.inc on line 139.
* user warning: query: ALTER TABLE d_audio ADD COLUMN remote_size int NOT NULL default 0 in drupal/includes/database.pgsql.inc on line 666.
* warning: pg_query() [function.pg-query]: Query failed: ERROR: syntax error at or near "INNER" LINE 1: UPDATE d_audio a INNER JOIN d_audio_file af ON a.vid = af.vi... ^ in drupal/includes/database.pgsql.inc on line 139.
* user warning: query: UPDATE d_audio a INNER JOIN d_audio_file af ON a.vid = af.vid SET a.file_name = af.filename, a.file_path = af.filepath, a.file_mime = af.filemime, a.file_size = af.filesize in /home/severinghaus/drupal/modules/audio/audio.install on line 299.
* warning: pg_query() [function.pg-query]: Query failed: ERROR: column "fid" contains null values in drupal/includes/database.pgsql.inc on line 139.
* user warning: query: ALTER TABLE d_audio ALTER fid SET NOT NULL in drupal/includes/database.pgsql.inc on line 674.
Update #5202
# Failed: ALTER TABLE {audio} ADD COLUMN remote_size int NOT NULL default 0
# Failed: UPDATE {audio} a INNER JOIN {audio_file} af ON a.vid = af.vid SET a.file_name = af.filename, a.file_path = af.filepath, a.file_mime = af.filemime, a.file_size = af.filesize
Update #6002
# Failed: ALTER TABLE {audio} ALTER fid SET NOT NULL
Comment | File | Size | Author |
---|---|---|---|
#3 | audio-6.x-1.0-unstable6_upgrade_5202-6002_2.patch | 3.27 KB | HorsePunchKid |
#1 | audio-6.x-1.0-unstable6_upgrade_5202-6002_0.patch | 2.53 KB | HorsePunchKid |
Comments
Comment #1
HorsePunchKid CreditAttribution: HorsePunchKid commentedHere is a patch that fixes the various problems I had. I have only tested it against Postgres.
* removed duplicate {audio}.remote_size declaration
* don't make {audio}.fid NOT NULL until it's done being populated
* don't attempt to UPDATE {audio}.fid if nothing is inserted into {files}
Comment #2
HorsePunchKid CreditAttribution: HorsePunchKid commentedUnrelated, but it would also be nice if you did:
chmod a-x README.txt audio.*
Comment #3
HorsePunchKid CreditAttribution: HorsePunchKid commentedHere is a revised patch taking #473860: audio install >= update 5202 tries to create multiple remote_size records under consideration.
Comment #4
Liam Morland