Дмитрий

global ${$arParams["FILTER_NAME"]};

Передача в функцию по ссылке убрана еще в 5.4

function insertData($arr, $names, $path) {
    $pathArr = explode('.', $path);
    $lastKey = array_pop($pathArr);
    foreach ($names as $lang => $value) {
        $branch = &$arr[$lang];
        foreach($pathArr as $pathPart) {
            if(!isset($branch[$pathPart])) {
                $branch = null;
                break;
            }
            
            $branch = &$branch[$pathPart];
        }
        
        if(!$branch) continue;

        if($lastKey === 'goods') {
            if(!isset($branch['goods'])) $branch['goods'] = [];
            $branch['goods'][] = $value;
        } else if (!isset($branch[$lastKey])) {
            $branch[$lastKey] = $value;
        }
    }
}

Разве что происками иудеев можно объяснить, потому что в самом запросе никаких упоминаний про день недели нет, и никаких причин не показывать субботы тоже нет. Код абсолютно корректный. Возможно вы не все показали и отфильтровывание происходит при переборе.

Экранировать кавычки
<?php echo addslashes($row['a_name']) ?>

php.net/manual/ru/function.addslashes.php

foreach ($arr as $ind=>$usersBlock) {
	$keys = [];
	foreach ($usersBlock as $user) {
		$key = $user['user_id'] . '#' . $user['date'];
		list($hours, $minutes, $seconds) = explode(':', $user['time']);

		$seconds += $minutes * 60 + $hours * 3600;

		if(!isset($keys[$key]))	{
			$keys[$key] = $user;
			$keys[$key]['seconds'] = $seconds;
		} else {
			$keys[$key]['seconds'] += $seconds;
		}
	}	

	$arr[$ind] = [];
	foreach ($keys as $key => $value) {
		$value['time'] = convertToTime($value['seconds']);
		unset($value['seconds']);
		$arr[$ind][] = $value;
	}
}

function convertToTime($seconds) {
	// Напишите конвертацию секунд в полноценное время сами
	return $seconds;
}

1. Нормально. ООП для того и создан, что метод с одним названием может быть в разных классах, но работать по своему, если это согласуется с общей логикой.

2. По хорошему модель должна просто описывать данные, в крайнем случае возвращать связанные с нею наборы данных, но не возвращать экземпляры сторонних классов

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

new ErrorView($data)

чтобы конструктор errorView возвращал представление по имени ошибки, а функции wrongLogin и wrongPwd были бы приватными.

4) А зачем там вообще die?

При использовании $this->$x в $x может содержаться имя свойства, например

echo $this->name

идентично

$x = 'name';
echo $this->$x;