Разработчикам

Форум

Ошибка при обновлении php до 8

Пользователь 20753 Посетитель Сообщений: 30 Баллов: 4 Регистрация: 13.02.2008	#11 0 28.12.2022 00:40:33 Интересно сколько времени ещё пройдёт до тех пор пока кто-нибудь объяснит на примере как надо переделать код, чтоб рабочий код в php 7.4 оставался рабочим и в php 8...?

Пользователь 1441

Гуру

Сообщений: 1126 Баллов: 569 Регистрация: 23.03.2005

#12

28.12.2022 10:03:38

Цитата
написал: Интересно сколько времени ещё пройдёт до тех пор пока кто-нибудь объяснит на примере как надо переделать код, чтоб рабочий код в php 7.4 оставался рабочим и в php 8...?

Да чего объяснять то?
Я же дал ссылку.
Проблема основная одна - статически вызывается метод класса, в котором не указано static в коде.
Соответственно, проходимся по всем сторонним модулям и прописываем static.

студия Leveret

Пользователь 1149855

Заглянувший

Сообщений: 7 Баллов: 2 Регистрация: 27.07.2018

#13

28.12.2022 19:29:07

Цитата

написал:

Цитата
написал: Интересно сколько времени ещё пройдёт до тех пор пока кто-нибудь объяснит на примере как надо переделать код, чтоб рабочий код в php 7.4 оставался рабочим и в php 8...?

Проблема в том что поставив костыль, потом так и пойдешь с ним, и где ниб он вылезет опять, тогда зачем вообще битрикс и маркет и все модули, т.е. это не решено в самой системе. Переходить на php8 чтобы потом всё работало на костылях? отличный план!

https://php.watch/versions/8.0/non-static-static-call-fatal-error

А админ молчит, видно зубы лечит

Пользователь 12947

Постоянный посетитель

Сообщений: 111 Баллов: 14 Регистрация: 21.06.2007

#14

28.12.2022 20:23:09

Цитата
написал: я на всех сайтах использовал код приведенный здесь в документации Битрикса в качестве примера

Вероятно, Вы брали код когда-то давно. Сейчас сходил по указанной ссылке - код в примере имеет ключевое слово static в объявлении метода, всё ок.

Лично я не понимаю, почему разработчики Битрикса страдали этой глупостью ( не указывали у явно статических методов static, и при этом вызывали эти методы как статические ) более десятка лет, но имеем, что имеем.

Пользователь 12947

Постоянный посетитель

Сообщений: 111 Баллов: 14 Регистрация: 21.06.2007

#15

28.12.2022 20:24:09

Цитата
написал: Проблема в том что поставив костыль

Это не костыль. Это исправление бага, который раньше жить не мешал, а теперь мешает.

Пользователь 1149855

Заглянувший

Сообщений: 7 Баллов: 2 Регистрация: 27.07.2018

#16

30.12.2022 10:45:56

Цитата

написал:

Цитата
написал: Проблема в том что поставив костыль

Это не костыль. Это исправление бага, который раньше жить не мешал, а теперь мешает.

Я может не понимаю, но у меня 12 модулей при переводе на php 8 осталось работать только 3, но мне нужен весь комплект. Хорошо я поправил баг, при любом другом обновлении все начнется сначала. Хорошо отделю модули и вынесу отдельно, тогда модули перестанут обновляться.

По любому битриксоидам нужно решать это в общем комплексе и для всех, а не только предупреждать что с 1 февраля не будет поддержки ( ".... а кто не купит лотерею отключим газ")

Мое мнение такое, что сначала предупредить разработчиков - кто не обновит до поддержки php8 удалим с маркета, или разделить маркет до и маркет после (может кто то захочет остаться и не обновляться вообще, я знаю таких). И после этого клиентам - - - Обновитесь.

Вот логика по крайней мере моя, как учили.

Пользователь 108089

Заглянувший

Сообщений: 12 Баллов: 1 Регистрация: 21.11.2011

#17

01.01.2023 20:54:30

Цитата

написал:

Цитата
написал: Интересно сколько времени ещё пройдёт до тех пор пока кто-нибудь объяснит на примере как надо переделать код, чтоб рабочий код в php 7.4 оставался рабочим и в php 8...?

Добрый вечер, пытаюсь перейти на PHP 8.1 с решением от aspro

Код
function aspro_allcorp2(){

заменил на

Код
function __construct(){

ошибка в обновлении пропала, но сам сайт - белая страницы, в коде:

Код

var arBasketItems = <pre>[Error]
Non-static method CAllUser::GetID() cannot be called statically (0)
/public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php:3672#0: CAllcorp2::getBasketItems()   
/public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php:3662#1: CAllcorp2::checkBasketItems()   
/public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php:1711#2: CAllcorp2::SetJSOptions()   
/public_html/bitrix/templates/aspro-allcorp2/header.php:31#3: include_once(string)   
/public_html/bitrix/modules/main/include/prolog_after.php:107#4: require(string)   
/public_html/bitrix/modules/main/include/prolog.php:11#5: require_once(string)   
/public_html/bitrix/header.php:1#6: require(string)   
/public_html/index.php:2
----------</pre>

с 3667 строки в файле public_html\bitrix\modules\aspro.allcorp2\classes\general\CAllcorp2.php код:

Код

public static function getBasketItems(){      
global $APPLICATION, $arSite, $USER;
      CModule::IncludeModule('iblock');
      if(!defined('ADMIN_SECTION')){
         $userID = CUser::GetID();
         $userID = ($userID > 0 ? $userID : 0);
         $arBackParametrs = self::GetFrontParametrsValues(SITE_ID);
         $bOrderViewBasket = ($arBackParametrs['ORDER_VIEW'] == 'Y' ? true : false);
         if($bOrderViewBasket && isset($_SESSION[SITE_ID][$userID]['BASKET_ITEMS']) && is_array($_SESSION[SITE_ID][$userID]['BASKET_ITEMS']) && $_SESSION[SITE_ID][$userID]['BASKET_ITEMS']){
            $arIBlocks = $arBasketItemsIDs = array();
            foreach($_SESSION[SITE_ID][$userID]['BASKET_ITEMS'] as $arBasketItem){
               if(isset($arBasketItem['IBLOCK_ID']) && intval($arBasketItem['IBLOCK_ID']) > 0 && !in_array($arBasketItem['IBLOCK_ID'], $arIBlocks))
                  $arIBlocks[] = $arBasketItem['IBLOCK_ID'];
               $arBasketItemsIDs[] = $arBasketItem['ID'];
            }
            $dbRes = CIBlockElement::GetList(array(), array('IBLOCK_ID' => $arIBlocks, 'ID' => $arBasketItemsIDs, 'PROPERTY_FORM_ORDER_VALUE' => false), false, false, array('ID'));
            while($arRes = $dbRes->Fetch()){
               unset($_SESSION[SITE_ID][$userID]['BASKET_ITEMS'][$arRes['ID']]);
            }
            return $_SESSION[SITE_ID][$userID]['BASKET_ITEMS'];
         }
         return array();
      }
      return false;
   }

Не понимаю куда там прописать static...

Пользователь 1149855

Заглянувший

Сообщений: 7 Баллов: 2 Регистрация: 27.07.2018

#18

02.01.2023 10:15:32

У меня другие модули, но думаю правила одни. (могу ошибаться)

Не установлено соединение с сервером обновлений. [Ux11] Ошибка описания модуля "название.модуля"
При возникновении данной ошибки нужно либо обновить модуль до последней версии, либо в файле
/bitrix/modules/название.модуля/install/index.php заменить название функции function название_модуля() на function __construct()
После этого ошибка должна уйти

Необходимо найти код данного модуля по следующему пути (/bitrix/modules/название.модуля), определиться с классом в котором возникает данная ошибка и к данному классу приписать служебное слово static, примерно так: static function МетодКласса()

/bitrix/modules/название.модуля/include.php

Пользователь 108089

Заглянувший

Сообщений: 12 Баллов: 1 Регистрация: 21.11.2011

#19

03.01.2023 18:31:18

Цитата
Non-static method CAllUser::GetID() cannot be called statically (0)

Какой модуль смотреть?
GetID?

Пользователь 43953

Заглянувший

Сообщений: 9 Баллов: 1 Регистрация: 12.09.2012

#20

04.01.2023 00:36:46

Цитата

написал:

Цитата

написал:

Цитата
написал: Интересно сколько времени ещё пройдёт до тех пор пока кто-нибудь объяснит на примере как надо переделать код, чтоб рабочий код в php 7.4 оставался рабочим и в php 8...?

Добрый вечер, пытаюсь перейти на PHP 8.1 с решением от aspro

Код
function aspro_allcorp2 () {

заменил на

Код
function __construct () {

ошибка в обновлении пропала, но сам сайт - белая страницы, в коде:

Код

  var  arBasketItems = <pre>[Error]
Non- static  method CAllUser::GetID() cannot be called statically ( 0 )
/public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php: 3672  #0: CAllcorp2::getBasketItems()    
/public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php: 3662  #1: CAllcorp2::checkBasketItems()    
/public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php: 1711  #2: CAllcorp2::SetJSOptions()    
/public_html/bitrix/templates/aspro-allcorp2/header.php: 31  #3: include_once(string)    
/public_html/bitrix/modules/main/ include /prolog_after.php: 107  #4: require(string)    
/public_html/bitrix/modules/main/ include /prolog.php: 11  #5: require_once(string)    
/public_html/bitrix/header.php: 1  #6: require(string)    
/public_html/index.php: 2 
----------</pre>

с 3667 строки в файле public_html\bitrix\modules\aspro.allcorp2\classes\general\CAllcorp2.php код:

Код

  public   static    function   getBasketItems  () {       
 global   $APPLICATION ,  $arSite ,  $USER ;
      CModule::IncludeModule( 'iblock' );
       if (!defined( 'ADMIN_SECTION' )){
          $userID  = CUser::GetID();
          $userID  = ( $userID  >  0  ?  $userID  :  0 );
          $arBackParametrs  =  self ::GetFrontParametrsValues(SITE_ID);
          $bOrderViewBasket  = ( $arBackParametrs [ 'ORDER_VIEW' ] ==  'Y'  ?  true  :  false );
          if ( $bOrderViewBasket  &&  isset ( $_SESSION [SITE_ID][ $userID ][ 'BASKET_ITEMS' ]) && is_array( $_SESSION [SITE_ID][ $userID ][ 'BASKET_ITEMS' ]) &&  $_SESSION [SITE_ID][ $userID ][ 'BASKET_ITEMS' ]){
             $arIBlocks  =  $arBasketItemsIDs  =  array ();
             foreach ( $_SESSION [SITE_ID][ $userID ][ 'BASKET_ITEMS' ]  as   $arBasketItem ){
                if ( isset ( $arBasketItem [ 'IBLOCK_ID' ]) && intval( $arBasketItem [ 'IBLOCK_ID' ]) >  0  && !in_array( $arBasketItem [ 'IBLOCK_ID' ],  $arIBlocks ))
                   $arIBlocks [] =  $arBasketItem [ 'IBLOCK_ID' ];
                $arBasketItemsIDs [] =  $arBasketItem [ 'ID' ];
            }
             $dbRes  = CIBlockElement::GetList( array (),  array ( 'IBLOCK_ID'  =>  $arIBlocks ,  'ID'  =>  $arBasketItemsIDs ,  'PROPERTY_FORM_ORDER_VALUE'  =>  false ),  false ,  false ,  array ( 'ID' ));
             while ( $arRes  =  $dbRes ->Fetch()){
                unset ( $_SESSION [SITE_ID][ $userID ][ 'BASKET_ITEMS' ][ $arRes [ 'ID' ]]);
            }
             return   $_SESSION [SITE_ID][ $userID ][ 'BASKET_ITEMS' ];
         }
          return   array ();
      }
       return   false ;
   }

Не понимаю куда там прописать static...

Вам не static писать надо. Он пишет, что CUser::GetID() вызван статически, а должно быть

Код
global $USER; $USER->GetID();

То есть где то вот здесь /public_html/bitrix/modules/aspro.allcorp2/classes/general/CAllcorp2.php: 3672 есть такой вызов CUser::GetID() - вместо него должен быть вызов вида, который описан выше

Форум

Центр поддержки

Продукты

Управление сайтом

Битрикс24

Интернет-магазин + CRM

Решения

Для интернет-магазинов

Каталог готовых решений

Внедрение

Выбрать партнера

Проверить партнера

Стать партнером