Модификатор для system/library/request.php

[ElenaPr]

Привет!

Могу ли я без последствий изменить (модификатором) код в классе Request?

Т.е. нарушит ли это работу магазина? Читала, что ядро нельзя трогать

 

В поля БД попадают всевозможные пробельные символы (в частности, табуляция, которая не обрезается функцией mysql trim)

Для уже введенных данных использую

SELECT ......  WHERE TRIM('\t' FROM TRIM(model)) IN (...,...,....)

 

В system/library/request.php

было

$data = htmlspecialchars($data, ENT_COMPAT, 'UTF-8');

хочу сделать

$data = htmlspecialchars(trim($data), ENT_COMPAT, 'UTF-8');

 

class Request {
    public $get = array();
    public $post = array();
    public $cookie = array();
    public $files = array();
    public $server = array();
    
    /**
     * Constructor
     */
    public function __construct() {
        $this->get = $this->clean($_GET);
        $this->post = $this->clean($_POST);
        $this->request = $this->clean($_REQUEST);
        $this->cookie = $this->clean($_COOKIE);
        $this->files = $this->clean($_FILES);
        $this->server = $this->clean($_SERVER);
    }
    
    /**
     * 
     * @param    array    $data
     *
     * @return    array
     */
    public function clean($data) {
        if (is_array($data)) {
            foreach ($data as $key => $value) {
                unset($data[$key]);

                $data[$this->clean($key)] = $this->clean($value);
            }
        } else {
            $data = htmlspecialchars($data, ENT_COMPAT, 'UTF-8'); //$data = htmlspecialchars(trim($data), ENT_COMPAT, 'UTF-8');
        }

        return $data;
    }
}

 

[mpn2005]

Добрый день.

Менять можно, никто не запрещает.

Но нужно понимать, что это действует абсолютно на весь сайт. 

И чисто в теории - может где-то подрезать и нужные пробелы.

[ElenaPr]

Спасибо. Не буду рисковать. А так заманчиво.

[mpn2005]

19 минут назад, ElenaPr сказал:

Спасибо. Не буду рисковать. А так заманчиво.

Заменяйте на уровне нужных контроллеров.

Или только для нужных post переменных.

[nikifalex]

7 часов назад, ElenaPr сказал:

Спасибо. Не буду рисковать. А так заманчиво.

программист не получится с таким подходом

[ElenaPr]

13 часов назад, mpn2005 сказал:

Заменяйте на уровне нужных контроллеров.

Или только для нужных post переменных.

Да. Так и планирую