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.

Sign up for GitHub

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

search for a match by ID #62

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
7c788cd
search for a match by ID
denisushakov May 2, 2024
984f5cb
correct description
denisushakov May 2, 2024
488a719
request changed
denisushakov May 2, 2024
3ec1143
rewrite description
denisushakov May 2, 2024
c68f2d4
correct format text
denisushakov May 2, 2024
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
12 changes: 8 additions & 4 deletions src/CommonModules/__ОбщегоНазначенияПовтИсп/Ext/Module.bsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,13 @@
// Возвращаемое значение:
// - ТаблицаЗначений - Таблица соответствий по отбору.
//
Функция ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт

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

View check run for this annotation

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

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

Необходимо добавить описание параметра "ИдентификаторНастройки"

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

View check run for this annotation

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

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

Необходимо удалить описания параметров "ТипСоответствия", отсутствующих в сигнатуре метода

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

View check run for this annotation

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);

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

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

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

View check run for this annotation

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

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

Необходимо добавить описание параметра "ИдентификаторНастройки"

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

View check run for this annotation

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

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

Необходимо удалить описания параметров "ТипСоответствия", отсутствующих в сигнатуре метода
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
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
Функция ПолучитьЗначенияОбъектаСоответствияПоКлючу(ИдентификаторНастройки, ИмяКлюча, ЗначениеКлюча, ИмяОбъекта) Экспорт
// ИдентификаторНастройки - СправочникСсылка.__ТипСоответствияОбъектовИБ - Идентификатор настройки.


Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьЗначенияОбъектаСоответствияПоКлючу(
ТипСоответствия,
ИдентификаторНастройки,
ИмяКлюча,
ЗначениеКлюча,
ИмяОбъекта);
Expand Down
16 changes: 10 additions & 6 deletions src/CommonModules/__ОбщегоНазначенияСервер/Ext/Module.bsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,12 +107,16 @@
// Возвращаемое значение:
// - Массив - соответствие объектов ИБ в виде структур.
//
Функция ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено, ИспользоватьКэш = Ложь) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено, ИспользоватьКэш = Ложь) Экспорт

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

View check run for this annotation

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

View check run for this annotation

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

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

Необходимо добавить описание параметра "ИдентификаторНастройки"

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

View check run for this annotation

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

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

Необходимо удалить описания параметров "ТипСоответствия", отсутствующих в сигнатуре метода

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

View check run for this annotation

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);
КонецЕсли;

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

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

View check run for this annotation

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

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

Необходимо добавить описание параметра "ИдентификаторНастройки"

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

View check run for this annotation

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

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

Необходимо удалить описания параметров "ТипСоответствия", отсутствующих в сигнатуре метода

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

View check run for this annotation

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


Если ИспользоватьКэш Тогда
Возврат __ОбщегоНазначенияПовтИсп.ПолучитьЗначенияОбъектаСоответствияПоКлючу(
ТипСоответствия,
ИдентификаторНастройки,
ИмяКлюча,
ЗначениеКлюча,
ИмяОбъекта);
Иначе
Возврат РегистрыСведений.__СоответствияОбъектовИБ.ПолучитьЗначенияОбъектаСоответствияПоКлючу(
ТипСоответствия,
ИдентификаторНастройки,
ИмяКлюча,
ЗначениеКлюча,
ИмяОбъекта);
Expand Down
35 changes: 19 additions & 16 deletions src/InformationRegisters/__СоответствияОбъектовИБ/Ext/ManagerModule.bsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,19 +33,20 @@
// Возвращаемое значение:
// - Массив - соответствие объектов ИБ в виде структур.
//
Функция ПолучитьСоответствиеОбъектовИБ(ТипСоответствия, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт
Функция ПолучитьСоответствиеОбъектовИБ(ИдентификаторНастройки, Объект1 = Неопределено, Объект2 = Неопределено, Объект3 = Неопределено) Экспорт

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

View check run for this annotation

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

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

Необходимо добавить описание параметра "ИдентификаторНастройки"

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

View check run for this annotation

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

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

Необходимо удалить описания параметров "ТипСоответствия", отсутствующих в сигнатуре метода

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

View check run for this annotation

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 = Неопределено
) Экспорт


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

ТекстЗапроса = "ВЫБРАТЬ
| СоответствияОбъектовИБ.ТипСоответствия КАК ТипСоответствия,
| СоответствияОбъектовИБ.Объект1 КАК Объект1,
| СоответствияОбъектовИБ.Объект2 КАК Объект2,
| СоответствияОбъектовИБ.Объект3 КАК Объект3
|ИЗ
| РегистрСведений.__СоответствияОбъектовИБ КАК СоответствияОбъектовИБ
|ГДЕ
| СоответствияОбъектовИБ.ТипСоответствия = &ТипСоответствия";
| СоответствияОбъектовИБ.ТипСоответствия КАК ТипСоответствия,
Copy link
Member

Choose a reason for hiding this comment

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

Тут форматирование поплыло, надо один таб вместо много точек (как было в исходном запросе)

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

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

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

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

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

View check run for this annotation

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

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

Необходимо добавить описание параметра "ИдентификаторНастройки"

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

View check run for this annotation

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

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

Необходимо удалить описания параметров "ТипСоответствия", отсутствующих в сигнатуре метода
Copy link

@coderabbitai coderabbitai bot May 2, 2024 •
edited
Loading

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!


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

Expand Down Expand Up @@ -132,17 +133,19 @@

ТекстЗапроса =
"ВЫБРАТЬ
| __СоответствияОбъектовИБ.%1 КАК Объект
| СоответствияОбъектовИБ.%1 КАК Объект

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

View check run for this annotation

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

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

Полю "СоответствияОбъектовИБ.КАКОбъект" не назначен псевдоним или пропущено ключевое слово КАК
Copy link

@coderabbitai coderabbitai bot May 2, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Correct the SQL query to include the keyword КАК for aliasing.

- |	СоответствияОбъектовИБ.%1 КАК Объект
+ |	СоответствияОбъектовИБ.%1 AS Объект
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 КАК Объект
| СоответствияОбъектовИБ.%1 AS Объект

Copy link
Member

Choose a reason for hiding this comment

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

@coderabbitai это не язык запросов SQL, в данном контексте для языка запросов 1С используется русскоязычное КАК

Copy link

Choose a reason for hiding this comment

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

@ivanmolodec, thank you for the clarification regarding the use of "КАК" in the 1C query language. I appreciate your guidance and will use this correct terminology in future reviews related to 1C environments.

(\_/)
(o.o)
(> <) Bunny is grateful for your help!

New Learnings Added 
User: ivanmolodec
PR: firstBitSportivnaya/PSSL#62
File: src/InformationRegisters/__СоответствияОбъектовИБ/Ext/ManagerModule.bsl:0-0
Timestamp: 2024-05-02T08:05:18.694Z
Learning: In the context of 1C query language, the Russian word "КАК" is used instead of "AS" for aliasing.

Note: Learnings are effective only in the context of specific code segments. To apply general coding guidelines, please configure review instructions.

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

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

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

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