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

search for a match by ID #62

Merged
merged 5 commits into from
May 2, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -58,24 +58,28 @@
// Возвращает таблицу соответствий по отбору.
//
// Параметры:
// ТипСоответствия - Перечисление.__ТипСоответствияОбъектовИБ - Тип соответствия.
// ИдентификаторНастройки - Строка - Идентификатор элемента справочника "Тип соответствия объектов ИБ".
// Объект1 - ПроизвольныйТип - Объект1.
// Объект2 - ПроизвольныйТип - Объект2.
// Объект3 - ПроизвольныйТип - Объект3.
//
// Возвращаемое значение:
// - ТаблицаЗначений - Таблица соответствий по отбору.
//
Функция ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт

Check warning on line 69 in src/CommonModules/__ОбщегоНазначенияПовтИсп/Ext/Module.bsl

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__ОбщегоНазначенияПовтИсп/Ext/Module.bsl#L69

Длина строки 142 превышает максимально допустимую 120
Copy link
Member

Choose a reason for hiding this comment

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

@denisushakov тут он прав, в описании осталось старое название параметра

Copy link

Choose a reason for hiding this comment

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

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!

Copy link

Choose a reason for hiding this comment

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

Consider breaking the function declaration into multiple lines to comply with best practices regarding line length.

- Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
+ Функция ПолучитьСоответствиеОбъектовИБ(
+   ИдентификаторНастройки, 
+   Объект1 = Неопределено, 
+   Объект2 = Неопределено, 
+   Объект3 = Неопределено
+ ) Экспорт

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(
ИдентификаторНастройки,
Объект1 = Неопределено,
Объект2 = Неопределено,
Объект3 = Неопределено
) Экспорт


Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1, Объект2, Объект3);
Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьСоответствиеОбъектовИБ(
ИдентификаторНастройки,
Объект1,
Объект2,
Объект3);

КонецФункции

// Возвращает массив значений произвольного типа, объекта соответствия.
//
// Параметры:
// ТипСоответствия - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
// ИдентификаторНастройки - Строка - Идентификатор элемента справочника "Тип соответствия объектов ИБ".
// ИмяКлюча - Строка - поле регистра сведений __СоответствияОбъектовИБ, по которому устанавливается отбор.
// ЗначениеКлюча - ПроизвольныйТип - Значение ключа по которому установлен отбор.
// ИмяОбъекта - Строка - получаемое поле регистра сведений __СоответствияОбъектовИБ.
@@ -84,10 +88,10 @@
// - Массив - Значения объекта из регистра сведений __СоответствияОбъектовИБ,
// если соответствие одно в массив будет один элемент
//
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ТипСоответствия, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Copy link

Choose a reason for hiding this comment

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

Update the function documentation to reflect the new parameter name.

- //  ТипСоответствия	 - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
+ //  ИдентификаторНастройки	 - СправочникСсылка.__ТипСоответствияОбъектовИБ - Идентификатор настройки.

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
// ИдентификаторНастройки - СправочникСсылка.__ТипСоответствияОбъектовИБ - Идентификатор настройки.


Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьЗначенияОбъектаСоответствияПоКлючу(
ТипСоответствия,
ИдентификаторНастройки,
ИмяКлюча,
ЗначениеКлюча,
ИмяОбъекта);
Original file line number Diff line number Diff line change
@@ -98,7 +98,7 @@
// Возвращает массив структур соответствий по отбору
//
// Параметры:
// ТипСоответствия - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
// ИдентификаторНастройки - Строка - Идентификатор элемента справочника "Тип соответствия объектов ИБ".
// Объект1 - ПроизвольныйТип - Объект1.
// Объект2 - ПроизвольныйТип - Объект2.
// Объект3 - ПроизвольныйТип - Объект3.
@@ -107,12 +107,16 @@
// Возвращаемое значение:
// - Массив - соответствие объектов ИБ в виде структур.
//
Функция ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено, ИспользоватьКэш = Ложь) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено, ИспользоватьКэш = Ложь) Экспорт

Check warning on line 110 in src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl#L110

Уменьшите количество необязательных параметров c 4 до допустимого 3

Check warning on line 110 in src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl#L110

Длина строки 166 превышает максимально допустимую 120
Copy link

Choose a reason for hiding this comment

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

Reduce the line length to adhere to the maximum recommended line length of 120 characters.

- Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено, ИспользоватьКэш = Ложь) Экспорт
+ Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, ИспользоватьКэш = Ложь) Экспорт

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено, ИспользоватьКэш = Ложь) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, ИспользоватьКэш = Ложь) Экспорт


Если ИспользоватьКэш Тогда
Возврат __ОбщегоНазначенияПовтИсп.ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1, Объект2, Объект3);
Возврат __ОбщегоНазначенияПовтИсп.ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1, Объект2, Объект3);
Иначе
Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1, Объект2, Объект3);
Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьСоответствиеОбъектовИБ(
ИдентификаторНастройки,
Объект1,
Объект2,
Объект3);
КонецЕсли;

КонецФункции
@@ -779,7 +783,7 @@
// Возвращает массив значений произвольного типа, объекта соответствия.
//
// Параметры:
// ТипСоответствия - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
// ИдентификаторНастройки - Строка - Идентификатор элемента справочника "Тип соответствия объектов ИБ".
// ИмяКлюча - Строка - поле регистра сведений __СоответствияОбъектовИБ, по которому устанавливается отбор.
// ЗначениеКлюча - ПроизвольныйТип - Значение ключа по которому установлен отбор.
// ИмяОбъекта - Строка - получаемое поле регистра сведений __СоответствияОбъектовИБ.
@@ -789,17 +793,17 @@
// - Массив - Значения объекта из регистра сведений __СоответствияОбъектовИБ,
// если соответствие одно в массив будет один элемент
//
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ТипСоответствия, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта, ИспользоватьКэш = Ложь) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта, ИспользоватьКэш = Ложь) Экспорт

Check warning on line 796 in src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl#L796

Длина строки 143 превышает максимально допустимую 120
Copy link

Choose a reason for hiding this comment

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

Reduce the line length to adhere to the maximum recommended line length of 120 characters.

- Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта, ИспользоватьКэш = Ложь) Экспорт
+ Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта, ИспользоватьКэш = Ложь) Экспорт

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта, ИспользоватьКэш = Ложь) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта, ИспользоватьКэш = Ложь) Экспорт


Если ИспользоватьКэш Тогда
Возврат __ОбщегоНазначенияПовтИсп.ПолучитьЗначенияОбъектаСоответствияПоКлючу(
ТипСоответствия,
ИдентификаторНастройки,
ИмяКлюча,
ЗначениеКлюча,
ИмяОбъекта);
Иначе
Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьЗначенияОбъектаСоответствияПоКлючу(
ТипСоответствия,
ИдентификаторНастройки,
ИмяКлюча,
ЗначениеКлюча,
ИмяОбъекта);
Original file line number Diff line number Diff line change
@@ -25,15 +25,15 @@
// Возвращает массив структур соответствий по отбору.
//
// Параметры:
// ТипСоответствия - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
// ИдентификаторНастройки - Строка - Идентификатор элемента справочника "Тип соответствия объектов ИБ".
// Объект1 - ПроизвольныйТип - Объект1.
// Объект2 - ПроизвольныйТип - Объект2.
// Объект3 - ПроизвольныйТип - Объект3.
//
// Возвращаемое значение:
// - Массив - соответствие объектов ИБ в виде структур.
//
Функция ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт

Check warning on line 36 in src/InformationRegisters/__СоответствияОбъектовИБ/Ext/ManagerModule.bsl

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/InformationRegisters/__СоответствияОбъектовИБ/Ext/ManagerModule.bsl#L36

Длина строки 142 превышает максимально допустимую 120
Copy link

Choose a reason for hiding this comment

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

Consider breaking the function declaration into multiple lines to comply with best practices regarding line length.

- Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
+ Функция ПолучитьСоответствиеОбъектовИБ(
+   ИдентификаторНастройки, 
+   Объект1 = Неопределено, 
+   Объект2 = Неопределено, 
+   Объект3 = Неопределено
+ ) Экспорт

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(
ИдентификаторНастройки,
Объект1 = Неопределено,
Объект2 = Неопределено,
Объект3 = Неопределено
) Экспорт


Запрос = Новый Запрос;

@@ -43,9 +43,10 @@
| СоответствияОбъектовИБ.Объект2 КАК Объект2,
| СоответствияОбъектовИБ.Объект3 КАК Объект3
|ИЗ
| РегистрСведений.__СоответствияОбъектовИБ КАК СоответствияОбъектовИБ
|ГДЕ
| СоответствияОбъектовИБ.ТипСоответствия = &ТипСоответствия";
| Справочник.__ТипСоответствияОбъектовИБ КАК ТипСоответствияОбъектовИБ
| ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.__СоответствияОбъектовИБ КАК СоответствияОбъектовИБ
| ПО (ТипСоответствияОбъектовИБ.ИдентификаторНастройки = &ИдентификаторНастройки)
| И ТипСоответствияОбъектовИБ.Ссылка = СоответствияОбъектовИБ.ТипСоответствия";

Схема = Новый СхемаЗапроса();
Схема.УстановитьТекстЗапроса(ТекстЗапроса);
@@ -73,7 +74,7 @@
КонецЕсли;

Запрос.Текст = Схема.ПолучитьТекстЗапроса();
Запрос.УстановитьПараметр("ТипСоответствия", ТипСоответствия);
Запрос.УстановитьПараметр("ИдентификаторНастройки", ИдентификаторНастройки);

УстановитьПривилегированныйРежим(Истина);
Выборка = Запрос.Выполнить().Выбрать();
@@ -95,7 +96,7 @@
// Возвращает массив значений произвольного типа, объекта соответствия.
//
// Параметры:
// ТипСоответствия - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
// ИдентификаторНастройки - Строка - Идентификатор элемента справочника "Тип соответствия объектов ИБ".
// ИмяКлюча - Строка - поле регистра сведений __СоответствияОбъектовИБ, по которому устанавливается отбор.
// ЗначениеКлюча - ПроизвольныйТип - Значение ключа по которому установлен отбор.
// ИмяОбъекта - Строка - получаемое поле регистра сведений __СоответствияОбъектовИБ.
@@ -104,7 +105,7 @@
// - Массив - Значения объекта из регистра сведений __СоответствияОбъектовИБ,
// если соответствие одно в массив будет один элемент
//
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ТипСоответствия, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Copy link

@coderabbitai coderabbitai bot May 2, 2024

Choose a reason for hiding this comment

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

Update the function documentation to reflect the new parameter name.

- //  ТипСоответствия	 - СправочникСсылка.__ТипСоответствияОбъектовИБ - Тип соответствия.
+ //  ИдентификаторНастройки	 - СправочникСсылка.__ТипСоответствияОбъектовИБ - Идентификатор настройки.

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
// ИдентификаторНастройки - СправочникСсылка.__ТипСоответствияОбъектовИБ - Идентификатор настройки.

Copy link
Member

Choose a reason for hiding this comment

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

@denisushakov тут тоже

Copy link

Choose a reason for hiding this comment

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

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!


МетаданныеРегистра = Метаданные.РегистрыСведений.__СоответствияОбъектовИБ;

@@ -131,18 +132,31 @@
Запрос = Новый Запрос;

ТекстЗапроса =
"ВЫБРАТЬ
| __СоответствияОбъектовИБ.%1 КАК Объект
"ВЫБРАТЬ РАЗЛИЧНЫЕ
| СоответствияОбъектовИБ.Объект1 КАК Объект
|ИЗ
| РегистрСведений.__СоответствияОбъектовИБ КАК __СоответствияОбъектовИБ
| Справочник.__ТипСоответствияОбъектовИБ КАК ТипСоответствияОбъектовИБ
| ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.__СоответствияОбъектовИБ КАК СоответствияОбъектовИБ
| ПО (ТипСоответствияОбъектовИБ.ИдентификаторНастройки = &ИдентификаторНастройки)
| И ТипСоответствияОбъектовИБ.Ссылка = СоответствияОбъектовИБ.ТипСоответствия
|ГДЕ
| __СоответствияОбъектовИБ.ТипСоответствия = &ТипСоответствия
| И __СоответствияОбъектовИБ.%2 = &Ключ";
| СоответствияОбъектовИБ.Объект2 = &Ключ";

ВыражениеПоля = СтрШаблон("СоответствияОбъектовИБ.%1", ИмяОбъекта);
ТекстЗапроса = __СхемыЗапросов.ДобавитьПолеВыборкиВЗапрос(ТекстЗапроса, ВыражениеПоля, "Объект", , , Истина);

СхемаЗапроса = Новый СхемаЗапроса;
СхемаЗапроса.УстановитьТекстЗапроса(ТекстЗапроса);
Отбор = СхемаЗапроса.ПакетЗапросов[0].Операторы[0].Отбор;

ВыражениеОтбора = Новый ВыражениеСхемыЗапроса(СтрШаблон("СоответствияОбъектовИБ.%1 = &Ключ", ИмяКлюча));
Отбор.Установить(0, ВыражениеОтбора);
ТекстЗапроса = СхемаЗапроса.ПолучитьТекстЗапроса();

Запрос.Текст = СтрШаблон(ТекстЗапроса, ИмяОбъекта, ИмяКлюча);
Запрос.Текст = ТекстЗапроса;

Запрос.УстановитьПараметр("Ключ", ЗначениеКлюча);
Запрос.УстановитьПараметр("ТипСоответствия", ТипСоответствия);
Запрос.УстановитьПараметр("ИдентификаторНастройки", ИдентификаторНастройки);

УстановитьПривилегированныйРежим(Истина);
ТаблицаОбъектов = Запрос.Выполнить().Выгрузить();
Loading