Multistore email template URLs

[completepresta]

I had a bug in classes/Mail.php with multistore.

When i select 'All Stores' and change the status the e-mail urls always use the url of main store.
To fix this around line 170:
 

Change:

$template_vars['{shop_url}'] = Context::getContext()->link->getPageLink('index', true, Context::getContext()->language->id);

$template_vars['{my_account_url}'] = Context::getContext()->link->getPageLink('my-account', true, Context::getContext()->language->id);

$template_vars['{guest_tracking_url}'] = Context::getContext()->link->getPageLink('guest-tracking', true, Context::getContext()->language->id);

$template_vars['{history_url}'] = Context::getContext()->link->getPageLink('history', true, Context::getContext()->language->id);

 

To:

$template_vars['{shop_url}'] = Context::getContext()->link->getPageLink('index', true, Context::getContext()->language->id, null, false, (int)$id_shop);

$template_vars['{my_account_url}'] = Context::getContext()->link->getPageLink('my-account', true, Context::getContext()->language->id, null, false, (int)$id_shop);

$template_vars['{guest_tracking_url}'] = Context::getContext()->link->getPageLink('guest-tracking', true, Context::getContext()->language->id, null, false, (int)$id_shop);

$template_vars['{history_url}'] = Context::getContext()->link->getPageLink('history', true, Context::getContext()->language->id, null, false, (int)$id_shop);

 

[cedricfontaine]

Did you report it on Git Hub ?

[completepresta]

Did you report it on Git Hub ?

 

No not yet, didn't know to do that

[beyonceuk1]

When I do this, the shop URL in the mails is appearing only as "http" - any idea how I can have the shop URL used?

[beyonceuk1]

When I do this, the shop URL in the mails is appearing only as "http" - any idea how I can have the shop URL used?

 

I didn't ever receive a reply to this. Anyone had the same problem?