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