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

Исправить алгоритм определения поддержки асинхронных вызовов #413

Merged
merged 3 commits into from Feb 18, 2019

Conversation

Projects
None yet
3 participants
@VladFrost
Copy link
Contributor

commented Feb 18, 2019

сделанные изменения

  • исправил алгоритм определения поддержки асинхронных вызовов
  • удалил странный алгоритм определения защиты от опасных действий

@silverbulleters/Collaborators - просьба прокомментировать и проверить

@artbear artbear added this to the next milestone Feb 18, 2019


Запрет = Метаданные.РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент = Метаданные.СвойстваОбъектов.РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент.НеИспользовать;
Возврат Не Запрет;
Если Метаданные.РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент = Метаданные.СвойстваОбъектов.РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент.НеИспользовать Тогда

This comment has been minimized.

Copy link
@artbear

artbear Feb 18, 2019

Collaborator

Вот здесь ошибка все-таки
метод называется КонфигурацияПоддерживаетАсинхронныеВызовы
а условие ниже возвращает Ложь, если в конфигурации не поддерживаются синхронные вызовы,
хотя должно бы Истину :)

Если Метаданные.РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент = Метаданные.СвойстваОбъектов.РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент.НеИспользовать Тогда
		Возврат Ложь;
	КонецЕсли;

This comment has been minimized.

Copy link
@VladFrost

VladFrost Feb 18, 2019

Author Contributor

Да, тогда + ещё сюда надо будет добавить условие про поддержку асинхронности
https://github.com/silverbulleters/add/blob/master/epf/xddTestRunner/xddTestRunner/Forms/УправляемаяФорма/Ext/Form/Module.bsl#L1183

This comment has been minimized.

Copy link
@VladFrost

VladFrost Feb 18, 2019

Author Contributor

Или здесь что-то сделать https://github.com/silverbulleters/add/blob/master/epf/xddTestRunner/xddTestRunner/Forms/УправляемаяФорма/Ext/Form/Module.bsl#L1209
Потому что сейчас в синхронном режиме сломались отчеты в пакетном режиме :(

This comment has been minimized.

Copy link
@artbear

artbear Feb 18, 2019

Collaborator

вроде бы все в порядке, у нас на закрытом CI тесты ОФ проходят, или я что-то не вижу :)

@artbear

This comment has been minimized.

Copy link
Collaborator

commented Feb 18, 2019

тут вообще что-то странное
смотрю в конфигураторе 8.3.10
не вижу через поиск РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент (как сейчас в коде)
а находится РежимИспользованияСинхронныхВызововРасширенийПлатформыИВнешнихКомпонент ("Платформы" добавилось)

как текущий код вообще сейчас работает на 8.3.10? :) - а ведь точно работает.

@VladFrost

This comment has been minimized.

Copy link
Contributor Author

commented Feb 18, 2019

тут вообще что-то странное
смотрю в конфигураторе 8.3.10
не вижу через поиск РежимИспользованияСинхронныхВызововРасширенийИВнешнихКомпонент (как сейчас в коде)
а находится РежимИспользованияСинхронныхВызововРасширенийПлатформыИВнешнихКомпонент ("Платформы" добавилось)

как текущий код вообще сейчас работает на 8.3.10? :) - а ведь точно работает.

Переименовали в 8.3.8 https://dl04.1c.ru/content/Platform/8_3_14_1565/1cv8upd_8_3_14_1565.htm

@artbear

This comment has been minimized.

Copy link
Collaborator

commented Feb 18, 2019

Тогда придется добавлять 2 ветки - меньше 8.3.8 и (больше или равно) :(

@silverbulleters-ci-admin

This comment has been minimized.

Copy link
Collaborator

commented Feb 18, 2019

@artbear
Copy link
Collaborator

left a comment

Отличные исправления, чуть-чуть поправишь?

@artbear artbear merged commit c62589e into silverbulleters:master Feb 18, 2019

0 of 2 checks passed

continuous-integration/jenkins/pr-merge This commit has test failures
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.