User Manager Plugin для Siena
Dayver |
|
---|---|
#39159 Kabak: Мне кажется это не нужное усложнение - использовать отдельную таблицу когда можно обойтись 4-мя доп. полями. Потому судя по всему вам иногда приходится делать выборку из вашей таблицы в то время когда достаточно было бы использовать данные из стандартной таблицы пользователя. Ничего критического но на будущее как разработчику наверное стоит учесть.
Тем что ссылки которые прописаны жестко в коде, не смогу быть обработанны url менеджером который превращает ссылки в ЧПУшные, ничего от этого не сломается конечно но если кто то захочет прописать ЧПУшные правила и с учетом вашего плагина то спотыкнется об эту жесткую привязку. А то что html в пхп коде, а не в отдельном ресурсом файле просто усложнит изменение если кто то что то захочет поменять в плане html разметки (просто нарушается концепция, а так ничего страшного). Pavlo Tkachenko aka Dayver
|