Setup canonical by default not like google wants it?

[Meton]

Hi Guys,

If you look at this page you can see that as it is the current canonical setup in virtuemart appears to be "not how google wants it". Any thoughts on this?

https://support.google.com/webmasters/answer/139066?hl=en

Referring to this part:
-----------------------------------------------
Avoid errors: use absolute paths rather than relative paths with the rel="canonical" link element.

Use this structure:  https://www.example.com/dresses/green/greendresss.html
Not this structure: /dresses/green/greendress.html).
-----------------------------------------------


Kind regards,

Meton

[PRO]

I thought this was fixed.

The solutions is here
http://forum.virtuemart.net/index.php?topic=129421.0

[Meton]

Thx a mil pro!

[Meton]

I thought this was fixed.

The solutions is here
http://forum.virtuemart.net/index.php?topic=129421.0

Hi pro,

This worked in categoryview by using:

Code Select Expand


$document->addHeadLink( JRoute::_($link, true, -1) , 'canonical', 'rel', '' );


But in my productdetails page there is a totally different structure that i don't understand enough to dive in to this:

Code Select Expand


if($isCustomVariant !==false and !empty($isCustomVariant->usecanonical) and !empty($product->product_parent_id)){
$parent = $product_model ->getProduct($product->product_parent_id);
$document->addHeadLink($parent->canonical, 'canonical', 'rel', '');
} else {
$document->addHeadLink($product->canonical, 'canonical', 'rel', '');
}

Do you know how this is done?

Kind regards,

Meton

[PRO]

// Set Canonic link
            if($isCustomVariant !==false and !empty($isCustomVariant->usecanonical) and !empty($product->product_parent_id)){
               $parent = $product_model ->getProduct($product->product_parent_id);
               $document->addHeadLink(JRoute::_($parent->canonical, true, -1), 'canonical', 'rel', '');
            } else {
               $document->addHeadLink(JRoute::_($product->canonical, true, -1), 'canonical', 'rel', '');
            }

[Meton]

// Set Canonic link
            if($isCustomVariant !==false and !empty($isCustomVariant->usecanonical) and !empty($product->product_parent_id)){
               $parent = $product_model ->getProduct($product->product_parent_id);
               $document->addHeadLink(JRoute::_($parent->canonical, true, -1), 'canonical', 'rel', '');
            } else {
               $document->addHeadLink(JRoute::_($product->canonical, true, -1), 'canonical', 'rel', '');
            }

Ill give this a go when i get back home, thanks Pro!

[Meton]

Worked like a charm! I have gone live! Cheers m8

[alatak]

Hello

I have fixed it, but a different way.
I did

Code Select Expand


    $document->addHeadLink(JUri::getInstance()->toString(array('scheme', 'host', 'port')).JRoute::_($product->canonical, FALSE), 'canonical', 'rel', '');


because the JRoute($link, false, -1)

-1 is for $ssl parameter.
If it is $ssl=-1 then the canonical URL will be http://dresses/green/greendress.html even if your website is in SSL (set in joomla)

[lindapowers]

Hello

I have fixed it, but a different way.
I did

Code Select Expand


    $document->addHeadLink(JUri::getInstance()->toString(array('scheme', 'host', 'port')).JRoute::_($product->canonical, FALSE), 'canonical', 'rel', '');


because the JRoute($link, false, -1)

-1 is for $ssl parameter.
If it is $ssl=-1 then the canonical URL will be http://dresses/green/greendress.html even if your website is in SSL (set in joomla)

Can you please tell me file to make the modification? Is this in the router file? location?

Regards

[GJC Web Design]

If this is a product details page I assume she means here

// Set Canonic link
            if($isCustomVariant !==false and !empty($isCustomVariant->usecanonical) and !empty($product->product_parent_id)){
               $parent = $product_model ->getProduct($product->product_parent_id);
               $document->addHeadLink($parent->canonical, 'canonical', 'rel', '');
            } else {
               $document->addHeadLink($product->canonical, 'canonical', 'rel', '');
            }

components\com_virtuemart\views\productdetails\view.html.php ~ line 219

[lindapowers]

If this is a product details page I assume she means here

// Set Canonic link
            if($isCustomVariant !==false and !empty($isCustomVariant->usecanonical) and !empty($product->product_parent_id)){
               $parent = $product_model ->getProduct($product->product_parent_id);
               $document->addHeadLink($parent->canonical, 'canonical', 'rel', '');
            } else {
               $document->addHeadLink($product->canonical, 'canonical', 'rel', '');
            }

components\com_virtuemart\views\productdetails\view.html.php ~ line 219

Thanks man!

[Milbo]

Latest SVN has already:

Code Select Expand


// Set Canonic link
if($isCustomVariant !==false and !empty($isCustomVariant->usecanonical) and !empty($product->product_parent_id)){
$parent = $product_model ->getProduct($product->product_parent_id);
$document->addHeadLink( JUri::getInstance()->toString(array('scheme', 'host', 'port')).JRoute::_($parent->canonical), 'canonical', 'rel', '');
} else {
$document->addHeadLink( JUri::getInstance()->toString(array('scheme', 'host', 'port')).JRoute::_($product->canonical), 'canonical', 'rel', '');
}