Jump to content

Multilanguage bug.


erwu

Recommended Posts

Hello I'm using 1.2.4 with "polish" language as a default. Other languages are disabled. I found that Google boot doesn't seems to support cookies ( and language settings ). I get all languages at the same time, that's why I get warnings.


       /* Automatically detect language if not already defined */
       if (!$cookie->id_lang AND isset($_SERVER['HTTP_ACCEPT_LANGUAGE']))
       {
           // FIX
           $lang = new Language(3);
           if (!Validate::isLoadedObject($lang) OR !$lang->active)
               $cookie->id_lang = NULL;
           /*


           $array = explode(',', Tools::strtolower($_SERVER['HTTP_ACCEPT_LANGUAGE']));
           if (Validate::isLanguageIsoCode($array[0]))
           {
               $lang = new Language(intval(Language::getIdByIso(strval($array[0]))));
               if (Validate::isLoadedObject($lang) AND $lang->active)
                   $cookie->id_lang = intval($lang->id);
           }*/
       }



and

Warning: htmlentities() expects parameter 1 to be string, array given in /home/miwa/domains/copymedia.pl/public_html/classes/Tools.php on line 582

Warning: htmlentities() expects parameter 1 to be string, array given in /home/miwa/domains/copymedia.pl/public_html/classes/Tools.php on line 575

Warning: htmlentities() expects parameter 1 to be string, array given in /home/miwa/domains/copymedia.pl/public_html/classes/Tools.php on line 575



this is how it crashes website for google BOT.
http://webcache.googleusercontent.com/search?q=cache:rn8IKvJ2HzgJ:www.copymedia.pl/6873-KM-1635+site:copymedia.pl&cd=64&hl=pl&ct=clnk&gl=pl

http://webcache.googleusercontent.com/search?q=cache:5Va0Mb1wJQcJ:www.copymedia.pl/317-a0fp021-toner-czarny-oryginalny.html+site:http://www.copymedia.pl&cd=409&hl=pl&ct=clnk&gl=pl





I know that my skin is modified, but You should without physical cookie's param value: $cookie->lang_id for DEFAULT

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...