Jump to content

Recommended Posts

Здравствуйте. Обнаружила ошибку в коде, но сама справится не могу, прошу помощи. Опенкарт русская сборка 2.3.0.2 (rs.6)

 

 

PHP Warning:  in_array() expects parameter 2 to be array, null given in ... /www/catalog/controller/extension/module/sms_alert.php on line 15

 

В строке 15 указан следующее: 

 

if (in_array($status, $this->config->get('sms_alert_processing_status'))) {

 

В целом выглядит так: 

<?php
class ControllerExtensionModuleSmsAlert extends Controller {
    public function index($route, $data) {
        
        if (isset($data[0]) && !empty($data[0])) {
            
            $this->load->language('extension/module/sms_alert');

            $this->load->model('extension/module/sms_alert');
            
            $order_id = $data[0];

            $status = $this->model_extension_module_sms_alert->getOrder($order_id);

            if (in_array($status, $this->config->get('sms_alert_processing_status'))) {

                $req = "http://sms.ru/sms/send?api_id=" . $this->config->get('sms_alert_id') . "&to=" . $this->config->get('sms_alert_tel') . "&text=".urlencode($this->language->get('text_order') . $order_id);
                file_get_contents($req);
                
                // тест запроса
                // $this->log->write($req);
            }
        }
    }

}

 

Share this post


Link to post
Share on other sites
1 час назад, pinkmandarin сказал:

В строке 15 указан следующее:  

 


if (in_array($status, $this->config->get('sms_alert_processing_status'))) {

 

 

чтобы быстро и просто ответить - сделайте вот так в данной строке - добавьте в начало проверки -

 

$this->config->get('sms_alert_processing_status') &&

то есть должно быть :

 

if ($this->config->get('sms_alert_processing_status') && in_array($status, $this->config->get('sms_alert_processing_status'))) {

 

!!!! со скобками не напутайте.

  • +1 1

Share this post


Link to post
Share on other sites
5 часов назад, vier сказал:

 

чтобы быстро и просто ответить - сделайте вот так в данной строке - добавьте в начало проверки -

 


$this->config->get('sms_alert_processing_status') &&

то есть должно быть :

 


if ($this->config->get('sms_alert_processing_status') && in_array($status, $this->config->get('sms_alert_processing_status'))) {

 

!!!! со скобками не напутайте.

 

Сделала. Получилось так: 

<?php
class ControllerExtensionModuleSmsAlert extends Controller {
    public function index($route, $data) {
        
        if (isset($data[0]) && !empty($data[0])) {
            
            $this->load->language('extension/module/sms_alert');

            $this->load->model('extension/module/sms_alert');
            
            $order_id = $data[0];

            $status = $this->model_extension_module_sms_alert->getOrder($order_id);

            if ($this->config->get('sms_alert_processing_status') && in_array($status, $this->config->get('sms_alert_processing_status'))) {

                $req = "http://sms.ru/sms/send?api_id=" . $this->config->get('sms_alert_id') . "&to=" . $this->config->get('sms_alert_tel') . "&text=".urlencode($this->language->get('text_order') . $order_id);
                file_get_contents($req);
                
                // тест запроса
                // $this->log->write($req);
            }
        }
    }

}

 

Я вижу, что в следующих строках добавился код, а что с этим дальше делать?

Share this post


Link to post
Share on other sites
34 минуты назад, pinkmandarin сказал:

Я вижу, что в следующих строках добавился код, а что с этим дальше делать?

всё. - пользоваться.

я не разработчик данного модуля, - я лишь указал на "пробелы" в коде.

Share this post


Link to post
Share on other sites
9 минут назад, vier сказал:

всё. - пользоваться.

я не разработчик данного модуля, - я лишь указал на "пробелы" в коде.

 Прошу прощения за свои ограниченные знания кода :oops:, но теперь просто так оставить и все? Все будет работать? 

Share this post


Link to post
Share on other sites
1 минуту назад, pinkmandarin сказал:

 Прошу прощения за свои ограниченные знания кода :oops:, но теперь просто так оставить и все? Все будет работать? 

ошибки не должно быть, а будет ли работать сам модуль и что он делать будет - это не ко мне.

Вы попробуйте - работает модуль, так как я даже не знаю, что это за модуль и что он делает - мне это не к чему.

Share this post


Link to post
Share on other sites
59 минут назад, vier сказал:

ошибки не должно быть, а будет ли работать сам модуль и что он делать будет - это не ко мне.

Вы попробуйте - работает модуль, так как я даже не знаю, что это за модуль и что он делает - мне это не к чему.

 

Благодарю от всей души! :eek:

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
You are posting as a guest. If you have an account, please sign in.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    No registered users viewing this page.

×

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.