Hi.

The "Download location for METAR data is not accessible." message is visible to all. It would be great to only show that message to admins.

Thanks.

CommentFileSizeAuthor
#10 weather-836582.patch665 bytesaaronelborg

Comments

dokuf’s picture

Subscribe!

I now get the error since a few days and everybody can see it...

johnnoc’s picture

Category: support » feature
jboeger’s picture

Same here!! very frustrating.

http://www.americascupalameda.com

MrNetworker’s picture

Yes Im also noticing lots of messages over the last few weeks.
Is it me, it seems the caching of the METAR data is not working?
I can refresh the page after 2 minutes and its redownloaded the data.
Is NOAA throttling the data?

Cheers

MrNetworker’s picture

Update: Just noticed one thing with my sites.
Make sure the date and time is set correctly. Administer » Site configuration » Date and Time
I noticed this when I was looking through the logs.
Mine was on +13 hours when it should have been +12 hours. It look like the updates are working better now since it’s not always 1 hour ahead and updating every page refresh.

sopia’s picture

Version: 6.x-5.9 » 5.x-6.3

I'm having this same problem. "Download location for METAR data is not accessible" message on many pages. Many more than usual in the last couple of weeks. Had to turn off the block temporarily (hopefully!).

Would be good to be able to switch off the message to all non-admins so we don't have to worry about this message showing up on old pages as we look into the problem.

toddy’s picture

Version: 5.x-6.3 » 6.x-5.9
Assigned: Unassigned » toddy

Hi,

it's no problem to remove the error message. What would you like instead? Would an entry in the watchdog tables be enough? That way, any administrator could see the problem, while end users do not get that message.

Regards,
Tobias

joetheyeti’s picture

I kept getting the same METAR error message after the module working well for weeks. I get the error fairly frequently. I couldn't get the error message to go away even if I told Drupal to write all errors to the log and not the screen. Here's my solution:

I hacked the module code and get the error to stop showing up on my web site. I HIGHLY recommend making a backup of the original code before you do this, so you can revert if you need to.

To remove the METAR error message:

1) Open the file weather.module (from the sites>all>modules>weather folder) in a simple text editor (WordPad works well, DON'T use MS Word)
2) Scroll to the bottom
3) Delete the following code:

// check on errors
if ($metar_raw === FALSE) {
drupal_set_message(t('Download location for METAR data is not accessible.'), 'error');
}

4) Save the document
5) Upload the document to your Drupal site

I don't know if this hack has repercussions, so you it at your own risk. It worked great on my site. If the METAR data isn't available, the weather module simply displays a question mark graphic and a nice, discreet "No data" message underneath it. But you DON'T get that ugly-ass error message in the red box at the top of the page.

izmeez’s picture

Version: 6.x-5.x-dev » 6.x-5.9
Assigned: Unassigned » toddy
Status: Needs review » Active

I was redirected to here from the discussion at http://drupal.org/node/838280

[Edit] I have posted more updates to the other issue queue and editing some of this post. [/edit]

I don't know which is the best thread to discuss this in.

In my case, yes the METAR data not accessible error is displayed. But the bigger problem is the module is not working now so I presume the error message is correct.

I'm not looking for a feature to hide the error message except for admins but to find a way for the METAR data to be accessible again.

Is this a bug or is it because I am using private file downloads?

I have gone back to 6.x-5.11 and it works. I have also posted a brief note in the other thread.

Sorry, I did not want to hijack this thread unless they are related issues.

Thanks.

aaronelborg’s picture

Version: 6.x-5.9 » 6.x-5.x-dev
Assigned: toddy » Unassigned
Status: Active » Needs review
StatusFileSize
new665 bytes

If the user has the permission of 'administer custom weather block', this message should only be shown to them.

toddy’s picture

Version: 6.x-5.9 » 6.x-5.x-dev
Status: Active » Fixed

This is now fixed in CVS, thanks a lot for the patch. It'll be released shortly.

Regards,
Tobias

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.