1. PHP / Говнокод #8461

    +160

    1. 1
    2. 2
    unset($this->session->data['order_id']);
$this->data['pap4_orderid'] = $this->session->data['order_id'];

    Не разбирался зачем сохраняется order_id, но в любом слусае - где логика?
    opencart, /catalog/controller/checkout/success.php, 13-14 строки

    Запостил: antongorodezkiy, 09 Ноября 2011

    Комментарии (12) RSS

    • ava Easy_proger 10.11.2011 00:14 # 0

      интересно кто минусанул ?

      в первой строчке ключ из массива будет удален и во второй строке будет сохранен null
      Ответить

      • ava istem 10.11.2011 00:24 # +1

        Да что вы вообще понимаете в PHP???
        Ответить

        • ava Easy_proger 10.11.2011 00:30 # −1

          видимо не чего 
          <?php
$array = array('order_id'=>"someFoo");
unset($array['order_id']);
echo $array['order_id'];
?>


          == "" ну не null, хорошо … смысл был в том, что запись логически неверная
          Ответить

          • ava istem 10.11.2011 00:36 # 0

            ...
            unset($array['order_id'])
            eггог_геpoгting($array['order_id']); // делов-то...
            :)
            Ответить

            • ava Easy_proger 10.11.2011 00:38 # 0

              да я не об этом … логика то где в коде ТС ?
              $this->data['pap4_orderid'] всегда будет =="" … 
              Ответить

              • ava istem 10.11.2011 00:40 # 0

                Вы тут походите, присмотритесь. #8461 покажется вам мелочью.
                Ответить

          • ava AxisPod 10.11.2011 10:35 # 0

            <?php
            $array = array('order_id'=>"someFoo");
            $element = &$array['order_id'];
            unset($array['order_id']);
            echo isset($array['order_id']);
            ?>
            Ответить

            • ava AxisPod 10.11.2011 10:42 # 0

              А нет, как-то иначе стало работать. Раньше при сохранении ссылки unset не удалял объект.
              Ответить

      • ava antongorodezkiy 10.11.2011 00:48 # 0

        Нет, не интересует.
        Миссией Говнокод.ру считаю предупреждение разработчиков, что-то типа крика "Не делайте так никогда!". И вопросы собственной кармы в этом деле находятся в конце очереди приоритетов, извините.
        Ответить

    • ava lucidfoxGovno 13.11.2011 18:16 # 0 

      $this->data['pap4_orderid'] = $this->session->data['order_id'];
unset($this->session->data['order_id']);
      нет говна
      автор ошибся
      Ответить

    • ava guest8 09.04.2019 11:57 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий