_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

Comments

tr’s picture

tr commented
Status: Active » Needs review
StatusFileSize
new stock-D6.patch735 bytes
new 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 commented
Status: Needs review » Reviewed & tested by the community

works fine

bwynants’s picture

bwynants commented

tested and works

tr’s picture

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.