audio install >= update 5202 tries to create multiple remote_size records

gnassar - May 27, 2009 - 04:55
Project:Audio
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

Update 5202 tries to create the 'remote_size' field in the 'audio' module twice -- the first time as a varchar, the second as an int. This gives a warning on install/upgrade. Easy fix, except for the part of knowing which it *should* be, which I have no idea so I'll leave the patch to someone else.

#1

drewish - May 27, 2009 - 16:14

ah, yeah. the varchar should be named 'remote_path' but since they're not used it might make more sense to just drop both of them.

#2

tbartels - September 8, 2009 - 18:53

Patches for both conditions attached, someone just needs to decide which path is correct, I guess.

AttachmentSize
alter remote_size ('type' => 'varchar') to remote_path 366 bytes
remove both remote_size field definitions 795 bytes
 
 

Drupal is a registered trademark of Dries Buytaert.