User Blog Mod: доработки

[Alecto]

Sheer
а зачем оно нужно? разве у таблицы комментариев нет своей колонки?
phpbb_blogs_reply.reply_time - сортировка комментов по этому полю дает все последние комментарии.
или у вас отображаются только последние комментарии для последних записей?

... кажется я понял, ваш аддон выводит не последние комментарии, а последние комментарии для последних записей блогов, по одному комментарию для каждой записи.

Добавлено спустя 4 минуты 28 секунд:
сделал так:
запустил /install/install_add_blog.php, а затем убил поле user_last_blog_mark в phpbb_users - портал стало ругаться на отсутствие индекса user_last_blog_mark
затем в последней вашей упрощенной версии top10_blogs.php удалил строку

Код: Выделить всё

    $user_last_blog_mark = $user->data['user_last_blog_mark'];

все заработало.

завтра... точнее уже сегодня, на свежую голову, попробую откатить сайт и выполнить всю установку заново, чтобы убедиться в правильности установки и работоспособности аддона.
как по мне получилось функционально и просто.

[Sheer]

ваш аддон выводит не последние комментарии, а последние комментарии для последних записей блогов, по одному комментарию для каждой записи

Да. Иначе нафиг это надо. Увидеть последние 5 комментариев в одном блоге?
...
Я подумаю... кажется есть идея, как обойтись без модификации таблицы.

Добавлено спустя 1 минуту 36 секунд:

удалил строку

Да, это лишнее. Прозевал.

[Alecto]

Sheer
урррааа!
а я попробую сделать альтернативный вариант вывода в центральной части портала. покажу что получится, может кому понравится.

[Sheer]

Возьми тут User Blog Mod: доработки новый файл с твоими правками и кое-какими моими. А то я там лишнего удалил, и получилось, что если нет комментариев вообще, то не будет выводиться сообщение "Нет комментариев".

Добавлено спустя 7 минут 48 секунд:

Я подумаю...

Придумал. Может завтра сделаю.

[Alecto]

не пойму чего, но FF молчит, а IE и Opera ругаются

Код: Выделить всё

[phpBB Debug] PHP Notice: in file /portal/block/top10_blogs.php on line 45: Undefined index: NEW_BLOG

хотя в файле /language/ru/mods/blog/common.php переменная объявлена

Код: Выделить всё

	'NEW_BLOG'						=> 'Добавить запись',

вот такой блок получился:

пока немного через ж.... через таблицы и дивы нарисовал, что-то с тегами DT и DD для двух циклов сразу у меня не заладилось, может на свежую голову что придумаю.

[Sheer]

не пойму чего, но FF молчит

Вот это и правда непонятно, потому как обязан. Скрипт портала не цепляет языковые переменные из
/language/ru/mods/blog/common.php. Надо объявить в /language/ru/common.php, а из /language/ru/mods/blog/common.php убрать.

[Alecto]

в принципе так и сделал - заработало везде, только из /language/ru/mods/blog/common.php не убрал.
скажите пожалуйста, а если делать выборку последних комментариев к заметкам не создавая дополнительное поле в баазе - вы говорили, что это возможно. как такой запрос повлияет на нагрузку к базе? ведь скорее всего придется опрашивать не одну а две таблицы? или разницы не будет?

еще конструктивное предложение - возможно ли кэшировать запросы и к записям блогов и к комментария эдак минут на 15-30? портал и так тяжелый, еще не считал сколько дополнительных запросов добавляет аддон.

[Sheer]

В любом случае две:blogs и blogs_reply. Плюс еще и users.

возможно ли кэшировать запросы

Дык запросы вроде все кэшируются.

[Alecto]

Sheer тогда, если я вас правильно понял - есть смысл полностью избавиться от вмешательства в базу аддоном, при этом количество запросов и нагрузка на базу не изменится.

[Sheer]

Смысл может и есть, а насчет нагрузки не знаю... Запросы делать все равно придется, а дополнительные поля в таблице на нагрузку не очень-то влияют.

[Alecto]

помогите пожалуйста поправить шаблон.
решил что через div и table - это через ж... неправильно, переделываю через UL, LI, DL, DD, DT - как в других шаблонах (за основу взял шаблон списка форумов).

но, никак не могу разобраться, как мне правильно вывести два цикла в два столбца?
с таблицами все понятно, один цикл (вывод записей дневников) в одном столбике, другой (вывод комментариев) - во втором.
здесь не получается.

хоть убейте не пойму.
сейчас выводится в в два столбика, но нету строк. шаблон прилагаю.

[Sheer]

Так попробуй

Код: Выделить всё

<fieldset class="fields1 column1">
   тут первый цикл
</fieldset>
<fieldset class="fields1 column2">
   тут второй цикл
</fieldset>

[Alecto]

да, спасибо, на две колонки разделить удалось.
по сути получился табличный вариант.
тот скрин, что я выкладывал ранее построен на таблице и добавлении в циклы div с margin-... со всех сторон.

сейчас по сути надо тоже каждый блок вставить в свои div span с подгонкой смещений со всех сторон?

Код: Выделить всё

<fieldset class="fields1 column1">
	<!-- BEGIN blog_row -->
<div class="panel">
	<div class="inner"><span class="corners-top"><span></span></span>
                             цикл
	<span class="corners-bottom"><span></span></span></div>
</div>
	<!-- END blog_row -->
</fieldset>

без div получается полностью синий фон.

Добавлено спустя 6 минут 47 секунд:
вот такой код дал желаемый результат, не знаю насколько он корректен. выкинул все лишние на мой взгляд DIV и SPAN

Код: Выделить всё

<div class="post bg1" style="margin-right:-7px; margin-bottom:1px;">
	<span class="corners-top"></span>
                        ЗНАЧЕНИЯ ДЛЯ СТРОКИ ПЕРВОГО ЦИКЛА
	<span class="corners-bottom"></span>
</div>

<div class="post bg1" style="margin-left:-7px; margin-bottom:1px;">
	<span class="corners-top"></span>                
                                                          ЗНАЧЕНИЯ ДЛЯ СТРОКИ ВТОРОГО ЦИКЛА
	<span class="corners-bottom"></span>
</div>

Добавлено спустя 8 минут 3 секунды:
при этом, в опере тоже все нормально, но в IE середина расползается (1) и, не могу понять, почему, но если разлогинишься, но миникартинки (2) становятся красными.

[Sheer]

если разлогинишься, но миникартинки (2) становятся красными.

Потому что

Код: Выделить всё

        $new = (($blog_row['blog_time'] > $user->data['user_lastvisit'])) ? true : false;
        $icon_topic_latest = ($new) ? 'icon_topic_newest' : 'icon_topic_latest'; 

и $new принимает значение true
Если хочешь, можешь поставить условие проверки пользователя, если гость$icon_topic_latest = 'icon_topic_latest';

Ну так вот. Облегченный вариант для "стандартных" таблиц блога

top10_blogs_light20110907182255.rar

[Alecto]

заменил старый файл. вот что вывалило:

Код: Выделить всё

[phpBB Debug] PHP Notice: in file /portal/block/top10_blogs.php on line 20: Undefined index: user_last_blog_mark