PHP MySQL

Викторович писал(а):...
$qr = "SELECT * FROM images WHERE idclient=$idclient ORDER BY id";
...

Вот это называется "SQL injection"

levsha
апиридил

bykovdima
levsha
И? Теперь я знаю как это называется)))) а как сделать что бы строка перескакивала при выводе картинок?

Викторович писал(а):bykovdima
levsha
И? Теперь я знаю как это называется))))

А когда через твой сайт устроят спам-рассылку и захостят на нём парочку вирусов, еще и узнаешь чем это плохо

Викторович писал(а):а как сделать что бы строка перескакивала при выводе картинок?

Например вставлять <br> каждые 10 картинок. Но так как ты там темплейты используешь, то всё немного сложнее: нужно в темплейте это реализовать

levsha
Это не сайт. Это клиентская база которая стоит на локальном компе в офисе. На компе апач и майскл.
Писалась она давно, это я уже переписываю под себя. Автора и не найти))) Так что учусь сам.
Вот код темплейта;
<form name="img{IDIMG}" action="" method="post" onSubmit="">
<td style="padding-right: 5px;"><a href="../inc/imgs.php?idimg={IDIMG}" target="_blank"><img src="../img/c/{IMGNM}" width="50" height="50" border=1 ></a></td>
<input type="hidden" name="idimg" value="{IDIMG}">
</form>

levsha писал(а):Гм, подключать jquery только чтобы сделать один post, выглядит как-то слишком жирно

Во первых jquery скорее всего уже есть на странице
Во вторых jquery давно закеширован браузером, этот CDN используется очень часто, даже если прийдется его подключать на страницу он будет браться из кеша
в третих этот сайт уже ничем не испортить судя по коду )

Выглядит ужасно )) я тебе сочувствую
Из за такого кода PHP не любят
Выводить это таблицей это кошмар, надо выкинуть таблицу из шаблона, сдеать обычный список картинок и с помощью CSS его переносить по строкам.
На выходе список должен выглядеть как то так:

Код:

<div class="picture-list">
    <a href="../inc/imgs.php?idimg=1" target="_blank"><img src="../img/c/1.png" width="50" height="50" border=1 ></a>
    <a href="../inc/imgs.php?idimg=2" target="_blank"><img src="../img/c/2.png" width="50" height="50" border=1 ></a>
    <a href="../inc/imgs.php?idimg=3" target="_blank"><img src="../img/c/3.png" width="50" height="50" border=1 ></a>
    <a href="../inc/imgs.php?idimg=4" target="_blank"><img src="../img/c/4.png" width="50" height="50" border=1 ></a>
    ....
</div>


CSS код должен выглядеть как то так:

Код:

.picture-list a {
    display: inline-block;
    width: 50px;
    height: 50px;
    margin: 5px;
    overflow: hidden;
    float: left;
    text-decoration: none;
}


+1 к igor'OK - не надо использовать таблицы для форматирования!

igor'OK
Большое спасибо!!!!!!! Получилось

Коллеги, нужна помощь.
Есть такой код:
<?
$dt_cur = date("Ymd");
$qr = "SELECT min(dt_alert) AS dt_alert FROM alerts WHERE idusr=$idusr AND dt_alert<$dt_cur LIMIT 1";
$rs = mysql_query($qr);
while($rw = mysql_fetch_object($rs)):
echo ($rw->dt_alert);
endwhile;
if( вот тут что то надо придумать с $dt_alert < $dt_cur ):
?>
Тут будет выводиться HTML. Но он должен выводиться при условии что $dt_alert < $dt_cur. Но у меня не получается...
<?
endif;
?>
echo ($rw->dt_alert); работает... но это мне не нужно. Вывожу это для проверки что правильно с таблицы берется наименьшая дата.
А вообще смысл такой - если в таблице нет даты меньше чем текущая, то HTML не должен выводиться на странице. Даже если в таблице есть даты равные текущей и позже текущей. А если есть дата меньше текущей, то должен выводиться HTML код.
Посмотрите плиз. Может что то посоветуете. Как получить эту самую $dt_alert что бы сравнить ее с $dt_cur .

Пхп не знаю

Но если эхо правильно выводит, почему не получается это же с текущей датой сравнить?

qwertyua писал(а):Но если эхо правильно выводит, почему не получается это же с текущей датой сравнить?

по тому что я пхп тоже практически не знаю)))) вот и не знаю как это прописать))

If ($rw->dt_alert < $dt_cur ):

Так нельзя?

qwertyua писал(а):If ($rw->dt_alert < $dt_cur ):

Так нельзя?

пробовал. не работает.