Order Admin page does not pass $order context to uc_price [#621134]

I've got an ubercart site which has a different currency sign for each order - I handle this by storing the order's currency sign in the data serialized field of uc_order, and pulling it out again in a price handler I assigned via hook_uc_price_handler.

However, inside of the order page at admin/store/order, which is created via uc_order_admin in uc_order.admin.inc, the $context that is passed to uc_price doesn't contain the $order object, even though you do have some order information in that function. I'm going to reply to this with a patch that does include that information.

Comment	File	Size	Author
#2	ubercart-621134.patch	634 bytes	jcfiala
#1	ubercart-621134.patch	607 bytes	jcfiala

Comments

Comment #1

jcfiala commented 2 November 2009 at 17:07

Status:

Active

» Needs review

Status	File	Size
new	ubercart-621134.patch	607 bytes

So, here's the patch that I came up with. it's really just doing one thing - adding the existing $order to the existing $context.

Now, it's not a complete order object, so anyone working with a price alterer will have to pick up on that and fetch the data they *do* need, but it at least gives the order_id, which is, I think, necessary.

Thank you.