Skip to content

Conversation

alroniks
Copy link
Member

@alroniks alroniks commented Apr 2, 2023

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

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

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

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

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

N/A

@alroniks alroniks requested a review from sergant210 April 2, 2023 12:58
@sergant210
Copy link
Contributor

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

@alroniks
Copy link
Member Author

alroniks commented Apr 2, 2023

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

@alroniks
Copy link
Member Author

alroniks commented Apr 2, 2023

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

@sergant210
Copy link
Contributor

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

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

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

@alroniks
Copy link
Member Author

alroniks commented Apr 2, 2023

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

Это как?

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

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

@sergant210
Copy link
Contributor

Это как?

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants