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

Исправление ошибок и поддержка режима совместимости < 8.3 #1107

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sfaqer
Copy link
Contributor

@sfaqer sfaqer commented Jul 3, 2023

  1. Исправлена загрузка плагинов в обычных формах
  2. Адаптация форм к работе в режиме совместимости < 8.3
  3. Исправлена ошибка при формировании представления теста в обычной форме, при не переданной настройке добавлять имя пользователя
  4. Для дымового теста проверки общих модулей учтена настройка "Используется"
  5. Исправлена ошибка дымового теста открытия форм в случае если в конфигурации есть общий модуль "ОбщегоНазначения" но в нём нет метода "ПодсистемаСуществует"

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

2. Адаптация форм к работе в режиме совместимости < 8.3
3. Исправлена ошибка при формировании представления теста в обычной форме, при не переданной настройке добавлять имя пользователя
4. Для дымового теста проверки общих модулей учтена настройка "Используется"
5. Исправлена ошибка дымового теста открытия форм в случае если в конфигурации есть общий модуль "ОбщегоНазначения" но в нём нет метода "ПодсистемаСуществует"
@artbear
Copy link
Collaborator

artbear commented Jul 16, 2023

Адаптация форм к работе в режиме совместимости < 8.3

@sfaqer основная проблема, что в этом режиме перестанут работать полезнейшие платформенные методы СтрХХХ

поэтому я и ушел в свое время от поддержки подобного режима совместимости, т.к. это затрудняет разработку

напиши, зачем тебе нужна подобная совместимость?
УПП вроде бы позволяет работать с 8.3.5 или 8.3.6

@sfaqer
Copy link
Contributor Author

sfaqer commented Jul 17, 2023

@artbear Ну, конкретно у меня УПП 1.2 которую с горем пополам получилось на 8.2.16 перевести.)
бтв, методы СтрXXX я в глобальный модуль закинул, поэтому у меня проблемы нет, скорее у меня проблема возникает на уровне ЭтотОбъект -> ЭтаФорма, ну и целенаправленно этот режим можно не поддерживать, я сам могу время от времени фиксы подкидывать.)

Copy link
Collaborator

@artbear artbear left a comment

Choose a reason for hiding this comment

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

не согласен только с этим кодом, остальные изменения устраивают

Результат = Плагины[Имя];
Если ТипЗнч(Результат) <> Тип("Строка") Тогда
Возврат Результат;
КонецЕсли;
Copy link
Collaborator

Choose a reason for hiding this comment

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

предлагаю этот код откатить, т.к. я уже внес исправления для работы в ОФ.

посмотри, мы проверяли в УПП 1.3
тесты в ОФ работают

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

чёт не растёт

Choose a reason for hiding this comment

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

Проблема остается. Правда стреляет не всегда.

На клиент-серверной в Windows работает
На файловой в Linux ошибка сохраняется.

ПС: проверено на дев и 6.9.5

Если ПустаяСтрока(АдресВременногоХранилища) Тогда
НастройкиПродукта = Неопределено;
Иначе
НастройкиПродукта = ПолучитьИзВременногоХранилища(АдресВременногоХранилища);
Copy link
Collaborator

Choose a reason for hiding this comment

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

предлагаю и этот код откатить, т.к. я уже внес исправления для работы в ОФ.

посмотри, мы проверяли в УПП 1.3
тесты в ОФ работают

@artbear artbear removed the question label Jul 19, 2023
@artbear artbear added this to the 6.9.0 milestone Jul 19, 2023
@artbear artbear added tdd ОФ Обычные формы labels Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tdd ОФ Обычные формы
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants