PHP MySQL

[Викторович]

igor'OK
Получилось, правда не работает в ФФ,но то мелочи.
Вот еще вопрос.
На странице клиента миниатюры картинок выводятся вот таким кодом:
// --- Data out about images
$i=0;
$qr = "SELECT * FROM images WHERE idclient=$idclient ORDER BY id";
$rs = mysql_query($qr);
while($rw = mysql_fetch_object($rs)):
if($i==5)
print'</tr><tr><td height="5"></td></tr><tr>';
$tpl->assign(
array(
IDIMG => $rw -> id,
IMGNM => urlencode( $rw->nm ),
W => $rw->w,
H => $rw->h,
DESCR => $rw->description
));
$tpl->parse(RW_IMGS,".rw_imgs");
$i++;
endwhile;
if($i<1){
$tpl->define( array( rw_imgs=>"empty.htm" ));
$tpl->parse(RW_IMGS,".rw_imgs");
};

Картинки выводятся подряд в один ряд. Но когда их становится много они начинают растягивать страничку. Нужно что то поменять в этом коде что бы при достижении какого то количества (например 10) начиналась следующая строка итд.
Посмотри пожалуйста что не так написано.

[levsha]

...
$qr = "SELECT * FROM images WHERE idclient=$idclient ORDER BY id";
...

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

[bykovdima]

levsha
апиридил

[Викторович]

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

[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>

[igor'OK]

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

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

[igor'OK]

Выглядит ужасно )) я тебе сочувствую
Из за такого кода 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;
}


[levsha]

+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]

Пхп не знаю

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

[Викторович]

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

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

[qwertyua]

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

Так нельзя?

[Викторович]

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

Так нельзя?

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