1.x is still stuck using `key` as the field name in uc_file_users, which is a source of problems. Patch attached ports the change.

CommentFileSizeAuthor
#3 uc_file.key_change.1.x.patch7.15 KBcha0s
#1 uc_file.1.x.patch7.05 KBcha0s
uc_file.1.x.patch7.05 KBcha0s
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cha0s’s picture

cha0s CreditAttribution: cha0s commented
FileSize
uc_file.1.x.patch7.05 KB

Well, might be useful to update the name of the function...

Island Usurper’s picture

Island Usurper CreditAttribution: Island Usurper commented
Status: Needs review » Needs work

Use db_change_column() for the Postgres branch of the update function.

cha0s’s picture

cha0s CreditAttribution: cha0s commented
Status: Needs work » Needs review
FileSize
uc_file.key_change.1.x.patch7.15 KB

Rerolled with pgsql support.

Island Usurper’s picture

Island Usurper CreditAttribution: Island Usurper commented
Status: Needs review » Fixed

Great. Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Passer’s picture

Passer CreditAttribution: Passer commented
Passer’s picture

Passer CreditAttribution: Passer commented

thanks

torgosPizza’s picture

torgosPizza
he/him
Primary language English
Location Portland, OR
CreditAttribution: torgosPizza commented

Any chance of rolling this into a 1.8? :)