Fix microdata breadcrumb phpbb 3.1.x

[владимир1983]

Не знаю почему сделано так как сделано, но оно не работает из коробки. Проверено много где. Да и по коду видно что необходимых обязательных атрибутов нет.
Предлагаемый рабочий фикс
styles/prosilver/template/navbar_header.html

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

	<ul id="nav-breadcrumbs" class="linklist navlinks" role="menubar">
		<!-- DEFINE $MICRODATA = ' itemtype="http://data-vocabulary.org/Breadcrumb" itemscope=""' -->
		<!-- EVENT overall_header_breadcrumbs_before -->
		<li class="small-icon icon-home breadcrumbs">
			<!-- IF U_SITE_HOME --><span class="crumb"><a href="{U_SITE_HOME}"{$MICRODATA} data-navbar-reference="home">{L_SITE_HOME}</a></span><!-- ENDIF -->
			<!-- EVENT overall_header_breadcrumb_prepend -->
			<span class="crumb"><a href="{U_INDEX}" accesskey="h"{$MICRODATA} data-navbar-reference="index">{L_INDEX}</a></span>
			<!-- BEGIN navlinks -->
				<!-- EVENT overall_header_navlink_prepend -->
				<span class="crumb"><a href="{navlinks.U_VIEW_FORUM}"{$MICRODATA}<!-- IF navlinks.MICRODATA --> {navlinks.MICRODATA}<!-- ENDIF -->>{navlinks.FORUM_NAME}</a></span>
				<!-- EVENT overall_header_navlink_append -->
			<!-- END navlinks -->
			<!-- EVENT overall_header_breadcrumb_append -->
		</li>
		<!-- EVENT overall_header_breadcrumbs_after -->

		<!-- IF S_DISPLAY_SEARCH and not S_IN_SEARCH -->
			<li class="rightside responsive-search" style="display: none;"><a href="{U_SEARCH}" title="{L_SEARCH_ADV_EXPLAIN}" role="menuitem">{L_SEARCH}</a></li>
		<!-- ENDIF -->
	</ul>

Заменить на

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

   <ul id="nav-breadcrumbs" class="linklist navlinks" role="menubar">
      
      <!-- EVENT overall_header_breadcrumbs_before -->
      <li class="small-icon icon-home breadcrumbs">
         <!-- IF U_SITE_HOME --><span class="crumb" itemscope itemtype="http://data-vocabulary.org/Breadcrumb"><a href="{U_SITE_HOME}" data-navbar-reference="home" itemprop="url"><span itemprop="title">{L_SITE_HOME}</span></a></span><!-- ENDIF -->
         <!-- EVENT overall_header_breadcrumb_prepend -->
         <span class="crumb" itemscope itemtype="http://data-vocabulary.org/Breadcrumb"><a href="{U_INDEX}" accesskey="h" data-navbar-reference="index" itemprop="url"><span itemprop="title">{L_INDEX}</span></a></span>
         <!-- BEGIN navlinks -->
            <!-- EVENT overall_header_navlink_prepend -->
         <span class="crumb" itemscope itemtype="http://data-vocabulary.org/Breadcrumb"><a href="{navlinks.U_VIEW_FORUM}" itemprop="url"><span itemprop="title">{navlinks.FORUM_NAME}</span></a></span>
            <!-- EVENT overall_header_navlink_append -->
         <!-- END navlinks -->
         <!-- EVENT overall_header_breadcrumb_append -->
      </li>
      <!-- EVENT overall_header_breadcrumbs_after -->

      <!-- IF S_DISPLAY_SEARCH and not S_IN_SEARCH -->
         <li class="rightside responsive-search" style="display: none;"><a href="{U_SEARCH}" title="{L_SEARCH_ADV_EXPLAIN}" role="menuitem">{L_SEARCH}</a></li>
      <!-- ENDIF -->
   </ul>

Проверить микроформаты можно тут https://developers.google.com/structure ... ting-tool/
Например это без фикса

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

http://anderson.phpbbguru.net/viewtopic.php?f=5&t=61

и это

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

https://area51.phpbb.com/phpBB/viewtopic.php?f=99&t=47746

Ну и как итог для примеров вид раз и два
А вот с фиксом

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

http://forum.volnistye.ru/viewtopic.php?t=6273

Фикс сделан неделю назад, но уже можно видеть результат.
p.s. На домене forum.volnistye.ru сейчас идет обратная миграция с seo url и соответственно происходит переиндексация. Поэтому не удивлятся наличию и дефолтных и seo url в серпе.
pp.ss кому не ясно про что я, то лучше не спрашивать тут а читать там https://developers.google.com/structure ... readcrumbs
ppp.sss на офе мне так и не удалось найти информации которая проясняла бы почему сделано именно так. И вообще никто эту проблему вроде как и не поднимал. Есть тема на https://area51.phpbb.com/phpBB/viewtopi ... 7&start=10 но она вроде как пылью покрылась.

[xisp]

Всё больше убеждаюсь, что разрабы phpBB не умеют в СЕО.

[LavIgor]

Предлагаемый рабочий фикс

А пробовали просто у itemscope="" стереть =""?
Заметил, что, кроме отказа от DEFINE, фикс отличается именно этим.

[владимир1983]

отличается именно этим

Не только. Если присмотришься найдешь ещё отличия). Нет например обязательного параметра itemprop="url". Есть ещё отличия)

[LavIgor]

Есть ещё отличия)

Точно, прошу прощения.
Но всё равно, я думаю, что DEFINE можно оставить, добавив остальные правки. Буковок поменьше.

[владимир1983]

я думаю, что DEFINE можно оставить

Тот кто понимает как работает DEFINE $VAR и что можно передать такой переменой напишет свой вариант без труда глядя пример валидного кода что я привел.
На мой взгляд код в том виде что я его привел более понятен-читабелен большему количеству людей.

[COB16]

владимир1983, это только к стандартному стилю относиться?

[владимир1983]

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

[COB16]

владимир1983, у меня вобще не находит

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

<nav role="navigation">
	<div class="inventea-posts-menu">
		<ul class="inventea-menu" role="menubar">
			<!-- EVENT navbar_header_quick_links_before -->

			<!-- IF S_DISPLAY_SEARCH -->
				<!-- IF S_REGISTERED_USER --><li><a href="{U_SEARCH_SELF}" role="menuitem">{L_SEARCH_SELF}</a></li><!-- ENDIF -->
				<!-- IF S_USER_LOGGED_IN --><li><a href="{U_SEARCH_NEW}" role="menuitem">{L_SEARCH_NEW}</a></li><!-- ENDIF -->
				<!-- IF S_LOAD_UNREADS --><li><a href="{U_SEARCH_UNREAD}" role="menuitem">{L_SEARCH_UNREAD}</a></li><!-- ENDIF -->
				<li><a href="{U_SEARCH_UNANSWERED}" role="menuitem">{L_SEARCH_UNANSWERED}</a></li>
				<li><a href="{U_SEARCH_ACTIVE_TOPICS}" role="menuitem">{L_SEARCH_ACTIVE_TOPICS}</a></li>
			<!-- ENDIF -->

			<!-- EVENT navbar_header_quick_links_after -->
		</ul>
	</div>

	<div class="inventea-forum-menu">
		<ul class="inventea-menu" role="menubar">
			<!-- EVENT overall_header_navigation_prepend -->

			<li><a href="{U_FAQ}" rel="help" title="{L_FAQ_EXPLAIN}" role="menuitem">{L_FAQ}</a></li>
			<!-- IF S_DISPLAY_SEARCH --><li><a href="{U_SEARCH}" role="menuitem">{L_SEARCH}</a></li><!-- ENDIF -->

			<!-- IF not S_IS_BOT -->
				<!-- IF S_DISPLAY_MEMBERLIST --><li><a href="{U_MEMBERLIST}" role="menuitem">{L_MEMBERLIST}</a></li><!-- ENDIF -->
				<!-- IF U_TEAM --><li><a href="{U_TEAM}" role="menuitem">{L_THE_TEAM}</a></li><!-- ENDIF -->
			<!-- ENDIF -->

			<!-- IF U_MCP --><li><a href="{U_MCP}" title="{L_MCP_SHORT}" role="menuitem">{L_MCP}</a></li><!-- ENDIF -->
			<!-- IF U_ACP --><li><a href="{U_ACP}" title="{L_ACP_SHORT}" role="menuitem">{L_ACP}</a></li><!-- ENDIF -->

			<!-- EVENT overall_header_navigation_append -->
		</ul>
	</div>
</nav>

[владимир1983]

Значит код breadcrumb у тебя в другом файле стиля.
Ссылку на стиль дай.

[COB16]

владимир1983, https://www.phpbb.com/community/viewtop ... &t=2315976

[владимир1983]

В overall_header.html смотри.

[COB16]

владимир1983, нашел, строка 135

[rxu]

Дошли руки https://tracker.phpbb.com/browse/PHPBB3-14115

[rxu]

Пофиксено в phpBB 3.1.6-RC1 (будущем).