From d8681ecc92769792c283cef1a51cdb306590741c Mon Sep 17 00:00:00 2001
From: Jacek Kowalski <Jacek@jacekk.info>
Date: Wed, 29 Aug 2012 13:19:35 +0000
Subject: [PATCH] Poprawka związana z błędnym umiejscowieniem wywołania API GeoNames w module pogoda.
---
modules/30_pogoda/handler.php | 43 +++++++++++++++++++++++++++----------------
1 files changed, 27 insertions(+), 16 deletions(-)
diff --git a/modules/30_pogoda/handler.php b/modules/30_pogoda/handler.php
index 3a71211..39beb6a 100644
--- a/modules/30_pogoda/handler.php
+++ b/modules/30_pogoda/handler.php
@@ -7,7 +7,7 @@
$arg = trim($msg->args);
$out = new BotMsg();
-
+ $loc = FALSE;
if(empty($arg)) {
$msg->session->setClass('pogoda');
@@ -43,16 +43,6 @@
}
$out->a('<p>Nie ustawiono miasta (pomoc - wpisz: help miasto) - '.(!$forced ? 'na podstawie danych z katalogu publicznego ' : '').'wybieram miasto '.$arg.'</p>'."\n\n");
-
- $loc = new api_geonames();
- $loc = $loc->search($arg);
-
- if($loc === FALSE) {
- return new BotMsg('Nie udało się pobrać danych o podanym miejscu - spróbuj ponownie za około 10 minut.');
- }
- elseif($loc === NULL) {
- return new BotMsg('Dla podanego miejsca nie udało się uzyskać współrzędnych geograficznych - spróbuj wpisać inną nazwę.');
- }
}
else
{
@@ -66,6 +56,18 @@
}
}
+ if($loc === FALSE) {
+ $loc = new api_geonames();
+ $loc = $loc->search($arg);
+
+ if($loc === FALSE) {
+ return new BotMsg('Nie udało się pobrać danych o podanym miejscu - spróbuj ponownie za około 10 minut.');
+ }
+ elseif($loc === NULL) {
+ return new BotMsg('Dla podanego miejsca nie udało się uzyskać współrzędnych geograficznych - spróbuj wpisać inną nazwę.');
+ }
+ }
+
$api = yrno_weather($loc['lat'], $loc['lng']);
if($api == FALSE) {
return new BotMsg('Nie udało się pobrać danych o pogodzie - spróbuj ponownie za około 10 minut.');
@@ -73,13 +75,14 @@
$out->a('<p>Pogoda dla '.htmlspecialchars($loc['name']).', '.htmlspecialchars($loc['countryName']).'.</p>'."\n\n");
- $icon = $api->symbols[$api->getCurrentIcon()];
+ $icon = (int)$api->getCurrentIcon();
$weather = $api->getCurrentWeather();
$out->a('<p><b>Teraz</b><br />'."\n"
- . $icon.'<br />'."\n"
+ . '<img src="./data/pogoda/'.$icon.'.png" /><br />'."\n"
+ . api_yrno_parse::$symbols[$icon].'<br />'."\n"
. 'Temp.: '.htmlspecialchars($weather['temp']).'°C<br />'."\n"
- . 'Wiatr: '.htmlspecialchars($weather['wind']).' km/h, '.$api->wind($weather['wind']).'<br />'."\n"
+ . 'Wiatr: '.htmlspecialchars($weather['wind_speed']).' km/h, '.api_yrno_parse::wind($weather['wind_dir']).'<br />'."\n"
. 'Ciśnienie: '.htmlspecialchars($weather['pressure']).' hPa</p>'."\n\n");
$when = time();
@@ -104,12 +107,20 @@
function getHTMLforWeather($name, $icons, $weather) {
$html = '<p><b>'.$name.'</b><br />'."\n";
+ $desc = array();
+ $curr = 0;
foreach($icons as $icon) {
- if(is_file('./data/pogoda/'.htmlspecialchars($icon).'.png')) {
- $html .= '<img src="./data/pogoda/'.htmlspecialchars($icon).'.png" alt="" /> ';
+ $icon = (int)$icon;
+ if(is_file('./data/pogoda/'.$icon.'.png')) {
+ $html .= '<img src="./data/pogoda/'.$icon.'.png" alt="" /> ';
+ if($icon != $curr) {
+ $desc[] = api_yrno_parse::$symbols[$icon];
+ $curr = $icon;
+ }
}
}
$html .= '<br />'."\n"
+ . implode(' / ', $desc).'<br />'."\n"
. 'Temp.: '.$this->getHTMLforRange($weather['temp']['day']).'°C (w nocy: '.$this->getHTMLforRange($weather['temp']['night']).'°C)<br />'."\n"
. 'Wiatr: '.$this->getHTMLforRange($weather['wind']['day']).' km/h (w nocy: '.$this->getHTMLforRange($weather['wind']['night']).' km/h)</p>'."\n\n";
--
Gitblit v1.9.1