1) PSR. Названия функций - это одно из первых что увидит тот кто будет проверять. Они как минимум должны быть с маленькой буквы.
2) Отступы, а точнее их отсутствие. Самому то глаз не режет это? Я бы тоже не стал смотреть.
3) Мешанина из кода. Всех интересует знаешь и понимаешь ли ты что такое патерны и mvc а тут им и не пахнет даже.
Никакой фильтрации данных, подготовленных запросов, композера, фреймворков, шаблонов ООП, стандартов PSR, форматировании кода и т.д.
Мне не понять зачем это написано. Основные стандарты PSR соблюдаются в голове без какой либо постподготовки. Человек с опытом так просто не напишет.