[SOLVED] All friendly URLs work except Manufacturers

[DaveBr]

Hopefully someone can assist with this.

 

Friendly urls are turned on and all appear to work as they should, except for that for the Manufacturer which returns a 404 error.

 

Also, this is specific to the Manufacturer link on product pages. Links from a scroller which appears on all pages work ok.

 

On product pages when the link is hovered over the url contains no data after the basic shop url and looks like this.

http://domainname.co.uk/shop/en/_

In SEO & URLs, the route to manufacturer is setup as {id}_{rewrite}

 

Presumably there's a simple reason it's failing to pickup the ID & rewrite it, but I'm struggling to work out why, when all the others are OK.

 

htaccess has been regenerated, caches cleared etc.

 

Edit: This is v1.5.4.0

Edited January 2, 2014 by vekia (see edit history)

[Sharak]

URL of your shop would be helpful. What did you change in manufacturer page?

[vekia]

 

 

In SEO & URLs, the route to manufacturer is setup as {id}_{rewrite}

have you got any other similar pattern to this {id}_{rewrite} ?

[DaveBr]

Hi. Yes, both Route to Category & Route to Supplier both also use {id}_{rewrite} and seem to be working normally.

 

Sharak, I sent a message with the URL. Thanks.

[vekia]

they haven't got the same pattern

 

{id}_{rewrite} http://robertsonsofpitlochry.co.uk/shop/en/7_an-cnoc-knockdhu

{id}-{rewrite} http://robertsonsofpitlochry.co.uk/shop/en/7-whisky

 

it works well

[DaveBr]

That's strange. Can you try here?

http://robertsonsofpitlochry.co.uk/shop/en/whisky/10-aberlour-12-year-old.html

 

Where it says

 

Availability:

Distiller/Brewer Aberlour

these links on product pages give me a 404 error.

[vekia]

ohh now i understand the matter of problem.

you've got problem with link, not with pattern.

 

can you show contents of your product.tpl file? especially part where manufacturer link appears

[DaveBr]

I'm not sure how to a attach a file, but here's the only lines I can see in the product.tpl which relates to manufactures.

 

{if isset($product->manufacturer_name) && $product->manufacturer_name != ""}

<p class="product-manufacturer">

<span class="value_name">{l s='Manufacturer:'}</span> <a href="{$link->getManufacturerLink($product->manufacturer_name)}">{$product->manufacturer_name}</a>

</p>

{/if}

[vekia]

it's because you have to use manufacturer id instead of manufacturer name

 

try with this, instead of:

{$link->getManufacturerLink($product->manufacturer_name)}

 

use

{$link->getManufacturerLink($product->id_manufacturer)}

[Sharak]

Code shows you don't have rewrite implemented in this link. Check manufacturer-list.tpl, links there look like this:

<a href="{$link->getmanufacturerLink($manufacturer.id_manufacturer, $manufacturer.link_rewrite)|escape:'htmlall':'UTF-8'}">

[DaveBr]

it's because you have to use manufacturer id instead of manufacturer name

 

try with this, instead of:

{$link->getManufacturerLink($product->manufacturer_name)}

 

use

{$link->getManufacturerLink($product->id_manufacturer)}

Should I replace all instances of manufacturer_name with id_manufacturer in all lines? Or just that section you've given?

[vekia]

Should I replace all instances of manufacturer_name with id_manufacturer in all lines? Or just that section you've given?

only in href="" param

 

href="{$link->getManufacturerLink($product->manufacturer_name)}"

[DaveBr]

Fantastic. That's it sorted.

 

Many thanks

 

Thanks Sharak too.

[vekia]

glad to hear that Sharak & I could help you a little

i marked whole topic as [solved]

 

with regards,

Milos