_uc_stock_send_mail calls $account = user_load($order); this call will fail!

creating some entries in the log...

Recoverable fatal error: Object of class stdClass could not be converted to string in DatabaseStatementBase->execute
Warning: array_flip() [function.array-flip]: Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->load() (

the $account is not even used in the function so i think the call should just be removed...

CommentFileSizeAuthor
#1 stock.patch691 bytesTR
#1 stock-D6.patch735 bytesTR
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR’s picture

TR CreditAttribution: TR commented
Status: Active » Needs review
FileSize
stock-D6.patch735 bytes
stock.patch691 bytes

I kept the $account in so it can be used for user tokens. Patch attached for D7 and for D6, which has a similar problem.

bwynants’s picture

bwynants CreditAttribution: bwynants commented
Status: Needs review » Reviewed & tested by the community

works fine

bwynants’s picture

bwynants CreditAttribution: bwynants commented

tested and works

TR’s picture

TR CreditAttribution: TR commented
Status: Reviewed & tested by the community » Fixed

Committed to 6.x-2.x and 7.x-3.x.

Status: Fixed » Closed (fixed)

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