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