Download: Weather

[dr. Jekyll]

...thing being that everything looks too close to the edges of the block (screenshot below with red lines to show not much room). You should maybe consider adding some spaces to make it look more consistent with the rest of the sidebar blocks that come with the board. Also, maybe you can add a line break before where it says "Forecast for Sunday night". Still a great mod though.

' alt='' class='ipsImage' >

+ 1 ' alt='' class='ipsImage' >


I have also a couple of questions:

• is it possible to self change the icons?
• is it possible to have this mod to self recognize the users location by IP as on Yahoo! ?
  

[chuckf]

Sorry I didn't reply sooner. I wasn't getting notified of replies here. Hopefully that has been resolved.

Noticed you updated 2.0 to work with 3.2 IPB. If we install 2.0 on 3.1.4 IPB, will it still work?

Thank you.

JB

NO. Weather v2.0.0 (what I'm calling the v2 series) requires IPB v3.2.0. Please continue to use the v1 series (latest is v1.3.0) for IPB v3.1.4.

[chuckf]

This looks like a really nice Mod, but one thing makes it a bit unattractive to me. That thing being that everything looks too close to the edges of the block ...

I'm very glad to hear you like Weather.

Coming up with one layout that pleases everyone is near impossible. That's why I provide the weather css, language, and skin. You can use the facilities built into IPB to tweak any or all of them.

I decided to use all the available space for the block because it can get long at times. However, you can add the spacing you want by editing the weather.css. I suggest you try adding the following at the top of weather.css:

.weather_block {

   padding: 0px 10px

}

You can add a break before the forecast sub title by editing the weather language bit called 'forecast'. Just set the new value to "<br>Forecast for" (without the quotes).

[chuckf]

I have also a couple of questions:

• is it possible to self change the icons?
  

The weather icons come from weatherbug so they can't be changed.

• is it possible to have this mod to self recognize the users location by IP as on Yahoo! ?

I thought about that but dismissed it because of proxies. Also an ISP where the IP address is registered can be quite far from ones home. WeatherBug has many stations to choose from for any particular location. I was surprised to find one only a mile from my home.

I may consider adding it as an option if there's a significant demand for it.

[chuckf]

Many thanks for new upgrade [img]

[/img]

Since i installed it on Version 3.2.0 i see the following message in the acp

Argh. I test and test and test and still there's a bug. Nice find and thanks for reporting it.

That's the old dashboard notification mechanism running. I don't supply it in the v2.0.0 release but I guess I need to remove it when upgrading from the v1 series.

For now you can just delete the file \admin\applications_addon\other\weather\extensions\dashboardNotifications.php.

[simplymesobe5280]

Hello chuckf,

There seems to be something wrong with the 2.0.1 . I uploaded the new content folders but there isn´t any "upgrade...." shown on the Installed Applications part on the acp. Only an "FURL..." error message is shown at the top.

[chuckf]

Hello chuckf,

There seems to be something wrong with the 2.0.1 . I uploaded the new content folders but there isn´t any "upgrade...." shown on the Installed Applications part on the acp. Only an "FURL..." error message is shown at the top.

The updated versions.xml file didn't make it into the zip file. I have corrected the zip file in the MarketPlace and on my site. You can fix this manually to save time and bandwidth by replacing the contents of \admin\applications_addon\other\weather\xml\versions.xml with the following:

<?xml version="1.0" encoding="UTF-8"?>

<versions>

   <version>

 	<human>2.0.0</human>

 	<long>20000</long>

   </version>

   <version>

  	<human>2.0.1</human>

   	<long>20001</long>

   </version>

</versions>

You really just need to add the version info for release 2.0.1 as indicated above.

Once you make this change refresh your Manage Applications & Modules page and you should see an Upgrade Available button on Weather. Just click it to install 2.0.1. The FURL error should go away then.

Sorry for the incovenience and thank you for reporting this problem.

[simplymesobe5280]

Many thanks for the fast fix :D

[simplymesobe5280]

Well i download and uploaded the whole new content.

Now i get the following errors

Warning: DOMDocument::load() [domdocument.load]: expected '>' in /home/xxxxxx/domains/mysite.net/admin/applications_addon/other/weather/xml/versions.xml, line: 12 in /home/xxxxxx/domains/mysite.net.net/ips_kernel/classXML.php on line 331

Warning: DOMDocument::load() [domdocument.load]: Premature end of data in tag versions line 2 in /home/xxxxxx/domains/mysite.net.net/admin/applications_addon/other/weather/xml/versions.xml, line: 13 in /home/xxxxxx/domains/mysite.net.net/ips_kernel/classXML.php on line 331

On the Installed Applications part on the acp it showed

Weather vinstall Up to date

Had to uninstall it :(

[chuckf]

This is what I get for trying to do a quick fix before work. The versions.xml file had an extra <version> line in it. I've corrected it in the zip file.

[simplymesobe5280]

All worked great now - many thanks again :D

[Buskopan]

any chance to make it work on other language ? Russian for example

[chuckf]

The conditions and forecasts from Weatherbug are only in English now. When they support other languages I'll add it to Weather.

[Dmacleo]

when a guest looks at board if the zip code for default starts with a 0 (04401 example) the current show ok but the forcast says cant retrieve for 4401.
its dropping the first 0.

btw I enjoy the app, I have some suggestions if thats ok?

[chuckf]

when a guest looks at board if the zip code for default starts with a 0 (04401 example) the current show ok but the forcast says cant retrieve for 4401.

its dropping the first 0.

Thank you for reporting this issue. I have reproduced the problem and am working on a fix.

btw I enjoy the app, I have some suggestions if thats ok?

I'm glad you like it. I'm always open to suggestions. You can post them here for your convenience but I would prefer you posted them as Feature Requests in my tracker or as feedback posts in the Weather forums at www.fischeronline.net.

[Dmacleo]

ok will visit the tracker today and post an idea.
thanks

[chuckf]

I'd hoped to be able to release a fix for this today but something has come up.

In the mean time you can fix it yourself if you don't mind editing admin\applications_addon\other\weather\sources\classes\gateway.php as follows:

Find this code:

      // if guest or member hasn't set their weather settings use defaults

      if ( !isset($weather_location_code) || ($weather_location_code == '0' && $weather_station_id == '0') )

      {

         $weather_station_id = $this->settings['weather_default_station_id'];

         $code = $this->settings['weather_default_location_code'];

         $weather_location_type = intval(substr($code,0,1));

         $weather_location_code = intval(substr($code,1));

         $weather_unit_type = $this->settings['weather_default_settings_units'];

      }

replace the line:

$weather_location_code = intval(substr($code,1));

with:

$weather_location_code = substr($code,1);

[Dmacleo]

seems to have fixed it, thanks.

[chuckf]

Weather v2.0.2 for IP.Board 3.20+ and v1.3.1 for IP.Board 3.1.4 are available now!

This release fixes a bug where the forecast could be "unavailable" for guests. The bug would occur if the admin set the default location to a zip code that started with a zero like 04401.

[Azoun]

Hi,

I have noticed a slight layout issue in the block on the side as can be seen below:



Any idea what to change where to fix this please?

Thanks,

Andy

[Dmacleo]

mine does not do that is it pure skin issue?
I am board 3.2.2 tough.

[chuckf]

I reproduced your issue with the Chrome browser. You can fix it by adding two lines to weather skin.

In the AdminCP go to Look & Feel. Press the down arrow to the right of the skin set you want to change (e.g. Ip.Board). Select Manage Templates & CSS.

Ensure the Templates tab is selected and scroll down to skin_weather and lift click it. Left click weatherShowWeather.

Find the these two lines:

<dl class='weather_info'>
<dt>{$this->lang->words['temp']}:</dt>

and put a <br/> between them so the result looks like this:

<dl class='weather_info'>
<br/>
<dt>{$this->lang->words['temp']}:</dt>

Also find these two lines:

<dl class='weather_info'>
<dt>{$this->lang->words['high']}:</dt>

and put a <br/> between them.

Press the Save button and refresh your browser.

[Dmacleo]

would it be good idea for all to do that just in case?
I use FF so not an issue here but I know some user use chrome.

[Azoun]

Many thanks for the fast fix :)

[Dmacleo]

fwiw I applied this to all skins even though I saw no issues on FF.
it worked fine and on IE9, FF5 and 6, chrome and opera it seemed to work fine. it was limited quick view but should be fine.