Implementing hreflang in a multilanguage store

[DrByte]

After reading a number of sources, it seems best to skip if only 1 language, or display for all languages if more than 1.

So, proposing this: https://github.com/zencart/zencart/pull/1314

[enzo-ita]

After reading a number of sources, it seems best to skip if only 1 language, or display for all languages if more than 1.

So, proposing this: https://github.com/zencart/zencart/pull/1314

Hi,
I have implemented the mod proposed so the html_header section becomes

// BOF hreflang for multilingual sites

if (isset($language_list) || $language_list = $lng->get_available_languages() && sizeof($language_list) > 1) {
     foreach($language_list as $key=>$value) {
echo '    <link rel="alternate" href="' . ($this_is_home_page ? zen_href_link(FILENAME_DEFAULT, 'language=' . $value['code'], $request_type) : $canonicalLink . (strpos($canonicalLink, '?') ? '&amp;' : '?') . 'language=' . $value['code']) . '" hreflang="' . $value['code'] . '" />' . "\n";
     }
   }
   // EOF hreflang for multilingual sites 

But I get a blank page and the following error log


PHP Fatal error: Call to a member function get_available_languages() on a non-object in /home2/xxxxx/public_html/includes/templates/responsive_classic/common/html_header.php on line 71

I have noticed that in github the file lines to modified start with 48 while in my file the lines were

// BOF hreflang for multilingual sites 

is starts at 73.

I would like to understand this.
Thanks.

Happy New Year from Italy

enzo

[DrByte]

PHP Fatal error: Call to a member function get_available_languages() on a non-object in /home2/xxxxx/public_html/includes/templates/responsive_classic/common/html_header.php on line 71

I have noticed that in github the file lines to modified start with 48 while in my file the lines were

// BOF hreflang for multilingual sites 

is starts at 73.

I would like to understand this.

That's because the change posted is for v1.6.0.

For your v1.5 site, do what torvista posted above to get it to output the tags for all languages. And since you're supporting multiple languages already you won't need the other part of the change I mentioned in v160.

[enzo-ita]

That's because the change posted is for v1.6.0.

For your v1.5 site, do what torvista posted above to get it to output the tags for all languages. And since you're supporting multiple languages already you won't need the other part of the change I mentioned in v160.

Thanks. I di it and it works. For my other site that is mono language I do not need to touch the html_header.php right?
Ciao ciao

[bramf]

Is the code in html_header already corrected in 1.55e (I copied the code from zen-cart 1.55e) or should I still apply the fix described in this forum ( I have 3 languages installed)?