search

sadr0b0t / vkurse

Automatically exported from code.google.com/p/vkurse
0 stars 0 forks source link

Правки кода людьми, не отвечающими за данную часть проекта #58

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
В ревизии r347 пользователем euroaika был 
исправлен один из исходных файлов и 
пересобран VkurseAPI. Комментарий к ревизии: 
"Убраны баги, связанные с орфографией и 
англоязычными словами. Исправлен формат 
времени. Добавлена проверка на пустоту 
расписания с соответствующим выводом" - он 
относится к VkurseME, но никак не к тем правкам, 
которые были сделаны в API.
В model.Schedule было добавлено поле examTypeID, хотя 
оно должно находиться (и находится) в 
model.Lecture, что в принципе является грубой 
ошибкой нарушения структуры данных. Не 
говоря уж о том, что подобные изменения 
кода "посторонними" людьми без ведома 
отвечающей за этот код группы, могут 
привести к серьезным проблемам и ошибкам.

Я откатил изменения и восстановил эти 
файлы из ревизии r284.

Original issue reported on code.google.com by apx%phys...@gtempaccount.com on 21 Dec 2010 at 10:27

GoogleCodeExporter commented 9 years ago 
Просьба прокомментировать автора 
изменения для чего оно было нужно и к чему 
ведет его откат. Поломает ли это клиент JavaME, 
Android, XML-сервис, Web Interface? Какая версия сейчас 
у нас на новом сервере, с какой версие 
собраны последние версии мобильных 
клиентов и тд. и т.п.

Original comment by bender...@gmail.com on 21 Dec 2010 at 10:41

GoogleCodeExporter commented 9 years ago 
XML-сервис это не поломает, т.к. там и так 
использовалась старая версия (r284), потому 
что я не знал об этих изменениях и файл на 
сервере, соответственно, не обновил. С 
новым сервером аналогично - туда я пытался 
и пытаюсь залить старую версию.

Original comment by apx%phys...@gtempaccount.com on 21 Dec 2010 at 10:47

GoogleCodeExporter commented 9 years ago 
>В model.Schedule было добавлено поле examTypeID, хотя 
оно должно находиться (и находится) в 
model.Lecture

Я правильно понимаю, что добавление нового 
поля examType уже было отражено в базе данных и 
во VkurseAPI, в том числе в старой версии 
библиотеки, которая сейчас используется на 
обоих серверах? Т.е. examTypeId уже 
существтвовал в правильном месте в той 
версии библиотеки, которая сейчас работает 
на серверах, а это изменение добавило его 
еще и в неправильное?

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

В общем, нужно прояснить ситуацию.

Original comment by bender...@gmail.com on 21 Dec 2010 at 11:17

GoogleCodeExporter commented 9 years ago 
нет, неправильные изменения коснулись 
только файлов на svn, БД и XML на сервере никто 
не менял
возможно, кривую версию api успел забрать с 
svn и начать использовать кто-то из других 
групп, за них ответить не могу
сейчас на svn последние, исправленные, 
рабочие версии API и XML

Original comment by apx%phys...@gtempaccount.com on 21 Dec 2010 at 11:46