Order file uploads not saved

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #131947

Actually, I have just looked at the link in the email between the old ones that worked, and the new ones that don't. On the new ones, the word "administrator" is missing from the link before /index.php. I'll look into that and see if I can add it into the php file.
Would that work do you think?

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

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132107

ok, that didn't work. Here is the difference between links from a old order that worked and a new one that doesn't.

WORKING LINK:
www.quickcheapprint.co.nz/administrator/...bV81OTkzODk3NTEucG5n

NOT WORKING LINK THAT WE GET NOW:
www.quickcheapprint.co.nz/index.php/pape...X18zMDU2ODMyNzEucGRm

Does that help you?

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

  • Posts: 26146
  • Thank you received: 4026
  • MODERATOR
10 years 11 months ago #132148

Hi,

Can you just precise the email please.
It might be a problem with the new template.

Regards,

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.

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132265

Hi, I'm sorry, what do you mean "precise the email" ?? I have attached a PDF of the email if that is what you meant?

Attachments:

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

  • Posts: 26146
  • Thank you received: 4026
  • MODERATOR
10 years 11 months ago #132291

Hi,

I can't open your PDF (I just got some ads for adobe).
Please precise the context of your email... You know, there are several emails send by HikaShop, I just want to know : which email you are talking about :)

Regards,

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.

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132296

Sorry about that, seems when I saved the email as a PDF it did some wierd format.

The email is the notification email sent to the administrator to say there has been an order created on our website. It has a summary of the order and a link to the file the user has uploaded as part of their order.

(we are a printing business so when a user makes a order, they upload their print file as part of the order).

We then try to click on the link to get the file - this is where the problem is.

I have attached a new PDF of the email, this one will work :)

Attachments:

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

  • Posts: 26146
  • Thank you received: 4026
  • MODERATOR
10 years 11 months ago #132328

Thanks, it is much better :)

So the email is : order_admin_notification
The fact is that, the link to the product download weren't the in the previous emails.
When I made the new template, I made some copy/paste between several emails and the administrator email shouldn't have it.

In the file "media/com_hikashop/mail/order_admin_notification.preload.php", you will find this line:

$t .= '<a href="'.hikashop_frontendLink('index.php?option=com_hikashop&ctrl=order&task=download&file_id='.$file->file_id.'&order_id='.$data->order_id.$file_pos.$url_itemid).'">'.$fileName.'</a><br/>';
You can replace it by:
$t .= '<a href="'.JRoute::_('administrator/index.php?option=com_hikashop&ctrl=order&task=download&file_id='.$file->file_id.'&order_id='.$data->order_id.$file_pos).'">'.$fileName.'</a><br/>';
and it should work.

Regards,

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.

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132353

Hi Jerome,
Thanks for your help with this. After looking at the code I can totally understand mistakes like that are easy to make! You all do a fantastic job creating this component, I envy your code skills.

Anyway, I'll try that alteration in the morning when I get in to work and let you know the outcome.

Thanks again.
Alan

The following user(s) said Thank You: Mohamed Thelji

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

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132488

Unfortunately it didn't make any difference, the link is still the same as mentioned previously. Its like its not even looking at that code...

I have attached the screenshots so you can confirm the code is correctly inserted. Also showing the email link.

Attachments:
Last edit: 10 years 11 months ago by ahcopyart. Reason: added screenshots

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

  • Posts: 26146
  • Thank you received: 4026
  • MODERATOR
10 years 11 months ago #132715

Hi,

Nicolas made a fix last week and I though it was early (so, include in the latest package).
Please change the line

$t .= '<p>'.$fieldsClass->getFieldName($field).': '.$fieldsClass->show($field,$item->$namekey).'</p>';
By:
$t .= '<p>'.$fieldsClass->getFieldName($field).': '.$fieldsClass->show($field,$item->$namekey,'admin_email').'</p>';
It will fix the way that the custom item fields are displayed.
My fix was for the product files (not the item custom fields).

Regards,

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.

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132756

Hi Jerome,

So far I have made a code edit in that file like Mohamed asked me to, then I have done your code edit, now I need to do another code edit, I think we need to clarify should the php file have all 3 code edits or just this latest one suggested by you?

Last edit: 10 years 11 months ago by ahcopyart. Reason: clarification

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

  • Posts: 26146
  • Thank you received: 4026
  • MODERATOR
10 years 11 months ago #132758

Hi,

You can read my previous post. It explain the modifications.

Regards,

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.

  • Posts: 118
  • Thank you received: 3
10 years 11 months ago #132809

I didn't quite understand your previous post which is why I was asking. Anyway, I have guessed and kept all the code modifications in this thread and just tested it. IT WORKS :)
Thanks for your help and patience again.

Alan

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

Time to create page: 0.051 seconds
Powered by Kunena Forum