95shuma / crm

3 stars 0 forks source link

Стандартизация кода #28

Closed kempl95 closed 4 years ago

kempl95 commented 4 years ago

Проблемы:

  1. Наименования таблиц doctor и hospitals_doctor отличаются от других
  2. Наименования переменных в моделях и репозиториях не стандартизированы согласно CamelCase стилю

Решение:

  1. Изменить наименования таблиц:

    • наименование таблицы doctor - изменить на doctors
    • наименование таблицы hospitals_doctor - изменить hospitals_doctors
  2. Поменять в Моделях и Репозиториях названия переменных. Например заменить “registration_type“ на ”registrationType”

Планируемое время: 20 мин

urmat-mederbekov commented 4 years ago

1:30 час заняло, так как мы не учли что надо и обьекты формы тоже менять, соответственно name атрибуты тоже, соответственно requestparam название тоже

kempl95 commented 4 years ago

Не выполнен пункт №1

kempl95 commented 4 years ago

Изменение таблиц может потребовать дополнительное время, так как указанные таблицы использованы в других таблицах, что может вызвать некоторое осложнения при их изменении.

urmat-mederbekov commented 4 years ago

потратил еще 15 мин на первый пункт

urmat-mederbekov commented 4 years ago

еще 10 мин изменений селектов