Forums / National / Russian / Модули и плагины / User Manager Plugin

User Manager Plugin для Siena

Dayver
#39160 2014-02-05 21:30
#39159 Kabak:

Да, глупость...  это чем грозит ?

Мне кажется это не нужное усложнение - использовать отдельную таблицу когда можно обойтись 4-мя доп. полями. Потому судя по всему вам иногда приходится делать выборку из вашей таблицы в то время когда достаточно было бы использовать данные из стандартной таблицы пользователя. Ничего критического но на будущее как разработчику наверное стоит учесть.

это чем грозит ?

Тем что ссылки которые прописаны жестко в коде, не смогу быть обработанны url менеджером который превращает ссылки в ЧПУшные, ничего от этого не сломается конечно но если кто то захочет прописать ЧПУшные правила и с учетом вашего плагина то спотыкнется об эту жесткую привязку. А то что html в пхп коде, а не в отдельном ресурсом файле просто усложнит изменение если кто то что то захочет поменять в плане html разметки (просто нарушается концепция, а так ничего страшного).

Pavlo Tkachenko aka Dayver