BiTel

Форум BiTel
bgbilling.ru     docs.bitel.ru     wiki.bitel.ru     dbinfo.bitel.ru     bgcrm.ru     billing.bitel.ru     bitel.ru    
Текущее время: 29 мар 2024, 01:30

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




Начать новую тему Ответить на тему  [ Сообщений: 5 ] 
Автор Сообщение
 Заголовок сообщения: ClosedDateChecker
СообщениеДобавлено: 11 окт 2018, 19:01 
Не в сети
Клиент
Аватара пользователя

Зарегистрирован: 29 мар 2010, 23:11
Сообщения: 5854
Карма: 472
Доброго времени суток, уважаемые.
Все время устанавливая дату закрытого периода, был уверен что там дата с учетом того числа которое указано.
Ну типа все закрыто учитывая эта дату.
Ну и дока располагает какбэ к этому
Цитата:
Для закрытого периода устанавливается его правая граница.

Считал что в биллинге правая граница считается как dd.MM.yyyy 23:59:59

Но сегодня один из бухов уточнил у меня почему он смог занести платеж от 30.09.2018 (такого вопроса не возникало, т.к. дополнительно висит скрипт запрещающий практически всем заносить платежи ранее текущей даты).

Полез ковырять и увидел что
Код:
public static void checkDatesForUpdate(Calendar closedDate, Calendar date, Calendar newDate) throws BGMessageException {
        if (date != null && TimeUtils.dateBefore(date, closedDate)) {
            throw new BGMessageException("Дата лежит в закрытом периоде - изменение невозможно!");
        } else if (newDate != null && TimeUtils.dateBefore(newDate, closedDate)) {
            throw new BGMessageException("Новая дата лежит в закрытом периоде - изменение невозможно!");
        }
    }


ИМХО более правильным будет использовать TimeUtils.dateBeforeOrEqual();

Если конечно такое поведение неправильное :)

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


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ClosedDateChecker
СообщениеДобавлено: 11 окт 2018, 19:48 
Не в сети
Клиент
Аватара пользователя

Зарегистрирован: 20 апр 2009, 12:03
Сообщения: 3092
Откуда: Иркутск
Карма: 338
Хы, я наоборот, помню обратную ситуацию - закрываешь последним числом месяца, тогда не даёт закрывать АП этим числом.

Т.е. закрываешь период 30.09.2018
АП стоит с 01.09.2018 по null
Хочешь закрыть АП 30.09.2018 - запрещает, хотя по логике всё ок должно быть.
Писал разработчикам где-то, можно найти даже, но не пофиксили, поэтому закрываю период на день меньше, т.е. 29.09.

Получается, в функции проверки одной даты нужно сделать equal, а для проверки двух дат, наоборот - убрать equal


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ClosedDateChecker
СообщениеДобавлено: 11 окт 2018, 19:49 
Не в сети
Клиент
Аватара пользователя

Зарегистрирован: 20 апр 2009, 12:03
Сообщения: 3092
Откуда: Иркутск
Карма: 338
АП - абонплата. Хотя справедливо для любой сущности с 2 датами.


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ClosedDateChecker
СообщениеДобавлено: 11 окт 2018, 21:42 
Не в сети
Клиент
Аватара пользователя

Зарегистрирован: 29 мар 2010, 23:11
Сообщения: 5854
Карма: 472
Скорее достаточно будет просто сделать чтобы дата закрытого периода указывалась/сохранялась не просто датой а датой и временем 23:59:59как

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


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ClosedDateChecker
СообщениеДобавлено: 12 окт 2018, 07:41 
Не в сети
Клиент
Аватара пользователя

Зарегистрирован: 20 апр 2009, 12:03
Сообщения: 3092
Откуда: Иркутск
Карма: 338
Во, 2010г:
https://forum.bitel.ru/viewtopic.php?f= ... 5&p=34279&


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

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


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

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


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

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