Order status change

hugojacobs
Topic Author
Offline

13 years 8 months ago #83645 by hugojacobs

Order status change was created by hugojacobs

Hello,

I have installed Hikamarket. everything is working fine, however I can not change the orderstatus in the frontend.

When I change the status in the screen and save it, it does not actually change the status and it returns with the original status.

Other fields (like the address in the invoice screen) I can change.

Thanks for your help

Hugo

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #83654 by Jerome

Replied by Jerome on topic Order status change

Hi,

Right, there is a missing field in the view "show_general".
Please edit the file and replace the last lines:

Code:

if(isset($this->edit) && $this->edit === true ) {
	echo JHTML::_('form.token')."\r\n";
}

By:

Code:

if(isset($this->edit) && $this->edit === true ) {
	echo '<input type="hidden" name="data[general]" value="1"/>';
	echo JHTML::_('form.token')."\r\n";
}

It would fix the problem. This fix would be include in the next package.

Regards and thanks for the report.

Jerome - Obsidev.com
HikaMarket & HikaSerial developer / HikaShop core dev team.

Also helping the HikaShop support team when having some time or couldn't sleep.
By the way, do not send me private message, use the "contact us" form instead.

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #83659 by hugojacobs

Replied by hugojacobs on topic Order status change

Hi Jerome,

Thanks a lot for your quick reply. This is working fine now.

I have noticed some other issues in the same functionality:
- the emailnotification to the customer does not seem to be working. I do not receive the status change mail.
- The order status in the list and the orders is displayed in English. My front-end language is Dutch
- sorting the orders in the list is not working
- Filtering of the orders in the list is not working. It would be nice if you could filter on order status.

Have a good evening!

Hugo

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #83666 by Jerome

Replied by Jerome on topic Order status change

Hi,

Notification should be in the first release of HikaMarket. It was something that I forgot to extract from the package.
The notification would be finish/fix in the next release like the sorting/filtering for products and orders.

Regards,

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #83728 by hugojacobs

Replied by hugojacobs on topic Order status change

Hi Jerome,

Thanks. When will this new release be available?

Hugo

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #83732 by Jerome

Replied by Jerome on topic Order status change

Hi Hugo,

I have update the component without modifying the version number.
You can re-download HikaMarket it would update your website with the last improvements.

Regards,

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #84005 by hugojacobs

Replied by hugojacobs on topic Order status change

Hi Jerome,

Thanks a lot.

2 issues mentioned earlier have not been resolved yet:
- sorting the orders in the list is not working - this is still the case
- Filtering of the orders in the list is not working. It would be nice if you could filter on order status. - Filtering on order status works if you use the status description in English.

I think it would be usefull to add a filtering option on the order status with a drill-down menu.

Regards
Hugo

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #84022 by Jerome

Replied by Jerome on topic Order status change

Hi,

I have installed the latest version on the demo website and I can change the sorting in the orders listing.
I tried it with Firefox and Google Chrome. I think it should be the same with Internet Explorer, Opera, etc.

Filtering on "translated order status" is not possible so yes, we would move it to a drop down like the backend order listing.

Regards,

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #84053 by hugojacobs

Replied by hugojacobs on topic Order status change

Hi,

I am not sure what is going wrong - I have installed the latest version (4/1/2013).

The sorting on the different fields does not work.
I tried with IE9, firefox 17.01 and IE8.
Also I set the front-end language to EN.

Hugo

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #84121 by Jerome

Replied by Jerome on topic Order status change

Hi,

I am working on a new package but I have to finish the new feature implementation before.

Are you sure that you have redownload the package ?
I already see one time someone who couldn't redownload new package because his browser cached the file.

Regards,

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #84435 by hugojacobs

Replied by hugojacobs on topic Order status change

Hi Jerome,

I am not lucky on this one..

I have tried this on 3 computers with 3 browsers (IE 8/9, FF) and the sorting doesn't work.
Also I cleared the cache an to be sure I have installed the latest version (from 7/1).

Anyway, I can probably wait for the release where filtering on the project status is included (similar to the backend order screen).

When do you expect this will be available?

Thanks
Hugo

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #84439 by hugojacobs

Replied by hugojacobs on topic Order status change

I looked a bit further and with another user the sorting worked.
I re-created the first user and now it is working for this user as well.

Anyway.

Still the filtering option is the better solotion :-)

Hugo

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #84440 by Jerome

Replied by Jerome on topic Order status change

Hi,

If you want, I can send you a preview package of HikaMarket by email.

Regards,

Please Log in or Create an account to join the conversation.

hugojacobs
Topic Author
Offline

13 years 8 months ago #84442 by hugojacobs

Replied by hugojacobs on topic Order status change

Hi Jerome,

That would be great!
(I suppose you can see my e-mail address in my profile - if not I will send it seperately)

Hugo

Please Log in or Create an account to join the conversation.

gabriela
Offline

13 years 8 months ago - 13 years 8 months ago #85533 by gabriela

Replied by gabriela on topic Order status change

Hi,

I notice that when the vendor changes the order status and selects the checkbox to notify the customer, the notification is not being sent.

I checked on my show_general.php and I do see the last lines mentioned above:

Code:

<?php
if(isset($this->edit) && $this->edit === true ) {
?>
	<input type="hidden" name="data[general]" value="1"/>
	<?php echo JHTML::_('form.token')."\r\n";
}

What could be going on with the notification e-mail?

Also, I need to limit the vendors so that they are only able to change the order status to "Shipped". Any guess on how can I do this?

Regards,

Last edit: 13 years 8 months ago by gabriela. Reason: marked code

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago - 13 years 8 months ago #85537 by Jerome

Replied by Jerome on topic Order status change

Hi,

Only the main vendor could send a notification.
Regarding the current view, the limitation is not set

Code:

if(isset($this->edit) && $this->edit === true && hikamarket::acl('order_edit_notify')) {

Should be

Code:

if(isset($this->edit) && $this->edit === true && $this->vendor->vendor_id == 0 && hikamarket::acl('order_edit_notify')) {

A vendor could just update a partial order (a sub order) and, because it is not completely defined, I preferred to forbidden the mail notification for the vendors.
That's why you would find in the order class

Code:

if($vendor_id == 0 && !empty($data['notify']) && hikamarket::acl('order_edit_notify')) {

I guess that we can put an option but I can't yet guarantee that the result would be directly the right one.

Regards,

Last edit: 13 years 8 months ago by Jerome.

Please Log in or Create an account to join the conversation.

solia
Offline

13 years 8 months ago #87172 by solia

Replied by solia on topic Order status change - Notification email

Dear Jerome: Thank you for your work, as it seems that order status can be changed and sorting looks good. Like others, I would like to ensure that the notification email is sent if selected.

I will serve as administrator for HikaShop, and have my client be the only other vendor for front end access using HikaMarket. I see where the vendor number is shown next to the vendor in the configuration.

I see also your suggested code change, to add a vendor ID to permit the notification. However, I am not sure which file I must modify. Can you please clarify which php file I must make the change to?

Thank you very much.

Nadine at Solia.

Please Log in or Create an account to join the conversation.

Jerome
Offline
HikaShop core developer

13 years 8 months ago #87191 by Jerome

Replied by Jerome on topic Order status change

Hi,

For the first part of the code, where I suggest to add the check on the vendor_id, the view is "ordermarket | show_general".
You can override the view or you can modify the file directly (components/com_hikamarket/view/ordermarket/views/show_general.php)

I am currently working on HikaMarket 1.1.0 which would be compatible with HikaShop 2.1.0
The release should be publish few days after the validation of the next HikaShop package (when HikaShop would be accessible thanks to auto-update).

In the new release, I have made some improvements in the notification but some tests are still required.

Regards,

The following user(s) said Thank You: solia

Please Log in or Create an account to join the conversation.

Moderators: Obsidev

Time to create page: 0.296 seconds