Closed (fixed)
Project:
Weather
Version:
6.x-5.18
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
19 May 2010 at 22:38 UTC
Updated:
3 Jan 2014 at 01:42 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
majdi commentedSubscribe
Comment #2
steinmb commentedHad a test on my sandbox install but also I was unable to make it work.
Weather version 6.x-5.12
Comment #3
Dennis-1 commentedNot sure if this is your problem, but are you using Drupal's block cache (enabled)? I was experiencing this issue yesterday, but managed to get it to work by disabling the block cache in admin - site configuration - performance. Though, my issue was only affecting anonymous users. Registered users were able to see the location block without issue prior to that.
Comment #4
asb commentedHi,
I just tried weather 6.x-5.13 from 2010-Jul-03 in combination with the 'Node Map' module, and this doesn't work either for geocoded nodes.
However, as annoying this issue might be, it is by no means critical since in neither damages Drupal core, nor does it disturb the main functionality of the 'Whether' module itself. It's simply an advertised feature (advertised on the project page) which seems to not work, or might require additional configuration steps that are undocumented. Any success report would be nice, though ;)
Comment #5
Pierre Lannoy commentedSame with 6.x-5.15. No news ? That's an important functionality for my site...
Comment #6
Stacy Prendeville commentedAnyone had any success getting this to work?
I do not have the block appearing...
Comment #7
toddy commentedHi,
this should be fixed in weather 6.x-5.17.
Regards,
Tobias
Comment #8
asb commentedShould it? If I enable the "Weather" block ("Wetter: Orte von Nodes (benötigt Location- oder Node-Map-Modul)") on a node with Nodemap data, I'm getting this fatal error:
And yes, this is 5.17:
Comment #9
asb commentedChanging status to active with priotity "critical" since enabling the Weather block affects the functionality of Drupal core. Please feel free to modify this if you think that it's not appropriate.
Can anyone else confirm either that the issue is fixed, or is anybody else getting the fatal error as well?
Comment #10
toddy commentedHi,
thanks for notifying me about this. It seems that the vast majority of users installed the Location module, not the NodeMap module, therefore this bug has not been discovered earlier. However, now the dependency on the Location module (which provides the missing function earth_distance()) has been removed, and you should be able to use Weather with NodeMap.
Regards,
Tobias
Comment #11
asb commentedThank you for the quick update. I can confirm that (a) the initial issue is fixed for me ("Weather: location of nodes (requires Location or Node Map module) block dont work") and (b) the fatal error reported in #8 is gone as well.
The only issue remaining is strange characters in the "weather" block (see attached screenshot and online examples 1, 2, 3, 4, inner block, bottom right).
Except for this minor formatting issue the module appears to be working quite well, actually it is pretty cool ;)
Thank you!
Edit: There's just another gotcha I just ran into: There seems to be a serious problem with block caching. At least on my site, the 'Weather' block wasn't shown to anonymous users, as well as the 'Node Map' block (comment on the 'Node Map' issue queue: #302933: Map Not Appearing). Please look out for these block caching issues for anonymous users!
I believe to have found a workaround by installing the Block Cache Alter module and changing the caching settings for the 'Weather' block to "per page" (as well as for the 'Node Map' block). However I'm not sure yet about the performance ramifications of this setting, and I don't know if this will play well together with the 'Boost' static page cache (which I had to wipe completely to see any results for anonymous users).
-asb
Comment #12
toddy commentedHi,
I cannot see the strange characters you're describing in the four links you posted. Could you attach a screenshot, please?
Regards,
Tobias
Comment #13
asb commentedMissing screenshot. I'm referring to the squares between the values (e.g. 26) and the unit (e.g. °C).
Comment #14
toddy commentedAh, ok. Those are thin spaces ( ) used to separate numbers from their units (typografically correct). It seems to be a problem with your browser. Do you use a modern browser for the site?
Comment #15
asb commentedI would think so (Opera 10.63 Build 6450 for Linux on 64 bit Ubuntu "Maverick"). OK, I checked with Chromium and Firefox which don't have this issue. Opera not honoring properly encoded HTML entities, that's a first.
Here's a comparison about browser support for this entity: Schmaler, nicht umbrechbarer Zwischenraum and a discussion at Opera forums. Also there seems to be a related issue with Chromium and Webkit/Safari. Nasty :-(