BiTel

Форум BiTel
bgbilling.ru     docs.bitel.ru     wiki.bitel.ru     dbinfo.bitel.ru     bgcrm.ru     billing.bitel.ru     bitel.ru    
Текущее время: 06 июл 2025, 04:31

Часовой пояс: UTC + 5 часов [ Летнее время ]




Начать новую тему Ответить на тему  [ Сообщений: 9 ] 
Автор Сообщение
 Заголовок сообщения: Изменение реквизитов на договоре
СообщениеДобавлено: 17 июн 2015, 15:21 
Не в сети
Клиент

Зарегистрирован: 07 мар 2012, 15:02
Сообщения: 932
Откуда: Воронеж
Карма: 35
Добавил на несколько договоров реквизиты в модуле Bill, потом попытался изменить дату закрытия одного из реквизитов на одном из договоров и... Все реквизиты со всех договоров перескочили на один договор, на котором я пытался изменить реквизит, и стали абсолютно одинаковыми! С остальных договоров реквизиты, соответственно, исчезли.

Раз уж мы начали смотреть код, то в классе AttributeManager нашёл такой код:
Код:
query = new StringBuilder((id <= 0) ? "INSERT INTO" : "UPDATE")
    .append(" bill_attribute_").append(this.mid)
    .append(" SET cid=?, name=?, value=?, start=?, end=?");
if (id <= 0)
{
    query.append(" WHERE id=").append(id);
}


Видно, что условие WHERE добавляются когда id <= 0, т. е. когда нужно вставлять новую запись, не обновлять существующую. Соответственно, добавление новых параметров тоже не работате.

Вот это точно поломалось после последнего обновления.

_________________

Клиент: вер. 6.2.873 / 04.12.2017 19:38:11
os: Windows 7; java: Java HotSpot(TM) 64-Bit Server VM, v.1.8.0_65
Сервер: вер. 6.2.1202 / 04.12.2017 19:39:21
os: Linux; java: Java HotSpot(TM) 64-Bit Server VM, v.1.8.0_91


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 17 июн 2015, 19:31 
Не в сети
Разработчик

Зарегистрирован: 08 ноя 2007, 01:05
Сообщения: 8343
Откуда: Уфа
Карма: 238
vkulakov писал(а):
Вот это точно поломалось после последнего обновления.


Мне всегда нравится эта фраза. в 80 % случаев это не так. Тут класс с 2012 года не трогали. Он либо всегда не работал или что-то не так, но обновление скорее всего никак не влияет


Сходу не нашел ошибки в коде, надо проверять. В оригинале такой код:
Код:
                                if (id <= 0)
            {
               ps = con.prepareStatement( "INSERT INTO bill_attribute_" + mid
                     + " (cid, name, value, start, end) " + " VALUES (?, ?, ?, ?, ?)" );
            }
            else
            {
               ps = con.prepareStatement( "UPDATE bill_attribute_" + mid
                     + " SET cid=?, name=?, value=?, start=?, end=?" + " WHERE id=?" );
               ps.setInt( 6, id );
            }

            ps.setInt( 1, cid );
            ps.setString( 2, name );
            ps.setString( 3, value );
            ps.setDate( 4, TimeUtils.convertCalendarToSqlDate( start ) );
            ps.setDate( 5, TimeUtils.convertCalendarToSqlDate( end ) );

            ps.executeUpdate();

            ps.close();


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 17 июн 2015, 19:36 
Не в сети
Разработчик

Зарегистрирован: 08 ноя 2007, 01:05
Сообщения: 8343
Откуда: Уфа
Карма: 238
stark писал(а):
vkulakov писал(а):
Вот это точно поломалось после последнего обновления.


Мне всегда нравится эта фраза. в 80 % случаев это не так. Тут класс с 2012 года не трогали. Он либо всегда не работал или что-то не так, но обновление скорее всего никак не влияет



Хм, прошу прощения. Тут действительно сломали недавно.У меня старый код был.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 17 июн 2015, 19:44 
Не в сети
Разработчик

Зарегистрирован: 08 ноя 2007, 01:05
Сообщения: 8343
Откуда: Уфа
Карма: 238
Исправлено,обновление выкладывается.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 17 июн 2015, 19:45 
Не в сети
Клиент
Аватара пользователя

Зарегистрирован: 29 мар 2010, 23:11
Сообщения: 5854
Карма: 472
stark писал(а):
Хм, прошу прощения. Тут действительно сломали недавно.У меня старый код был.

Изображение

_________________
Цитаты великих людей :umnik:
Напишите в helpdesk © stark
повторяю: => хелпдеск => доработка => профит © dimOn
свершилось... © skn
Мой код изящен, лёгок, оригинален, краток. Как прохладный весенний ветерок, как звонкий ручей! © dimOn
Вежливый разработчик © Artur
Эти баги тоже исправлены, как и те, которые еще не написаны © Artur
ну т.е. существует воркэраунд, ок © dimOn


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 17 июн 2015, 21:46 
Не в сети
Клиент

Зарегистрирован: 07 мар 2012, 15:02
Сообщения: 932
Откуда: Воронеж
Карма: 35
Может ещё в соседней ветке посмотрите проблему с y и m в запросе счетов?

А ещё интересно, зачем надо было лезть в класс, который не менялся с 2012 года и что-то там ломать???

_________________

Клиент: вер. 6.2.873 / 04.12.2017 19:38:11
os: Windows 7; java: Java HotSpot(TM) 64-Bit Server VM, v.1.8.0_65
Сервер: вер. 6.2.1202 / 04.12.2017 19:39:21
os: Linux; java: Java HotSpot(TM) 64-Bit Server VM, v.1.8.0_91


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 18 июн 2015, 14:22 
Не в сети
Разработчик

Зарегистрирован: 08 ноя 2007, 01:05
Сообщения: 8343
Откуда: Уфа
Карма: 238
vkulakov писал(а):
Может ещё в соседней ветке посмотрите проблему с y и m в запросе счетов?

А ещё интересно, зачем надо было лезть в класс, который не менялся с 2012 года и что-то там ломать???

Так по вашей же просьбе туда функционал добавили. Зачем вам сортировка понадобилась?
А с y m не считаю ошибкой . Можем исправить 6.2 .


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 18 июн 2015, 14:53 
Не в сети
Клиент

Зарегистрирован: 07 мар 2012, 15:02
Сообщения: 932
Откуда: Воронеж
Карма: 35
Ещё интересно, изменения, которые делаются в 6.0 попадают в остальные версии? В 6.1 и в 6.2? Например, в 6.0 добавили сортировку, в 6.1 она тоже будет? С upload_status тоже интересно.

Не получится ли так, что после обновления на 6.2 у нас всё поломается?

_________________

Клиент: вер. 6.2.873 / 04.12.2017 19:38:11
os: Windows 7; java: Java HotSpot(TM) 64-Bit Server VM, v.1.8.0_65
Сервер: вер. 6.2.1202 / 04.12.2017 19:39:21
os: Linux; java: Java HotSpot(TM) 64-Bit Server VM, v.1.8.0_91


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: 18 июн 2015, 15:02 
Не в сети
Разработчик

Зарегистрирован: 08 ноя 2007, 01:05
Сообщения: 8343
Откуда: Уфа
Карма: 238
vkulakov писал(а):
Ещё интересно, изменения, которые делаются в 6.0 попадают в остальные версии? В 6.1 и в 6.2? Например, в 6.0 добавили сортировку, в 6.1 она тоже будет? С upload_status тоже интересно.

Не получится ли так, что после обновления на 6.2 у нас всё поломается?



Изменения для 6.0 всегда попадают в более старшие версии. Если этого вдруг не происходит, то это косяк . Часто изменения делаются в ветке разработки(Master) и потом последовательно спускаются до нужной версии.

Именно поэтому мы не хотим добавлять новый функционал в 6.0, так как приходится потом патчить много версий , разруливая конфикты в каждой из них. Ошибки исправлять - это неизбежное зло. А вот функционал новый - не будем добавлять.


Вернуться к началу
 Профиль  
 
Показать сообщения за:  Поле сортировки  
Начать новую тему Ответить на тему  [ Сообщений: 9 ] 

Часовой пояс: UTC + 5 часов [ Летнее время ]


Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 1


Вы не можете начинать темы
Вы не можете отвечать на сообщения
Вы не можете редактировать свои сообщения
Вы не можете удалять свои сообщения
Вы не можете добавлять вложения

Найти:
Перейти:  
POWERED_BY
Русская поддержка phpBB
[ Time : 0.038s | 34 Queries | GZIP : On ]