miniShop2 icon indicating copy to clipboard operation
miniShop2 copied to clipboard

Отказ от старых версий php

Open CrazyBoy49z opened this issue 4 years ago • 9 comments

Запрос функции / Feature request

Резюме / Summary

Отказаться от версий php 5.3, 5.4, 5.5

Зачем это нужно / Why is it needed?

На дворе 2021 год, и думаю уже поддерживать эти версии не нужно, а уже есть версия 8.0.1. Что не устраивает MODX через поддержку старых версий очень отстает. Так как в минишопе можно было сделать рефакторинг кода или перенести уже в 3?

CrazyBoy49z avatar Feb 12 '21 11:02 CrazyBoy49z

Конечно стоит отказаться) В документации docs.modx.com в требованиях: image

meshkov avatar Feb 12 '21 15:02 meshkov

Без нормальной статистики трудно сказать. Тут только если просить Леню как-то собрать ее хотя бы с modxhost, на каких версиях там все работает, в разрезе, где установлен minishop2. И я не знаю, собирает ли modstore статистику по версиям php в момент установки пакета. Но можно попробовать узнать.

alroniks avatar Mar 16 '21 21:03 alroniks

Без нормальной статистики трудно сказать. Тут только если просить Леню как-то собрать ее хотя бы с modxhost, на каких версиях там все работает, в разрезе, где установлен minishop2. И я не знаю, собирает ли modstore статистику по версиям php в момент установки пакета. Но можно попробовать узнать.

Здесь соглашусь потому и создал issue c статусом обсуждения

CrazyBoy49z avatar Mar 19 '21 09:03 CrazyBoy49z

Как нашёл в коде Minishop не полностью работает на php 5.3 вопрос удаляем поддержку от него или восстанавливаем поддержку

https://github.com/Ibochkarev/miniShop2/blob/master/core/components/minishop2/processors/mgr/orders/product/get.class.php#L8

https://github.com/Ibochkarev/miniShop2/blob/master/core/components/minishop2/controllers/resource_create.class.php#L65

https://github.com/Ibochkarev/miniShop2/blob/master/core/components/minishop2/controllers/resource_create.class.php#L70

CrazyBoy49z avatar Apr 01 '21 23:04 CrazyBoy49z

Предлагаю пока что придерживаться php 5.6, потому что есть прецеденты обращений, при попытке отказаться от этой версии. Все что ниже 5.6 удаляем. А есть какие то сервисы валидации кода по заданной версии языка?

biz87 avatar Apr 11 '21 08:04 biz87

Нашел сочетание PHP_CodeSniffer + PHPCompatibility https://github.com/PHPCompatibility/PHPCompatibility Нужно попробовать

biz87 avatar Apr 11 '21 08:04 biz87

Предлагаю пока что придерживаться php 5.6, потому что есть прецеденты обращений, при попытке отказаться от этой версии. Все что ниже 5.6 удаляем. А есть какие то сервисы валидации кода по заданной версии языка?

Понял если буду встречать такой код буду обновлять

CrazyBoy49z avatar Apr 11 '21 16:04 CrazyBoy49z

@biz87 Есть такая штука https://github.com/rectorphp/rector, он умеет даже текущий код приводить к заданной версии языка

alroniks avatar Apr 11 '21 18:04 alroniks

@biz87 Есть такая штука https://github.com/rectorphp/rector, он умеет даже текущий код приводить к заданной версии языка

Спасибо интересная штука

CrazyBoy49z avatar Apr 11 '21 20:04 CrazyBoy49z