Added reverse assignment #88
Conversation
WalkthroughThe recent updates to the "Event Subscription" subsystem greatly enhance clarity and usability by refining event handling procedures. Key modifications include improved parameter management, clearer delineation of roles for modules and handlers, and a focus on avoiding type mixing in subscriptions. These changes empower users to implement and manage event subscriptions more effectively. 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 as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
Analysis Details7 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: 0
Outside diff range, codebase verification and nitpick comments (1)
docs/ПодпискиНаСобытия.md (1)
7-10: Fix unordered list indentation issues.The unordered list items are not properly indented. Fix the indentation to adhere to Markdown best practices.
- - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") - - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") - - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") - - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)** + - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") + - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") + - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") + - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)** - - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. - **Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). - Добавить их в подсистему **пбп_ПодпискиНаСобытия**. - - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. - - Структура общего модуля обработчиков: - - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам. + - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. + **Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). + Добавить их в подсистему **пбп_ПодпискиНаСобытия**. + - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. + - Структура общего модуля обработчиков: + - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам. - - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры: + - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры:Also applies to: 32-32, 35-37, 99-99
Tools
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- docs/ПодпискиНаСобытия.md (3 hunks)
- src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl (8 hunks)
- src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl (6 hunks)
Additional context used
Markdownlint
docs/ПодпискиНаСобытия.md
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
32-32: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
35-35: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
36-36: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
37-37: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
22-22: Column: 1
Hard tabs(MD010, no-hard-tabs)
24-24: Column: 1
Hard tabs(MD010, no-hard-tabs)
99-99: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl
[warning] 121-121: src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl#L121
Не следует присваивать параметру Отказ значение отличное от Истина
[warning] 154-154: src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl#L154
Не следует присваивать параметру Отказ значение отличное от Истина
[warning] 190-190: src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl#L190
Не следует присваивать параметру Отказ значение отличное от Истинаsrc/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl
[warning] 124-124: src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl#L124
Не следует присваивать параметру Отказ значение отличное от Истина
[warning] 157-157: src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl#L157
Не следует присваивать параметру Отказ значение отличное от Истина
[warning] 193-193: src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl#L193
Не следует присваивать параметру Отказ значение отличное от Истина
[warning] 229-229: src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl#L229
Не следует присваивать параметру Отказ значение отличное от Истина
Additional comments not posted (14)
docs/ПодпискиНаСобытия.md (7)
4-4: Typographical correction and clarification approved.The correction of "отстутствия" to "отсутствия" and the clarification on the necessity of using the event subscription subsystem improve the document's clarity.
6-10: Improved structure and clarity approved.The changes provide a clearer delineation of the roles of common modules and event handlers, enhancing the document's readability and usability.
Tools
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
12-13: Emphasis on parameter reassignment approved.The emphasis on reassigning parameters after method execution is crucial for ensuring the correct functioning of event handlers.
15-28: Example of parameter reassignment approved.The example clearly demonstrates the correct method for parameter reassignment in the
ПередЗаписьюprocedure.Tools
Markdownlint
22-22: Column: 1
Hard tabs(MD010, no-hard-tabs)
24-24: Column: 1
Hard tabs(MD010, no-hard-tabs)
31-37: Procedural instructions approved.The instructions for verifying the existence of subscriptions and creating new ones are clear and emphasize the importance of not mixing object types and events within a single subscription.
Tools
Markdownlint
32-32: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
35-35: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
36-36: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
37-37: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
Line range hint
72-98: Example of event handler approved.The example clearly demonstrates the correct method for handling events in the
пбп_ДокументыОбработкаСобытийmodule.
99-99: Instructions for adding functions approved.The instructions for adding a function to map object types to procedures in the
пбп_ОбщегоНазначенияПовтИспmodule are clear and detailed.Tools
Markdownlint
99-99: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl (5)
49-50: Reassignment ofИсточникapproved.The reassignment of
ИсточникfromПараметрыМетода[0]ensures that the parameter is correctly updated after method execution.
86-88: Reassignment ofИсточникandСтандартнаяОбработкаapproved.The reassignment of
ИсточникandСтандартнаяОбработкаfromПараметрыМетодаensures that the parameters are correctly updated after method execution.
120-122: Reassignment ofИсточникandОтказapproved, but verifyОтказvalue.The reassignment of
ИсточникandОтказfromПараметрыМетодаensures that the parameters are correctly updated after method execution. However, ensure thatОтказis not assigned a value other thanИстина.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 121-121: src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl#L121
Не следует присваивать параметру Отказ значение отличное от Истина
154-155: Reassignment ofОтказapproved, but verifyОтказvalue.The reassignment of
ОтказfromПараметрыМетода[1]ensures that the parameter is correctly updated after method execution. However, ensure thatОтказis not assigned a value other thanИстина.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 154-154: src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl#L154
Не следует присваивать параметру Отказ значение отличное от Истина
190-192: Reassignment ofОтказandПроверяемыеРеквизитыapproved, but verifyОтказvalue.The reassignment of
ОтказandПроверяемыеРеквизитыfromПараметрыМетодаensures that the parameters are correctly updated after method execution. However, ensure thatОтказis not assigned a value other thanИстина.Tools
GitHub Check: 1C: Project SubSystems Library Sonarqube Results
[warning] 190-190: src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl#L190
Не следует присваивать параметру Отказ значение отличное от Истинаsrc/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl (2)
49-50: Reassignment ofИсточникapproved.The reassignment of
ИсточникfromПараметрыМетода[0]ensures that the parameter is correctly updated after method execution.
84-86: Reassignment ofИсточникandСтандартнаяОбработкаapproved.The reassignment of
ИсточникandСтандартнаяОбработкаfrom `Пара
|
Убрать проверку на Отказ до выполнение процедуры конкретного объекта. Согласно концепции 1С, в документации описать, если нужно ускорить и не выполнять подписки при отказе, вставьте кусок кода ... |
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 7
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- docs/ПодпискиНаСобытия.md (3 hunks)
- src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl (12 hunks)
- src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl (9 hunks)
Files skipped from review as they are similar to previous changes (2)
- src/cf/CommonModules/пбп_ДокументыОбработкаСобытий/Ext/Module.bsl
- src/cf/CommonModules/пбп_СправочникиОбработкаСобытий/Ext/Module.bsl
Additional context used
LanguageTool
docs/ПодпискиНаСобытия.md
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
Markdownlint
docs/ПодпискиНаСобытия.md
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
19-19: Column: 1
Hard tabs(MD010, no-hard-tabs)
20-20: Column: 1
Hard tabs(MD010, no-hard-tabs)
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Additional comments not posted (1)
docs/ПодпискиНаСобытия.md (1)
4-4: Fix typographical error.The word "в место" should be written as "вместо".
- добавьте проверку этого условия в место проверки **ОбменДанными.Загрузка**. + добавьте проверку этого условия вместо проверки **ОбменДанными.Загрузка**.Likely invalid or redundant comment.
docs/ПодпискиНаСобытия.md
Outdated
| Процедура ПередЗаписью(Источник, Отказ, РежимЗаписи, РежимПроведения) Экспорт | ||
| Если Источник.ОбменДанными.Загрузка Или Отказ Тогда | ||
| Возврат; | ||
| КонецЕсли; | ||
| КонецПроцедуры |
There was a problem hiding this comment.
Replace hard tabs with spaces.
Hard tabs should be replaced with spaces for consistent formatting.
- Если Источник.ОбменДанными.Загрузка Или Отказ Тогда
- Возврат;
- КонецЕсли;
+ Если Источник.ОбменДанными.Загрузка Или Отказ Тогда
+ Возврат;
+ КонецЕсли;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
Markdownlint
19-19: Column: 1
Hard tabs(MD010, no-hard-tabs)
20-20: Column: 1
Hard tabs(MD010, no-hard-tabs)
| @@ -77,7 +105,7 @@ | |||
| КонецПроцедуры | |||
| ``` | |||
|
|
|||
| - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найти функцию с названием объекта и события и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры: | |||
| - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры: | |||
There was a problem hiding this comment.
Fix unordered list indentation.
The unordered list items should be indented consistently.
- - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры:
+ - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры: 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
Markdownlint
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
| **Общая структура подписок на события:** | ||
| - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") | ||
| - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") | ||
| - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") | ||
| - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)** | ||
|
|
There was a problem hiding this comment.
Fix unordered list indentation.
The unordered list items should be indented consistently.
- **Общая структура подписок на события:**
- - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий")
- - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи")
- - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи")
- - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)**
+**Общая структура подписок на события:**
+ - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий")
+ - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи")
+ - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи")
+ - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)**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
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
| **Пример обратного присвоения:** | ||
| ```1C (BSL) | ||
| Процедура ПередЗаписью(Источник, Отказ, РежимЗаписи, РежимПроведения) Экспорт | ||
| // ... | ||
| ПараметрыМетода = Новый Массив; | ||
| ПараметрыМетода.Добавить(Источник); | ||
| ПараметрыМетода.Добавить(Отказ); | ||
|
|
||
| пбп_ОбщегоНазначенияСервер.ВыполнитьМетодКонфигурации(ИмяМетода, ПараметрыМетода); | ||
|
|
||
| Источник = ПараметрыМетода[0]; | ||
| Отказ = ПараметрыМетода[1]; | ||
| // ... | ||
| КонецПроцедуры |
There was a problem hiding this comment.
Replace hard tabs with spaces.
Hard tabs should be replaced with spaces for consistent formatting.
-
+
-
+ 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.
| **Пример обратного присвоения:** | |
| ```1C (BSL) | |
| Процедура ПередЗаписью(Источник, Отказ, РежимЗаписи, РежимПроведения) Экспорт | |
| // ... | |
| ПараметрыМетода = Новый Массив; | |
| ПараметрыМетода.Добавить(Источник); | |
| ПараметрыМетода.Добавить(Отказ); | |
| пбп_ОбщегоНазначенияСервер.ВыполнитьМетодКонфигурации(ИмяМетода, ПараметрыМетода); | |
| Источник = ПараметрыМетода[0]; | |
| Отказ = ПараметрыМетода[1]; | |
| // ... | |
| КонецПроцедуры | |
| **Пример обратного присвоения:** | |
| ```1C (BSL) | |
| Процедура ПередЗаписью(Источник, Отказ, РежимЗаписи, РежимПроведения) Экспорт | |
| // ... | |
| ПараметрыМетода = Новый Массив; | |
| ПараметрыМетода.Добавить(Источник); | |
| ПараметрыМетода.Добавить(Отказ); | |
| пбп_ОбщегоНазначенияСервер.ВыполнитьМетодКонфигурации(ИмяМетода, ПараметрыМетода); | |
| Источник = ПараметрыМетода[0]; | |
| Отказ = ПараметрыМетода[1]; | |
| // ... | |
| КонецПроцедуры |
Tools
Markdownlint
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
| Процедура ПередЗаписью(Источник, Отказ, РежимЗаписи, РежимПроведения) Экспорт | ||
| // ... | ||
| ПараметрыМетода = Новый Массив; | ||
| ПараметрыМетода.Добавить(Источник); | ||
| ПараметрыМетода.Добавить(Отказ); | ||
|
|
||
| пбп_ОбщегоНазначенияСервер.ВыполнитьМетодКонфигурации(ИмяМетода, ПараметрыМетода); | ||
|
|
||
| Источник = ПараметрыМетода[0]; | ||
| Отказ = ПараметрыМетода[1]; | ||
| // ... | ||
| КонецПроцедуры |
There was a problem hiding this comment.
Replace hard tabs with spaces.
Hard tabs should be replaced with spaces for consistent formatting.
-
+
-
+ Committable suggestion was skipped due to low confidence.
Tools
Markdownlint
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
| **Порядок действий при работе с подсистемой:** | ||
| - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. | ||
| **Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). | ||
| Добавить их в подсистему **пбп_ПодпискиНаСобытия**. | ||
| - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. | ||
| - Структура общего модуля обработчиков: | ||
| - В области **СлужебныйПрограммныйИнтерфейс** располагать обработчики объектов, в ней располагаются подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс**, для группировки по подсистемам. | ||
| - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам. |
There was a problem hiding this comment.
Fix unordered list indentation.
The unordered list items should be indented consistently.
-**Порядок действий при работе с подсистемой:**
- - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее.
-**Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль).
-Добавить их в подсистему **пбп_ПодпискиНаСобытия**.
- - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров.
- - Структура общего модуля обработчиков:
- - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам.
+**Порядок действий при работе с подсистемой:**
+ - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее.
+**Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль).
+Добавить их в подсистему **пбп_ПодпискиНаСобытия**.
+ - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров.
+ - Структура общего модуля обработчиков:
+ - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам.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
Markdownlint
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (5)
docs/ПодпискиНаСобытия.md (5)
7-10: Fix unordered list indentation.The unordered list items should be indented consistently.
- - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") - - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") - - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") - - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)** + - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") + - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") + - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") + - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)**Tools
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
14-14: Fix typographical error.The word "вместо" should be written as one word.
- добавьте проверку этого условия в место проверки **ОбменДанными.Загрузка**. + добавьте проверку этого условия вместо проверки **ОбменДанными.Загрузка**.Tools
LanguageTool
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
31-33: Replace hard tabs with spaces.Hard tabs should be replaced with spaces for consistent formatting.
- + - +Tools
Markdownlint
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
41-46: Fix unordered list indentation.The unordered list items should be indented consistently.
- - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. -**Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). -Добавить их в подсистему **пбп_ПодпискиНаСобытия**. - - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. - - Структура общего модуля обработчиков: - - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам. + - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. +**Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). +Добавить их в подсистему **пбп_ПодпискиНаСобытия**. + - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. + - Структура общего модуля обработчиков: + - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам.Tools
Markdownlint
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
108-108: Fix unordered list indentation.The unordered list items should be indented consistently.
- - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры: + - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры:Tools
Markdownlint
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- docs/ПодпискиНаСобытия.md (3 hunks)
Additional context used
Learnings (1)
docs/ПодпискиНаСобытия.md (1)
Learnt from: DenisUshakov URL: https://github.com/firstBitSportivnaya/PSSL/pull/52 Timestamp: 2024-04-19T12:01:38.966Z Learning: Hard tabs are used intentionally for highlighting blocks of code in the `ПереопределениеМетодовБСП.md` file and possibly other files in the repository.
LanguageTool
docs/ПодпискиНаСобытия.md
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
Markdownlint
docs/ПодпискиНаСобытия.md
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Additional comments not posted (5)
docs/ПодпискиНаСобытия.md (5)
4-4: Clarification added for event subscription usage.The added clarification improves the understanding of when to use the event subscription subsystem.
17-22: Example procedure with "Export" annotation added.The example is clear and aligns with the 1C documentation guidelines.
Line range hint
81-106:
Example of a common module handler added.The example is clear and aligns with the 1C documentation guidelines.
Line range hint
110-112:
Example of calling a procedure to fill in the correspondence added.The example is clear and aligns with the 1C documentation guidelines.
Tools
Markdownlint
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Line range hint
114-126:
Example of filling in the correspondence between object types and the called procedure added.The example is clear and aligns with the 1C documentation guidelines.
Tools
LanguageTool
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (6)
docs/ПодпискиНаСобытия.md (6)
6-11: Fix unordered list indentation.The unordered list items should be indented consistently.
- **Общая структура подписок на события:** - - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") - - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") - - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") - - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)** +**Общая структура подписок на события:** + - Для каждого класса **общий модуль** (пример: "пбп_ДокументыОбработкаСобытий") + - В каждом общем модуле распологаются процедуры-обработчики для данного класса (пример: "ОбработкаПроведения", "ПриЗаписи") + - В области **СлужебныйПрограммныйИнтерфейс** добавляются одноименные процедуры для обработки конкретных событий (пример: "СписаниеБезналичныхДенежныхСредствПриЗаписи") + - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** необходимо добавить соответствие типа объекта и вызываемой процедуры в одноименной функции **(Указывать только имя процедуры!)**Tools
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
12-14: Fix typographical issue.The word "вместо" should be written as one word.
- - В обработчиках, указанных в подписках на события, реализован возврат параметра **Отказ** согласно концепции фирмы 1С. При необходимости ускорить работу и избежать выполнения кода при условии **Отказ = Истина**, добавьте проверку этого условия в место проверки **ОбменДанными.Загрузка**. + - В обработчиках, указанных в подписках на события, реализован возврат параметра **Отказ** согласно концепции фирмы 1С. При необходимости ускорить работу и избежать выполнения кода при условии **Отказ = Истина**, добавьте проверку этого условия вместо проверки **ОбменДанными.Загрузка**.Tools
LanguageTool
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
15-22: Replace hard tabs with spaces.Hard tabs should be replaced with spaces for consistent formatting.
- Если Источник.ОбменДанными.Загрузка Или Отказ Тогда - Возврат; - КонецЕсли; + Если Источник.ОбменДанными.Загрузка Или Отказ Тогда + Возврат; + КонецЕсли;Tools
Markdownlint
19-19: Column: 1
Hard tabs(MD010, no-hard-tabs)
20-20: Column: 1
Hard tabs(MD010, no-hard-tabs)
24-38: Replace hard tabs with spaces.Hard tabs should be replaced with spaces for consistent formatting.
- + - +Tools
Markdownlint
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
40-46: Fix unordered list indentation.The unordered list items should be indented consistently.
-**Порядок действий при работе с подсистемой:** - - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. -**Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). -Добавить их в подсистему **пбп_ПодпискиНаСобытия**. - - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. - - Структура общего модуля обработчиков: - - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам. +**Порядок действий при работе с подсистемой:** + - Необходимо проверить, существует ли общая подписка на нужный тип объекта и событие. Например: **ОбработкаПроведения**, **ПриЗаписи** и так далее. +**Важно!** Не допускается смешение типов объектов и событий в одной подписке. Если подписка не существует, нужно её создать, а так же добавить общий модуль с аналогичным названием (напр. **пбп_ДокументыПриЗаписи** - подписка, **пбп_ДокументыОбработкаСобытий** - общий модуль). +Добавить их в подсистему **пбп_ПодпискиНаСобытия**. + - Если создана новая подписка и модуль, то в модуле создается обработчик для подписки и выбрать его в подписке. У каждого события свой набор параметров. + - Структура общего модуля обработчиков: + - В области **СлужебныйПрограммныйИнтерфейс** располагаются обработчики объектов. Внутри этой области могут быть подобласти по обработчикам. Допустимо добавлению произвольных областей внутри **СлужебныйПрограммныйИнтерфейс** для группировки по подсистемам.Tools
Markdownlint
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
108-108: Fix unordered list indentation.The unordered list items should be indented consistently.
- - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры: + - В общем модуле **пбп_ОбщегоНазначенияПовтИсп** найдите функцию с названием объекта и события, и добавить в соответствие тип объекта и вызываемую процедуру **(Указывать только имя процедуры!)**. Если функции для данного набора нет, создать её, перед возвратом добавить вызов заранее созданной процедуры:Tools
Markdownlint
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- docs/ПодпискиНаСобытия.md (3 hunks)
Additional context used
Learnings (1)
docs/ПодпискиНаСобытия.md (1)
Learnt from: DenisUshakov URL: https://github.com/firstBitSportivnaya/PSSL/pull/52 Timestamp: 2024-04-19T12:01:38.966Z Learning: Hard tabs are used intentionally for highlighting blocks of code in the `ПереопределениеМетодовБСП.md` file and possibly other files in the repository.
LanguageTool
docs/ПодпискиНаСобытия.md
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
Markdownlint
docs/ПодпискиНаСобытия.md
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
19-19: Column: 1
Hard tabs(MD010, no-hard-tabs)
20-20: Column: 1
Hard tabs(MD010, no-hard-tabs)
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Additional comments not posted (2)
docs/ПодпискиНаСобытия.md (2)
4-5: LGTM!The new paragraph enhances clarity on when to use the event subscription subsystem.
81-81: LGTM!The example handler enhances the documentation by providing a practical example.
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- docs/ПодпискиНаСобытия.md (3 hunks)
Additional context used
Learnings (1)
docs/ПодпискиНаСобытия.md (1)
Learnt from: DenisUshakov URL: https://github.com/firstBitSportivnaya/PSSL/pull/52 Timestamp: 2024-04-19T12:01:38.966Z Learning: Hard tabs are used intentionally for highlighting blocks of code in the `ПереопределениеМетодовБСП.md` file and possibly other files in the repository.
LanguageTool
docs/ПодпискиНаСобытия.md
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
Markdownlint
docs/ПодпискиНаСобытия.md
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
19-19: Column: 1
Hard tabs(MD010, no-hard-tabs)
20-20: Column: 1
Hard tabs(MD010, no-hard-tabs)
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Additional comments not posted (9)
docs/ПодпискиНаСобытия.md (9)
4-5: LGTM!The change improves clarity by emphasizing the use of the event subscription subsystem in the absence of standard subscriptions.
6-11: LGTM!The change enhances clarity and provides a clearer definition of roles for common modules and event handlers.
Tools
Markdownlint
7-7: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
8-8: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
9-9: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
10-10: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
12-14: LGTM!The change emphasizes the need for reassigning parameters and checking the refusal condition, aligning with performance optimization and documentation standards.
Tools
LanguageTool
[uncategorized] ~14-~14: Слово пишется слитно: «вместо».
Context: ...тина**, добавьте проверку этого условия в место проверки ОбменДанными.Загрузка. ...(Pravopisanie_slitno1)
15-22: LGTM!The code example illustrates the proper handling of the refusal condition, improving clarity and usability.
Tools
Markdownlint
19-19: Column: 1
Hard tabs(MD010, no-hard-tabs)
20-20: Column: 1
Hard tabs(MD010, no-hard-tabs)
24-37: LGTM!The code example demonstrates the correct procedure for reassigning parameters after executing methods, enhancing clarity and usability.
Tools
Markdownlint
31-31: Column: 1
Hard tabs(MD010, no-hard-tabs)
33-33: Column: 1
Hard tabs(MD010, no-hard-tabs)
40-46: LGTM!The change improves clarity and provides detailed steps for verifying the existence of subscriptions and avoiding the mixing of object types and events.
Tools
Markdownlint
41-41: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
44-44: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
45-45: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
46-46: Expected: 2; Actual: 3
Unordered list indentation(MD007, ul-indent)
Line range hint
81-97:
LGTM!The code example provides a clear illustration of how to implement an event handler in the common module, improving clarity and usability.
108-109: LGTM!The change provides a clear and detailed procedure for adding a function to the common module, enhancing clarity and usability.
Tools
Markdownlint
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Line range hint
111-123:
LGTM!The code example provides a clear illustration of how to fill the correspondence between object types and methods, improving clarity and usability.
Tools
Markdownlint
108-108: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- src/cf/CommonModules/пбп_ОбновлениеИнформационнойБазыПБП/Ext/Module.bsl (1 hunks)
Files skipped from review due to trivial changes (1)
- src/cf/CommonModules/пбп_ОбновлениеИнформационнойБазыПБП/Ext/Module.bsl
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- src/cf/CommonModules/пбп_ОбновлениеИнформационнойБазыПБП/Ext/Module.bsl (1 hunks)
- src/cf/Configuration.xml (1 hunks)
Files skipped from review due to trivial changes (1)
- src/cf/Configuration.xml
Files skipped from review as they are similar to previous changes (1)
- src/cf/CommonModules/пбп_ОбновлениеИнформационнойБазыПБП/Ext/Module.bsl
Summary by CodeRabbit
Documentation
Bug Fixes