Кидала ZeVz

sm996, Ну,Так он тебе и не заплатит,за не робочий скрипт+ты его уже розкрыл...

Автор надо было живой ему пример а не скидывать скрипт((( а так уже ничего тут не докажешь, тем более скрипт ты засветил(

спс поржал с темы... школа в отрыве... он не заплатил-в пабл...

Sokoloff, у пацана горе а ты поржал:))

Tera, горебеда.гиф7.

Зато,настроение теперь хорошое=)))

Tera, Привет,Дружище=)

Автор дурак если бы не засветил код и дал бы живой пример я бы купил без проблем так как нужен этот скрипт

1. Ребята, не тупите, я скрипт выложыл, только потому, что он не заплатил.
2. Я, дал ему демонстрационную часть скрипта, потому, что негде поставить пример, работаю на Денвере.
3. Если бы он заплатил, я, дал бы ему весь скрипт и не выкладывал бы часть кода.
По демонстрационному скрипту, видно, что скрипт работает, было замечено только 1 ошибка, строка "Нет новостей" выводилась под дизайн, ошибка, была сразу исправленна. Я, предложыл ему заплатить за работу, он отказался, сказал дайте мне весь скрипт, я, расмотрел это как, попитку получить весь скрипт, без оплаты, я, предложил заплатить 50% я, отдам весь скрипт и он заплатит остольные 50%, он отказался. Это не кидальство?

И се ты вреш строка нет новостей выходящая за дизайн не была исправлена и хватит чесать тут балабол
Я да сказал дай скрипт на проверку ты не дал а за воздух я портить не собераюсь и за то что ты мне давал там было все криво

За воздух плотить не буду гребанное т9

Скрин 7.
удали </table> на 170 строке и поставь </table> на 173 строку

Что бы было так
{
$kom=mysql_result(mysql_query("SELECT COUNT(*) FROM `kom` WHERE `news`='".$post['id']."' "),0);
$date = date(' y/d/m в H:i', $post['date']);
echo '
<tr>
<b>'.$post['name'].'</b>
</a><small>'.$date.'</small><br />
'.$post['text'].'
</br>
<a href="?act=kom&id='.$post['id'].'">Коментарии('.$kom.')</a><br />
';
}
echo '</table>';

Я, перепутал строки и написал тебе это (117 и 151 строка выниси </table> за })

sm996, Надо было исправлять ты автор скрипта ты и должен исправлять а не писать мне исправь та или там
Умник блин может мне вообще переписать скрипт и заплатить ещё тебе

Я лишь гляну первые две строчки


Дальше всё стало ясно :D за это я бы не заплатил и рубля) Одним словом дырявый нечитабельный код.

СеРжЧеРНиК, Я же говорил дырка автора в баню за такое

Почему это?

Ты же кодер или кто?))
Ну во 1х $act фильтровать как раз не нужно ибо это бессмысленно, далее $id фильтруется htmlspecialchars - что тоже абсолютно тупо ибо htmlspecialchars не спасет от sql injection да и фильтровать цыфры нужно inval'ом/abs'ом... Далее чуть нижу еще парочка переменнок также нефильрованы как следует... Вообщем нет слов товарищь:D
P.S. это вообще жесть) ну и зачем проверять переменную цыфры ли в ней и потом фильтровать?) А если в переменной еще и кавычки помимо цыфр тогда она останется не фильтрованной? :D Для того и создан intval. Сразу интвалишь и никаких ис_нумерик) не перестаю удивляться говнокоду дцмс-а:D

СеРжЧеРНиК, htmlspecialchars — преобразует спец. символы в html сущности.
Вы правы.
В полном скрипте вот так.
$act = htmlspecialchars(strip_tags(stripslashes(trim(urldecode(mysql_escape_string($_GET['act']))))));
$id = htmlspecialchars(strip_tags(stripslashes(trim(urldecode(mysql_escape_string($_GET['id']))))));

Ахаха :D спасибо за настроение)))
P.S. Я думал лет 5 назад только так фильтровали новички в программировании)
Агрументирую: зачем strip_tags?! если htmlspecialchars сделает свою работу и strip_tags уже безполезен будет. mysql_escape_string уже наверное никто не использует, вместо него _real.
urldecode зачем декодировать в url? о_О это ваще пздц товарищи извиняюсь за прямолинейность, но как я уже сказал для $id достаточно будет invtal'а, а $act не нужно вообще трогать, он безопасен и в случае неверного параметра в $act - пользователь будет просто направлен на default.

Думаю больше смысла нет в этой теме.