-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Переход на новую версию Twig #13
Comments
Уже отвечал на этот вопрос в задаче #11. Твиг 2+ - это лок на php 7.0, тогда как минимальная версия, с которой работает битрикс - это 5.6. Формально нам ничего не мешает переписать библиотеку и сделать ее совместимой с php 7 и twig 2+, но пока не до этого, к сожалению. План переработать библиотеку и выпустить обновленную версию есть, но вот со временем проблемы. Если у вас есть желание и возможности - вы можете помочь проекту. |
Переход на Twig 2 (или 3) невозможен в принципе: битрикс требует |
Друзья, приятные новости. Кажется переход на новую версию Twig становится вполне реален. Начиная с версии 20.5.393 (помечена на данный момент как beta) Bitrix позволяет использовать mbstring.func_overload=0. В качестве эксперимента, попробовал в форке https://github.com/lozunoff/tools.twig подключить Twig последний версии (на текущий момент это 3.0.5). Заменил названия старых классов Twig 1.X (без пространства имен) на новые классы Twig 3.X (с пространством имен), внес косметические правки в методы getCacheKey() и isFresh() и библиотека вполне себе заработала. С выводом кириллицы проблем нет. |
Отличная новость! @lozunoff, может быть тогда и PR заодно сделаете? |
@mmjurov отправил PR с имеющимися на данный момент изменениями, просьба посмотреть. |
Спасибо! Потребуется некоторое количество времени, чтобы проверить, а также внести несколько косметических правок |
Влил правки в ветку разработки версии 2.x библиотеки. |
Возможно безболезненно перейти на свежую версию Twig`a ?
The text was updated successfully, but these errors were encountered: