Hi, I found that your module is quite cool, but the language filter is not working with your module. If you click the link below:-
http://indiecom.net/archive

You will see two nodes in different languages appear in the Archives page. This is not right since I'm using English mode in my website. The right behavior is the Archives page should only shows the Enlgish nodes whenever I'm using English mode in my website.

I hope you understand.

www.indiecom.net
Skype:1984

Comments

gábor hojtsy’s picture

Project: Archive » Localizer

I don't think this is a problem in the archive module. We do wrap our calls in the standard query rewrite calls supported by Drupal. What else should be there to support localizer? I don't think it is our problem, recategorizing.

sun’s picture

Title: Localizer Not Supported » Archived nodes displayed in all languages
Category: bug » support

Is multilingual content support enabled? If archive module is really using db_rewrite_sql() then this should be the only reason why contents are displayed in more than one language.

Roberto Gerola’s picture

i have to take a look at the sql used by this module.
it could be that localizer doesn't consider correctly it.

Roberto Gerola’s picture

i have to take a look at the sql used by this module.
it could be that localizer doesn't consider correctly it.

Roberto Gerola’s picture

Exactly what version are you using now of localizer ?

1.8 introduces optional support for content type.
Can you check it under localizer settings ?

Roberto Gerola’s picture

I've taken a look at archive module.
Perhaps there is a little problem with archive_page function.

Can you change the line # 64 in this way :

$result = pager_query(db_rewrite_sql(array_shift($query)), $nodes, 0, NULL, $query);

and let me know if this solves the problem ?

It seems to me that a call to db_rewrite_sql is missing.

gábor hojtsy’s picture

Project: Localizer » Archive
Status: Active » Fixed

Oh, excuse me, it was my fault. The Drupal 5.x-dev version of the module did have some fixes, including this "node permission related" fix. These were not released in an 5.x-1.1 bugfix release. Now done the release, the new tarball will be available later today when the release script runs. Excuse me.

hanief84’s picture

Version: 5.x-1.0 » 5.x-1.2
Status: Fixed » Closed (fixed)

Cool! It has been fix. FYI, I 'm using your Archive module with the 5.x-1.7 Localizer Module

Way to go! Thank You So Much!

http://indiecom.net/archive (Test it here!)