First of all, thank You for this great module!

I would like to notice two minor things about Classified Ads:
-there is a bug in ads count, e.g. if I use taxonomy hierarchy like : Sell->Toys and I put 1 ad in this category the Sell (parent) category count will be 2 http://nunu.hu
-after I activated the ed_classified module the choosen Administration Theme is not working, all admin pages show up with the site Default theme. Have You some idea of this problems?

Thanks: Balazs

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Version: 5.x-1.5-8 » 6.x-3.x-dev

Problem 1 is fixed in 6.x-3.x.

Problem 2 is still there even in 6.x-3.x : with [ed_]classified enabled, the admin theme, and possibly other variables, are not applied until a cache clear. The admin/settings/admin page still shows the previous value of the admin_theme in the theme selection widget. Clearing cache shows the proper value.

Problem 2 does not exist on 7.x-3.x.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Title: Ads Count, Administration Theme » system_settings_form_submit() fails to save variables entry in cache (triggered by Classified Ads)
Project: Classified Ads » Cache Router
Version: 6.x-3.x-dev » 6.x-1.x-dev

More detail: with 6.x-3.x, this only happens when using cacherouter, and specifically using it in "file" mode. The problem is identical with all settings forms; not just admin theme.

With cacherouter on APC, or with core caching, problem does not happen. Accordingly, moving issue to cacherouter. Problem does not happen in the same configuration when Classified Ads is not enabled, though.

In more detail: with that specific configuration, the form triggers system_settings_form_submit, in which admin_theme is properly stored in the variables table, but when invoking variable_init as a shutdown handler, a file creation error occurs in cacherouter "file.inc" on variables-87cd8b8808600624d8c590cfc2e6e94b.perm, although that path exists and is writable, and is visible in watchdog.