Closed (won't fix)
Project:
Donations Thermometer
Version:
6.x-1.1
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Jun 2010 at 17:58 UTC
Updated:
24 Mar 2016 at 08:03 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dboulet commentedI can't recreate this bug. Are you sure that you haven't changed any of the block visibility settings?
Comment #2
mennonot commentedI had this problem too: the thermometer graphic didn't show up for anonymous users. I had to turn on "administer progress bar" for anonymous users to get it to work.
I wander if caching has something to do with it. I have normal caching turned on.
Comment #3
dboulet commentedHi mennonot,
Does anything show up at all for anonymous users? Do they see nothing, or only "Current amount" and "Target"?
Comment #4
dboulet commentedI think that what's happening is that the CSS and JS files aren't being added for anonymous users when block caching is turned on, because of the way the files are added (see related issue #214856: CSS and JS for Cached Blocks (drupal_add_css incompatible with blocks)). It seems like there is no easy way, in Drupal 6, to add these files only when the block is displayed.
I guess the way to fix this is to add the files in hook_init(), but that means that the files would now be added on every page, whether the block is displayed or not. I guess that's not a big problem, as the files are pretty lightweight. Maybe we could also add a setting to make the JavaScript optional.
Comment #5
dboulet commented@mennonot, see if this patch fixes your issue.
Comment #6
ronaldmulero commentedFWIW:
You have to click on the "Save configuration" button of the Donations Thermometer Settings page after you install the module. Otherwise the graphic won't display in the block at all, only the "Current amount" and "Target".
Tested version 6.x-1.1 in Drupal 6.19
Comment #7
JezV commentedThanks @dboulet. I had the same issue as @mennonot but the patch seems to have resolved the problem.
Comment #8
nazarioa commentedNot supported at this point.