Skip to content
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

Upgrade php version to 8.3 #70

Open
CrazyTapok-bit opened this issue Nov 27, 2023 · 2 comments
Open

Upgrade php version to 8.3 #70

CrazyTapok-bit opened this issue Nov 27, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@CrazyTapok-bit
Copy link
Owner

Documentation

@CrazyTapok-bit CrazyTapok-bit added the enhancement New feature or request label Nov 27, 2023
@erickskrauch
Copy link

Задирая версию, вы лишь отрезаете от использования своей библиотеки разработчиков, кодовая база которых ещё не обновлена на последнюю версию языка. Насколько я могу видеть, текущая реализация всё ещё написана целиком и полностью на 8.1 и не использует никаких возможностей новых версий PHP. Вы можете быть принципиальным и добавить их. Но я бы хотел предложить обратное: понизьте минимальную версию обратно на 8.1 и сохраняйте её до тех пор, пока это не будет создавать для вас проблем поддержки.

@CrazyTapok-bit
Copy link
Owner Author

Частково ви маєте рацію, переважна більшість проекта зараз не має принципових змін з PHP 8.2, хоч має саме цю версію в залежностях. Зверніть увагу що частково проект підтримує readonly властивості, які доступні лише з >=8.2.

Наразі ми не можемо забезпечити повну підтримку readonly властивості для кожної сутності, оскільки перше значення має бути призначене в конструкторі класу в якому властивість оголошена, і відповідно не дозволяє зробити це з батьківському класі. Тобто стає питання або принципово змінювати підхід наповнення сутностей, або чекати виправлення цього багу (як на мою думку) в php. Якщо ви знаєте як це реалізувати, буду радий будь-якій допомозі 😉

Стосовно PHP 8.1. Ви можете користуватись бібліотекою tgWebValid <=4.1, всі вони цілком підтримують той самий функціонал що і : ^4.2 (який вже потребує php ^8.2). Але за можливості я б все ж радив оновитись до ^4.2.2 в якій був виправлений суттєвий баг

@CrazyTapok-bit CrazyTapok-bit self-assigned this Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants