Вярна ли е логиката в този код?

vaster

Registered
Ето го кода:

PHP:
<?php
$id = $_GET['id'];
$window_location = './?'.back_url($_GET['back_url']).'#'.$_GET['id'];

$Query = mysql_query('SELECT * FROM `estate` WHERE `id`="'.$id.'"')
or die(mysql_error()); $row = mysql_fetch_array($Query);

	$name = $row['seller_firstname'].' '.$row['seller_surname'].' '.$row['seller_family'];

if(!$_POST['Submit'] && $_SESSION['insert_id'] == null) {
	$content = $_POST['content'];
	$Event = mysql_query('INSERT INTO `estate_events` (`user_id`,`estate_id`,`type`,`time`)
	 VALUES ("'.SID.'","'.$id.'","call","'.time().'")')
	or die(mysql_error());

	if($_SESSION['insert_id'] == null) {
		$_SESSION['insert_id'] = mysql_insert_id();
	}
}



if(isset($_POST['Submit']) && $_SESSION['insert_id'] != null) {
	$Event_Update = mysql_query('UPDATE `estate_events` SET
	`content`="'.$_POST['content'].'",
	`allow`="'.$_POST['Submit'].'",
	`date`="'.$_POST['date'].'"
	WHERE `id`="'.$_SESSION['insert_id'].'"')
	or die(mysql_error());
		echo '<h3>Бележката е записана.</h3>';
		$_SESSION['insert_id'] = '';
		#header('location:'.$window_location);
} else {
	#$_SESSION['insert_id'] = '';
	echo '<b>Телефон за връзка с '.$name.'</b>: <span style="font-size:22px;">'.$row['seller_phone'].'</span>';
?>

<form action="" method="post">
              <div class="form-group">
                <label>Бележка</label>
                <textarea name="content" class="form-control" placeholder="Напиши причина и съдържание на разговора."></textarea>
              </div>

              <button value="1" name="Submit" type="submit" class="btn btn-default" onClick="window.top.location = '<?php echo $window_location; ?>'">Може оглед</button>
              <button value="0" name="Submit" type="submit" class="btn btn-default" onClick="window.top.location = '<?php echo $window_location; ?>'">Неможе оглед</button>
              <button value="00" name="Submit" type="submit" class="btn btn-default" onClick="window.top.location = '<?php echo $window_location; ?>'">Няма връзка</button>

</form>
<?php } ?>

Файлът се изпълнява чрез друг файл, в който има session_start(); и други неща. Този файл се отваря с fancybox ефект през линк. Идеята е, че когато се кликне линка системата трябва да логне действието, че е натиснат линка и АКО въведе нещо в бележката потребителя то тази бележка да се записва към вече добавения нов запис, не да се създава още един. Ако потребителя реши да затвори прозореца и пак да го отвори отново трябва да се създава нов запис и ако напише нещо в бележка да се записва към съответното id, не ново.

В момента кода работи, но не съм сигурен до колко е правилно написан, тъй като имах една малка грешка, която е коригирана вече и този път искам да съм 100% сигурен, че всичко ще е точно, затова и ще разчитам на вашата помощ! :?: +1
 
Има много дупки в заявките. Ескейпни си стойностите, които въвеждаш в ДБ.
 

Back
Горе