Архитектурный рефакторинг подсистемы управления интеграциями #99
Conversation
…ны / изменены реквизиты для типов интеграций помимо SOAP и REST. Добавлены / изменены обработчики форм справочников подсистемы под отрисовку разных типов интеграций
|
Warning Rate limit exceeded@thvvmas has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 12 minutes and 36 seconds before requesting another review. How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughThis pull request introduces significant enhancements to the integration settings management within the application. It includes the addition of a new XML form for configuring external component integration, along with a corresponding module that implements the functionality for loading and saving external components. The changes facilitate user interaction through a structured interface and event handling, improving the overall integration process. Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Analysis Details98 Issues
Coverage and Duplications
Project ID: PSSL |
Analysis Details95 Issues
Coverage and Duplications
Project ID: PSSL |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 8
Outside diff range, codebase verification and nitpick comments (5)
src/cf/Catalogs/пбп_ТипыИнтеграций/Ext/ManagerModule.bsl (1)
25-64: LGTM, but consider adding exception handling.The function is well-structured and the logic is clear. The query is using a
CASEexpression to determine the value ofПараметрыОтbased on the hierarchy ofТипИнтеграции. The function is handling the case when the query returns no results by returning an empty reference.However, consider adding exception handling to gracefully handle any exceptions that may be thrown by the query.
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl (2)
Line range hint
34-69: LGTM, but consider adding exception handling and a description of the return value and the parameters.The function is well-structured and the logic is clear. The query is retrieving the parameters of the integration flow. The function is building a structure of the parameters based on the result of the query. The function is handling the case when the query returns no results by returning an empty structure.
However, consider the following suggestions:
- Add exception handling to gracefully handle any exceptions that may be thrown by the query.
- Add a description of the return value and the parameters of the function.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 34-34: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl#L34
Добавьте описание возвращаемого значения функции
[warning] 34-34: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl#L34
Необходимо добавить описание всех параметров метода
61-62: LGTM, but consider handling the case when the JSON type is a string and adding exception handling.The function is well-structured and the logic is clear. The function is converting the default value of a parameter based on its JSON type. The function is handling the case when the default value is not filled. The function is handling the case when the JSON type is a boolean, date, or number.
However, consider the following suggestions:
- Handle the case when the JSON type is a string.
- Add exception handling to gracefully handle any exceptions that may be thrown by the conversion.
Also applies to: 79-103
src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl (2)
24-45: Optimize the loop using theПродолжитьoperator.The code changes are approved.
However, consider applying this diff to slightly optimize the loop:
Для Каждого ЭлементМассива Из НепроверяемыеРеквизиты Цикл ПорядковыйНомер = ПроверяемыеРеквизиты.Найти(ЭлементМассива); - Если ПорядковыйНомер <> Неопределено Тогда + Если ПорядковыйНомер = Неопределено Тогда + Продолжить; + КонецЕсли; ПроверяемыеРеквизиты.Удалить(ПорядковыйНомер); - КонецЕсли; КонецЦикла;
47-112: Optimize the conditional blocks using theИначеЕслиoperator.The code changes are approved.
However, consider applying this diff to slightly optimize the conditional blocks:
-Если ЭлементНаследованиия = Справочники.пбп_ТипыИнтеграций.Каталог Тогда +ИначеЕсли ЭлементНаследованиия = Справочники.пбп_ТипыИнтеграций.Каталог Тогда Порт = ""; ВнешняяКомпонента = Неопределено; УчетнаяЗапись = Неопределено; ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); ИмяCOMОбъекта = ""; ИмяФайлаВнешнейКомпоненты = ""; -ИначеЕсли ЭлементНаследованиия = Справочники.пбп_ТипыИнтеграций.FTPРесурсы Тогда +ИначеЕсли ЭлементНаследованиия = Справочники.пбп_ТипыИнтеграций.FTPРесурсы Тогда ВнешняяКомпонента = Неопределено; УчетнаяЗапись = Неопределено; ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); ИмяCOMОбъекта = ""; ИмяФайлаВнешнейКомпоненты = "";
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
src/cf/CommonTemplates/пбп_PinkRabbitMQ/Ext/Template.binis excluded by!**/*.bin
Files selected for processing (47)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки.xml (7 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl (2 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl (1 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаСписка/Ext/Form.xml (2 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаСписка/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form.xml (1 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_ИсторияИнтеграции.xml (7 hunks)
- src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаСписка/Ext/Form.xml (2 hunks)
- src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form.xml (1 hunks)
- src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (13 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции.xml (3 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты.xml (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form.xml (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаСписка/Ext/Form.xml (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form.xml (6 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (4 hunks)
- src/cf/Catalogs/пбп_ПользовательскиеФункции/Ext/ObjectModule.bsl (1 hunks)
- src/cf/Catalogs/пбп_ТипыИнтеграций.xml (1 hunks)
- src/cf/Catalogs/пбп_ТипыИнтеграций/Ext/ManagerModule.bsl (1 hunks)
- src/cf/Catalogs/пбп_ТипыИнтеграций/Ext/Predefined.xml (1 hunks)
- src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl (6 hunks)
- src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ОбщегоНазначенияКлиент/Ext/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ОбщегоНазначенияПовтИсп/Ext/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl (2 hunks)
- src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (5 hunks)
- src/cf/CommonModules/пбп_РегламентныеЗаданияСервер/Ext/Module.bsl (1 hunks)
- src/cf/CommonTemplates/пбп_PinkRabbitMQ.xml (1 hunks)
- src/cf/CommonTemplates/пбп_SimpleKafkaAdapter.xml (1 hunks)
- src/cf/Configuration.xml (2 hunks)
- src/cf/Constants/пбп_ИспользоватьПользовательскиеФункции.xml (1 hunks)
- src/cf/DefinedTypes/пбп_Пользователи.xml (1 hunks)
- src/cf/DefinedTypes/пбп_УчетныеЗаписиЭлектроннойПочты.xml (1 hunks)
- src/cf/Enums/пбп_МетодыЗапросаREST.xml (1 hunks)
- src/cf/Enums/пбп_МетодыЗапросаREST/Ext/ManagerModule.bsl (1 hunks)
- src/cf/Enums/пбп_ТипыАвторизации.xml (1 hunks)
- src/cf/Enums/пбп_ФорматыИнтеграций.xml (2 hunks)
- src/cf/FunctionalOptions/пбп_ИспользоватьПользовательскиеФункции.xml (1 hunks)
- src/cf/Roles/пбп_РедактированиеНастроекИнтеграции/Ext/Rights.xml (2 hunks)
- src/cf/Roles/пбп_РедактированиеПользовательскихФункций/Ext/Rights.xml (1 hunks)
- src/cf/ScheduledJobs/пбп_ВыполнениеПользовательскихФункцийФайловыхОбменов.xml (1 hunks)
- src/cf/Subsystems/пбп_КомандныйИнтерфейсУправлениеНастройками.xml (1 hunks)
- src/cf/Subsystems/пбп_УправлениеИнтеграциями.xml (2 hunks)
Files skipped from review due to trivial changes (6)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты.xml
- src/cf/Catalogs/пбп_ПользовательскиеФункции/Ext/ObjectModule.bsl
- src/cf/CommonTemplates/пбп_PinkRabbitMQ.xml
- src/cf/CommonTemplates/пбп_SimpleKafkaAdapter.xml
- src/cf/Enums/пбп_МетодыЗапросаREST/Ext/ManagerModule.bsl
- src/cf/Roles/пбп_РедактированиеПользовательскихФункций/Ext/Rights.xml
Additional context used
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl
[warning] 39-39: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L39
Удалите неиспользуемую переменную РезультатОбработки
[notice] 31-31: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L31
Добавьте описание метода программного интерфейсаsrc/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl
[warning] 34-34: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl#L34
Добавьте описание возвращаемого значения функции
[warning] 34-34: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl#L34
Необходимо добавить описание всех параметров методаsrc/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl
[warning] 62-62: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L62
Синтаксическая конструкция вида "Если...Тогда...ИначеЕсли..." должна содержать ветвь "Иначе".
[failure] 24-24: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L24
Уменьшите когнитивную сложность "ПередЗаписью" с 18 до 15
[warning] 58-58: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L58
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
[warning] 30-30: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L30
Длина строки 124 превышает максимально допустимую 120src/cf/CommonModules/пбп_ОбщегоНазначенияКлиент/Ext/Module.bsl
[warning] 167-167: src/cf/CommonModules/пбп_ОбщегоНазначенияКлиент/Ext/Module.bsl#L167
Нельзя использовать латинские и кириллические символы в одном идентификатореsrc/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl
[warning] 144-144: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L144
Необходимо избавиться от многократного использования строкового литерала "Kafka"
[warning] 139-139: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L139
Необходимо избавиться от многократного использования строкового литерала "ИдентификаторНастройки"
[warning] 254-254: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L254
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "100" и используйте эту константу вместо магического числа.src/cf/CommonModules/пбп_ОбщегоНазначенияПовтИсп/Ext/Module.bsl
[warning] 118-118: src/cf/CommonModules/пбп_ОбщегоНазначенияПовтИсп/Ext/Module.bsl#L118
Добавьте описание возвращаемого значения функции
[warning] 118-118: src/cf/CommonModules/пбп_ОбщегоНазначенияПовтИсп/Ext/Module.bsl#L118
Необходимо добавить описание всех параметров методаsrc/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl
[warning] 37-37: src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl#L37
Длина строки 123 превышает максимально допустимую 120src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl
[notice] 33-33: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L33
Избавьтесь от избыточного обращения внутри модуля через его имя или псевдоним ЭтотОбъект
[notice] 298-298: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L298
Избавьтесь от избыточного обращения внутри модуля через его имя или псевдоним ЭтотОбъект
[warning] 270-270: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L270
Длина строки 132 превышает максимально допустимую 120
[notice] 282-282: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L282
Возможная опечатка в "Наследованиия"
[notice] 164-164: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L164
Возможная опечатка в "Наследованиия"
[warning] 248-248: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L248
Синтаксическая конструкция вида "Если...Тогда...ИначеЕсли..." должна содержать ветвь "Иначе".
[warning] 196-196: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L196
Необходимо избавиться от многократного использования строкового литерала "ПараметрыВходаПараметрURL"src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl
[warning] 144-144: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L144
Длина строки 125 превышает максимально допустимую 120
[warning] 165-165: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L165
Длина строки 136 превышает максимально допустимую 120
[warning] 215-215: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L215
Необходимо добавить описание всех параметров метода
[warning] 215-215: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L215
Добавьте описание возвращаемого значения функции
[warning] 446-446: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L446
Длина строки 134 превышает максимально допустимую 120
[warning] 440-440: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L440
Длина строки 136 превышает максимально допустимую 120
Additional comments not posted (66)
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаСписка/Ext/Form/Module.bsl (1)
29-29: LGTM!The change in the argument passed to the method
ИнициализироватьПредопределенныеЗначенияaligns with the PR objective of refactoring the integration management subsystem to support a broader range of integration methods. The change is localized to this procedure and does not introduce any issues.src/cf/FunctionalOptions/пбп_ИспользоватьПользовательскиеФункции.xml (1)
1-22: LGTM!The functional option
пбп_ИспользоватьПользовательскиеФункцииis well-defined and its purpose is clear from the name and the content. The inclusion of the catalogпбп_ПользовательскиеФункцииand the attributes related to user functions in the content suggests that this functional option is used to control the usage of user functions in the integration subsystem. The file does not introduce any issues.src/cf/DefinedTypes/пбп_Пользователи.xml (1)
1-28: LGTM!The defined type
пбп_Пользователиis well-defined and its purpose is clear from the name. The type is a string with a length of 10 characters, which suggests that it is used to represent user identifiers or codes. The file does not introduce any issues.src/cf/ScheduledJobs/пбп_ВыполнениеПользовательскихФункцийФайловыхОбменов.xml (1)
1-22: LGTM!The new scheduled job
пбп_ВыполнениеПользовательскихФункцийФайловыхОбменовis defined correctly. The job properties, such as the name, method name, description, and restart settings, are specified as expected.src/cf/DefinedTypes/пбп_УчетныеЗаписиЭлектроннойПочты.xml (1)
1-28: LGTM!The new defined type
пбп_УчетныеЗаписиЭлектроннойПочтыfor email accounts is defined correctly. The type is based on thexs:stringtype with a variable length of up to 10 characters, which seems reasonable for storing email account information.src/cf/Subsystems/пбп_КомандныйИнтерфейсУправлениеНастройками.xml (1)
23-24: LGTM!The additions of the constant and functional option for controlling custom user functions align with the PR objective of enhancing the integration management subsystem.
src/cf/Enums/пбп_ТипыАвторизации.xml (1)
1-67: LGTM!The new enumeration for defining authorization types aligns with the PR objective of enhancing the integration management subsystem.
src/cf/Constants/пбп_ИспользоватьПользовательскиеФункции.xml (1)
1-58: LGTM!The new constant for controlling the usage of custom user functions aligns with the PR objective of enhancing the integration management subsystem.
src/cf/Subsystems/пбп_УправлениеИнтеграциями.xml (1)
29-29: LGTM!The changes are consistent with the list of alterations. The changes are adding new references to the subsystem definition and renaming an existing reference. The changes are not removing any existing references and are not introducing any syntax errors.
Also applies to: 32-32, 41-45
src/cf/Catalogs/пбп_ТипыИнтеграций/Ext/Predefined.xml (1)
1-121: LGTM!The file follows the expected format and conventions for defining predefined items. The changes are approved.
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl (1)
90-101: LGTM!The
УдалитьРегламентноеЗаданиеПоКлючуprocedure correctly deletes the scheduled job by the provided key. The changes are approved.src/cf/Roles/пбп_РедактированиеНастроекИнтеграции/Ext/Rights.xml (3)
46-46: LGTM!The renaming of the object from
Catalog.пбп_МетодыИнтеграцииtoCatalog.пбп_ТипыИнтеграцийis approved. The rights are preserved correctly.
85-122: LGTM!The addition of the new object
Catalog.пбп_ИнтеграционныеПотокиand the granted rights are approved. The rights are comprehensive and appropriate for the purpose of enhancing the integration management capabilities.
124-149: LGTM!The addition of the new object
Configuration.ПроектнаяБиблиотекаПодсистемand the granted rights are approved. The rights related to main window modes and analytics system client are appropriate for expanding the configuration options for user interface modes and analytics functionalities.src/cf/Catalogs/пбп_ТипыИнтеграций.xml (1)
1-96: LGTM!The metadata definition for the
пбп_ТипыИнтеграцийcatalog looks good.src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl (1)
1-158: LGTM!The form module for loading an external component looks good. The code is well-structured and follows the standard practices.
src/cf/Enums/пбп_ФорматыИнтеграций.xml (1)
5-5: LGTM!The changes to the enum definition for integration formats look good:
- The enum name and several enum values have been appropriately renamed for clarity.
- New enum values have been added to support a more comprehensive list of integration formats.
These changes enhance the functionality and usability of the integration system. The code changes are approved.
Also applies to: 9-9, 13-13, 19-19, 23-23, 93-176
src/cf/Enums/пбп_МетодыЗапросаREST.xml (1)
1-162: Looks good!Adding this new enum definition for REST request methods is a great idea. It provides a clean way to represent and work with common HTTP methods like GET, POST, PATCH, DELETE, PUT in the codebase.
The enum is well-structured and follows the naming conventions. The code changes are approved.
src/cf/CommonModules/пбп_ОбщегоНазначенияКлиент/Ext/Module.bsl (1)
167-173: LGTM!The new exported functions
ЭтоWindowsКлиентandТипПлатформыКлиентаare great additions to the module. They enhance its capability to identify the client's operating system and platform type, which can be crucial for conditional logic based on the environment.The functions are implemented correctly and follow the existing coding style. The code changes are approved.
Also applies to: 175-186
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 167-167: src/cf/CommonModules/пбп_ОбщегоНазначенияКлиент/Ext/Module.bsl#L167
Нельзя использовать латинские и кириллические символы в одном идентификатореsrc/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаСписка/Ext/Form.xml (2)
98-98: Verify the impact of theDataPathchange.The
DataPathfor theInputFieldnamed "Метод" has been changed fromСписок.Методto~Список.Метод. This change indicates a potential shift in how the method data is accessed or represented, possibly introducing a new context or scope.Ensure that this change is thoroughly tested to confirm that it behaves as expected and does not introduce any unintended side effects or bugs.
114-114: Verify the impact of theMainTablesetting update.The
MainTablesetting has been updated fromCatalog.пбп_МетодыИнтеграцииtoCatalog.пбп_ИнтеграционныеПотоки. This change suggests a shift in the underlying data source for the dynamic list.Ensure that this change is thoroughly tested to confirm that the correct integration methods are retrieved and displayed in the form as expected.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаСписка/Ext/Form.xml (1)
97-101: LGTM!The new
LabelFieldelement designated for integration type enhances the user interface by providing more context and options related to integration types, thereby improving the overall functionality of the form.src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (4)
84-95: LGTM!The addition of new systems—"Rabbit Mq," "Kafka," and "Active Directory"—significantly enhances the module's functionality by expanding its integration capabilities.
107-109: LGTM!The renaming of function
ПредопределенныеЗначенияМетодыИнтеграции()toПредопределенныеЗначенияИнтеграционныеПотоки()reflects a shift in terminology that aligns with the new integration methods being introduced. The logic within this function has also been updated to accommodate the new naming and to ensure that the correct integration methods are returned.Also applies to: 113-114
236-236: LGTM!The renaming of function
ТаблицаПредопределенныхМетодыИнтеграции()toТаблицаПредопределенныхИнтеграционныеПотоки()aligns the function name with the new terminology for integration methods.
252-256: LGTM!The additional columns added to the table returned by
ТаблицаПредопределенныхНастройкиИнтеграции()are essential for defining the characteristics of the new integrations and support the new integration methods.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 254-254: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L254
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "100" и используйте эту константу вместо магического числа.src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаСписка/Ext/Form.xml (2)
83-90: LGTM!The addition of the new
LabelFieldelement named "Ссылка" is approved. It enriches the form's functionality by providing a new field that users can interact with, potentially improving data management or user experience.
117-117: Verify the impact of modifying theDataPath.The
DataPathfor the existingLabelFieldnamed "МетодИнтеграции" has been modified by prefixing it with a tilde (~). This may indicate a change in how the data is referenced or processed, suggesting a shift in the underlying logic or data handling for this field.Run the following script to verify the impact of this change:
Verification successful
Localized Impact of
DataPathModificationThe modification of the
DataPathfor theLabelFieldnamed "МетодИнтеграции" is localized to theФормаСпискаform. There are no other references to thisDataPathelsewhere in the codebase, indicating that the change does not affect other components or logic. The associatedContextMenuandExtendedTooltipare reused in another form, but theDataPathchange itself is specific to theФормаСпискаform.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the impact of modifying the `DataPath` for the `LabelField` named "МетодИнтеграции". # Test: Search for the usage of the `DataPath`. Expect: Only occurrences of the new `DataPath`. rg --type xml -A 5 $'Список\.МетодИнтеграции'Length of output: 760
Script:
#!/bin/bash # Search for references to the LabelField named "МетодИнтеграции" in the codebase. rg --type xml -A 5 'LabelField name="МетодИнтеграции"' # Search for references to the ContextMenu associated with "МетодИнтеграции". rg --type xml -A 5 'ContextMenu name="МетодИнтеграцииКонтекстноеМеню"' # Search for references to the ExtendedTooltip associated with "МетодИнтеграции". rg --type xml -A 5 'ExtendedTooltip name="МетодИнтеграцииРасширеннаяПодсказка"'Length of output: 3848
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form.xml (1)
1-240: LGTM!The addition of the new form for the catalog named
пбп_ИнтеграционныеПотокиis approved. The form is well-structured and follows the standard conventions for defining forms in 1C:Enterprise. The UI elements are properly named and have appropriate data paths. The attributes and commands are defined correctly.src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form.xml (1)
1-317: LGTM!The new form for loading an external component looks good. It follows the standard practices and includes the necessary controls and commands.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form.xml (7)
10-12: LGTM!The changes to the event handlers look good:
- The new
OnReadAtServerevent will allow additional processing when reading data from the server.- Renaming
OnWriteAtServertoBeforeWriteAtServerclarifies that the event occurs before writing data.
48-56: LGTM!The new
ТипИнтеграцииinput field is a good addition. It allows specifying the integration type and reacts to changes via theOnChangeevent.
81-88: LGTM!The renaming of
СерверtoСтрокаПодключенияis an improvement. It clarifies the purpose of the field.The new
StartChoiceevent will be useful for any additional processing when the user starts selecting a connection string.
99-103: LGTM!The new input fields
УчетнаяЗапись,ТипАвторизации, andИмяCOMОбъектаare good additions. They allow specifying additional integration settings and support editing multiple values.Also applies to: 105-110, 111-116
117-123: LGTM!The new
ИспользоватьПользовательскиеФункцииcheckbox field is a useful addition. It allows the user to easily enable or disable the use of custom functions.
124-128: LGTM!The new
ПодключитьВнешнююКомпонентуbutton is a good addition. It allows the user to easily connect an external component from a file by invoking the associated command.
268-282: LGTM!The new attributes
АдресВнешнейКомпонентыandВнешняяКомпонентаЗагруженаИзФайлаare necessary additions. They allow storing the address of the external component and tracking whether it was loaded from a file.Also applies to: 283-293
src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl (1)
36-37: LGTM!The changes to the conditional logic look good:
- Replacing the check for "СправочникМенеджер.пбп_МетодыИнтеграции" with "СправочникМенеджер.пбп_ИнтеграционныеПотоки" reflects the updated handling of predefined values based on the integration flows.
- The new conditional block that checks for the constant "пбп_ИспользоватьПользовательскиеФункции" and modifies the query string enhances the specificity of the query by excluding records based on the "ИдентификаторНастройки" when user-defined functions are used. This improves data retrieval accuracy.
Also applies to: 142-148
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 37-37: src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl#L37
Длина строки 123 превышает максимально допустимую 120src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (6)
180-234: LGTM!The new procedure
УстановитьСвойстваЭлементовenhances the control flow and logic of the integration process by setting the correct properties based on the integration type.
33-33: LGTM!The changes in the
ПриСозданииНаСервереprocedure, including invoking the newУстановитьСвойстваЭлементовprocedure and refactoring references to integration formats, are consistent and align with the new terminology.Also applies to: 44-44, 46-46
70-72: LGTM!The changes in the
ПриОткрытииprocedure, refactoring references to integration formats, are consistent and align with the new terminology.
92-93: LGTM!The changes in the
ЗапросИсходящийДокументСформирован_Подключаемыйprocedure, refactoring references to integration formats, are consistent and align with the new terminology.
104-105: LGTM!The changes in the
ЗапросВходящийДокументСформирован_Подключаемыйprocedure, refactoring references to integration formats, are consistent and align with the new terminology.
352-358: LGTM!The changes to the
ПолучитьФорматИнтеграцииfunction, updating it to acceptФорматИнтеграцииinstead ofФорматЗапросаИнтеграции, align with the new terminology for integration formats.src/cf/CommonModules/пбп_РегламентныеЗаданияСервер/Ext/Module.bsl (1)
265-291: Verify the impact of the conditional check and consider error handling.The introduction of the conditional check adds complexity and flexibility to the task processing logic. However, it's important to verify the impact of these changes:
Ensure that the behavior of the task processing logic remains consistent and correct in both scenarios - when the standard subsystem library exists and when it doesn't.
Consider adding error handling and logging mechanisms to handle potential exceptions or unexpected scenarios gracefully, especially in the case where the library doesn't exist and the alternative path is taken.
Verify that the changes do not introduce any unintended side effects or impact the overall functionality of the task processing system.
To verify the impact of the changes, run the following script:
Consider extracting the task deletion logic into a separate procedure to improve code reusability and maintainability. This would encapsulate the branching logic and provide a single point of entry for deleting tasks, making the code easier to understand and modify in the future.
Verification successful
Conditional Check and Error Handling Verification
The conditional check for the existence of the standard subsystem library is consistently used across the codebase, ensuring a standardized approach. However, consider enhancing the error handling by adding logging to capture the context of exceptions for better traceability.
- File:
src/cf/CommonModules/пбп_РегламентныеЗаданияСервер/Ext/Module.bsl- Lines: 265-291
Ensure that the task processing logic remains consistent and correct in both scenarios, with and without the library. Consider adding logging mechanisms to improve error traceability.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the impact of the conditional check on task processing. # Test 1: Search for usages of the `пбп_ОбщегоНазначенияВызовСервера.СуществуетБиблиотекаСтандартныхПодсистем` function. # Expect: The function should be used consistently across the codebase. ast-grep --lang bsl --pattern $'пбп_ОбщегоНазначенияВызовСервера.СуществуетБиблиотекаСтандартныхПодсистем()' # Test 2: Search for error handling and logging mechanisms related to task processing. # Expect: Adequate error handling and logging should be implemented. rg --type bsl -A 5 $'(ОтменитьТранзакцию|ВызватьИсключение|ЗаписьЖурналаРегистрации)' # Test 3: Verify the consistency of task processing behavior. # Expect: The behavior should remain consistent regardless of the presence of the library. ast-grep --lang bsl --pattern $'Задание = РегламентныеЗадания.НайтиПоУникальномуИдентификатору($_); $$$ Если Задание <> Неопределено Тогда Задание.Удалить(); КонецЕсли;'Length of output: 611
Script:
#!/bin/bash # Description: Verify the impact of the conditional check on task processing using alternative methods. # Test 1: Search for usages of the `СуществуетБиблиотекаСтандартныхПодсистем` function. rg 'СуществуетБиблиотекаСтандартныхПодсистем' -A 5 # Test 2: Search for error handling and logging mechanisms related to task processing. rg '(ОтменитьТранзакцию|ВызватьИсключение|ЗаписьЖурналаРегистрации)' -A 5 # Test 3: Verify the consistency of task processing behavior. rg 'Задание = РегламентныеЗадания.НайтиПоУникальномуИдентификатору' -A 5Length of output: 129191
src/cf/Configuration.xml (1)
272-273: LGTM!The changes made to the
Configuration.xmlfile enhance the configuration capabilities and expand the functionality of the system. The additions of new templates, jobs, types, constants, and enumerations provide more options for customization and integration. The renaming of existing entries improves clarity and relevance.The modifications are well-structured and align with the overall architecture of the configuration.
Also applies to: 332-332, 335-337, 341-341, 352-354
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl (1)
3-4: LGTM!The code changes are approved.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (1)
26-27: LGTM!The change enhances the initialization process by setting up the UI elements before checking if the object reference is filled.
src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl (4)
73-75: LGTM!The changes to the keys within the
СтруктураЗаписиИсторииstructure enhance clarity regarding the data being handled in the integration process.
215-217: LGTM!The renaming of the function and the update to its internal logic to reference
Справочники.пбп_ИнтеграционныеПотокиare approved.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 215-215: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L215
Необходимо добавить описание всех параметров метода
[warning] 215-215: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L215
Добавьте описание возвращаемого значения функции
440-450: LGTM!The changes to the procedure
ДополнительноОбработатьЗапросыИнтеграцииПередЗаписьюВИсториюto reflect the new naming conventions for incoming and outgoing messages are approved. The logic for processing these messages now aligns with the updated structure.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 446-446: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L446
Длина строки 134 превышает максимально допустимую 120
[warning] 440-440: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L440
Длина строки 136 превышает максимально допустимую 120
229-229: Verify the impact of the change in functionМетодИнтеграции.The function has been modified to reference
Справочники.пбп_ИнтеграционныеПотокиinstead ofСправочники.пбп_МетодыИнтеграции, indicating a shift in the underlying data structure being utilized.Ensure that this change is thoroughly tested to confirm that it behaves as expected and does not introduce any regressions or compatibility issues with other parts of the system that may rely on the previous data structure.
Run the following script to verify the usage of
Справочники.пбп_ИнтеграционныеПотоки:Verification successful
Successful Transition to New Data Structure Verified
The codebase has successfully transitioned to using
Справочники.пбп_ИнтеграционныеПотоки, with no remaining references to the old data structureСправочники.пбп_МетодыИнтеграции. Ensure thorough testing to confirm that the new structure behaves as expected in all contexts.
References to
Справочники.пбп_ИнтеграционныеПотокиfound in:
src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bslsrc/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаСписка/Ext/Form/Module.bslNo references to
Справочники.пбп_МетодыИнтеграцииfound.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of `Справочники.пбп_ИнтеграционныеПотоки` in the codebase. # Test: Search for references to `Справочники.пбп_ИнтеграционныеПотоки`. # Expect: Only valid usages, no references to the old data structure `Справочники.пбп_МетодыИнтеграции`. rg --type bsl $'Справочники\.пбп_ИнтеграционныеПотоки' # Test: Search for any remaining references to the old data structure `Справочники.пбп_МетодыИнтеграции`. # Expect: No results found. rg --type bsl $'Справочники\.пбп_МетодыИнтеграции'Length of output: 163
Script:
#!/bin/bash # Description: Verify the usage of `Справочники.пбп_ИнтеграционныеПотоки` in the codebase. # Test: Search for references to `Справочники.пбп_ИнтеграционныеПотоки`. # Expect: Only valid usages, no references to the old data structure `Справочники.пбп_МетодыИнтеграции`. rg $'Справочники\.пбп_ИнтеграционныеПотоки' # Test: Search for any remaining references to the old data structure `Справочники.пбп_МетодыИнтеграции`. # Expect: No results found. rg $'Справочники\.пбп_МетодыИнтеграции'Length of output: 621
src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form.xml (2)
96-96: Verify the impact of changing the data source for the "МетодИнтеграции" input field.Ensure that switching the
DataPathfromОбъект.МетодИнтеграцииtoОбъект.ИнтеграционныйПотокdoes not introduce any unintended side effects or break existing functionality.Run the following script to check for potential issues:
Verification successful
Change to
DataPathis isolated and does not introduce issues.The modification of the
DataPathtoОбъект.ИнтеграционныйПотокis limited to the form in question and does not affect other parts of the codebase. This suggests that the change is intentional and does not introduce any unintended side effects.
- Location:
src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form.xmlScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage and impact of `Объект.ИнтеграционныйПоток`. # Test: Search for read/write access to `Объект.ИнтеграционныйПоток`. # Expect: References limited to this form; no unexpected usages. rg --type xml 'Объект.ИнтеграционныйПоток' -g '*.xml'Length of output: 355
102-102: Verify the impact of changing the data source for the "ФорматЗапросаИнтеграции" input field.Ensure that switching the
DataPathfromОбъект.ФорматЗапросаИнтеграцииtoОбъект.ФорматИнтеграцииdoes not introduce any unintended side effects or break existing functionality.Run the following script to check for potential issues:
Verification successful
No issues found with the change to the "ФорматЗапросаИнтеграции" input field.
The modification of the
DataPathfromОбъект.ФорматЗапросаИнтеграцииtoОбъект.ФорматИнтеграцииis localized to this form and does not impact other parts of the system.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage and impact of `Объект.ФорматИнтеграции`. # Test: Search for read/write access to `Объект.ФорматИнтеграции`. # Expect: References limited to this form; no unexpected usages. rg --type xml 'Объект.ФорматИнтеграции' -g '*.xml'Length of output: 328
src/cf/Catalogs/пбп_ИнтеграционныеПотоки.xml (4)
5-9: The renaming of the catalog object is approved.The change from "пбп_МетодыИнтеграции" to "пбп_ИнтеграционныеПотоки" is consistent with the overall refactoring and improves the semantic meaning of the catalog.
27-31: The renaming of the catalog and its synonym is approved.The changes align with the overall refactoring and enhance the clarity and meaning of the catalog's name and synonym.
366-376: Verify the impact of renaming the "ПараметрыREST" tabular section to "ПараметрыВхода".Ensure that changing the tabular section name from "ПараметрыREST" to "ПараметрыВхода" does not introduce any unintended side effects or break existing functionality.
Run the following script to check for potential issues:
Verification successful
Renaming of "ПараметрыREST" to "ПараметрыВхода" is consistent and localized.
The change is well-integrated across the relevant XML files, with no unexpected references outside the intended scope. This suggests that the renaming does not introduce any unintended side effects or break existing functionality.
- References are found in
src/cf/Catalogs/пбп_ИнтеграционныеПотоки.xmlandsrc/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form.xml.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage and impact of the "ПараметрыВхода" tabular section. # Test: Search for read/write access to the "ПараметрыВхода" tabular section. # Expect: References limited to this catalog; no unexpected usages. rg --type xml 'ПараметрыВхода' -g '*.xml'Length of output: 6337
106-110: Verify the impact of renaming the "Метод" attribute to "ТочкаВхода".Ensure that changing the attribute name from "Метод" to "ТочкаВхода" does not introduce any unintended side effects or break existing functionality.
Run the following script to check for potential issues:
src/cf/Catalogs/пбп_НастройкиИнтеграции.xml (3)
405-405: The increase in the length of the "СтрокаПодключения" attribute is approved.Allowing longer connection strings is a reasonable adjustment to accommodate the needs of the integration subsystem.
858-858: Provide more information about the purpose and functionality of the new "ФормаЗагрузкиВнешнейКомпоненты" form.To better understand the impact of adding this form, please provide details on:
- The specific purpose of the form
390-394: Verify the impact of renaming the "Сервер" attribute to "СтрокаПодключения".Ensure that changing the attribute name from "Сервер" to "СтрокаПодключения" does not introduce any unintended side effects or break existing functionality.
Run the following script to check for potential issues:
Verification successful
Renaming of "Сервер" to "СтрокаПодключения" is localized and should not introduce side effects.
The attribute "СтрокаПодключения" is consistently referenced within the
пбп_НастройкиИнтеграцииcatalog and its associated form. Ensure that the UI elements using this attribute reflect the intended changes for consistency.
- References found in:
src/cf/Catalogs/пбп_НастройкиИнтеграции.xmlsrc/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form.xmlScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage and impact of the "СтрокаПодключения" attribute. # Test: Search for read/write access to the "СтрокаПодключения" attribute. # Expect: References limited to this catalog; no unexpected usages. rg --type xml 'СтрокаПодключения' -g '*.xml'Length of output: 927
src/cf/Catalogs/пбп_ИсторияИнтеграции.xml (4)
Line range hint
540-553: LGTM!The attribute renaming and type change are approved.
584-588: LGTM!The attribute renaming is approved.
632-636: LGTM!The attribute renaming is approved.
Line range hint
776-789: LGTM!The attribute renaming and type change are approved.
src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl (1)
505-517: LGTM!The new exported function
ПолучитьЗначениеКонстантыlooks good. It provides a handy way to dynamically retrieve the value of a constant by its name.
| Процедура ВыполнениеПользовательскихФункцийФайловыхОбменов(НастройкаИнтеграции, ПользовательскаяФункция) Экспорт | ||
|
|
||
| пбп_ОбщегоНазначенияСлужебный.ПриНачалеВыполненияРегламентногоЗадания( | ||
| Метаданные.РегламентныеЗадания.пбп_ВыполнениеПользовательскихФункцийФайловыхОбменов); | ||
|
|
||
| ПараметрыФункции = Новый Структура; | ||
| ПараметрыФункции.Вставить("НастройкаИнтеграции", НастройкаИнтеграции); | ||
|
|
||
| РезультатОбработки = Неопределено; | ||
|
|
||
| РезультатОбработки = пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами( | ||
| ПользовательскаяФункция, ПараметрыФункции); | ||
|
|
||
| КонецПроцедуры |
There was a problem hiding this comment.
The new procedure is implemented correctly, but there are a couple of improvements to consider:
- The variable
РезультатОбработкиis declared but never used after being assigned a value. Consider removing this variable if the return value fromпбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрамиis not needed.
-РезультатОбработки = Неопределено;
-
-РезультатОбработки = пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами(
+пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами(
ПользовательскаяФункция, ПараметрыФункции);- Add a description for the procedure to document its purpose and usage. This will improve the maintainability and readability of the code.
+// Выполняет пользовательскую функцию файловых обменов с указанными параметрами.
+//
+// Параметры:
+// НастройкаИнтеграции - <Тип.Вид> - настройка интеграции файловых обменов.
+// ПользовательскаяФункция - <Тип.Вид> - пользовательская функция для выполнения.
Процедура ВыполнениеПользовательскихФункцийФайловыхОбменов(НастройкаИнтеграции, ПользовательскаяФункция) Экспорт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. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Процедура ВыполнениеПользовательскихФункцийФайловыхОбменов(НастройкаИнтеграции, ПользовательскаяФункция) Экспорт | |
| пбп_ОбщегоНазначенияСлужебный.ПриНачалеВыполненияРегламентногоЗадания( | |
| Метаданные.РегламентныеЗадания.пбп_ВыполнениеПользовательскихФункцийФайловыхОбменов); | |
| ПараметрыФункции = Новый Структура; | |
| ПараметрыФункции.Вставить("НастройкаИнтеграции", НастройкаИнтеграции); | |
| РезультатОбработки = Неопределено; | |
| РезультатОбработки = пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами( | |
| ПользовательскаяФункция, ПараметрыФункции); | |
| КонецПроцедуры | |
| // Выполняет пользовательскую функцию файловых обменов с указанными параметрами. | |
| // | |
| // Параметры: | |
| // НастройкаИнтеграции - <Тип.Вид> - настройка интеграции файловых обменов. | |
| // ПользовательскаяФункция - <Тип.Вид> - пользовательская функция для выполнения. | |
| Процедура ВыполнениеПользовательскихФункцийФайловыхОбменов(НастройкаИнтеграции, ПользовательскаяФункция) Экспорт | |
| пбп_ОбщегоНазначенияСлужебный.ПриНачалеВыполненияРегламентногоЗадания( | |
| Метаданные.РегламентныеЗадания.пбп_ВыполнениеПользовательскихФункцийФайловыхОбменов); | |
| ПараметрыФункции = Новый Структура; | |
| ПараметрыФункции.Вставить("НастройкаИнтеграции", НастройкаИнтеграции); | |
| пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами( | |
| ПользовательскаяФункция, ПараметрыФункции); | |
| КонецПроцедуры |
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 39-39: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L39
Удалите неиспользуемую переменную РезультатОбработки
[notice] 31-31: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L31
Добавьте описание метода программного интерфейса
| Процедура ПередЗаписью(Отказ) | ||
|
|
||
| Если ОбменДанными.Загрузка Тогда | ||
| Возврат; | ||
| КонецЕсли; | ||
|
|
||
| Если Не ЭтоНовый() И пбп_ОбщегоНазначенияПовтИсп.ПолучитьЗначениеКонстанты("пбп_ИспользоватьПользовательскиеФункции") Тогда | ||
| Запрос = Новый Запрос; | ||
| Запрос.УстановитьПараметр("Ссылка", Ссылка); | ||
| Запрос.Текст = "ВЫБРАТЬ | ||
| | пбп_ИнтеграционныеПотоки.ПользовательскаяФункция КАК ПользовательскаяФункция, | ||
| | пбп_ИнтеграционныеПотоки.НастройкаИнтеграции КАК НастройкаИнтеграции | ||
| |ИЗ | ||
| | Справочник.пбп_ИнтеграционныеПотоки КАК пбп_ИнтеграционныеПотоки | ||
| |ГДЕ | ||
| | пбп_ИнтеграционныеПотоки.Ссылка = &Ссылка"; | ||
| РезультатЗапроса = Запрос.Выполнить(); | ||
| ВыборкаРезультатаЗапроса = РезультатЗапроса.Выбрать(); | ||
|
|
||
| ПользовательскаяФункцияДоИзменения = Справочники.пбп_ПользовательскиеФункции.ПустаяСсылка(); | ||
| НастройкаИнтеграцииДоИзменения = Справочники.пбп_НастройкиИнтеграции.ПустаяСсылка(); | ||
| Если ВыборкаРезультатаЗапроса.Следующий() Тогда | ||
| ПользовательскаяФункцияДоИзменения = ВыборкаРезультатаЗапроса.ПользовательскаяФункция; | ||
| НастройкаИнтеграцииДоИзменения = ВыборкаРезультатаЗапроса.НастройкаИнтеграции; | ||
| КонецЕсли; | ||
|
|
||
| Если ЗначениеЗаполнено(ПользовательскаяФункцияДоИзменения) | ||
| И Не ЗначениеЗаполнено(ПользовательскаяФункция) Тогда | ||
| УдалитьРегламентноеЗаданиеПоКлючу(Строка(Ссылка.УникальныйИдентификатор())); | ||
| КонецЕсли; | ||
|
|
||
| НеобходимоОбновлениеПараметровЗадания = Ложь; | ||
| Если ЗначениеЗаполнено(ПользовательскаяФункцияДоИзменения) И ЗначениеЗаполнено(ПользовательскаяФункция) | ||
| И ПользовательскаяФункцияДоИзменения <> ПользовательскаяФункция Тогда | ||
| НеобходимоОбновлениеПараметровЗадания = Истина; | ||
| ИначеЕсли ЗначениеЗаполнено(НастройкаИнтеграцииДоИзменения) И ЗначениеЗаполнено(НастройкаИнтеграции) | ||
| И НастройкаИнтеграцииДоИзменения <> НастройкаИнтеграции Тогда | ||
| НеобходимоОбновлениеПараметровЗадания = Истина; | ||
| КонецЕсли; | ||
|
|
||
| Если НеобходимоОбновлениеПараметровЗадания Тогда | ||
| КлючЗадания = Строка(Ссылка.УникальныйИдентификатор()); | ||
|
|
||
| ОтборЗадания = Новый Структура; | ||
| ОтборЗадания.Вставить("Ключ", КлючЗадания); | ||
| МассивРегламентныхЗаданий = пбп_РегламентныеЗаданияСервер.НайтиЗадания(ОтборЗадания); | ||
|
|
||
| Если МассивРегламентныхЗаданий.Количество() Тогда | ||
| РегламентноеЗадание = МассивРегламентныхЗаданий[0]; | ||
|
|
||
| ПараметрыРегламентногоЗадания = Новый Массив; | ||
| ПараметрыРегламентногоЗадания.Добавить(НастройкаИнтеграции); | ||
| ПараметрыРегламентногоЗадания.Добавить(ПользовательскаяФункция); | ||
|
|
||
| РегламентноеЗадание.Параметры = ПараметрыРегламентногоЗадания; | ||
| РегламентноеЗадание.Записать(); | ||
| КонецЕсли; | ||
| КонецЕсли; | ||
| КонецЕсли; | ||
|
|
||
| КонецПроцедуры |
There was a problem hiding this comment.
Approved with suggestions
The ПередЗаписью event handler correctly updates the scheduled job based on the object changes. However, consider the following suggestions to improve the code:
-
Reduce the cognitive complexity by extracting the scheduled job update logic into a separate procedure. This will make the code more readable and maintainable.
-
Refactor the repeated code blocks in the
ИначеЕслиstatement to reduce duplication. Extract the common logic into a separate procedure or function. -
Although not critical, consider adding an empty
Иначеbranch to theЕслиstatements for completeness and clarity.
Please let me know if you would like assistance with implementing these suggestions.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 62-62: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L62
Синтаксическая конструкция вида "Если...Тогда...ИначеЕсли..." должна содержать ветвь "Иначе".
[failure] 24-24: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L24
Уменьшите когнитивную сложность "ПередЗаписью" с 18 до 15
[warning] 58-58: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L58
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
[warning] 30-30: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl#L30
Длина строки 124 превышает максимально допустимую 120
| // Возвращаемое значение: | ||
| // Булево - если нет клиентского приложения, возвращается Ложь. | ||
| // | ||
| Функция ЭтоWindowsКлиент() Экспорт |
There was a problem hiding this comment.
Исправьте смешение латинских и кириллических символов в имени идентификатора.
Статический анализ кода выявил следующую проблему:
Нельзя использовать латинские и кириллические символы в одном идентификаторе.
Предлагаю исправить имя функции, чтобы использовать только кириллические символы:
-Функция ЭтоWindowsКлиент() Экспорт
+Функция ЭтоКлиентWindows() Экспорт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. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Функция ЭтоWindowsКлиент() Экспорт | |
| Функция ЭтоКлиентWindows() Экспорт |
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 167-167: src/cf/CommonModules/пбп_ОбщегоНазначенияКлиент/Ext/Module.bsl#L167
Нельзя использовать латинские и кириллические символы в одном идентификаторе
| НоваяНастройка = Результат.Добавить(); | ||
| НоваяНастройка.Наименование = "Rabbit Mq"; | ||
| НоваяНастройка.ИдентификаторНастройки = "RabbitMq"; | ||
| НоваяНастройка.ИнтегрируемаяСистема = Справочники.пбп_ИнтегрируемыеСистемы.НайтиПоРеквизиту( | ||
| "ИдентификаторНастройки", "RabbitMq"); | ||
| НоваяНастройка.ТипИнтеграции = Справочники.пбп_ТипыИнтеграций.RabbitMq; | ||
| НоваяНастройка.ТипАвторизации = Перечисления.пбп_ТипыАвторизации.Базовая; | ||
|
|
||
| НоваяНастройка = Результат.Добавить(); | ||
| НоваяНастройка.Наименование = "Kafka"; | ||
| НоваяНастройка.ИдентификаторНастройки = "Kafka"; | ||
| НоваяНастройка.ИнтегрируемаяСистема = Справочники.пбп_ИнтегрируемыеСистемы.НайтиПоРеквизиту( | ||
| "ИдентификаторНастройки", "Kafka"); | ||
| НоваяНастройка.ТипИнтеграции = Справочники.пбп_ТипыИнтеграций.Kafka; | ||
| НоваяНастройка.ТипАвторизации = Перечисления.пбп_ТипыАвторизации.Базовая; | ||
|
|
||
| НоваяНастройка = Результат.Добавить(); | ||
| НоваяНастройка.Наименование = "Active directory"; | ||
| НоваяНастройка.ИдентификаторНастройки = "ActiveDirectory"; | ||
| НоваяНастройка.СтрокаПодключения = "Provider=""ADsDSOObject"""; | ||
| НоваяНастройка.ИнтегрируемаяСистема = Справочники.пбп_ИнтегрируемыеСистемы.НайтиПоРеквизиту( | ||
| "ИдентификаторНастройки", "ActiveDirectory"); | ||
| НоваяНастройка.ТипИнтеграции = Справочники.пбп_ТипыИнтеграций.COM; | ||
| НоваяНастройка.ТипАвторизации = Перечисления.пбп_ТипыАвторизации.Базовая; | ||
| НоваяНастройка.ИмяCOMОбъекта = "ADODB.Connection"; |
There was a problem hiding this comment.
Address the following issues to improve code quality:
- Avoid repeated use of string literal "Kafka".
- Avoid repeated use of string literal "ИдентификаторНастройки".
- Replace magic numbers "200" and "100" with named constants.
To address these issues:
- Extract the repeated string literals into named constants.
- Replace the magic numbers with named constants that convey their meaning.
For example:
// Named constants for string literals
Const KAFKA_IDENTIFIER = "Kafka";
Const SETTING_IDENTIFIER = "ИдентификаторНастройки";
// Named constants for magic numbers
Const MAX_CONNECTION_STRING_LENGTH = 200;
Const MAX_COM_OBJECT_NAME_LENGTH = 100;Then, use these named constants in place of the string literals and magic numbers throughout the code.
Do you want me to submit a PR with these changes?
Also applies to: 254-254, 256-256
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 144-144: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L144
Необходимо избавиться от многократного использования строкового литерала "Kafka"
[warning] 139-139: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L139
Необходимо избавиться от многократного использования строкового литерала "ИдентификаторНастройки"
| // См. пбп_ОбщегоНазначенияСервер.ПолучитьЗначениеКонстанты. | ||
| // | ||
| Функция ПолучитьЗначениеКонстанты(ИмяКонстанты) Экспорт | ||
|
|
||
| Возврат пбп_ОбщегоНазначенияСервер.ПолучитьЗначениеКонстанты(ИмяКонстанты); | ||
|
|
||
| КонецФункции |
There was a problem hiding this comment.
LGTM, but address the static analysis hints.
The addition of the new exported function named ПолучитьЗначениеКонстанты is approved. It enhances the module's functionality by providing a direct interface for obtaining constant values, thereby improving code modularity and reusability.
However, please address the following static analysis hints to improve the code quality:
- Add a description of the return value of the function.
- Add a description of all parameters of the function.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 118-118: src/cf/CommonModules/пбп_ОбщегоНазначенияПовтИсп/Ext/Module.bsl#L118
Добавьте описание возвращаемого значения функции
[warning] 118-118: src/cf/CommonModules/пбп_ОбщегоНазначенияПовтИсп/Ext/Module.bsl#L118
Необходимо добавить описание всех параметров метода
| ИначеЕсли ТипЗначения = Тип("СправочникМенеджер.пбп_МетодыИнтеграции") Тогда | ||
| ТЗПредопределенныхЗначений = пбп_ПредопределенныеЗначенияПереопределяемый.ПредопределенныеЗначенияМетодыИнтеграции(); | ||
| ИначеЕсли ТипЗначения = Тип("СправочникМенеджер.пбп_ИнтеграционныеПотоки") Тогда | ||
| ТЗПредопределенныхЗначений = пбп_ПредопределенныеЗначенияПереопределяемый.ПредопределенныеЗначенияИнтеграционныеПотоки(); |
There was a problem hiding this comment.
Fix the line length issue.
The static analysis tool reports that the length of the line at 37 is 123, which exceeds the maximum allowed length of 120.
To improve code readability and maintainability, please break the long line into multiple lines. For example:
ИначеЕсли ТипЗначения = Тип("СправочникМенеджер.пбп_ИнтеграционныеПотоки") Тогда
ТЗПредопределенныхЗначений =
пбп_ПредопределенныеЗначенияПереопределяемый.ПредопределенныеЗначенияИнтеграционныеПотоки();Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 37-37: src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl#L37
Длина строки 123 превышает максимально допустимую 120
| @@ -0,0 +1,418 @@ | |||
| // Библиотека проектных подсистем для упрощения разработки архитектуры на 1С: Предприятие 8, | |||
There was a problem hiding this comment.
Reminder: Add tests.
The TODO comment indicates that tests are missing.
Do you want me to generate the unit testing code or open a GitHub issue to track this task?
| @@ -141,7 +141,7 @@ | |||
|
|
|||
| УстановитьПривилегированныйРежим(Истина); | |||
|
|
|||
| СтруктураНастроек = пбп_ОбщегоНазначенияСлужебный.ЗначенияРеквизитовОбъекта(НастройкаИнтеграции, "Сервер, Порт"); | |||
| СтруктураНастроек = пбп_ОбщегоНазначенияСлужебный.ЗначенияРеквизитовОбъекта(НастройкаИнтеграции, "СтрокаПодключения, Порт"); | |||
There was a problem hiding this comment.
Address warnings from static analysis tools.
The following warnings from static analysis tools should be addressed:
-
Line length warnings:
- Lines 144, 165, 440, and 446 exceed the maximum allowed limit of 120 characters.
Wrap these long lines to adhere to the line length limit and improve code readability.
-
Missing documentation warnings:
- Function
ПолучитьСтруктуруПараметровВходаis missing documentation for its parameters and return value.
Add missing documentation for the function parameters and return value to improve code maintainability.
- Function
Also applies to: 165-165, 215-215, 440-440, 446-446
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 144-144: src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl#L144
Длина строки 125 превышает максимально допустимую 120
Analysis Details29 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 8
Outside diff range and nitpick comments (2)
src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (2)
Line range hint
229-235: LGTM with suggestions
The changes to use the new terminology
ФорматИнтеграцииinstead ofФорматЗапросаИнтеграцииare approved.Consider adding the
Значmodifier to theТекстСообщенияИнтеграцииparameter to indicate that it is passed by value.-Функция ОтформатироватьСообщениеИнтеграции(ТекстСообщенияИнтеграции) +Функция ОтформатироватьСообщениеИнтеграции(Знач ТекстСообщенияИнтеграции)
- The line length exceeds the maximum allowed limit of 120 characters. Consider breaking the long line into multiple lines to improve readability and adhere to the coding style guidelines.
-Если Объект.ФорматИнтеграции = Перечисления.пбп_ФорматыИнтеграций.XML И СтрНайти(ТекстСообщенияИнтеграции, "xml") <> 0 Тогда +Если Объект.ФорматИнтеграции = Перечисления.пбп_ФорматыИнтеграций.XML + И СтрНайти(ТекстСообщенияИнтеграции, "xml") <> 0 ТогдаTools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 200-200: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L200
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
[warning] 229-229: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L229
Установите модификатор "Знач" для параметра ТекстСообщенияИнтеграции метода ОтформатироватьСообщениеИнтеграции
[warning] 231-231: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L231
Длина строки 125 превышает максимально допустимую 120
321-323: LGTM with a suggestion
The changes to use the new terminology
ВходящееСообщениеandИсходящееСообщениеinstead ofЗапросВходящийandЗапросИсходящийare approved.The line lengths exceed the maximum allowed limit of 120 characters. Consider breaking the long lines into multiple lines to improve readability and adhere to the coding style guidelines.
-Элементы.ЗапросИсходящий.Документ.defaultView.start(ОтформатироватьСообщениеИнтеграции(Объект.ИсходящееСообщение), "nerd"); +Элементы.ЗапросИсходящий.Документ.defaultView.start( + ОтформатироватьСообщениеИнтеграции(Объект.ИсходящееСообщение), + "nerd" +); -Элементы.ЗапросВходящий.Документ.defaultView.start(ОтформатироватьСообщениеИнтеграции(Объект.ВходящееСообщение), "nerd"); +Элементы.ЗапросВходящий.Документ.defaultView.start( + ОтформатироватьСообщениеИнтеграции(Объект.ВходящееСообщение), + "nerd" +);Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 323-323: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L323
Длина строки 123 превышает максимально допустимую 120
[warning] 321-321: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L321
Длина строки 125 превышает максимально допустимую 120
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (13)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl (3 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl (1 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (13 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (4 hunks)
- src/cf/Catalogs/пбп_ТипыИнтеграций/Ext/ManagerModule.bsl (1 hunks)
- src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl (6 hunks)
- src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl (2 hunks)
- src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (5 hunks)
- src/cf/CommonModules/пбп_РегламентныеЗаданияСервер/Ext/Module.bsl (2 hunks)
Files skipped from review as they are similar to previous changes (5)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl
- src/cf/Catalogs/пбп_ТипыИнтеграций/Ext/ManagerModule.bsl
- src/cf/CommonModules/пбп_ИнтеграцииСервер/Ext/Module.bsl
- src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl
Additional context used
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl
[warning] 46-46: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L46
Удалите неиспользуемую переменную РезультатОбработкиsrc/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl
[warning] 97-97: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L97
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 87-87: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L87
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 60-60: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L60
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 92-92: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L92
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 66-66: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L66
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 73-73: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L73
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 101-101: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L101
Нельзя использовать латинские и кириллические символы в одном идентификатореsrc/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl
[warning] 100-100: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl#L100
Уберите инициализацию параметров метода "Вставить" вложенными методамиsrc/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl
[warning] 146-146: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L146
Необходимо избавиться от многократного использования строкового литерала "Kafka"
[warning] 258-258: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L258
Создайте константу с понятным названием, присвойте ей значение "100" и используйте эту константу вместо магического числа.
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl
[warning] 200-200: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L200
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
[warning] 229-229: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L229
Установите модификатор "Знач" для параметра ТекстСообщенияИнтеграции метода ОтформатироватьСообщениеИнтеграции
[warning] 231-231: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L231
Длина строки 125 превышает максимально допустимую 120
[warning] 323-323: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L323
Длина строки 123 превышает максимально допустимую 120
[warning] 321-321: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L321
Длина строки 125 превышает максимально допустимую 120src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl
[warning] 206-206: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L206
Необходимо избавиться от многократного использования строкового литерала "ru = 'Конечная точка'"src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl
[notice] 66-66: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L66
Избавьтесь от избыточного обращения внутри модуля через его имя или псевдоним ЭтотОбъект
[warning] 172-172: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L172
Длина метода "УстановитьСвойстваЭлементов" равна 202, что больше установленного лимита в 200 строк
[warning] 445-445: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L445
Установите модификатор "Знач" для параметра Результат метода ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере
[warning] 181-181: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L181
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 275-275: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L275
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
[warning] 275-275: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L275
Необходимо избавиться от многократного использования строкового литерала "ru = 'Сервер'"
Additional comments not posted (46)
src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl (2)
38-49: Skipped generating a similar comment as the existing comment from the previous review is still valid.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 46-46: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L46
Удалите неиспользуемую переменную РезультатОбработки
31-37: Add a description for the procedure.The procedure is missing a description. Add a description to document its purpose and usage. This will improve the maintainability and readability of the code.
Apply this diff to add a description:
+// Выполняет пользовательскую функцию файловых обменов с указанными параметрами. +// +// Параметры: +// НастройкаИнтеграции - СправочникСсылка.пбп_НастройкиИнтеграции - настройка интеграции интеграционного +// потока, для которого выполняется регламентное задание. +// ПользовательскаяФункция - СправочникСсылка.пбп_ПользовательскиеФункции - выполняемая пользовательская функция. Процедура ВыполнениеПользовательскихФункцийФайловыхОбменов(НастройкаИнтеграции, ПользовательскаяФункция) ЭкспортLikely invalid or redundant comment.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl (1)
24-45: LGTM!The code changes are approved.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl (4)
23-40: LGTM!The code changes are approved.
47-63: LGTM!The code changes are approved.
66-94: LGTM!The code changes are approved.
97-123: LGTM!The code changes are approved.
The static analysis warning for line 100 is a false positive and can be ignored. The code is not using nested methods to initialize the parameters of the "Вставить" method. It is using a ternary operator to conditionally set the value of the "ИзмененВручную" parameter, which is a valid and concise way to initialize the parameter.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 100-100: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl#L100
Уберите инициализацию параметров метода "Вставить" вложенными методамиsrc/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (4)
84-95: LGTM!The code changes are approved.
107-114: LGTM!The code changes are approved.
134-161: LGTM!The code changes are approved.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 146-146: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L146
Необходимо избавиться от многократного использования строкового литерала "Kafka"
254-259: LGTM!The code changes are approved.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 258-258: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L258
Создайте константу с понятным названием, присвойте ей значение "100" и используйте эту константу вместо магического числа.
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (10)
33-33: LGTM!The changes to invoke the new procedure
УстановитьСвойстваЭлементов()at the beginning ofПриСозданииНаСервереare approved.
70-73: LGTM!The changes to use the new terminology
ФорматИнтеграцииinstead ofФорматЗапросаИнтеграцииare approved.
92-93: LGTM!The changes to use the new terminology
ФорматИнтеграцииinstead ofФорматЗапросаИнтеграцииare approved.
104-105: LGTM!The changes to use the new terminology
ФорматИнтеграцииinstead ofФорматЗапросаИнтеграцииare approved.
120-121: LGTM!The changes to use the new terminology
ФорматИнтеграцииinstead ofФорматЗапросаИнтеграцииare approved.
139-140: LGTM!The changes to use the new terminology
ФорматИнтеграцииinstead ofФорматЗапросаИнтеграцииare approved.
163-165: LGTM!The changes to use the new terminology
ВходящееСообщениеandИсходящееСообщениеinstead ofЗапросВходящийandЗапросИсходящийare approved.
179-218: LGTM!The new procedure
УстановитьСвойстваЭлементовis approved. It correctly sets the properties of elements based on the integration flow type, enhancing the control flow and logic of the integration process.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 200-200: src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L200
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
220-226: LGTM!The new function
НаправлениеИнтеграцииВыгруженоis approved. It correctly determines if the integration direction is exported based on the exchange status.
Line range hint
287-296: LGTM!The changes to use the new terminology
ВходящееСообщениеandИсходящееСообщениеinstead ofЗапросВходящийandЗапросИсходящийare approved.Also applies to: 304-312
src/cf/CommonModules/пбп_РегламентныеЗаданияСервер/Ext/Module.bsl (3)
305-324: LGTM!The code changes are approved.
326-337: LGTM!The code changes are approved.
265-269: LGTM, but verify the conditional check.The code changes are approved.
However, ensure that this change is thoroughly tested to confirm that the conditional check is working as expected and that the correct function is called based on the existence of the standard subsystem library.
Run the following script to verify the conditional check:
Verification successful
Conditional Check Verified
The conditional logic in the code correctly calls the appropriate function based on the existence of the standard subsystem library. Both functions,
УдалитьРегламентноеЗаданиеСИспользованиемКэшаandУдалитьРегламентноеЗаданиеБезКэша, are present and used as expected.
УдалитьРегламентноеЗаданиеСИспользованиемКэшаis defined and used.УдалитьРегламентноеЗаданиеБезКэшаis defined and used.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the conditional check is working as expected. # Test 1: Search for the usage of `УдалитьРегламентноеЗаданиеСИспользованиемКэша` function. # Expect: At least one occurrence if the standard subsystem library exists. rg --type bsl -A 5 $'УдалитьРегламентноеЗаданиеСИспользованиемКэша' # Test 2: Search for the usage of `УдалитьРегламентноеЗаданиеБезКэша` function. # Expect: At least one occurrence if the standard subsystem library does not exist. rg --type bsl -A 5 $'УдалитьРегламентноеЗаданиеБезКэша'Length of output: 729
Script:
#!/bin/bash # Description: Verify the conditional check is working as expected by searching for the functions without specifying the file type. # Test 1: Search for the usage of `УдалитьРегламентноеЗаданиеСИспользованиемКэша` function. rg -F -A 5 'УдалитьРегламентноеЗаданиеСИспользованиемКэша' # Test 2: Search for the usage of `УдалитьРегламентноеЗаданиеБезКэша` function. rg -F -A 5 'УдалитьРегламентноеЗаданиеБезКэша'Length of output: 3036
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl (11)
23-28: LGTM!The procedure looks good. It's preparing the form by calling two other procedures.
31-37: LGTM!The procedure looks good. It's setting a flag on the current object if it's modified and the identifier is not empty.
40-49: LGTM!The procedure looks good. It's creating or updating the scheduled job based on the
ИспользоватьРегламентноеЗаданиеflag.
52-64: LGTM!The procedure looks good. It's prompting the user to save the catalog item if the reference is not filled in, setting the visibility of the schedule setup link, and updating the scheduled job on the server.
67-71: LGTM!The procedure looks good. It's setting element properties by calling
УстановитьСвойстваЭлементовwhen the integration setting is changed.
74-86: LGTM!The procedure looks good. It's setting selection parameters for the integration setting if the identifier is empty by adding a new selection parameter and assigning it to the
ПараметрыВыбораproperty of the passed element.
89-100: LGTM!The procedure looks good. It's populating selection data for the integration setting if the identifier is empty by loading values into a value list and assigning it to the
ДанныеВыбораparameter.
107-118: LGTM!The procedure looks good. It's opening a dialog to configure the schedule of the scheduled job if the reference is filled in.
121-130: LGTM!The procedure looks good. It's setting the visibility and availability of the scheduled job settings group based on the visibility of the schedule settings and whether the reference is filled in.
137-149: LGTM!The procedure looks good. It's preparing the form on the server by setting the visibility of the scheduled job settings group and retrieving the scheduled job if the reference is filled in and the schedule settings are visible.
277-290: LGTM!The procedure looks good. It's setting the visibility of form elements to the appropriate values for the file exchange integration type.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (11)
26-27: LGTM!The code changes are approved.
49-56: LGTM!The code changes are approved.
59-70: LGTM!The code changes are approved.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[notice] 66-66: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L66
Избавьтесь от избыточного обращения внутри модуля через его имя или псевдоним ЭтотОбъект
95-99: LGTM!The code changes are approved.
102-117: LGTM!The code changes are approved.
152-165: LGTM!The code changes are approved.
381-396: LGTM!The code changes are approved.
399-410: LGTM!The code changes are approved.
413-422: LGTM!The code changes are approved.
455-461: LGTM!The code changes are approved.
66-66: Remove the redundant module referenceTools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[notice] 66-66: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L66
Избавьтесь от избыточного обращения внутри модуля через его имя или псевдоним ЭтотОбъект
| РезультатОбработки = пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами( | ||
| ПользовательскаяФункция, ПараметрыФункции); |
There was a problem hiding this comment.
Remove the unused variable РезультатОбработки.
The variable РезультатОбработки is declared but never used after being assigned a value. Remove this variable as the return value from пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами is not needed.
Apply this diff to remove the unused variable:
-РезультатОбработки = пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами(
+пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами(
ПользовательскаяФункция, ПараметрыФункции);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. Thoroughly test & benchmark the code to ensure it meets the requirements.
| РезультатОбработки = пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами( | |
| ПользовательскаяФункция, ПараметрыФункции); | |
| пбп_ОбщегоНазначенияСервер.ВыполнитьПользовательскуюФункциюСПараметрами( | |
| ПользовательскаяФункция, ПараметрыФункции); |
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 46-46: src/cf/CommonModules/пбп_МетодыРегламентныхЗаданийСервер/Ext/Module.bsl#L46
Удалите неиспользуемую переменную РезультатОбработки
| Процедура ПередЗаписью(Отказ) | ||
|
|
||
| Если ОбменДанными.Загрузка Тогда | ||
| Возврат; | ||
| КонецЕсли; | ||
|
|
||
| ЭлементНаследования = Справочники.пбп_ТипыИнтеграций | ||
| .ПолучитьПредопределенныйЭлементНаследованияНастроекТипаИнтеграции(ТипИнтеграции); | ||
| Если ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.Каталог Тогда | ||
| Порт = ""; | ||
| ВнешняяКомпонента = Неопределено; | ||
| УчетнаяЗапись = Неопределено; | ||
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | ||
| ИмяCOMОбъекта = ""; | ||
| ИмяФайлаВнешнейКомпоненты = ""; | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.FTPРесурсы Тогда | ||
| ВнешняяКомпонента = Неопределено; | ||
| УчетнаяЗапись = Неопределено; | ||
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | ||
| ИмяCOMОбъекта = ""; | ||
| ИмяФайлаВнешнейКомпоненты = ""; | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ПочтовыйКлиент Тогда | ||
| СтрокаПодключения = ""; | ||
| Порт = ""; | ||
| ВнешняяКомпонента = Неопределено; | ||
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | ||
| ИмяCOMОбъекта = ""; | ||
| ИмяФайлаВнешнейКомпоненты = ""; | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.COM | ||
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.КоманднаяСтрока Тогда | ||
| Порт = ""; | ||
| ВнешняяКомпонента = Неопределено; | ||
| УчетнаяЗапись = Неопределено; | ||
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | ||
| ИспользоватьПользовательскиеФункции = Ложь; | ||
| ИмяФайлаВнешнейКомпоненты = ""; | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.RESTAPI | ||
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.SOAP Тогда | ||
| ВнешняяКомпонента = Неопределено; | ||
| УчетнаяЗапись = Неопределено; | ||
| ИмяCOMОбъекта = Неопределено; | ||
| ИспользоватьПользовательскиеФункции = Ложь; | ||
| ИмяФайлаВнешнейКомпоненты = ""; | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ВнешняяКомпонента Тогда | ||
| УчетнаяЗапись = Неопределено; | ||
| ИмяCOMОбъекта = Неопределено; | ||
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | ||
| ИспользоватьПользовательскиеФункции = Ложь; | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.БрокерыСообщений Тогда | ||
| УчетнаяЗапись = Неопределено; | ||
| ИмяCOMОбъекта = Неопределено; | ||
| ИспользоватьПользовательскиеФункции = Ложь; | ||
| Иначе | ||
| УчетнаяЗапись = Неопределено; | ||
| ИмяCOMОбъекта = Неопределено; | ||
| КонецЕсли; | ||
|
|
||
| КонецПроцедуры |
There was a problem hiding this comment.
Fix the identifier naming issues.
The code changes are approved.
However, please address the following issues identified by static analysis:
- Line 60: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта. - Line 66: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта. - Line 73: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта. - Line 87: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта. - Line 92: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта. - Line 97: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта. - Line 101: Mixing Latin and Cyrillic characters in identifier
ИмяCOMОбъекта.
Apply this diff to fix the identifier naming issues:
- ИмяCOMОбъекта = "";
+ ИмяCOMОбъекта = "";- ИмяCOMОбъекта = "";
+ ИмяCOMОбъекта = "";- ИмяCOMОбъекта = "";
+ ИмяCOMОбъекта = "";- ИмяCOMОбъекта = Неопределено;
+ ИмяCOMОбъекта = Неопределено;- ИмяCOMОбъекта = Неопределено;
+ ИмяCOMОбъекта = Неопределено;- ИмяCOMОбъекта = Неопределено;
+ ИмяCOMОбъекта = Неопределено;- ИмяCOMОбъекта = Неопределено;
+ ИмяCOMОбъекта = Неопределено;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. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Процедура ПередЗаписью(Отказ) | |
| Если ОбменДанными.Загрузка Тогда | |
| Возврат; | |
| КонецЕсли; | |
| ЭлементНаследования = Справочники.пбп_ТипыИнтеграций | |
| .ПолучитьПредопределенныйЭлементНаследованияНастроекТипаИнтеграции(ТипИнтеграции); | |
| Если ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.Каталог Тогда | |
| Порт = ""; | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИмяCOMОбъекта = ""; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.FTPРесурсы Тогда | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИмяCOMОбъекта = ""; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ПочтовыйКлиент Тогда | |
| СтрокаПодключения = ""; | |
| Порт = ""; | |
| ВнешняяКомпонента = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИмяCOMОбъекта = ""; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.COM | |
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.КоманднаяСтрока Тогда | |
| Порт = ""; | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.RESTAPI | |
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.SOAP Тогда | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ВнешняяКомпонента Тогда | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.БрокерыСообщений Тогда | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| Иначе | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| КонецЕсли; | |
| КонецПроцедуры | |
| Процедура ПередЗаписью(Отказ) | |
| Если ОбменДанными.Загрузка Тогда | |
| Возврат; | |
| КонецЕсли; | |
| ЭлементНаследования = Справочники.пбп_ТипыИнтеграций | |
| .ПолучитьПредопределенныйЭлементНаследованияНастроекТипаИнтеграции(ТипИнтеграции); | |
| Если ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.Каталог Тогда | |
| Порт = ""; | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИмяCOMОбъекта = ""; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.FTPРесурсы Тогда | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИмяCOMОбъекта = ""; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ПочтовыйКлиент Тогда | |
| СтрокаПодключения = ""; | |
| Порт = ""; | |
| ВнешняяКомпонента = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИмяCOMОбъекта = ""; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.COM | |
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.КоманднаяСтрока Тогда | |
| Порт = ""; | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.RESTAPI | |
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.SOAP Тогда | |
| ВнешняяКомпонента = Неопределено; | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| ИмяФайлаВнешнейКомпоненты = ""; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ВнешняяКомпонента Тогда | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| ТипАвторизации = Перечисления.пбп_ТипыАвторизации.ПустаяСсылка(); | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.БрокерыСообщений Тогда | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| ИспользоватьПользовательскиеФункции = Ложь; | |
| Иначе | |
| УчетнаяЗапись = Неопределено; | |
| ИмяCOMОбъекта = Неопределено; | |
| КонецЕсли; | |
| КонецПроцедуры |
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 97-97: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L97
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 87-87: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L87
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 60-60: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L60
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 92-92: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L92
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 66-66: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L66
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 73-73: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L73
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 101-101: src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl#L101
Нельзя использовать латинские и кириллические символы в одном идентификаторе
| НоваяНастройка.ТипАвторизации = Перечисления.пбп_ТипыАвторизации.Базовая; | ||
|
|
||
| НоваяНастройка = Результат.Добавить(); | ||
| НоваяНастройка.Наименование = "Kafka"; |
There was a problem hiding this comment.
Extract the repeated string literal into a named constant.
The static analysis tool hints at repeated use of the string literal "Kafka".
To improve code quality, extract the repeated string literal into a named constant:
// Named constant for the string literal
Const KAFKA_IDENTIFIER = "Kafka"; Then, use this named constant in place of the string literal throughout the code.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 146-146: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L146
Необходимо избавиться от многократного использования строкового литерала "Kafka"
| @@ -211,6 +251,12 @@ | |||
|
|
|||
| ОбщиеКолонкиТаблицыПредопределенныхЗначений(Результат); | |||
|
|
|||
| Результат.Колонки.Добавить("ИнтегрируемаяСистема" , Новый ОписаниеТипов("СправочникСсылка.пбп_ИнтегрируемыеСистемы")); | |||
| Результат.Колонки.Добавить("ТипИнтеграции" , Новый ОписаниеТипов("СправочникСсылка.пбп_ТипыИнтеграций")); | |||
| Результат.Колонки.Добавить("СтрокаПодключения" , пбп_ОбщегоНазначенияСервер.ОписаниеТипаСтрока(200)); | |||
There was a problem hiding this comment.
Replace magic numbers with named constants.
The static analysis tool hints at using named constants instead of magic numbers "200" and "100".
To improve code quality, replace the magic numbers with named constants that convey their meaning:
// Named constants for magic numbers
Const MAX_CONNECTION_STRING_LENGTH = 200;
Const MAX_COM_OBJECT_NAME_LENGTH = 100;Then, use these named constants in place of the magic numbers:
Результат.Колонки.Добавить("СтрокаПодключения", пбп_ОбщегоНазначенияСервер.ОписаниеТипаСтрока(MAX_CONNECTION_STRING_LENGTH));
Результат.Колонки.Добавить("ИмяCOMОбъекта", пбп_ОбщегоНазначенияСервер.ОписаниеТипаСтрока(MAX_COM_OBJECT_NAME_LENGTH));Also applies to: 258-258
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.
| .ПолучитьПредопределенныйЭлементНаследованияНастроекТипаИнтеграции(ТипИнтеграции); | ||
| Если ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.Каталог | ||
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.FTPРесурсы | ||
| Или ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ПочтовыйКлиент Тогда | ||
| УстановитьВидимостьЭлементовДляТипаФайловыеОбмены(); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.COM Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Функция / запрос'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Ложь); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.RESTAPI Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Ресурс'"); | ||
| Элементы.ПараметрыВхода.Заголовок = НСтр("ru = 'Параметры запроса / URL'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВходаПараметрURL, УстанавливаемоеСвойство, Истина); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.SOAP Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Конечная точка'"); | ||
| Элементы.ПараметрыВхода.Заголовок = НСтр("ru = 'Параметры запроса'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВходаПараметрURL, УстанавливаемоеСвойство, Ложь); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ВнешняяКомпонента Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Функция'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Ложь); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.КоманднаяСтрока Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Команда'"); | ||
| Элементы.ПараметрыВхода.Заголовок = НСтр("ru = 'Аргументы команды'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВходаПараметрURL, УстанавливаемоеСвойство, Ложь); | ||
|
|
||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.БрокерыСообщений Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Конечная точка'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Ложь); | ||
|
|
||
| Иначе | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Конечная точка'"); | ||
| Элементы.ПараметрыВхода.Заголовок = НСтр("ru = 'Параметры'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТочкаВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, МетодЗапроса, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПользовательскаяФункция, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВхода, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВходаПараметрURL, УстанавливаемоеСвойство, Ложь); | ||
| КонецЕсли; | ||
| КонецЕсли; | ||
|
|
||
| КонецПроцедуры |
There was a problem hiding this comment.
LGTM, but consider refactoring.
The procedure looks good, correctly setting the visibility and other properties of form elements based on the integration type.
However, the procedure is quite large and complex, with many conditional branches. Consider refactoring it into smaller, more focused procedures for each integration type to improve readability and maintainability.
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 206-206: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L206
Необходимо избавиться от многократного использования строкового литерала "ru = 'Конечная точка'"
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПараметрыВходаПараметрURL, УстанавливаемоеСвойство, Истина); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.SOAP Тогда | ||
| Элементы.ТочкаВхода.Заголовок = НСтр("ru = 'Конечная точка'"); |
There was a problem hiding this comment.
Consider extracting the duplicated string literal into a constant or variable.
The string literal "ru = 'Конечная точка'" is used multiple times in the УстановитьСвойстваЭлементов procedure, at lines 206, 246, and 258.
To avoid duplication and improve maintainability, consider extracting this string literal into a constant or variable.
For example, you could define a constant at the beginning of the module:
Const КонечнаяТочка = "ru = 'Конечная точка'";And then replace the string literal with the constant:
Элементы.ТочкаВхода.Заголовок = КонечнаяТочка;Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 206-206: src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L206
Необходимо избавиться от многократного использования строкового литерала "ru = 'Конечная точка'"
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроек, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Ложь); | ||
|
|
||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.SOAP Тогда | ||
|
|
||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Сервер'"); | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроек, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Ложь); | ||
|
|
||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ВнешняяКомпонента Тогда | ||
|
|
||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Имя компоненты'"); | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроек, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Истина); | ||
|
|
||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.КоманднаяСтрока Тогда | ||
|
|
||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Исполняемый файл'"); | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроек, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Ложь); | ||
|
|
||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.БрокерыСообщений Тогда | ||
|
|
||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Сервер'"); | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроек, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Истина); | ||
|
|
||
| Иначе | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СтрокаПодключения, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИспользоватьПользовательскиеФункции, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, УчетнаяЗапись, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроек, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Истина); | ||
| КонецЕсли; | ||
| КонецЕсли; | ||
|
|
||
| КонецПроцедуры |
There was a problem hiding this comment.
Approved with notes
The procedure is quite long and contains repeated code blocks for setting up UI elements for different integration types. However, extracting these blocks into separate procedures may not significantly improve readability due to the specifics of each case. The use of the helper procedure пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы is a good practice to avoid code duplication.
Rename the identifier to use only Cyrillic characters
The use of both Latin and Cyrillic characters in the identifier ИмяCOMОбъекта violates the naming convention.
Rename the identifier to use only Cyrillic characters, for example:
-ИмяCOMОбъекта
+ИмяВнешнегоОбъектаExtract the repeated string literal into a constant
The string literal "ru = 'Сервер'" is repeated multiple times in the procedure.
Extract it into a constant:
Const ЗаголовокСервер = "ru = 'Сервер'";Then replace the occurrences of the literal with the constant:
-Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Сервер'");
+Элементы.СтрокаПодключения.Заголовок = НСтр(ЗаголовокСервер); Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 172-172: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L172
Длина метода "УстановитьСвойстваЭлементов" равна 202, что больше установленного лимита в 200 строк
[warning] 181-181: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L181
Нельзя использовать латинские и кириллические символы в одном идентификаторе
[warning] 275-275: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L275
Синтаксическая конструкция "Если...Тогда...ИначеЕсли..." содержит повторяющиеся блоки кода
[warning] 275-275: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L275
Необходимо избавиться от многократного использования строкового литерала "ru = 'Сервер'"
| Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпоненты( | ||
| Результат, Знач ДополнительныеПараметры) Экспорт | ||
|
|
||
| Если Результат = Неопределено Тогда | ||
| Возврат; | ||
| КонецЕсли; | ||
|
|
||
| Если Результат.ИзмененВручную Тогда | ||
| Объект.ИмяФайлаВнешнейКомпоненты = Результат.ИмяФайла; | ||
| ВнешняяКомпонентаЗагруженаИзФайла = Истина; | ||
|
|
||
| ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере(Результат, ДополнительныеПараметры); | ||
|
|
||
| Модифицированность = Истина; | ||
| КонецЕсли; | ||
|
|
||
| КонецПроцедуры | ||
|
|
||
| &НаСервере | ||
| Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере( | ||
| Результат, Знач ДополнительныеПараметры) | ||
|
|
||
| АдресВнешнейКомпоненты = ПоместитьВоВременноеХранилище(Результат.ДвоичныеДанные, Новый УникальныйИдентификатор); | ||
|
|
||
| КонецПроцедуры | ||
|
|
There was a problem hiding this comment.
Approved with a note
The procedures correctly handle the external component selection and update the object state accordingly.
Add the Знач modifier to the parameter
The Результат parameter of the ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере procedure should have the Знач modifier to ensure the parameter value is not modified inside the procedure.
Add the modifier as follows:
-Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере(Результат, Знач ДополнительныеПараметры)
+Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере(Знач Результат, Знач ДополнительныеПараметры)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. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпоненты( | |
| Результат, Знач ДополнительныеПараметры) Экспорт | |
| Если Результат = Неопределено Тогда | |
| Возврат; | |
| КонецЕсли; | |
| Если Результат.ИзмененВручную Тогда | |
| Объект.ИмяФайлаВнешнейКомпоненты = Результат.ИмяФайла; | |
| ВнешняяКомпонентаЗагруженаИзФайла = Истина; | |
| ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере(Результат, ДополнительныеПараметры); | |
| Модифицированность = Истина; | |
| КонецЕсли; | |
| КонецПроцедуры | |
| &НаСервере | |
| Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере( | |
| Результат, Знач ДополнительныеПараметры) | |
| АдресВнешнейКомпоненты = ПоместитьВоВременноеХранилище(Результат.ДвоичныеДанные, Новый УникальныйИдентификатор); | |
| КонецПроцедуры | |
| Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпоненты( | |
| Результат, Знач ДополнительныеПараметры) Экспорт | |
| Если Результат = Неопределено Тогда | |
| Возврат; | |
| КонецЕсли; | |
| Если Результат.ИзмененВручную Тогда | |
| Объект.ИмяФайлаВнешнейКомпоненты = Результат.ИмяФайла; | |
| ВнешняяКомпонентаЗагруженаИзФайла = Истина; | |
| ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере(Результат, ДополнительныеПараметры); | |
| Модифицированность = Истина; | |
| КонецЕсли; | |
| КонецПроцедуры | |
| &НаСервере | |
| Процедура ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере( | |
| Знач Результат, Знач ДополнительныеПараметры) | |
| АдресВнешнейКомпоненты = ПоместитьВоВременноеХранилище(Результат.ДвоичныеДанные, Новый УникальныйИдентификатор); | |
| КонецПроцедуры |
Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 445-445: src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl#L445
Установите модификатор "Знач" для параметра Результат метода ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыНаСервере
… с русским и латинским наименованием
Analysis Details28 Issues
Coverage and Duplications
Project ID: PSSL |
…d в настройки проверки сонара
Analysis Details6 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 2
Outside diff range and nitpick comments (2)
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl (1)
Line range hint
36-71: LGTM! Consider using a constant for the parameter name.The changes to the function are approved. They align with the shift in focus from REST method parameters to integration flow parameters and reflect the updated data structure.
Consider replacing the hardcoded string
"ИнтеграционныйПоток"with a constant for better maintainability.+Const ИнтеграционныйПотокПараметр = "ИнтеграционныйПоток"; ... - Запрос.УстановитьПараметр("ИнтеграционныйПоток", ИнтеграционныйПоток); + Запрос.УстановитьПараметр(ИнтеграционныйПотокПараметр, ИнтеграционныйПоток);src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (1)
254-258: LGTM, but consider extracting magic numbers into named constants.The code changes that add the new columns to support the new integration methods look good.
However, please consider the following suggestion to improve code quality:
Extract the magic numbers "200" and "100" into named constants to improve readability and maintainability. For example:
// Named constants for magic numbers Const MAX_CONNECTION_STRING_LENGTH = 200; Const MAX_COM_OBJECT_NAME_LENGTH = 100; ... Результат.Колонки.Добавить("СтрокаПодключения", пбп_ОбщегоНазначенияСервер.ОписаниеТипаСтрока(MAX_CONNECTION_STRING_LENGTH)); ... Результат.Колонки.Добавить("ИмяCOMОбъекта", пбп_ОбщегоНазначенияСервер.ОписаниеТипаСтрока(MAX_COM_OBJECT_NAME_LENGTH));Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.
[warning] 258-258: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L258
Создайте константу с понятным названием, присвойте ей значение "100" и используйте эту константу вместо магического числа.
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (10)
- .bsl-language-server.json (1 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl (3 hunks)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (13 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form.xml (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form.xml (6 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (4 hunks)
- src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (5 hunks)
Files skipped from review as they are similar to previous changes (5)
- src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Forms/ФормаЭлемента/Ext/Form/Module.bsl
- src/cf/Catalogs/пбп_ИсторияИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form.xml
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl
- src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl
Additional context used
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl
[warning] 256-256: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L256
Создайте константу с понятным названием, присвойте ей значение "200" и используйте эту константу вместо магического числа.
[warning] 258-258: src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L258
Создайте константу с понятным названием, присвойте ей значение "100" и используйте эту константу вместо магического числа.
Additional comments not posted (26)
.bsl-language-server.json (1)
12-15: LGTM!The new configuration section
LatinAndCyrillicSymbolInWordis a valuable addition that enhances the functionality of the language server by allowing more nuanced handling of words that contain both Latin and Cyrillic symbols. This improves the language server's adaptability to mixed-language contexts.The
excludeWordsproperty correctly lists various terms related to XML, JSON, and other technical objects that should be excluded from certain processing or checks related to the mixing of Latin and Cyrillic symbols in a word.The
allowTrailingPartsInAnotherLanguageproperty set totrueappropriately permits trailing parts of words to be in a different language.These changes align with the AI-generated summary and the PR objectives.
src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ManagerModule.bsl (1)
Line range hint
74-107: Skipped review.This function is not part of the changes in this diff.
src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (5)
84-95: LGTM!The code changes that add the new integration systems look good.
107-109: LGTM!The renaming of the function and the corresponding changes look good.
113-114: LGTM!The code changes that update the integration method properties look good.
135-161: LGTM!The code changes that add the new integration settings look good. The use of
НайтиПоРеквизитуto find the corresponding integrated system and the setting of theТипИнтеграцииandТипАвторизацииproperties using references to existing directories and enumerations is the correct approach.
238-238: LGTM!The renaming of the function looks good.
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form.xml (8)
10-10: LGTM!The event changes are approved:
- Adding the
OnReadAtServerevent allows additional processing when reading data from the server.- Renaming
OnWriteAtServertoBeforeWriteAtServerclarifies that the event is triggered before writing data to the server.Also applies to: 12-12
49-57: LGTM!The new input field
ТипИнтеграцииis approved. It enhances the form's functionality by allowing users to specify the integration type and handle changes to the selected type.
82-89: LGTM!The changes to the input field are approved:
- Renaming
СерверtoСтрокаПодключенияprovides a more precise description of the field's purpose.- Adding the
StartChoiceevent allows custom handling of the selection process initiation, enhancing user interaction capabilities.
100-104: LGTM!The new input fields are approved. They enhance the form's functionality by allowing users to specify additional settings:
УчетнаяЗаписьfor specifying the account.ТипАвторизацииfor specifying the authorization type.ИмяCOMОбъектаfor specifying the COM object name.Also applies to: 106-111, 112-117
118-124: LGTM!The new checkbox field
ИспользоватьПользовательскиеФункцииis approved. It enhances the form's functionality by allowing users to enable or disable the usage of custom functions.
125-129: LGTM!The new button
ПодключитьВнешнююКомпонентуis approved. It provides users with a new action option to connect an external component from a file.
269-283: LGTM!The new attributes are approved. They expand the metadata associated with the form, allowing to store and manage additional information related to external components:
АдресВнешнейКомпонентыrepresents the address of the external component.ВнешняяКомпонентаЗагруженаИзФайлаindicates whether the external component is loaded from a file.Also applies to: 284-294
301-301: LGTM!The command changes are approved. They enhance the form's functionality:
- Changing the title of
ДобавитьОбщиеНастройкиto "Параметры аутентификации" indicates a shift in focus towards authentication-related settings.- Adding the new command
ПодключитьВнешнююКомпонентуИзФайлаprovides users with a new action option to connect an external component from a file.Also applies to: 327-341
src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЭлемента/Ext/Form/Module.bsl (11)
26-27: LGTM!The changes to the
ПриСозданииНаСервереprocedure are approved. Calling theУстановитьСвойстваЭлементовprocedure during form creation improves the initialization process.
49-56: LGTM!The new
ПриЧтенииНаСервереprocedure is approved. It correctly retrieves the external component data and stores it in a temporary storage when the object is read.
59-70: LGTM!The new
ПередЗаписьюНаСервереprocedure is approved. It correctly saves the external component data and sets theИзмененВручнуюflag before the object is written.
95-99: LGTM!The changes to the
ТипИнтеграцииПриИзмененииprocedure are approved. Calling theУстановитьСвойстваЭлементовprocedure when the integration type is changed improves the user experience by dynamically adjusting the form.
102-117: LGTM!The new
СтрокаПодключенияНачалоВыбораprocedure is approved. It correctly opens a dialog to select a directory for the connection string and handles the result, improving the user experience.
152-165: LGTM!The new
ПодключитьВнешнююКомпонентуИзФайлаprocedure is approved. It correctly opens a form to select an external component file and handles the result, improving the functionality.
172-348: Approved with notesThe
УстановитьСвойстваЭлементовprocedure is quite long and contains repeated code blocks for setting up UI elements for different integration types. However, extracting these blocks into separate procedures may not significantly improve readability due to the specifics of each case. The use of the helper procedureпбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормыis a good practice to avoid code duplication.
351-366: LGTM!The new
УстановитьВидимостьЭлементовДляТипаФайловыеОбменыprocedure is approved. It correctly sets the visibility of form elements for the file exchange integration type, improving the user experience.
369-380: LGTM!The new
УстановитьВидимостьЭлементовДляОстальныхТиповprocedure is approved. It correctly sets the visibility of form elements for other integration types, improving the user experience.
383-392: LGTM!The new
ОбработчикВыбораФайловогоКаталогаЗавершениеПослеВыбораВДиалогеprocedure is approved. It correctly handles the result of selecting a directory in a dialog and sets the connection string.
395-411: LGTM!The new
ОбработчикПодключенияВнешнейКомпонентыЗавершениеПослеВыбораКомпонентыprocedure is approved. It correctly handles the result of selecting an external component and updates the object state.
| Элементы.СтрокаПодключения.Заголовок = СерверСтрока; | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроекСтрока, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Ложь); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.ВнешняяКомпонента Тогда | ||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Имя компоненты'"); | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроекСтрока, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Истина); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.КоманднаяСтрока Тогда | ||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Исполняемый файл'"); | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроекСтрока, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Ложь); | ||
| ИначеЕсли ЭлементНаследования = Справочники.пбп_ТипыИнтеграций.БрокерыСообщений Тогда | ||
| Элементы.СтрокаПодключения.Заголовок = СерверСтрока; | ||
|
|
||
| УстановитьВидимостьЭлементовДляОстальныхТипов(); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СписокНастроекСтрока, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ПодключитьВнешнююКомпоненту, УстанавливаемоеСвойство, Истина); | ||
| Иначе | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, СтрокаПодключения, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИспользоватьПользовательскиеФункции, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, УчетнаяЗапись, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, Порт, УстанавливаемоеСвойство, Истина); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Истина); |
There was a problem hiding this comment.
Extract the repeated string literal into a constant
The string literal "ru = 'Сервер'" is repeated multiple times in the procedure.
Extract it into a constant:
Const ЗаголовокСервер = "ru = 'Сервер'";Then replace the occurrences of the literal with the constant:
-Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Сервер'");
+Элементы.СтрокаПодключения.Заголовок = НСтр(ЗаголовокСервер); | ИмяCOMОбъекта = "ИмяCOMОбъекта"; | ||
| ИспользоватьПользовательскиеФункции = "ИспользоватьПользовательскиеФункции"; | ||
| СписокНастроекСтрока = "СписокНастроек"; | ||
| ПодключитьВнешнююКомпоненту = "ПодключитьВнешнююКомпоненту"; | ||
|
|
||
| СерверСтрока = НСтр("ru = 'Сервер'"); | ||
|
|
||
| УстанавливаемоеСвойство = "Видимость"; | ||
|
|
||
| Если Не ЗначениеЗаполнено(Объект.ТипИнтеграции) Тогда | ||
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Каталог'"); | ||
|
|
||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ГруппаОсновныеНастройки, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, УчетнаяЗапись, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | ||
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | ||
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); |
There was a problem hiding this comment.
Rename the identifier to use only Cyrillic characters
The use of both Latin and Cyrillic characters in the identifier ИмяCOMОбъекта violates the naming convention.
Rename the identifier to use only Cyrillic characters, for example:
-ИмяCOMОбъекта
+ИмяВнешнегоОбъекта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. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ИмяCOMОбъекта = "ИмяCOMОбъекта"; | |
| ИспользоватьПользовательскиеФункции = "ИспользоватьПользовательскиеФункции"; | |
| СписокНастроекСтрока = "СписокНастроек"; | |
| ПодключитьВнешнююКомпоненту = "ПодключитьВнешнююКомпоненту"; | |
| СерверСтрока = НСтр("ru = 'Сервер'"); | |
| УстанавливаемоеСвойство = "Видимость"; | |
| Если Не ЗначениеЗаполнено(Объект.ТипИнтеграции) Тогда | |
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Каталог'"); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, ГруппаОсновныеНастройки, УстанавливаемоеСвойство, Ложь); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, УчетнаяЗапись, УстанавливаемоеСвойство, Ложь); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, ИмяCOMОбъекта, УстанавливаемоеСвойство, Ложь); | |
| ИмяВнешнегоОбъекта = "ИмяCOMОбъекта"; | |
| ИспользоватьПользовательскиеФункции = "ИспользоватьПользовательскиеФункции"; | |
| СписокНастроекСтрока = "СписокНастроек"; | |
| ПодключитьВнешнююКомпоненту = "ПодключитьВнешнююКомпоненту"; | |
| СерверСтрока = НСтр("ru = 'Сервер'"); | |
| УстанавливаемоеСвойство = "Видимость"; | |
| Если Не ЗначениеЗаполнено(Объект.ТипИнтеграции) Тогда | |
| Элементы.СтрокаПодключения.Заголовок = НСтр("ru = 'Каталог'"); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, ГруппаОсновныеНастройки, УстанавливаемоеСвойство, Ложь); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, УчетнаяЗапись, УстанавливаемоеСвойство, Ложь); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, ТипАвторизации, УстанавливаемоеСвойство, Ложь); | |
| пбп_ОбщегоНазначенияКлиентСервер.УстановитьСвойствоЭлементаФормы( | |
| Элементы, ИмяВнешнегоОбъекта, УстанавливаемоеСвойство, Ложь); |
Analysis Details5 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range and nitpick comments (2)
src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl (2)
Line range hint
1-1: TODO: Add testsThis comment indicates that unit tests are currently missing for this module. Adding tests is an important best practice that helps ensure code correctness and prevent regressions. Please prioritize writing unit tests to improve the overall quality and maintainability of this code.
Let me know if you would like any help with writing the unit tests for this module. I'd be happy to assist!
Line range hint
13-25: Reconsider the flat $20 service fee added to discounted billsThe discounts are calculated correctly per the business rules. However, adding a flat $20 service fee to any discounted bill can lead to some unexpected negative experiences for customers:
- For smaller bills, the $20 fee may exceed the discount amount. For example, a $100 bill with a 10% loyalty discount would become $110 after the service fee, which is more than the original undiscounted amount!
- Customers would be frustrated to see a higher total even after a "discount" was applied.
I recommend revisiting the service fee logic to avoid this. Some alternatives:
- Make the fee a percentage of the bill instead of a flat amount
- Only add the fee if the discounted total is above some minimum amount
- Deduct the fee from the bill before applying the discount percent
The goal should be that the loyalty discounts always result in a lower final bill than without the discount. This provides a better, less confusing experience for loyal customers.
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (8)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form.xml (1 hunks)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl (1 hunks)
- src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl (1 hunks)
- src/cf/Configuration.xml (2 hunks)
- src/cf/Roles/пбп_РедактированиеНастроекИнтеграции/Ext/Rights.xml (3 hunks)
- src/cf/Roles/пбп_РедактированиеПользовательскихФункций/Ext/Rights.xml (3 hunks)
- src/cf/Roles/пбп_ЧтениеКонстантыИспользоватьПользовательскиеФункции.xml (1 hunks)
- src/cf/Roles/пбп_ЧтениеКонстантыИспользоватьПользовательскиеФункции/Ext/Rights.xml (1 hunks)
Files skipped from review due to trivial changes (2)
- src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form/Module.bsl
- src/cf/Roles/пбп_ЧтениеКонстантыИспользоватьПользовательскиеФункции.xml
Files skipped from review as they are similar to previous changes (3)
- src/cf/Configuration.xml
- src/cf/Roles/пбп_РедактированиеНастроекИнтеграции/Ext/Rights.xml
- src/cf/Roles/пбп_РедактированиеПользовательскихФункций/Ext/Rights.xml
Additional comments not posted (8)
src/cf/Roles/пбп_ЧтениеКонстантыИспользоватьПользовательскиеФункции/Ext/Rights.xml (3)
1-2: LGTM!The XML declaration and root element are correctly defined.
3-5: LGTM!The
Rightselement attributes are correctly defined.
6-12: LGTM!The
objectelement correctly defines the read right for the constantпбп_ИспользоватьПользовательскиеФункции.src/cf/Catalogs/пбп_НастройкиИнтеграции/Forms/ФормаЗагрузкиВнешнейКомпоненты/Ext/Form.xml (5)
1-2: LGTM!The XML declaration and root
Formelement are correctly defined.
3-18: LGTM!The
AutoCommandBarelement and its childButtonelements are correctly defined.
22-186: LGTM!The form's main content, including the
UsualGroup,RadioButtonField,InputField, andButtonelements, is correctly defined and well-structured.
188-244: LGTM!The
Attributeselement and its childAttributeelements are correctly defined.
245-316: LGTM!The
Commandselement and its childCommandelements are correctly defined.
Analysis Details5 Issues
Coverage and Duplications
Project ID: PSSL |
src/cf/Configuration.xml
Outdated
| <CommonTemplate>пбп_BSLEditor</CommonTemplate> | ||
| <CommonTemplate>пбп_JSONEditor</CommonTemplate> | ||
| <CommonTemplate>пбп_XMLEditor</CommonTemplate> | ||
| <CommonTemplate>пбп_SimpleKafkaAdapter</CommonTemplate> |
There was a problem hiding this comment.
По всем типам метаданных сделать сортировку по наименованию
| @@ -0,0 +1,15 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
There was a problem hiding this comment.
Добавить роль в нужную подсистему. В целом проверить все объекты не входящие в подсистемы и разместить их в нужные
| @@ -139,6 +140,14 @@ | |||
| |ИТОГИ ПО | |||
| | Ссылка ИЕРАРХИЯ"; | |||
|
|
|||
| Если пбп_ОбщегоНазначенияПовтИсп.ПолучитьЗначениеКонстанты("пбп_ИспользоватьПользовательскиеФункции") Тогда | |||
There was a problem hiding this comment.
Тут нужно переделать, логика такая что мы вызываем функцию которая в случае существования БСП вызывает метод БСП, а в случае отсутствия БСП вызывает метод библиотеки. Посмотри тут
|
Ещё нужно повысить версию конфигурации до 1.0.2.3 и в общем модуле |
…добавлены в составы подсистем. Добавлены настройки для подключения прямого подключения к базе данных
Analysis Details9 Issues
Coverage and Duplications
Project ID: PSSL |
Analysis Details8 Issues
Coverage and Duplications
Project ID: PSSL |
…ка данных на форме настроек интеграции
Analysis Details6 Issues
Coverage and Duplications
Project ID: PSSL |
1 similar comment
Analysis Details6 Issues
Coverage and Duplications
Project ID: PSSL |
ivanmolodec
deleted the
architectural-refactoring-to-integrations-management-subsystem
branch
#85 Добавлены / изменены реквизиты для типов интеграций помимо SOAP и REST. Добавлены / изменены обработчики форм справочников подсистемы под отрисовку разных типов интеграций
Summary by CodeRabbit
New Features
Bug Fixes
Documentation