Ошибки сниппета — Undefined variable out и Fatal error Call to a member function get() on null

Комментарии (5)

1. 

   Игорь Сухинин 21.01.2018 23:11 #

   Привет!
   
   

   Notice: Undefined variable: out in .../core/cache/includes/elements/modsnippet/12.include.cache.php on line 11

   
   Вообще говоря, это не ошибка. Это предупреждение о том, что переменная не была определена. Хорошим правилом считается предопределять все переменные. Ну или поменяйте правила вывода ошибок в error_reporting, исключив E_NOTICE, хотя все же лучше инициализировать переменную перед циклом foreach. Например, как-то так
   

   $out = '';
   foreach($arr as $id){
       $res = $modx->getObject('modResource', $id);
       $pagetitle = $res->get('pagetitle');
       $uri = $res->get('uri');
       $out .= '<a href="'.$uri.'">'.$pagetitle.'</a>     ';
   }

   
   

   Fatal error: Call to a member function get() on null in .../core/cache/includes/elements/modsnippet/12.include.cache.php on line 9

   
   Тут посложнее, это на самом деле критическая ошибка. Дело в том, что у Вас происходит попытка вызова метода get() к переменной, которая не является объектом: $res->get('pagetitle');
   
   А получается это так потому, что вот эта строчка $modx->getObject('modResource', $id); возвращает null в строгом соответствии документации api.modx.com/revolution/2.2/#getObject. По каким-то причинам данный ресурс не находится… И уже вопрос, почему он не находится. Там выше у Вас в коде какие-то смутные процедуры производятся. Я думаю, там и надо искать проблему.

   ---

   1. 

      Зуля 22.01.2018 07:46 #

      Здравствуйте, Игорь!
      Спасибо, что ответили.
      В TV sposobi (список множественный выбор) у меня так:
      

      @EVAL return $modx->runSnippet('pdoResources',array(
      'parents'=>4,
      'depth'=>5,
      'limit'=>0,
      'sortby'=>'{"pagetitle":"ASC"}',
      'tpl'=>'@INLINE [[+pagetitle]]==#[[+id]]#','outputSeparator'=>'||',
      'templates' => 4
      ));

      
      
      Если этот TV в чанке вывожу так:

      [[+tv.sposobi]]

      , то на сайте отображаются #5#||#8#||#9#||#6#||#7#, а должно, чтобы отображались ресурсы с их url-ами вместо этих # и ||.
      А при использовании:

      [[+tv.sposobi:showcategories]]

      , та ошибка фатал.

      ---

      1. 

         Игорь Сухинин 22.01.2018 09:36 #

         Зуля, смотрите, что получается в результате вычислений Вашего кода
         

         $str = str_replace('##', '|', $input);
         $str = str_replace('#', '', $str);
         $arr = explode('|', $str);

         
         Мы получаем массив такого вида
         

         Array
         (
             [0] => 5
             [1] => 
             [2] => 8
             [3] => 
             [4] => 9
             [5] => 
             [6] => 6
             [7] => 
             [8] => 7
         )

         
         Как видно из примера выше, каждый второй элемент массива — пустой.
         
         Во-первых, очевидно, что есть ошибка в Ваших заменах символов строки.
         
         Во-вторых, всегда имеет смысл проверять возвращаемые значения. Например, переменную $res стоит проверять на null, раз уж согласно документации данный метод может его возвращать. Таким образом, надо дополнить в цикле проверкой как-то так
         

         foreach($arr as $id){
             $res = $modx->getObject('modResource', $id);
         	if (is_null($res)) continue;	
             $pagetitle = $res->get('pagetitle');
             $uri = $res->get('uri');
             $out .= '<a href="'.$uri.'">'.$pagetitle.'</a>     ';
         }

         
         Я даже почти уверен, что если Вы дополните такой проверкой свой код, то ничего больше исправлять не придется, т.к. нулевые результаты будут уже игнорироваться… Но это не лучший подход. Все-таки надо исправить код так, чтобы в массив не попадали пустые элементы.

         ---

      2. 

         Павел Романов 22.01.2018 10:32 #

         Этот сниппет работает при вызове на странице самого ресурса.
         То есть если в шаблоне так: [[*sposobi]], то выводится:

         #5##8##9##6##7#

         и все работает.
         
         Если надо выводить в чанке pdoResources или аналога, то там TV обрабатываются иначе и выводится дополнительно разделитель ||.
         Соответственно, в сниппете надо их сперва убрать:
         

         <?php
         $str = str_replace('||', '', $input); //убираем
         $str = str_replace('##', '|', $input);
         $str = str_replace('#', '', $str);
         $arr = explode('|', $str);
         foreach($arr as $id){
             $res = $modx->getObject('modResource', $id);
             if(!$res) continue; //да, проверяем на наличие объекта
             $pagetitle = $res->get('pagetitle');
             $uri = $res->get('uri');
             $out .= '<a href="'.$uri.'">'.$pagetitle.'</a>     ';
         }
         return $out;

         ---

2. 

   Зуля 22.01.2018 14:12 #

   Добрый день, друзья!
   Спасибо вам за помощь! Вопрос решен!

   ---