From 618783d7594fb460f52783ec37d8b832e984b695 Mon Sep 17 00:00:00 2001
From: Jacek Kowalski <Jacek@jacekk.info>
Date: Wed, 11 Jun 2014 20:59:29 +0000
Subject: [PATCH] [mod_kino] Poprawki możliwego użycia niezdefiniowanych zmiennych w skrajnych przypadkach

---
 modules/70_kino/handler.php |   18 +++++++-----------
 1 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/modules/70_kino/handler.php b/modules/70_kino/handler.php
index b22ddf9..6510e1c 100644
--- a/modules/70_kino/handler.php
+++ b/modules/70_kino/handler.php
@@ -135,7 +135,6 @@
 			MIASTO
 		*/
 		$miasta = self::getMiasta();
-		$found = FALSE;
 		$miasto_num = $miasto_nazw = '';
 		
 		if(!$miasta) {
@@ -145,7 +144,6 @@
 		foreach($miasta as $miasto => $numer) {
 			$szukaj = funcs::utfToAscii($miasto);
 			if(($pos = strpos($arg, $szukaj)) !== FALSE) {
-				$found = TRUE;
 				$miasto_nazw = htmlspecialchars($miasto);
 				$miasto_num = $numer;
 				
@@ -154,11 +152,10 @@
 			}
 		}
 		
-		if($found===FALSE && !empty($arg2)) {
+		if($miasto_num === '' && !empty($arg2)) {
 			foreach($miasta as $miasto => $numer) {
 				$szukaj = funcs::utfToAscii($miasto);
 				if(($pos = strpos($arg2, $szukaj)) !== FALSE) {
-					$found = TRUE;
 					$miasto_nazw = htmlspecialchars($miasto);
 					$miasto_num = $numer;
 					
@@ -168,8 +165,9 @@
 			}
 		}
 		
-		if($found === FALSE) {
+		if($miasto_num === '') {
 			$txt = 'Wybrane miasto nie został odnalezione. Obsługiwane miejscowości:';
+			$miasto = 'Warszawa';
 			foreach($miasta as $miasto => $num) {
 				$txt .= '<br />'."\n".htmlspecialchars($miasto);
 			}
@@ -210,7 +208,6 @@
 			KINO
 		*/
 		$kina = self::getKina($miasto_num, $czas);
-		$found = FALSE;
 		$kino_num = $kino_nazw = '';
 		
 		if(!$kina) {
@@ -228,7 +225,6 @@
 		if(!empty($arg)) {
 			foreach($kina as $kino => $kino_id) {
 				if(levenshtein(funcs::utfToAscii($kino), $arg, 1, 1, 0) < 2) {
-					$found = TRUE;
 					$kino_num = $kino_id;
 					$kino_nazw = htmlspecialchars($kino);
 					break;
@@ -236,10 +232,9 @@
 			}
 		}
 		
-		if($found===FALSE && !empty($arg2)) {
+		if($kino_num === '' && !empty($arg2)) {
 			foreach($kina as $kino => $kino_id) {
 				if(levenshtein(funcs::utfToAscii($kino), $arg2, 1, 1, 0) < 2) {
-					$found = TRUE;
 					$kino_num = $kino_id;
 					$kino_nazw = htmlspecialchars($kino);
 					break;
@@ -247,8 +242,9 @@
 			}
 		}
 		
-		if($found === FALSE) {
+		if($kino_num === '') {
 			$txt = (!empty($arg) ? 'Podany obiekt nie został znaleziony. ' : '').'Dostępne kina w pasujących miastach:';
+			$kino = '';
 			foreach($kina as $kino => $num) {
 				$txt .= '<br />'."\n".$miasto_nazw.' '.htmlspecialchars($kino);
 			}
@@ -262,7 +258,7 @@
 		/*
 			REPERTUAR
 		*/
-		$filmy = self::getKino($miasto_num, $kino_id, $czas);
+		$filmy = self::getKino($miasto_num, $kino_num, $czas);
 		
 		if(!$filmy) {
 			return new BotMsg('Przepraszamy, wystąpił bład przy pobieraniu listy wyświelanych filmów.');

--
Gitblit v1.9.1