Alex

Используйте ORM если нет сложных запросов. Это решает большую часть проблем с инъекциями

JS подключили?

vm.roleSelectOptions.push({name: 'очистить', id: -1})

vm.onRoleChange(role){
...
if(role.id === -1) {
  defaultRole();
}
...
}

Добавлю к предыдущим ораторам.
1. Если там все криво косо и все кто ругает разработки предыдущих не врут. То переписывание может поднять скорость генерации контента. Но это легко проверить воспользовавшись
https://developers.google.com/speed/pagespeed/insi...
2. Как уже говорили на seo особо не влияет переписывание. Но кривое переписывание может только навредить.
Как потеря контента, так и разметки (современные cms обычно умеют подставлять спец теги и атрибуты (meta, alt...))
3. Так же головняки с переписыванием интеграций с 1C, с платежными системами.
Любое переписывание сайта с нуля может привести как к поднятию позиций так и к потере.

https://angularfirebase.com/lessons/infinite-virtu...
Я так полагаю то что нужно?