Jump to content
Search In
  • More options...
Find results that contain...
Find results in...
  • Sign Up

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);
            }
        }
    }

}

 

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
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);
            }
        }
    }

}

 

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

Link to post
Share on other sites

34 минуты назад, pinkmandarin сказал:

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

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

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

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

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

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

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

Link to post
Share on other sites

1 минуту назад, pinkmandarin сказал:

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

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

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

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

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

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

 

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

Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...

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.