Да просто такой код труднее поддерживать, отлаживать. У тебя код по получению данных из модели будет находиться в двух местах: контроллере и вьюхе. Если ты захочешь те же данные еще в одном partial'e использовать, то станет совсем плохо. Другим программистам такой код будет сложно поддерживать т.к. они не ожидают запросов внутри вьюх.
Как мне кажется, ничего смертельного не произойдет. Просто это bad practice.