[beta] Topics as articles - Статьи форума

[Anvar]

Название: Topics as articles

Описание: Расширение позволяет назначить темам режим статьи, тем самым изменив вид отображения первого сообщения форума, с горизонтальным отображением профиля.

• Возможность пользователям создавать статьи устанавливается в правах доступа к форуму.

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

• В списке тем, статьи будут помечены соответствующей иконкой.

Идея портирования с мода для phpBB3.0: [BETA] Topics as article MOD

Репозиторий: https://github.com/bb3mobi/TopicsArticles

[xisp]

Думаешь, это возможно сделать из расширения?

[Anvar]

Shredder, Не получилось, нет событий. Были ещё задумки, надо будет всё же попробовать.

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

[rxu]

Нет событий - давайте добавлять.

[Anvar]

Тут если сделать способом как в моде 3.0, появятся запросы на то, что-бы можно было статью делать объявлением или прилепленной и вернёмся обратно к исходному.
К тому-же в том виде как сейчас, расширение само по себе достаточно легкое и дальнейшая миграция будет происходить без существенных проблем.
Ради личного интереса конечно буду пробовать и тот способ реализовать, а там как ляжет)))

Событие всё же одно необходимо, не только этому расширению. Что-бы можно было без дополнительных обращений к бд сохранять данные.

[Shredder]

Anvar
Просто когда я свою карту сайта портировал, то оставил обнаружение статей при помощи константы POST_ARTICLE, и эта константа вмурована там в сложные запросы, которые не так легко будет переделать (если вообще возможно). Можешь глянуть в core.php. Поэтому было бы удобнее, чтобы статьи были в "старом" виде.

[Anvar]

Shredder, Ну константу всё-равно придётся заменить, так как в 3.1 изменения вносить в constans.php мало кто захочет. И если захотят, то нет проблемы изменить поле.

Надо подумать будет как грамотней решить или событие добавить в карту.

[rxu]

Можно попробовать константу определить в сервисах, а другой экст будет ее опционально использовать, если существует.

[Alecto]

при цитировании (проверял в БО Татьяны) вместо имени автора подставляется undefined

[Shredder]

Можно попробовать константу определить в сервисах, а другой экст будет ее опционально использовать, если существует.

Есть ещё такое решение:
1) Константа из Topic Articles выносится в отдельный файл (пусть по аналогии с движком это будет constants.php), расположенный в папке экста
2) Другой экст (в данном случае мой Sitemap) проверяет наличие установленного и включённого экста Topic Articles, и если экст установлен и включён - подключает constants.php

Проверка осуществляется запросом типа этого:

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

		$sql = 'SELECT ext_name, ext_active
			FROM ' . EXT_TABLE . "
			WHERE ext_name = 'phpbbseo/usu' OR ext_name = 'shredder/seo_topic_url'";
		$result = $this->db->sql_query($sql);

		$seo_topic_url = 0;
		while ($row = $this->db->sql_fetchrow($result))
		{
			if ($row['ext_name'] == 'phpbbseo/usu' && $row['ext_active'])
			{
				$seo_core = $this->phpbb_container->get('phpbbseo.usu.core');
			}
			if ($row['ext_name'] == 'shredder/seo_topic_url')
			{
				$seo_topic_url = $row['ext_active'];
			}
		}
		$this->db->sql_freeresult($result);

Данный запрос успешно используется мной в карте для подключения двух других расширений. Переделать его под Topic Articles труда не составит.

[Татьяна5]

Проверка осуществляется

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

$this->phpbb_extension_manager->is_enabled('vendor/extname')

[Alecto]

пересобрал расширение Анвара с названием Topics as Articles (Light) за его авторством.

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

Без-имени-1.jpg

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

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

UPD: Анвар обновил свое расширение, лайт-версия так же обновлена до версии 0.0.2

[Alecto]

проблема [dev] Thanks for posts
решение [dev] AddonForThanksForPosts обновить расширение AddonForThanksForPosts до последней версии.
спасибо огромное Алг за оперативное решение.

[Вчерашний борщ]

Шапка достаточно давлеет над текстом. Хотелось бы какого-то более легкого дизайна: оставить лево- или правосторонний аватар (по настройкам) со стандартным набором инфо под ним, а сам текст делать с обтеканием блока аватара. Не всегда первый топик это статья, где важен автор, часто просто большой блок информации, который лучше воспринимается на широком поле без длинного продолжения с пустотой в колонке аватаров. Для тех, у кого не статьи, а так, статейки ))

[Erlang]

Шапка достаточно давлеет над текстом.

Согласен!
Было-бы лучше Шапку поместить ПОД статьёй.
Или добавить возможность в настройках выбирать размещение шапки верх/низ.
Можно такое дополнение сделать?

[rst]

Никто не реализовал уменьшение шапки или перенос вниз? Очень согласен с предыдущими мнениями. Переходя к статье пользователю нужен контент, а не широкая инфа об авторе.

Отправлено спустя 10 минут 1 секунду:
Расширение не работает на стиле от артодия. Кто-то пробовал решить эту проблему?
Странно, что стиль даже не поплыл. Всё так, будто расширения нет. Хотя метка темы в списке тем измененная, те экст работает.

Отправлено спустя 1 час 23 минуты 38 секунд:
Методом активного тыка было выяснено, что не отрабатывает условие:

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

postrow.POST_NUMBER == '1' 

Убираем условие - всё работает, но во всех сообщениях. В чем может быть причина? Как то можно по другому сформулировать условие проверки на первый пост?

Отправлено спустя 10 минут 1 секунду:
Совсем интересная хрень. Замена конструкции на:

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

postrow.S_ROW_COUNT eq 1 

работает, но рушит шаблон )))

Отправлено спустя 24 минуты 45 секунд:
Почти решен вопрос заменой условия на

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

postrow.S_FIRST_ROW

Всё отображается корректно, но выделяется первый пост на каждой странице, а мне нужен только первый пост темы, а не страницы. Как сделать?
Или как добавить условие проверки первой страницы?