АК
в плане производительности
Size: a a a
2019 December 19
AK
ну в целом да, с учётом того что для каждого элемента формы по отдельному шаблону идёт - ещё +1 будет не заметен на фоне общих тормозов ;)
AK
странно что подефалту не сделали это, ведь довольно часто нужно формы темизировать.
VP
Ну друпал же тогда каждый раз для каждой формы после прочистки кешей будет искать не один файл темы для каждой формы, а два. Хотя потом это закешируется и будет напрямую браться сразу нужный.
в любом случае будет использоваться только один, самый релевантный для конкретной страницы шаблон. Например, здесь используется (крестик)
X commerce-product--full.html.twig поскольку более специфичных шаблонов в данной теме не существует пока * (звёздочка)FILE NAME SUGGESTIONS:
* commerce-product--53--full.html.twig
* commerce-product--53.html.twig
* commerce-product--simple--full.html.twig
* commerce-product--simple.html.twig
x commerce-product--full.html.twig
* commerce-product.html.twig
AK
Vlad Proshin (drugan)
в любом случае будет использоваться только один, самый релевантный для конкретной страницы шаблон. Например, здесь используется (крестик)
X commerce-product--full.html.twig поскольку более специфичных шаблонов в данной теме не существует пока * (звёздочка)FILE NAME SUGGESTIONS:
* commerce-product--53--full.html.twig
* commerce-product--53.html.twig
* commerce-product--simple--full.html.twig
* commerce-product--simple.html.twig
x commerce-product--full.html.twig
* commerce-product.html.twig
использоваться-то да, но чтоб его найти - друпалу нужно сначала прошерстить все возможные папки на наличие файлов выше крестика по списку, а это довольно много io-операций
AK
Потом это конечно ляжет в кеш и будет браться сразу нужный, но до кеша - получим кучу лишних операций поиска файлов.
VP
использоваться-то да, но чтоб его найти - друпалу нужно сначала прошерстить все возможные папки на наличие файлов выше крестика по списку, а это довольно много io-операций
закатывать солнце вручную это не наша забота - Друпал с этим прекрасно сам справляется...
AK
Vlad Proshin (drugan)
закатывать солнце вручную это не наша забота - Друпал с этим прекрасно сам справляется...
да вот как раз надо понимать что он там и как делает, чтобы сайт не получился пошаговой стратегией. Чтобы найти файл темы - ему ведь не просто в 1 папку заглянуть надо, а ещё и по всем подпапкам пройтись в активной теме, а потом ещё и в модулях.
АК
Это называется преждевременная оптимизация
АК
Нужно сделать профайлинг и посмотреть, сколько ресурсов тратится на поиск шаблонов при сбросе кэша. И уже по результатам профайлинга делать выводы
VP
понимать надо не в чатах, а вот здесь ))
https://www.drupal.org/docs/8/api/cache-api
https://www.drupal.org/docs/8/api/cache-api
AI
Алексей Кузнецов
Это называется преждевременная оптимизация
+
AK
Ну мне кажется проще сразу добавить один "if" в хук, чтобы не засорять suggestions, чем потом когда сайт будет лагать - вставлять доп. кеширование и костыли для ускорения, благополучно забыв про этот отсутствующий "if"
AK
на рефакторинг и оптимизацию уже написанного кода обычно у клиентов нет ни времени ни доп денег ни желания, только когда уже совсем всё критично и плохо.
AK
Vlad Proshin (drugan)
понимать надо не в чатах, а вот здесь ))
https://www.drupal.org/docs/8/api/cache-api
https://www.drupal.org/docs/8/api/cache-api
ага, сначала пишешь как быстрее с надеждой на кеширование, а потом cache clear вываливает memory overflow :)
АК
Я вам скажу по секрету, что по той ссылке про кэш твига вообще ничего не написано
AK
Алексей Кузнецов
Я вам скажу по секрету, что по той ссылке про кэш твига вообще ничего не написано
ну а зачем нам кеш твига, если сразу уже готовый html лежит в page_cache :)
АК
Кто-то там хотел "онимать что он там и как делает"?
АК
Шаблон формы и готовый хтмл в page_cache - не вполне совместимые вещи
VP
Алексей Кузнецов
Я вам скажу по секрету, что по той ссылке про кэш твига вообще ничего не написано
здесь читал? https://www.drupal.org/docs/8/api/render-api