page 404 not show with i18n (Drupal 4.7-cvs)

show:

=========
Primary menu links
2
=========

all patches in menu - applied

Comments

kiev1.org’s picture

kiev1.org commented

not work
menu_set_active_item($path);
$return = menu_execute_active_handler();
I replaced it on drupal_goto($path) and works - but it is not correct, it is necessary to do right

jose reyero’s picture

jose reyero commented
Status: Active » Fixed

I don't know what caused this, but I think this is already fixed, please re-open if not

profix898’s picture

profix898 commented
Status: Fixed » Active

I have the same issue on a clean 4.7 install. Enter a nonexistent url, e.g. domain.com/?q=testurl (or domain.com/testurl with clean urls enabled), and you will get a 404 error page. Now enable i18n and try again and you will get the primary links page (as described above). This behaviour is reproducable on my install (cvs, i18n.module 1.25). We need a closer look on that.

profix898’s picture

profix898 commented

I realised that sometimes path parameter is given as Object (instead of string). The user is presented the primary links screen then and the url looks like that domain.com/en/Object. Error messages raised in that case are ...

    * warning: Illegal offset type in /www/htdocs/.../includes/path.inc on line 54.
    * warning: mysql_real_escape_string() expects parameter 1 to be string, object given in /www/htdocs/.../includes/database.mysql.inc on line 349.
    * warning: Illegal offset type in /www/htdocs/.../includes/path.inc on line 62.
jose reyero’s picture

jose reyero commented
Status: Active » Fixed

Yes, you were right. There was something wrong with i18n_url_rewrite which also caused some other troubles.

Thanks for the thourough testing.

jose reyero’s picture

jose reyero commented
Status: Fixed » Closed (fixed)