Владимир

На мой взгляд, это просто разные представления одного и того же. Если мысленно наложить прозрачное изображение слоев модели Three Tier на MVC, то у нас в слое данных окажется кусок модели отвечающий за запросы в базу данных, их структуру и т.п., в слое логики оставшийся кусок модели+контроллер, обеспечивающие бизнес-логику и все, что с этим связано, а в слое клиента - представление.

По поводу самой ошибки вам уже подсказали, хочу только подчеркнуть один момент: передавая внутрь функции значения глобальных и суперглобальных переменных в явном виде, вы поступаете правильно. Делайте так всегда и игнорируйте тех, кто говорит, что это - ошибка, т.к. они доступны везде. Это сэкономит вам кучу времени и нервов. И тем, кто будет поддерживать ваш код после вас тоже.