Closed bagart closed 11 years ago
dovg
commented
11 years ago Комментарии не нужны, тем более на русском. В диффе есть мусор и местами не выдержан coding style. Реквест желательно разбить на три, по одному на каждое дополнение.
По существу: Мне не очевидна необходимость данных изменений и не мне не понятно какую проблему они решают. Но это ИМХО.
AlexeyDsov
commented
11 years ago и +1 к разбиению реквеста на несколько
suquant
commented
11 years ago Может быть имеется в виду разделение времени от микросекунд точкой ? Я что-то не совсем понимаю, зачем точка, если не так.
bagart
commented
11 years ago вот и я посчитал это странным и сделал как нужно было в обычной ситуации + настраеваемый в конфиге делимер для обратной совместимости. ЗЫ: для меня имеет значение не это, в проект при каждом вызове поставить строгий делимер. а вот с именованием индексов и сиквенсами (может чтобы не усложнять добавить в новые конфиги булевый define BD_SEQ_NEW_NAME?) я реально столкнулся с пересесениями и мне показалдось это диким :) новые имена индексы можно применять - не влияет на обратную совместимость, вопрос скорее в идеалогии. я использовал подход классов где id подразумевается и удаляются все чтобы было понятно на 1 взгляд какие колонки используются
AlexeyDsov
commented
11 years ago Кстати, диффы выглядят так как будто бы используются пробелы вместо табов. Глянул файлы. Так оно и есть.
dovg
commented
11 years ago Я закрою. Все, включая автора согласны, что в текущем виде это вливать не нужно.
минимальные изменения: настраиваемый дефолтовый делимер для Date и Timestamp (устанавливаю в конфиге) Form::create()->import()-> drop() может лучше clear() ради очистки списка ошибок
в индексах для БД $name.'idx' заменил $table$column.'idx' для адекватной сортировки, убрал (UserInfo(класс) = user_info(table) = userinfo(индекс)) и _id (для читабельности)
в сиквенсы добавил _id_seq (очеводно, придётся переименовывать все имеющиеся сиквенсы в БД)
из спорного абстрактная прослойка BusinessAuto и public function getById