Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Доработаны методы получения информации из ИБ. #490 #534

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

KeynihAV
Copy link

No description provided.

|";
Запрос.УстановитьПараметр("Дата1", Дата1);
Запрос.УстановитьПараметр("Дата1", НачалоДня(Дата1));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для начальной даты я специально не использовал начало дня, чтобы проще было выполнять поиск от конкретного времени.
Например, последовательно в тестах создаю документы и в коде теста хочу получить только те документы, которые созданы на определенном шаге, а не все документы, которые созданы в этот день.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Другие изменения готов принять.
По НачалоДня предлагаю обсудить.

@KeynihAV
Copy link
Author

В таком случае можно убрать и КонецДня(), чтобы брать произвольный период.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

Наиболее популярный сценарий - указать период в виде двух дат без времени.
В этом случае первая дата сразу указывает на начало дня и это правильно.
А вот для второй даты нужно выполнять приведение к концу дня и для пользователя это неудобно.
Так что фича с преобразованием к концу дня полезна и ее лучше оставить.

…ментовМетаданногоПоОтбору теперь вызывает ПолучитьКоличествоЭлементовМетаданногоПоОтбору.
@KeynihAV
Copy link
Author

Опечатался в коммите. Там удален код из метода ПолучитьКоличествоЭлементовСправочникаПоОтбору.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

А можешь вкратце описать сценарии использования этих методов в своем коде?

@KeynihAV
Copy link
Author

Не доводилось их еще использовать, в рабочей конфигурации я пока всего с десяток тестов написал. Интересно посмотреть как внутри устроено, что за магические слова fork и pull request, заодно и помочь проекту. Поэтому я взял попавшуюся на глаза задачу.

@KeynihAV
Copy link
Author

Еще вижу задачу #297. Получается, что она давно сделана но не закрыта.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

По #297 подразумевалось наличие клиентских методов в управляемой форме раннера, т.к. сейчас из серверного кода теста при запуске в упр.режиме сложно достучаться до серверного кода раннера.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

Интересно посмотреть как внутри устроено, что за магические слова fork и pull request, заодно и помочь проекту. Поэтому я взял попавшуюся на глаза задачу.

Хорошее дело, приветствую.
Спасибо!

@artbear artbear modified the milestones: second, 4.3.0.0, 4.Х.0.0 Jul 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants