1
0
mirror of https://github.com/1C-Company/v8-code-style.git synced 2024-11-28 09:33:06 +02:00

Добавил ПолучитьФорму в проверку вызова нерекомендуемых методов

This commit is contained in:
Manaev Konstantin 2023-03-10 23:10:20 +03:00
parent f4095b1141
commit 715685c2fa
6 changed files with 19 additions and 2 deletions

View File

@ -16,6 +16,7 @@
#### Код модулей
- В проверку использования нерекомендуемых методов (use-non-recommended-method) добавлен метод ПолучитьФорму(GetForm)
#### Запросы

View File

@ -8,6 +8,8 @@
Сообщить("Текст сообщения");
Дата = ТекущаяДата();
ОбщаяФорма1 = ПолучитьФорму("ОбщаяФорма.ОбщаяФорма1");
```
## Правильно
@ -18,9 +20,12 @@
Сообщение.Сообщить();
Дата = ТекущаяДатаСеанса();
ОткрытьФорму("ОбщаяФорма.ОбщаяФорма1);
```
## См.
- [Ограничение на использование метода Сообщить](https://its.1c.ru/db/v8std#content:418:hdoc)
- [Работа в разных часовых поясах](https://its.1c.ru/db/v8std#content:643:hdoc:2.1)
- [Открытие форм](https://its.1c.ru/db/v8std#content:404:hdoc:1)

View File

@ -8,6 +8,8 @@ There are not recommended methods, instead of which either SSL methods or other
Message("Text");
Date = CurrentDate();
CommonForm1 = GetForm("CommonForm.CommonForm1");
```
## Compliant Solution
@ -18,9 +20,12 @@ Message.Text = ("Text");
Message.Message();
Date = CurrentSessionDate();
OpenForm("CommonForm.CommonForm1");
```
## See
- [Restriction on the use of the Message method](https://its.1c.ru/db/v8std#content:418:hdoc)
- [Working in different time zones](https://its.1c.ru/db/v8std#content:643:hdoc:2.1)
- [Opening forms](https://kb.1ci.com/1C_Enterprise_Platform/Guides/Developer_Guides/1C_Enterprise_Development_Standards/Designing_user_interfaces/Implementation_of_form/Opening_forms/?language=en)

View File

@ -51,7 +51,9 @@ public class UseNonRecommendedMethodCheck
private static final Set<String> NON_RECOMENDED_METHODS_LIST = Set.of("CurrentDate", //$NON-NLS-1$
"ТекущаяДата", //$NON-NLS-1$
"Message", //$NON-NLS-1$
"Сообщить"); //$NON-NLS-1$
"Сообщить", //$NON-NLS-1$
"ПолучитьФорму", //$NON-NLS-1$
"GetForm"); //$NON-NLS-1$
private static final String DELIMITER = ","; //$NON-NLS-1$

View File

@ -1,6 +1,7 @@
Function functionName(Parameters) Export
Message("Test");
date = CurrentDate();
CommonForm1 = GetForm("CommonForm.CommonForm1");
Return date;
EndFunction

View File

@ -48,12 +48,15 @@ public class UseNonRecommendedMethodsTest
updateModule(FOLDER_RESOURCE + "use-non-recommended-methods.bsl");
List<Marker> markers = getModuleMarkers();
assertEquals(2, markers.size());
assertEquals(3, markers.size());
Marker marker = markers.get(0);
assertEquals("2", marker.getExtraInfo().get(IExtraInfoKeys.TEXT_EXTRA_INFO_LINE_KEY));
marker = markers.get(1);
assertEquals("3", marker.getExtraInfo().get(IExtraInfoKeys.TEXT_EXTRA_INFO_LINE_KEY));
marker = markers.get(2);
assertEquals("4", marker.getExtraInfo().get(IExtraInfoKeys.TEXT_EXTRA_INFO_LINE_KEY));
}
}