From 3c97a446c7c170625e51b02058d2d560b227c4d0 Mon Sep 17 00:00:00 2001 From: Jacek Kowalski <Jacek@jacekk.info> Date: Mon, 17 Sep 2012 09:50:13 +0000 Subject: [PATCH] Poprawa błędu w klasie BotSession powodującego wielokrotne tworzenie wpisów w bazie danych sesyjnych (class=NULL, name=user_struct). --- modules/30_pogoda/handler.php | 29 ++++++++++++++++------------- 1 files changed, 16 insertions(+), 13 deletions(-) diff --git a/modules/30_pogoda/handler.php b/modules/30_pogoda/handler.php index 40a1691..ac77828 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'); @@ -37,22 +37,13 @@ catch(Exception $e) { } + $forced = FALSE; if(empty($arg)) { $arg = 'Warszawa'; $forced = TRUE; } $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 { @@ -63,6 +54,18 @@ 'lat' => $msg->session->geo['lat'], 'lng' => $msg->session->geo['lon'] ); + } + } + + 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ę.'); } } @@ -77,10 +80,10 @@ $weather = $api->getCurrentWeather(); $out->a('<p><b>Teraz</b><br />'."\n" - . '<img src="./data/pogoda/'.$icon.'.png" />'."\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_speed']).' km/h, '.api_yrno_parse::wind($weather['wind_direction']).'<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(); -- Gitblit v1.9.1