Простой чат

820
.
BigBen (04.05.2013 / 19:30)
Cталкер, в файле del.php это
if (isset($_GET['id'])) {
mysql_query("DELETE FROM `" . $table . "` WHERE `id_msg` = '" . $_GET['id'] . "'");

не совсем хорошо.
if (isset($_GET['id'])) {
mysql_query("DELETE FROM `" . $table . "` WHERE `id_msg` = '" . intval($_GET['id']) . "'");

Так нада??Или нет?
.
UA95 (05.05.2013 / 10:21)
if ($id) {
mysql_query("DELETE FROM `" . $table . "` WHERE `id_msg` = '$id'");
что за бред? откуда ты взял $id и где фильтрация?
.
Screamer
BigBen, В двиге эта переменная уже определена в incfiles/core.php и профильтрована.
.
andrei4ik (05.05.2013 / 10:18)
if (isset($_GET['id'])) {
mysql_query("DELETE FROM `" . $table . "` WHERE `id_msg` = '" . intval($_GET['id']) . "'");

Так нада??Или нет?
впринципе да, но лучше сделать так
$id=abs(intval($_GET['id']));
if(isset($id)){
mysql_query("DELETE FROM `" . $table . "` WHERE `id_msg` = '" . $id . "'");
.
Screamer (06.05.2013 / 16:09)
BigBen, В двиге эта переменная уже определена в incfiles/core.php и профильтрована.
это отдельный модуль дополнительный.
.
BigBen, index.php 22-я строка
.
Блиносвёрт ?
BigBen, верь
.
Таки да, ошибочка имеется. Одно радует, что обычный пользователь к ней не доберётся, необходимы права модератора.
.
BigBen (06.05.2013 / 16:12)
впринципе да, но лучше сделать так
$id=abs(intval($_GET['id']));
if(isset($id)){
mysql_query("DELETE FROM `" . $table . "` WHERE `id_msg` = '" . $id . "'");
а что это аbс()?
.
Если банет юзер пишет?
Всего: 35