Tasks
Tasks copied to clipboard
BlizD
Переработка метода ПолучитьМассивОписанияВозможныхКонтактов
добавленный код в метод ПолучитьМассивОписанияВозможныхКонтактов, сделанный коммитом https://github.com/BlizD/Tasks/commit/0b56b96c4adf2d8f5cf4e2b6794bb017b3b77378 необходимо перенести в общий модуль ВзаимодействияКлиентСерверПовтИспПереопределяемый.ДополнитьМассивОписанияВозможныхКонтактов
а вообще наверное надо сделать общий модуль ВзаимодействияКлиентСерверTasks или узВзаимодействияКлиентСервер и на него сослаться из ВзаимодействияКлиентСерверПовтИспПереопределяемый.ДополнитьМассивОписанияВозможныхКонтактов
Нет, конкретно эту доработку лучше так вынести, тем самым будет отделен код БСП от кода конфигурации tasks
и модуль ВзаимодействияКлиентСервер можно будет вернуть на поддержку
просто при беглом взгляде при сравнении tasks и БСП это что первое на глаза попалось, надо дальше еще смотреть что надо бы вынести в свои модули перед выполнением #117
Это про узОбщийМодульСервер или узОбщийМодульКлиент? Но надо тогда еще КлиентСервер иначе придется это писать в двух модулях и вызывать в зависимости от контекста приложения
Вообще надо позвать @BlizD и поинтересоваться стоит ли это или зря заморачиваемся
Думаю, пока не стоит с этим заморачиваться, код выделен "обрамлен", если надо будет вынести его в наши модули, тогда сделаем.
@pallid
кстати, зачем обрамлять, проект же версионируется?
В объектах, которые относятся к БСП, все же пока обрамляем, чтобы бы было легче обновить БСП. Что касается других мест, то если изменение проходит через pull request тогда комментарии не нужны. Но часть изменений приходит просто через CF файлы, поэтому там обрамление остается.
ну у бсп за частую есть переопределяемые модули для этого
Среда, 24 октября 2018, 14:13 +03:00 от Anton Ivanov [email protected]: @pallid
кстати, зачем обрамлять, проект же версионируется? В объектах, которые относятся к БСП, все же пока обрамляем, чтобы бы было легче обновить БСП. Что касается других мест, то если изменение проходит через pull request тогда комментарии не нужны. Но часть изменений приходит через просто CF файлы, поэтому там обрамление остается. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub , or mute the thread . -- с уважением, Василий Попов