Мы все делаем проекты на основе своего опыта и навыков - то что сейчас кажется сделанным идеально, через год нам же покажется говнокодом.
Объяснить что код плохой, объяснить что его сложно поддерживать сейчас а в дальнейшем будет еще сложнее, ....
сообщить клиенту о проблемах, которые в будущем станут еще больше.
Вы уверены, что вы со своим "не очень много опыта" сможете быстро и качественно переписать?
Зачем Вы используете $_GET и $_POST напрямую?
Зачем Вам в действии контроллера вообще $_GET?
Зачем Вы в виде так присваиваете переменные и подставляете в value?
Зачем Вы так заполняете модель?
$user->email=$_POST['EditUserAdmin']['email'];