pdoTools icon indicating copy to clipboard operation
pdoTools copied to clipboard

Upgrade Fenom to v3 with php 8 support

Open alroniks opened this issue 1 year ago • 6 comments

Что оно делает?

Обновляет зависимость для поддержки версии PHP от 8.0 и выше

Зачем это нужно?

Сейчас при использовании старой версии использование модификатора match роняет сайт с ошибкой 500

Связанные проблема(ы)/PR(ы)

N/A

alroniks avatar Apr 02 '23 12:04 alroniks

Я не знаю, что с этим делать? Автор Fenom пишет, что теперь PHP7 и ниже не поддерживается. Т.е. ломается обратная совместимость. Значит как минимум нужно поднимать мажорную версию. Мы с своё время решили, что pdoTools 2.х остается для MODX2, фиксим только вопросы безопасности. А для MODX3 делаем pdoTools 3 и её развиваем. Что делать?

sergant210 avatar Apr 02 '23 13:04 sergant210

@sergant210 в теории мы можем сделать без поднятия мажорной версии, но на modstore ограничить возможность ставить новую версию с php7, например

alroniks avatar Apr 02 '23 13:04 alroniks

Еще, как вариант, можно написать валидатор, который будет проверять версию php и не давать установить пакет

alroniks avatar Apr 02 '23 13:04 alroniks

Еще, как вариант, можно написать валидатор, который будет проверять версию php и не давать установить пакет

Думаю, это около 90% сайтов.

Наверно вторую версию нужно оставить как есть. Кому надо пусть подменяют файл фенома через расширение. Такая возможность заложена из коробки. Или как вариант, положить новые файлы Fenom 3 рядом и при инициализации pdoTools подключать соответствующую версию.

sergant210 avatar Apr 02 '23 14:04 sergant210

Кому надо пусть подменяют файл фенома через расширение. Такая возможность заложена из коробки.

Это как?

Или как вариант, положить новые файлы Fenom 3 рядом и при инициализации pdoTools подключать соответствующую версию.

Они в обоих случаях через композер ставятся, следовательно непонятно, как их разнести адекватно

alroniks avatar Apr 02 '23 15:04 alroniks

Это как?

Через системные настройки, где указываешь свой класс.

Они в обоих случаях через композер ставятся, следовательно непонятно, как их разнести адекватно

Костылем. Отдельную папку сделать.

sergant210 avatar Apr 03 '23 06:04 sergant210