4 replies [Last post]
zmove's picture
Offline
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer
Joined: 08/13/2007
Juice: 1192
Was this information Helpful?

Hi there,

When I send email notification to the customer on order status update, the icon under the "notification" column is a red cross whereas it should be a green tick.

The notification is successfully sended, I just tested, but the icon in admin panel is wrong.

Is the bug known, reproductible on the livetest or it's just me ? if just me, where I should search the origin of the problem ?

thanks

zmove

WesleyTx's picture
Offline
Bug Finder
Joined: 09/14/2007
Juice: 89
Re: Notification icon problem.

I just noticed the same thing, myself. Notification's sent, but the icon is still a red X.

The initial notification, however, when the order was placed, is marked with a green check.

zmove's picture
Offline
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer
Joined: 08/13/2007
Juice: 1192
Re: Re: Notification icon problem.

Yes, it's exactly the same thing, the originale notification made by the customer is a green tick.

Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: Re: Re: Notification icon problem.

Should be a simple fix... probably just a db column name was changed. I've put this on my bug list and will try to address it today. Smiling

EDIT: Fixed in Bazaar. Had to change line 647 of uc_order_order_pane.inc to the following:

<?php
  uc_order_comment_save
($form_values['order_id'], $user->uid, '-', 'order', $form_values['status'], $form_values['notify']);
?>

Notice I forgot to pass on the notify setting for the case of status updates without any message entered. Shocked

zmove's picture
Offline
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer
Joined: 08/13/2007
Juice: 1192
Re: Re: Re: Re: Notification icon problem.

It seems to work. Thank you Ryan !