Процессор для breadcrumbs

Давно подумывал о замене стандартных компонентов breadcrumbs, процессоры MODX привлекают меня гораздо больше, да и когда работаешь в одном окружении (smarty), необходимость переключаться на другой синтаксис слегка нервирует. И вот это случилось. Представляю на ваш суд процессор для вывода цепочки родителей.

Процессор breadcrumbs.class.php я кинул в папку (modxsite)site/web

class modSiteWebBreadcrumbsProcessor extends modProcessor{

	public function initialize(){
		$this->setDefaultProperties(array(
			'startId'       => 0,
			'excludeDocs'   => array(),
			'showHidden'    => true,
			'showUnpub'     => false,
			'showDeleted'   => false
		));
		return parent::initialize();
	}

	public function process() {
		$bc_path = array();
		$resource = $this->modx->resource;

		//стоим в голове, нечего выводить
		if($resource->get('id')==$this->getProperty('startId')){
			return array(
				'success'   => true,
				'message'   => '',
				'object'    => array()
			);
		}

		//соберем цепочку родителей
		$r=$resource->toArray();
		unset($r['content']);
		$bc_path[]=$r;

	    while($resource = $resource->getOne('Parent')) {
		    if (in_array($resource->id, $this->getProperty('excludeDocs'))
			    || !$this->getProperty('showHidden') && $resource->hidemenu
			    || !$this->getProperty('showUnpub') && !$resource->published
			    || !$this->getProperty('showDeleted') && $resource->deleted
		    ){
			    continue;
			}

		    $r=$resource->toArray();
		    unset($r['content']);
		    array_unshift($bc_path,$r);

		    if($resource->id==$this->getProperty('startId'))
				break;
		}
		return array(
			'success'   => true,
			'message'   => '',
			'object'    => $bc_path,
		);
	}
}
return 'modSiteWebBreadcrumbsProcessor';

для вывода использовал шаблон

{$params=['startId'=>37]} {* это голова нужной ветки *}
{processor action='site/web/breadcrumbs' ns=modxsite params=$params assign=result}

<div class="breadcrumbs">
	{$total = count($result['object'])}
	{$counter = 0}
	{foreach $result.object as $object}
		{$counter=$counter+1}
		{if $counter<$total}
			<span itemscope="itemscope" itemtype="http://data-vocabulary.org/Breadcrumb">
				<a itemprop="url" rel="{$object.pagetitle}" href="{$object.uri}">
					<span itemprop="title">{$object.pagetitle}</span></a></span>
			/
		{else}
			<span itemscope="itemscope">{$object.pagetitle}</span>
		{/if}
	{/foreach}
</div>

Буду благодарен любым советам по доработке.

Comments

Саша, привет!

Прежде всего: не стоит использовать для этого getdata-процессор, правильней просто расширить modProcessor и написать свой метод process.

  1. Сорри, что докапываюсь до синтаксиса, но

$modx=$this->modx; $resource = $modx->resource;

Просто чтобы один раз вызвать modx,создаватьдлянеепеременную?Чемненравитсяпростоmodx, создавать для нее переменную? Чем не нравится просто this->modx? И, хотя это объекты и все равно между ними будут ссылки, но все-таки хотя бы для наглядности не забываем про амперсанты. modx = & this->modx; resource = & modx->resource;

  1. Посмотри микроразметку, которую формирует родной Breadcrumbs, там не все так просто. Есть стандарты, и лучше их придерживаться, а то СЕОшники помидорами закидают.

modx=modx=this->modx; resource=resource = modx->resource;

исправил, но при добавлении ссылки

$modx= & $this->modx->resource;

перестает выводиться сам ресурс (пустой контент)

Посмотри микроразметку

спасибо за информацию, сейчас посмотрю

перестает выводиться сам ресурс (пустой контент)

Кстати да :) Ведь ты ниже перетираешь в цикле переменную resource:)Поэтомуresource :) Поэтому modx = & this−>modx—оправдано,авотсthis->modx — оправдано, а вот с resource здесь не оправдано. Но где логика позволяет, лучше конечно использовать, так как нагрузку снижает, не делает копию инстанса.

ОК. Ты бы это сразу куда-нибудь на гист выложил, чтобы всегда актуальный код был и можно было ПРы слать.

Круть! При случае обязательно погоняю. Но процессор все равно подумай на счет перевода на обычный modProcessor.

подумай на счет перевода на обычный modProcessor

Да я так и думал сначала, но потом решил, что зачем велосипед изобретать — есть процессор, который собирает все данные ресурсов. Кто знает — может, понадобится когда картинками из TV крошки отрисовать? :)

getdata-процессор рассчитан только на выборку документов за раз, он не рассчитан на выборку внутри него циклами. То есть его общий механизм тут не применим, а вставить список id-шников в prepareQueryBeforeCount здесь не годится, так как в какой-то момент ты получишь не тот результат просто из-за сортировки. Ты же не можешь гарантировать, что крошки все будут иметь id по порядку, или заголовки или типа того. Тебе надо просто получить их данные в цикле и набить в массив, и это делается за раз в одной функции.

И ТВ-шки добавить не сложно.

получишь не тот результат просто из-за сортировки

ну, для этого я и переопределил afterIteration. Возможно, и кривовато, но данных немного… А в целом я согласен, переделаю. Просто это-первое, что пришло в голову :)

Да, так лучше и нагляднее. Я пока такие мелочи упускаю. Но в любом случае, думаю, польза от него будет. По крайней мере, вывод теперь полностью под контролем :)