[pdoTools] 2.10.1 - исправление кэширования скриптов и стилей
Давненько я не писал про обновления pdoTools, ведь там не происходило ничего примечательного — обычная работа над некритичными ошибками и улучшения функционала.
Но сегодняшняя версия стоит отдельного объявления, ведь в ней наконец-то исправлена работа кэширования скриптов и стилей! За отличное расследование и исправление благодарим Сергея Шлокова, а я немного расскажу, как эта ошибка вообще появилась.
Дело в том, что изначально в ранних версиях pdoTools вызывать сниппеты можно было только одним методом — $_modx->runSnippet() и он напрямую вызывал метод modX::runSnippet().
Вызовы Fenom работают уже после того, как MODX парсит кэшированные тэги, поэтому все скрипты и стили, зарегистрированные через Fenom, не попадали в кэш ресурса. Чтобы исправить это, я запихивал в этот кэш вручную всё что было, после каждого запуска Fenom.
Это было крайне неудачным решением, но другого на тот момент я придумать не смог. Так что, с добавлением еще пары костылей оно более менее работало.
Но дело в том, что со временем запуск сниппетов через Fenom был переписан, и все вызовы давно шли через метод pdoTools::runSnippet(), что позволяло, в принципе, контролировать регистрацию скриптов и стилей кэшированных сниппетов — да никто об этом не подумал, кроме Сергея.
Так что в новой версии убраны все кривые костыли, а обработка скриптов и стилей кэшированных сниппетов, запускающихся через Fenom любым способом, происходит в одном его методе. И проблем с кэшированием больше нет.
Еще некоторые изменения:
— Улучшено регулярное выражение для определения синтаксиса Fenom. Теперь ситуаций, когда в чанке один просто тег и парсер не считает его достойным обработки, должно стать гораздо меньше. А ложных срабатываний, надеюсь, не прибавится.
— Пока проверял работу кэширования скриптов и стилей, заметил баг в pdoPage. Если ему указать &cache=`1`, то он перестаёт регистрировать канонические url со второй загрузки. Исправлено.
— Сами канонические url теперь всегда прописываются полным адресом вместе с доменом, независимо от настроек, как то советует Google.
— А pdoMenu теперь должен уметь использовать &conditionalTpls для чанка по умолчанию. Не знаю, насколько это полезно при наличии Fenom, но просили добавить.
— Еще добавили новую системную настройку pdotools_fenom_save_on_errors, которая сохраняет шаблоны при ошибках компиляции в core/cache/pdotools/error/, чтобы вам удобнее было их дебажить.
Остальные изменения можно посмотреть на GitHub.
Новая версия уже в репозиториях, можно обновляться!
Но сегодняшняя версия стоит отдельного объявления, ведь в ней наконец-то исправлена работа кэширования скриптов и стилей! За отличное расследование и исправление благодарим Сергея Шлокова, а я немного расскажу, как эта ошибка вообще появилась.
Дело в том, что изначально в ранних версиях pdoTools вызывать сниппеты можно было только одним методом — $_modx->runSnippet() и он напрямую вызывал метод modX::runSnippet().
Вызовы Fenom работают уже после того, как MODX парсит кэшированные тэги, поэтому все скрипты и стили, зарегистрированные через Fenom, не попадали в кэш ресурса. Чтобы исправить это, я запихивал в этот кэш вручную всё что было, после каждого запуска Fenom.
Это было крайне неудачным решением, но другого на тот момент я придумать не смог. Так что, с добавлением еще пары костылей оно более менее работало.
Но дело в том, что со временем запуск сниппетов через Fenom был переписан, и все вызовы давно шли через метод pdoTools::runSnippet(), что позволяло, в принципе, контролировать регистрацию скриптов и стилей кэшированных сниппетов — да никто об этом не подумал, кроме Сергея.
Так что в новой версии убраны все кривые костыли, а обработка скриптов и стилей кэшированных сниппетов, запускающихся через Fenom любым способом, происходит в одном его методе. И проблем с кэшированием больше нет.
Другие изменения
Еще некоторые изменения:
— Улучшено регулярное выражение для определения синтаксиса Fenom. Теперь ситуаций, когда в чанке один просто тег и парсер не считает его достойным обработки, должно стать гораздо меньше. А ложных срабатываний, надеюсь, не прибавится.
— Пока проверял работу кэширования скриптов и стилей, заметил баг в pdoPage. Если ему указать &cache=`1`, то он перестаёт регистрировать канонические url со второй загрузки. Исправлено.
— Сами канонические url теперь всегда прописываются полным адресом вместе с доменом, независимо от настроек, как то советует Google.
— А pdoMenu теперь должен уметь использовать &conditionalTpls для чанка по умолчанию. Не знаю, насколько это полезно при наличии Fenom, но просили добавить.
— Еще добавили новую системную настройку pdotools_fenom_save_on_errors, которая сохраняет шаблоны при ошибках компиляции в core/cache/pdotools/error/, чтобы вам удобнее было их дебажить.
Остальные изменения можно посмотреть на GitHub.
Новая версия уже в репозиториях, можно обновляться!
22 октября 2017, 10:25
Комментарии: 79
при обновлении зависание страницы и ошибка 500, сайт и админка в ауте
здесь подробнее описал modx.pro/help/13592/
здесь подробнее описал modx.pro/help/13592/
Добрый день!
Установил pdoTools.
Теперь выходит ошибка на сайте — HTTP ERROR 500.
В логах сервера показывает ошибку —
Установил pdoTools.
Теперь выходит ошибка на сайте — HTTP ERROR 500.
В логах сервера показывает ошибку —
Backend fatal error: PHP Fatal error: Cannot use [] for reading in /var/www/vhosts/site.ru/httpdocs/core/components/pdotools/model/pdotools/pdotools.class.php on line 373\nВерсия PHP — 5.4.16
Василий, спасибо огромное! Исчезли дубли каноникалов, про которые писали здесь modx.pro/help/12465/#comment-89059. Еще бы научится заставлять pdoPage делать каноникал в нижнем регистре, почему-то когда в файле snippet.pdopage.php делаю
$canurl = mb_strtolower($canurl);$modx->regClientStartupHTMLBlock('<link rel="canonical" href="' . $canurl . '"/>');
Нет конечно, проблема не в нем, какой url в адресной строке, такой он и использует. Просто если самому написать урл с большой буквы, то такой же каноникал и вставится в страницу. У меня из-за дублирования каноникала возникли проблемы с индексированием, описывал здесь modx.pro/help/12465/#comment-89059
И хотя теперь, после обновления pdoTools дублей быть не должно, в качестве превентивной меры хочу сделать принудительное приведение url к нижнему регистру, но не получается. Вот я и спрашиваю совета, как такое лучше сделать?
И хотя теперь, после обновления pdoTools дублей быть не должно, в качестве превентивной меры хочу сделать принудительное приведение url к нижнему регистру, но не получается. Вот я и спрашиваю совета, как такое лучше сделать?
пользуйтесь на здоровье
case 'OnHandleRequest':
// Приведение к нижнему регистру
$request = $_REQUEST;
$url = $request['q'];
if (!empty($url)) {
unset($_GET['q']);
$params = http_build_query($_GET);
if (strlen($params)) {
$params = '?' . $params;
}
if (preg_match('/[A-Z]/', $url)) {
$modx->sendRedirect(strtolower($url).$params);
}
}
break;
Кстати, ещё одно нововведение осталось незамеченным, которое появилось в версии 2.9.3 — возможность сохранять код в кэше в случае ошибки компиляции Fenom. Только сегодня выловил у себя ещё одну. Когда страниц много — хрен поймёшь где она. А щас открыл кэш и посмотрел. Мне кажется, было бы неплохо в статье это отменить.
Не знаю, у одного меня или нет. Но после обновления до последней pl2 версии. Рабочий сайт стал содержать в логах много ошибок:
Визуально причем вроде как все работает. Как выяснить на какое именно место ругается?
Кеш чистил…
[2017-10-24 08:33:17] (ERROR @ /var/www/data/www/poster/core/components/pdotools/model/pdotools/pdotools.class.php : 974) Unexpected tag 'id' in 7e3b12d6e990bd3d8307caca45f688c7 line 1, near '{id' <- thereВизуально причем вроде как все работает. Как выяснить на какое именно место ругается?
Кеш чистил…
Оказалось что ругань идет на
причем как выше Володя писал, это pdoTools выдергивает из источника файлов. Если бы он не трогал источник файлов, то все бы было нормально
[[migxResourceMediaPath? &pathTpl=`assets/MediaGallery/{id}/` &createFolder=`1`]]{id}причем как выше Володя писал, это pdoTools выдергивает из источника файлов. Если бы он не трогал источник файлов, то все бы было нормально
Понятно что сработает. Но это не выход.
Предполагается что все настройки медиасорс должны быть корректны для fenom.
И если это не так — идти их исправлять. Не всегда это вариант.
Как вариант решения этого можно ввести настройку ограничивающую обработку контекста в github.com/bezumkin/pdoTools/blob/master/core/components/pdotools/model/pdotools/pdoparser.class.php#L43
если такой вариант приемлим — я могу сделать pr.
Спасибо!
Предполагается что все настройки медиасорс должны быть корректны для fenom.
И если это не так — идти их исправлять. Не всегда это вариант.
Как вариант решения этого можно ввести настройку ограничивающую обработку контекста в github.com/bezumkin/pdoTools/blob/master/core/components/pdotools/model/pdotools/pdoparser.class.php#L43
если такой вариант приемлим — я могу сделать pr.
Спасибо!
Предполагается что все настройки медиасорс должны быть корректны для fenomОни обычно и так корректны, просто автор MIGX, зачем-то, использует фигурные скобки вместо квадратных, как в MODX принято.
И хозяин сайта включает использование Fenom везде, так что не вижу проблемы использовать его и в параметрах источника файлов в этом случае.
Как вариант решения этого можно ввести настройку ограничивающую обработку контекстаНу а если кто специально хочет использовать именно Fenom в настройках источника — им как быть?
В общем, посмотрим количество отзывов об этой проблеме.
Ну ок, в админке оно обрабатываться не будет, допустим. А на фронтенде разве настройки источника файлов не парсятся?
Если хоть один сниппет MIGX грузит этот источник и делает ему initialize() — будут ровно те же записи в логах, и всё равно придётся что-то менять: или регулярку синтаксиса, или настройки источника.
Я просто с MIGX не знаком, поясните, пожалуйста.
Если хоть один сниппет MIGX грузит этот источник и делает ему initialize() — будут ровно те же записи в логах, и всё равно придётся что-то менять: или регулярку синтаксиса, или настройки источника.
Я просто с MIGX не знаком, поясните, пожалуйста.
А на фронтенде разве настройки источника файлов не парсятся?в каком месте они там парсятся?
Если хоть один сниппет MIGX грузит этот источник и делает ему initialize() — будут ровно те же записи в логах, и всё равно придётся что-то менять: или регулярку синтаксиса, или настройки источника.согласен.
Я просто с MIGX не знаком, поясните, пожалуйста.я сам им не пользуюсь.
Я вообще конкретно MIGX в данном случае не касаюсь. Есть конкретная ситуация касаемо обработки настроек в админке парсером fenom и я лишь сказал о том что неплохо ввести ограничение на это и все.
Что с этим делать решать тебе.
По-моему такой вариант как в — 2.10.1-pl3 является оптимальным,
Fenom перестал детектить случаи когда фигурные скобки использовались в input-ах в атрибуте pattern — например
Просто если привыкаешь к Fenom, то уже само собой напрашивается его использование везде и повсюду и, порой, даже забываешь о местах где он не должен работать.
Сегодня обновил пакет на более чем 10 сайтах, полёт нормальный. Будем наблюдать дальше.
Fenom перестал детектить случаи когда фигурные скобки использовались в input-ах в атрибуте pattern — например
<input type="tel" pattern="8[0-9]{3}-[0-9]{3}">function(){}
//или
catch(err){}{googlemap}//так тег не работает
{googlemap }//так без проблемПросто если привыкаешь к Fenom, то уже само собой напрашивается его использование везде и повсюду и, порой, даже забываешь о местах где он не должен работать.
Сегодня обновил пакет на более чем 10 сайтах, полёт нормальный. Будем наблюдать дальше.
А на счёт работы Fenom в админке, Володин вариант сделать настройку для тех кому это очень надо, по-моему неплохая идея.Тем кому уж очень нужно могут создать плагин и отключить Fenom в админке
// Событие OnHandleRequest
if ($modx->context->key == 'mgr') $modx->getParser()->pdoTools->config['useFenom'] = false;
а если у меня весь сайт построен на тегах {mobile}{/mobile} и {desktop}{/desktop}, которые плагином добавляются, не обновляться теперь что ли? весь сайт покрашится?
по-моему проблема с источниками файлов такого не стоит вообще. Если человек включает феном на весь сайт, пофиксить пути в источнике должно быть его ответственностью, как ты и говорил
upd. кстати, ниже всплыл пример использования таких тегов в самом феноме — {ignore}{/ignore}. Крайне нужная штука порой.
по-моему проблема с источниками файлов такого не стоит вообще. Если человек включает феном на весь сайт, пофиксить пути в источнике должно быть его ответственностью, как ты и говорил
upd. кстати, ниже всплыл пример использования таких тегов в самом феноме — {ignore}{/ignore}. Крайне нужная штука порой.
по-моему это неправильно — это нативная работа феном вообще-то, почему ее нужно накостыливать? получается, Бруно ( который Мигас написал) как обычно без оглядки на всех делает по-своему, а мы из-за него убираем каноническую опцию из феном? Уж что-что, а ignore точно должен быть из коробки
по-моему это неправильно — это нативная работа феном вообще-то, почему ее нужно накостыливать?Потому что выше люди писали, что нативная работа, это когда можно указывать такие конструкции без боязни ошибки
<input pattern="\+7\s\(\d{3}\)\s\d{3}-\d{2}-\d{2}"/>Уж что-что, а ignore точно должен быть из коробкиВ фреймворках он работает без проблем. И если бы ты прочитал мою статью про тег ignore, ссылку на которую давали ниже, то ты бы понял, что Fenom не может работать нативно в MODX. Василий его более менее натянул, но специфика MODX ломает идеальную логику работы Fenom. И поэтому иногда возникают проблемы с выводом конструкции {тег}.
Если не понятно почему в фреймворках работает, а в MODX нет, могу объяснить поподробнее.
Уж что-что, а ignore точно должен быть из коробкиДобавил PR, который учитывает специфику парсинга MODX. Он решает проблему с ignore — где он бы не был указан (в шаблоне, ресурсе или чанке), он будет работать как от него и ожидается.
больше желчи :) я тут никого не подлавливаю, я про случаи когда надо заэкранить какой-нибудь js или css. В первых версиях игнор не справлялся с этим, потом стал работать. Что касается паттернов — тут я не спорю, но так как ими не пользуюсь никогда в связи с избирательной поддержкой браузерами этой фичи, то и не сталкивался с таким — ни подтвердить, ни опровергнуть не мог бы всё равно. Никакого подстёба в моих словах нет, только наблюдение на тему.
Не хотел отвечать — думал ты стебаешься. А сейчас думаю, ты скорее всего до конца не вник. Не имеет значения, что ты укажешь — паттерн или стили или еще чего с фигурными скобками. ignore не работает из-за специфики парсинга MODX. Попробуй добавить в примере выше (доступ я давал) в секцию HEAD следующий код
{ignore}<style>body {color: red}</style>{/ignore}
Здравствуйте! Подскажите пожалуйста, в чем может быть проблема, не работает Fenom если:
1. На сайт добавлены в хед критические стили CSS (для оптимизации скорости)
2. Стоит добавить на сайт код гугл тег менеджер или аналитику и все ломается(
Вот что получается prntscr.com/h18wxu и prntscr.com/h18x4k, системные настройки prntscr.com/h18x9x
И так на нескольких сайтах, PDO последней версии
1. На сайт добавлены в хед критические стили CSS (для оптимизации скорости)
2. Стоит добавить на сайт код гугл тег менеджер или аналитику и все ломается(
Вот что получается prntscr.com/h18wxu и prntscr.com/h18x4k, системные настройки prntscr.com/h18x9x
И так на нескольких сайтах, PDO последней версии
спасибо большое помогло. Но возник такой момент теперь, на страничках, где не использовался феном, теперь вылазит на сайте prntscr.com/h1a6ex и соответственно не работает аналитика (сервис показывает ошибку)
спасибо). Пробовал ранее { пробел для критических CSS, но у меня там было скобок 100-200, я и бросил эту затею (только сейчас вспомнил что можно было автозаменой сделать))). Но возник такой момент теперь, на страничках, где не использовался феном, теперь вылазит на сайте ignore prntscr.com/h1a6ex и соответственно не работает аналитика (сервис показывает ошибку)
Спасибо)
Там написано «Самый просто вариант — указать в шаблоне кэшируемый тег [[*content]]» — у меня в шаблоне так и есть, но теги всеравно вылазят prntscr.com/h1aenk
Там написано «Самый просто вариант — указать в шаблоне кэшируемый тег [[*content]]» — у меня в шаблоне так и есть, но теги всеравно вылазят prntscr.com/h1aenk
пробел для критических CSS, но у меня там было скобок 100-200, я и бросил эту затеюДелается это просто — устанавливаешь modDevTools, в поиске этого компонента вбиваешь открывающую скобку "{" и в поле замены скобку с пробелом "{ ". Заменяешь все совпадения оптом. Всего эти действия занимают минуту от силы.
Всегда это делаю перед тем как включить Fenom на страницах.
По поводу migxResourceMediaPath
Вот такая запись у меня не заработала:
Вот такая запись у меня не заработала:
{'migxResourceMediaPath' | snippet : ['pathTpl' => 'assets/MediaGallery/{id}/', 'createFolder' => 1]}{$_modx->runSnippet('migxResourceMediaPath', ['pathTpl' => '/images/gallery/{id}{alias}/', 'createFolder' => 1])}
Есть один момент не понятный:
Вот такой вызов
В чем может быть дело?
По отладки вижу лимит 10 стоит, но я его не ставил
Дописал 'limit' => 0, но не помню чтобы по умолчанию раньше был лимит в 10
Вот такой вызов
{'!pdoResources' | snippet : [
'tpl' => '@INLINE
<!-- item -->
<div class="col-md-2 col-sm-4 item">
<a href="{$uri}" class="">
{$pagetitle}
</a>
<div class="crest"></div>
<div class="crest2"></div>
</div>',
'showLog' => 1
]}0.0002561: pdoTools loaded
0.0000691: xPDO query object created
0.0006971: Added selection of modResource: SQL_CALC_FOUND_ROWS `id`, `type`, `contentType`, `pagetitle`, `longtitle`, `description`, `alias`, `link_attributes`, `published`, `pub_date`, `unpub_date`, `parent`, `isfolder`, `introtext`, `richtext`, `template`, `menuindex`, `searchable`, `cacheable`, `createdby`, `createdon`, `editedby`, `editedon`, `deleted`, `deletedon`, `deletedby`, `publishedon`, `publishedby`, `menutitle`, `donthit`, `privateweb`, `privatemgr`, `content_dispo`, `hidemenu`, `class_key`, `context_key`, `content_type`, `uri`, `uri_override`, `hide_children_in_tree`, `show_in_tree`, `properties`
0.0012379: Processed additional conditions
0.0017569: Added where condition: modResource.parent:IN(3,4,5,6,7,8,9,10,11,12,13,14,15), modResource.published=1, modResource.deleted=0
0.0001769: Sorted by modResource.publishedon, DESC
0.0000069: Limited to 10, offset 0
0.0004539: SQL prepared "SELECT SQL_CALC_FOUND_ROWS `modResource`.`id`, `modResource`.`type`, `modResource`.`contentType`, `modResource`.`pagetitle`, `modResource`.`longtitle`, `modResource`.`description`, `modResource`.`alias`, `modResource`.`link_attributes`, `modResource`.`published`, `modResource`.`pub_date`, `modResource`.`unpub_date`, `modResource`.`parent`, `modResource`.`isfolder`, `modResource`.`introtext`, `modResource`.`richtext`, `modResource`.`template`, `modResource`.`menuindex`, `modResource`.`searchable`, `modResource`.`cacheable`, `modResource`.`createdby`, `modResource`.`createdon`, `modResource`.`editedby`, `modResource`.`editedon`, `modResource`.`deleted`, `modResource`.`deletedon`, `modResource`.`deletedby`, `modResource`.`publishedon`, `modResource`.`publishedby`, `modResource`.`menutitle`, `modResource`.`donthit`, `modResource`.`privateweb`, `modResource`.`privatemgr`, `modResource`.`content_dispo`, `modResource`.`hidemenu`, `modResource`.`class_key`, `modResource`.`context_key`, `modResource`.`content_type`, `modResource`.`uri`, `modResource`.`uri_override`, `modResource`.`hide_children_in_tree`, `modResource`.`show_in_tree`, `modResource`.`properties` FROM `pmodx_site_content` AS `modResource` WHERE ( `modResource`.`parent` IN (3,4,5,6,7,8,9,10,11,12,13,14,15) AND `modResource`.`published` = 1 AND `modResource`.`deleted` = 0 ) ORDER BY modResource.publishedon DESC LIMIT 10 "
0.0008991: SQL executed
0.0001900: Total rows: 12
0.0002332: Rows fetched
0.0008690: Created inline "modChunk" with name "376a2e035e8b1e047dd42b7dca5e9d30"
0.0037138: Compiled Fenom chunk with name "modchunk/376a2e035e8b1e047dd42b7dca5e9d30"
0.0051739: Returning processed chunks
0.0106580: Total time
11 796 480: Memory usageВ чем может быть дело?
По отладки вижу лимит 10 стоит, но я его не ставил
Дописал 'limit' => 0, но не помню чтобы по умолчанию раньше был лимит в 10
К psoResources в доке так как ты говоришь написано, по умолчанию limit 10, а вот docs.modx.pro/components/pdotools/general-settings тут говорится в общих параметрах по умолчанию limit 0. Вот меня и сбило это столку. Ок понял спасибо
github.com/klkvsk/fenom-phpstorm-plugin вот плагин, он старенький и работает только подсветка, и немного глючный, но всё лучше чем ничего. Василий как-то предлагал подвсетку для смарти, но в этом случае всё в ошибках — всё-таки различия существенные. В этом плагине как раз синтаксис автоматически привязывается к файлам .fm
fenom — 0 ошибок
smarty — 3 ошибки.
Помимо extended он еще тильду не признает и выражения с вопросительным знаком, типа {if $a?} или {$a ?: $b}
Не спорю, сам плагин смарти гораздо лучше. Жаль что нормального под феном нет, постоянная краснота напрягает немного)
так что, смилостивишься добавить .fm? )
smarty — 3 ошибки.
Помимо extended он еще тильду не признает и выражения с вопросительным знаком, типа {if $a?} или {$a ?: $b}
Не спорю, сам плагин смарти гораздо лучше. Жаль что нормального под феном нет, постоянная краснота напрягает немного)
так что, смилостивишься добавить .fm? )
Слева Smarty, справа Fenom.
Второй, похоже, просто вообще не умеет детектить ошибки синтаксиса — вот их и нет. Что бы ты не писал — ему всё в порядке:
Я могу, конечно, добавить в pdoTools хоть какие расширения, но пусть хоть 5 человек об этом попросит. А то прям работа под заказ получается — никому не нужно, кроме тебя.
Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.
Здесь упомянуты:
| Компонент | Текущая версия | Закачки |
| pdoTools | 2.13.2-pl от 02.09.2021 | 54 170 |
| Tickets | 1.13.0-pl от 10.03.2020 | 19 558 |
| mSearch2 | 1.14.11-pl от 02.05.2023 | 18 904 |
| modDevTools | 1.2.1-pl от 27.02.2016 | 3 111 |
7 часов назад
У вас есть баг при изменении кол-ва позиции в корзине
20 ноября 2024, 16:25
В сниппете rcv3_html достаточно отложить загрузку через setTimeout (хотя кто-то делает через onClick). Не думаю что мой вариант самый правильный и что...
19 ноября 2024, 10:51
Решил свою проблему через имя пользователя, но хотелось бы через права пользователя «Неограниченные права»
<?php
/**
* Системное событие OnMan...
19 ноября 2024, 09:09
Спасибо, тоже очень интерестное решение.
18 ноября 2024, 15:21
'where' => [
'Data.price:!='=>'0'
]
18 ноября 2024, 14:19
miniShop2.Order.add('extfld_delivery_price','100', function() {
miniShop2.Order.getcost();
})
Это вот работает, но чтобы увид...
18 ноября 2024, 10:11
Благодарю за ответы.
16 ноября 2024, 21:12
Спасибо. Работает.
