miniShop2 icon indicating copy to clipboard operation
miniShop2 copied to clipboard

Работа ms2 с PHP8

Open biz87 opened this issue 3 years ago • 2 comments

Резюме / Summary

В целом нужно подготовить проект к работе с PHP8. Здесь буду вносить комментарии о найденных нюансах

biz87 avatar Oct 28 '21 12:10 biz87

Обязательный параметр принимается после необязательного

[2021-10-28 15:26:20] (ERROR @ /core/model/modx/modx.class.php: 1677) [OnMODXInit]
Deprecated: Required parameter $entry follows optional parameter $action in /core/components/minishop2/model/minishop2/minishop2.class.php on line 918

biz87 avatar Oct 28 '21 12:10 biz87

https://github.com/modx-pro/miniShop2/blob/master/core/components/minishop2/model/minishop2/minishop2.class.php#L918

php < 8.0

   public function orderLog($order_id, $action = 'status', $entry) 

php 8+

Варианты решений

 1.  public function orderLog($order_id, $entry, $action = 'status') // критическое изменение
 2.  public function orderLog($order_id, $action = 'status', $entry = 'здесь установить дефолтное значение') // менее критичное
 3. public function orderLog($order_id, $action, $entry) // тогда проверять $action на null
 4. Сделать отдельную функцию

CrazyBoy49z avatar Dec 09 '21 23:12 CrazyBoy49z