Без текста ошибки никто не сможет помочь. К тому же, кроме добавления RewardSiteVisit, ещё надо собственно вызывать эту логику из User->CreateSession и User->UpdateSession (иначе она просто мёртвым грузом будет лежать).
В моём JSFiddle меня удивляет зависимость от настроек JQuery — если подключить его на load, то в кастомный bind код заходит уже после того, как твиттеровский ембед «улучшен» (картинка появляется), и context не нулловый. Но если onDomready, то заходит до улучшения картики, и падает из-за нулла.
А проблема эта не возникала раньше наверное потому, что твиттеровский widgets.js был по-другому написан. Других отличий от ранее работающего кода я не вижу.
По сути получается, что они тоже рекомендуют делать «if (!Function.prototype.bind) {», т.е. bind() вообще не переопределяется на всех современных браузерах.
Отличные наработки, спасибо большое за продолжение движения продукта! А будет ли реализован код миграции баз существующей последней версии? Корректное слияние разных типов топиков в один.
Только шаблонные? Спасибо, хотя я не на это надеялся, эх.
К тому же, по коду похоже, что хуки есть или вызываемый на каждый выводимый пункт в списке, или общий — но почему-то вызываемый только если тип вызова — не топики и не комменты? Я по-другому этот кусок не могу понять:
Я же говорил, что на этой неделе посмотрю. Посмотрел, сделал, выложил новую версию в каталог (будет доступна после модерации), и залил на GitHub.
Эта логика (игнорировать исходный ALT изображения) включится, если вы проставите true для параметра $config['ignore_description'] в файле config.php плагина.
Надеюсь, вам это будет полезно. По умолчанию эта логика, естественно, отключена.
Даже с точки зрения пользователя — если картинка не показывается, то лучше вывести её описание, чем повторять один и тот же заголовок заметки для каждой картинки.
Так же и для поискового движка (тут я не знаком с тем, как именно Яндекс с этим работает) — лучше конкретное описание картинки (которое, надеюсь, как-то относится к теме заметки и т.п.), чем повторение одного и того же заголовка в разных ипостасях. По идее, за такие бездумные повторения надо бы наказывать, и если Яндекс этого ещё не делает — то рано или поздно научится.
Дополню, что режим «если нет описания — в альте будет заголовок заметки» я добавил как вынужденную меру. На самом деле предпочтительнее всегда иметь описания.
Ага, понял. С точки зрения SEO (как я её понимаю) это не очень полезно, но в принципе такую фичу можно добавить, через флажок в настройках, это нетрудно.
Наверное, это из стремления сократить количество запросов на сервер. Вам решать, стоит ли изменять этот параметр.
А проблема эта не возникала раньше наверное потому, что твиттеровский widgets.js был по-другому написан. Других отличий от ранее работающего кода я не вижу.
PS Способность удивляться — признак молодости духа, рад за вас ;) А сайт учту, спасибо.
К тому же, по коду похоже, что хуки есть или вызываемый на каждый выводимый пункт в списке, или общий — но почему-то вызываемый только если тип вызова — не топики и не комменты? Я по-другому этот кусок не могу понять:
{if $aReq.sType == 'topics'} {include file='topic_list.tpl'} {elseif $aReq.sType == 'comments'} {include file='comment_list.tpl'} {else} {hook run='search_result' sType=$aReq.sType} {/if}Но было бы неплохо поправить ошибку в названии (и в теге этой статьи), нужно двойное «м».
Вопрос в том, как это сделать через CSS, или куда прописать?
Через CSS можно сделать такими правилами (вот фиддл):
table { border: solid 1px red; padding: 0; border-collapse: collapse; } td { border: solid 1px green; }Цвета я задал разные, чтобы было понятно, что схлопывание приводит к использованию бордюра клеток, а не таблицы.
А сами эти стили наверное разумно будет поместить в файл css/topic.css шаблона (раз оно к топикам относится).
Эта логика (игнорировать исходный ALT изображения) включится, если вы проставите true для параметра $config['ignore_description'] в файле config.php плагина.
Надеюсь, вам это будет полезно. По умолчанию эта логика, естественно, отключена.
Так же и для поискового движка (тут я не знаком с тем, как именно Яндекс с этим работает) — лучше конкретное описание картинки (которое, надеюсь, как-то относится к теме заметки и т.п.), чем повторение одного и того же заголовка в разных ипостасях. По идее, за такие бездумные повторения надо бы наказывать, и если Яндекс этого ещё не делает — то рано или поздно научится.
Дополню, что режим «если нет описания — в альте будет заголовок заметки» я добавил как вынужденную меру. На самом деле предпочтительнее всегда иметь описания.
Я постараюсь на следующей неделе посмотреть.