Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added func to add folders #70

Merged
merged 3 commits into from
May 14, 2024
Merged

Added func to add folders #70

merged 3 commits into from
May 14, 2024

Conversation

denisushakov
Copy link
Contributor

@denisushakov denisushakov commented May 13, 2024

Summary by CodeRabbit

  • New Features

    • Improved explanations and examples for adding values and groups in the module __ПредопределенныеЗначенияПереопределяемый.
    • Added parameters to the Добавить() method for hierarchical structuring in the Результат collection.
  • Refactor

    • Updated the ПредопределенныеЗначения() function for adding new groups with specific properties for better data organization.

Copy link

coderabbitai bot commented May 13, 2024

Walkthrough

The recent updates involve enhancing the hierarchical structure and usage definitions in various modules, particularly focusing on refining the creation of predefined values in the BSL modules. These changes aim to improve data management processes and streamline the handling of predefined values effectively.

Changes

File Path Change Summary
docs/ПредопределенныеЗначения.md Added explanations and examples for adding values and groups in the module __ПредопределенныеЗначенияПереопределяемый, including parameters like ИдентификаторНастройки, Родитель, and УровеньИерархии.
.../__ПредопределенныеЗначенияПереопределяемый Updated Добавить() method with new parameters and added logic for creating a new group in the ПредопределенныеЗначения() function.

🐰✨
In the realm of code, where changes take flight,
Hierarchies bloom, making code bright!
Values and groups now align,
As the rabbit dances, in a code design!
Hurray for updates, a developer's delight! 🌟
🐰✨

Possibly related issues

  • github#57: The changes in this update align with the objective of enhancing predefined values by adding hierarchy support and associating elements with specific groups, addressing the need for hierarchical structuring in the system.

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?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

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)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

This comment has been minimized.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between 2f3422b and 3fdb447.
Files selected for processing (13)
  • src/cf/Catalogs/__ИнтегрируемыеСистемы.xml (2 hunks)
  • src/cf/Catalogs/__ИнтегрируемыеСистемы/Ext/ManagerModule.bsl (1 hunks)
  • src/cf/Catalogs/__МетодыИнтеграции.xml (2 hunks)
  • src/cf/Catalogs/__МетодыИнтеграции/Ext/ManagerModule.bsl (1 hunks)
  • src/cf/Catalogs/__НастройкиИнтеграции.xml (2 hunks)
  • src/cf/Catalogs/__НастройкиИнтеграции/Ext/ManagerModule.bsl (1 hunks)
  • src/cf/Catalogs/__ТипСоответствияОбъектовИБ.xml (2 hunks)
  • src/cf/Catalogs/__ТипСоответствияОбъектовИБ/Ext/ManagerModule.bsl (1 hunks)
  • src/cf/ChartsOfCharacteristicTypes/__ПредопределенныеЗначения.xml (1 hunks)
  • src/cf/CommonModules/__ИнтеграцииПереопределяемый/Ext/Module.bsl (3 hunks)
  • src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl (2 hunks)
  • src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (2 hunks)
  • src/cf/CommonModules/__ТипСоответствияОбъектовИБПереопределяемый/Ext/Module.bsl (1 hunks)
Additional Context Used
GitHub Check Runs (1)
1C: Project SubSystems Library Sonarqube Results success (4)

src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl: [warning] 50-50: src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl#L50
Текст запроса должен быть корректным и открываться конструктором запросов


src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl: [notice] 35-35: src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl#L35
Добавьте описание метода программного интерфейса


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [warning] 12-12: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L12
Необходимо избавиться от многократного использования строкового литерала "Интеграции"


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [warning] 17-17: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L17
Необходимо избавиться от многократного использования строкового литерала "Тестовая"

Additional comments not posted (20)
src/cf/Catalogs/__НастройкиИнтеграции/Ext/ManagerModule.bsl (1)

33-35: Change from creating single to multiple predefined values should enhance performance.

src/cf/Catalogs/__ИнтегрируемыеСистемы/Ext/ManagerModule.bsl (1)

33-35: Change from creating single to multiple predefined values should enhance performance.

src/cf/CommonModules/__ТипСоответствияОбъектовИБПереопределяемый/Ext/Module.bsl (1)

48-53: Addition of new columns for boolean and parent values aligns with enhancing data categorization and parent-child relationship management.

src/cf/Catalogs/__ТипСоответствияОбъектовИБ/Ext/ManagerModule.bsl (1)

60-62: Change from creating single to multiple predefined values should enhance performance.

src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (3)

11-20: Addition of new settings for integration and testing with parent settings specified.


27-35: Addition of new settings for integration and testing with parent settings specified.


43-43: Addition of new settings for integration and testing with parent settings specified.

src/cf/CommonModules/__ИнтеграцииПереопределяемый/Ext/Module.bsl (3)

76-81: Addition of new settings for integrated systems with parent settings specified.


92-97: Addition of new settings for integration methods with parent settings specified.


108-113: Addition of new settings for integration settings with parent settings specified.

src/cf/Catalogs/__МетодыИнтеграции/Ext/ManagerModule.bsl (1)

33-35: Change from creating single to multiple predefined values should enhance performance.

src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl (1)

35-75: Addition of procedures to create predefined values and groups.

src/cf/Catalogs/__ИнтегрируемыеСистемы.xml (2)

39-39: Updated <Hierarchical> tag from false to true to enable hierarchical data structure.


146-146: Updated <Use> tag from ForItem to ForFolderAndItem, expanding the scope to include both items and folders.

src/cf/Catalogs/__МетодыИнтеграции.xml (2)

39-39: Updated <Hierarchical> tag from false to true to enable hierarchical data structure.


238-238: Updated <Use> tag from ForItem to ForFolderAndItem, expanding the scope to include both items and folders.

src/cf/Catalogs/__ТипСоответствияОбъектовИБ.xml (2)

39-39: Updated <Hierarchical> tag from false to true to enable hierarchical data structure.


372-372: Updated <Use> tag from ForItem to ForFolderAndItem, expanding the scope to include both items and folders.

src/cf/Catalogs/__НастройкиИнтеграции.xml (2)

39-39: Change to hierarchical structure approved.

This change aligns with the PR's objective to enhance hierarchical structures within the system.


522-522: Change in tag usage approved.

The update from ForItem to ForFolderAndItem enhances data handling flexibility, consistent with the PR's goals.

@@ -576,7 +576,7 @@
<LinkByType/>
<ChoiceHistoryOnInput>Auto</ChoiceHistoryOnInput>
<Indexing>DontIndex</Indexing>
<Use>ForItem</Use>
<Use>ForFolderAndItem</Use>
Copy link

Choose a reason for hiding this comment

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

Tip

Codebase Verification

Several files in the codebase still use the old <Use> tag and have not been updated to the new <Use>ForFolderAndItem</Use> tag. Please review and update these files accordingly:

  • src/cf/Catalogs/__ПользовательскиеФункции.xml
  • src/cf/Catalogs/__НастройкиОтбораОбъектов.xml
  • src/cf/Catalogs/__ИсторияИнтеграции.xml
Analysis chain

Change in <Use> tag from ForItem to ForFolderAndItem aligns with the PR objectives to enhance flexibility in data handling.

Scripts Executed

The following scripts were executed for the analysis:


Script:

#!/bin/bash
# Description: Verify that the `<Use>` tag change is consistently applied across similar XML configurations.

# Test: Search for other occurrences of `<Use>` tags in XML files to ensure consistency.
rg --type xml '<Use>ForItem</Use>' -l | xargs rg '<Use>ForFolderAndItem</Use>'

Length of output: 326



Script:

#!/bin/bash
# Description: Verify that no XML files are missed in the update from `<Use>ForItem</Use>` to `<Use>ForFolderAndItem</Use>`.

# Search for all XML files and check for the presence of both `<Use>` tags to ensure complete update.
rg --type xml '<Use>ForItem</Use>' -l
rg --type xml '<Use>ForFolderAndItem</Use>' -l

Length of output: 605



Script:

#!/bin/bash
# Description: Verify specific files that might not have been updated to `<Use>ForFolderAndItem</Use>`.

# Check for the absence of the new `<Use>` tag in files that only appeared in the old tag list.
rg --type xml '<Use>ForFolderAndItem</Use>' src/cf/Catalogs/__ПользовательскиеФункции.xml
rg --type xml '<Use>ForFolderAndItem</Use>' src/cf/Catalogs/__НастройкиОтбораОбъектов.xml
rg --type xml '<Use>ForFolderAndItem</Use>' src/cf/Catalogs/__ИсторияИнтеграции.xml

Length of output: 719

This comment has been minimized.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 4

Out of diff range and nitpick comments (1)
src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (1)

65-65: Add a method description for ОбщиеКолонкиТаблицыПредопределенныхЗначений to improve code documentation.

Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between 3fdb447 and ee78729.
Files selected for processing (4)
  • src/cf/CommonModules/__ИнтеграцииПереопределяемый/Ext/Module.bsl (3 hunks)
  • src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl (2 hunks)
  • src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl (3 hunks)
  • src/cf/CommonModules/__ТипСоответствияОбъектовИБПереопределяемый/Ext/Module.bsl (1 hunks)
Files skipped from review as they are similar to previous changes (1)
  • src/cf/CommonModules/__ИнтеграцииПереопределяемый/Ext/Module.bsl
Additional Context Used
GitHub Check Runs (1)
1C: Project SubSystems Library Sonarqube Results success (14)

src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl: [warning] 40-40: src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl#L40
Длина строки 123 превышает максимально допустимую 120


src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl: [warning] 57-57: src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl#L57
Текст запроса должен быть корректным и открываться конструктором запросов


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 18-18: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L18
Возможная опечатка в "локал"


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 19-19: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L19
Возможная опечатка в "локал"


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 65-65: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L65
Добавьте описание метода программного интерфейса


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 67-67: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L67
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 67-67: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L67
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 67-67: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L67
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 73-73: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L73
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 73-73: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L73
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 73-73: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L73
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [notice] 73-73: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L73
Справа от ',' не хватает пробела


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [warning] 12-12: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L12
Необходимо избавиться от многократного использования строкового литерала "Интеграции"


src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl: [warning] 25-25: src/cf/CommonModules/__ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl#L25
Необходимо избавиться от многократного использования строкового литерала "Тестовая"

Additional comments not posted (2)
src/cf/CommonModules/__ТипСоответствияОбъектовИБПереопределяемый/Ext/Module.bsl (1)

47-47: The function call to __ПредопределенныеЗначенияПереопределяемый.ОбщиеКолонкиТаблицыПредопределенныхЗначений correctly integrates common columns into the predefined values table, aligning with the hierarchical enhancements described.

src/cf/CommonModules/__ПредопределенныеЗначения/Ext/Module.bsl (1)

29-33: The procedure ИнициализироватьПредопределенныеЗначения correctly initializes the predefined values as intended.

Comment on lines +18 to +19
НоваяНастройка.Наименование = "Интеграции локал";
НоваяНастройка.ИдентификаторНастройки = "Интеграции_локал";
Copy link

Choose a reason for hiding this comment

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

Consider correcting the potential typographical error in "локал" to "local" if that was the intended term.

- НоваяНастройка.Наименование = "Интеграции локал";
- НоваяНастройка.ИдентификаторНастройки = "Интеграции_локал";
+ НоваяНастройка.Наименование = "Интеграции local";
+ НоваяНастройка.ИдентификаторНастройки = "Интеграции_local";

Committable suggestion

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

Suggested change
НоваяНастройка.Наименование = "Интеграции локал";
НоваяНастройка.ИдентификаторНастройки = "Интеграции_локал";
НоваяНастройка.Наименование = "Интеграции local";
НоваяНастройка.ИдентификаторНастройки = "Интеграции_local";


Результат = ТаблицаПредопределенных();

ОписаниеЧисло = Новый ОписаниеТипов("Число",,, Новый КвалификаторыЧисла(10,0));

// Добавление
НоваяНастройка = Результат.Добавить();
НоваяНастройка.Наименование = "Интеграции";
Copy link

Choose a reason for hiding this comment

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

Refactor to avoid repeated use of the string literal "Интеграции". Consider defining it as a constant.

+ const GROUP_NAME = "Интеграции";
...
- НоваяНастройка.Наименование = "Интеграции";
- НоваяНастройка.ИдентификаторНастройки = "Интеграции";
+ НоваяНастройка.Наименование = GROUP_NAME;
+ НоваяНастройка.ИдентификаторНастройки = GROUP_NAME;

Committable suggestion

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

Suggested change
НоваяНастройка.Наименование = "Интеграции";
const GROUP_NAME = "Интеграции";
НоваяНастройка.Наименование = GROUP_NAME;
НоваяНастройка.ИдентификаторНастройки = GROUP_NAME;

Comment on lines +67 to +73
ОписаниеСтрока = Новый ОписаниеТипов("Строка",,,, Новый КвалификаторыСтроки(150));

Таблица.Колонки.Добавить("ИдентификаторНастройки", ОписаниеСтрока);
Таблица.Колонки.Добавить("Наименование", ОписаниеСтрока);
Таблица.Колонки.Добавить("ЭтоГруппа", Новый ОписаниеТипов("Булево"));
Таблица.Колонки.Добавить("Родитель", ОписаниеСтрока);
Таблица.Колонки.Добавить("УровеньИерархии", Новый ОписаниеТипов("Число",,,Новый КвалификаторыЧисла(2,0)));
Copy link

Choose a reason for hiding this comment

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

Correct the formatting by adding spaces after commas to maintain consistency and improve readability.

- ОписаниеСтрока = Новый ОписаниеТипов("Строка",,,, Новый КвалификаторыСтроки(150));
+ ОписаниеСтрока = Новый ОписаниеТипов("Строка", , , , Новый КвалификаторыСтроки(150));

Committable suggestion

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

Suggested change
ОписаниеСтрока = Новый ОписаниеТипов("Строка",,,, Новый КвалификаторыСтроки(150));
Таблица.Колонки.Добавить("ИдентификаторНастройки", ОписаниеСтрока);
Таблица.Колонки.Добавить("Наименование", ОписаниеСтрока);
Таблица.Колонки.Добавить("ЭтоГруппа", Новый ОписаниеТипов("Булево"));
Таблица.Колонки.Добавить("Родитель", ОписаниеСтрока);
Таблица.Колонки.Добавить("УровеньИерархии", Новый ОписаниеТипов("Число",,,Новый КвалификаторыЧисла(2,0)));
ОписаниеСтрока = Новый ОписаниеТипов("Строка", , , , Новый КвалификаторыСтроки(150));
Таблица.Колонки.Добавить("ИдентификаторНастройки", ОписаниеСтрока);
Таблица.Колонки.Добавить("Наименование", ОписаниеСтрока);
Таблица.Колонки.Добавить("ЭтоГруппа", Новый ОписаниеТипов("Булево"));
Таблица.Колонки.Добавить("Родитель", ОписаниеСтрока);
Таблица.Колонки.Добавить("УровеньИерархии", Новый ОписаниеТипов("Число",,,Новый КвалификаторыЧисла(2,0)));

// Параметры:
// Таблица - ТаблицаЗначений - см. __ПредопределенныеЗначенияПереопределяемыйТаблицаПредопределенных,
// __ТипСоответствияОбъектовИБПереопределяемый.ТаблицаПредопределенных
// МенеджерОбъекта - СправочникМенеджер, ДокументМенеджер, ОбработкаМенеджер, РегистрСведенийМенеджер - менеджер объекта.
Copy link

Choose a reason for hiding this comment

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

Reduce the line length to adhere to best practices, possibly by breaking the line into multiple lines.

- //  МенеджерОбъекта	 - СправочникМенеджер, ДокументМенеджер, ОбработкаМенеджер, РегистрСведенийМенеджер - менеджер объекта.
+ //  МенеджерОбъекта	 - СправочникМенеджер, ДокументМенеджер,
+ //                      ОбработкаМенеджер, РегистрСведенийМенеджер - менеджер объекта.

Committable suggestion

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

Suggested change
// МенеджерОбъекта - СправочникМенеджер, ДокументМенеджер, ОбработкаМенеджер, РегистрСведенийМенеджер - менеджер объекта.
// МенеджерОбъекта - СправочникМенеджер, ДокументМенеджер,
// ОбработкаМенеджер, РегистрСведенийМенеджер - менеджер объекта.

Copy link

Passed

Analysis Details

14 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 14 Code Smells

Coverage and Duplications

  • Coverage No coverage information (0.00% Estimated after merge)
  • Duplications 0.00% Duplicated Code (0.50% Estimated after merge)

Project ID: PSSL

View in SonarQube

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Out of diff range and nitpick comments (2)
docs/ПредопределенныеЗначения.md (2)

Line range hint 6-22: Fix the unordered list indentation for better readability.

- - **__ОбщегоНазначенияСервер.ПолучитьПредопределенноеЗначение(Имя, ИспользоватьКэш)** - если это единичное значение, возвращается сразу оно, если установлен флаг "Список значений" то возвращает массив значений. Для получения часто используемых предопределенных значений, установите ИспользоватьКэш = Истина.
-   - Параметры:
-     - Имя - Строка
-     - ИспользоватьКэш - Булево
-   - Возвращаемое значение:
-     - ПроизвольныйТип - Значение переменной ПВХ.
-     - Массив - Массив значений переменной ПВХ.
- - **__ОбщегоНазначенияСервер.ПолучитьПредопределенныеЗначения(Имена, ВРазрезеКлючей, ИспользоватьКэш)** - если "ВРазрезеКлючей" то возвращает соответствие нескольких элементов ПВХ, если ВРазрезеКлючей = ложь, то возвращает массив значений. Для получения часто используемых предопределенных значений, установите ИспользоватьКэш = Истина.
-   - Параметры:
-     - Имена - Строка - Имена переменных ПВХ, перечисленные через запятую.
-     - ВРазрезеКлючей - Булево
-     - ИспользоватьКэш - Булево
-   - Возвращаемое значение:
-     - Соответствие - Соответствие имен и значений переменных ПВХ.
-       - Ключ - Строка - имя значения
-       - Значение - ПроизвольныйТип
-     - Массив - ПроизвольныйТип
+ - **__ОбщегоНазначенияСервер.ПолучитьПредопределенноеЗначение(Имя, ИспользоватьКэш)** - если это единичное значение, возвращается сразу оно, если установлен флаг "Список значений" то возвращает массив значений. Для получения часто используемых предопределенных значений, установите ИспользоватьКэш = Истина.
+   - Параметры:
+     - Имя - Строка
+     - ИспользоватьКэш - Булево
+   - Возвращаемое значение:
+     - ПроизвольныйТип - Значение переменной ПВХ.
+     - Массив - Массив значений переменной ПВХ.
+ - **__ОбщегоНазначенияСервер.ПолучитьПредопределенныеЗначения(Имена, ВРазрезеКлючей, ИспользоватьКэш)** - если "ВРазрезеКлючей" то возвращает соответствие нескольких элементов ПВХ, если ВРазрезеКлючей = ложь, то возвращает массив значений. Для получения часто используемых предопределенных значений, установите ИспользоватьКэш = Истина.
+   - Параметры:
+     - Имена - Строка - Имена переменных ПВХ, перечисленные через запятую.
+     - ВРазрезеКлючей - Булево
+     - ИспользоватьКэш - Булево
+   - Возвращаемое значение:
+     - Соответствие - Соответствие имен и значений переменных ПВХ.
+       - Ключ - Строка - имя значения
+       - Значение - ПроизвольныйТип
+     - Массив - ПроизвольныйТип

Line range hint 34-74: Replace hard tabs with spaces for consistent indentation.

-	Результат = ТаблицаПредопределенных();
-	// Добавление
-	НоваяНастройка = Результат.Добавить();
-	НоваяНастройка.Наименование = "Пароль доступа на ИТС";
-	НоваяНастройка.ИдентификаторНастройки = "ПарольДоступаНаИТС";
-	НоваяНастройка.Пароль = Истина;
-	НоваяНастройка.СписокЗначений = Ложь;
-	НоваяНастройка.ТипЗначения = Новый ОписаниеТипов("Строка",,,, Новый КвалификаторыСтроки(150));
-	НоваяНастройка.Родитель = "Доступ_ИТС"; // Указывается идентификатор настройки группы
-	// КонецДобавления
-	
-	Возврат Результат;
-	
-КонецФункции
+    Результат = ТаблицаПредопределенных();
+    // Добавление
+    НоваяНастройка = Результат.Добавить();
+    НоваяНастройка.Наименование = "Пароль доступа на ИТС";
+    НоваяНастройка.ИдентификаторНастройки = "ПарольДоступаНаИТС";
+    НоваяНастройка.Пароль = Истина;
+    НоваяНастройка.СписокЗначений = Ложь;
+    НоваяНастройка.ТипЗначения = Новый ОписаниеТипов("Строка",,,, Новый КвалификаторыСтроки(150));
+    НоваяНастройка.Родитель = "Доступ_ИТС"; // Указывается идентификатор настройки группы
+    // КонецДобавления
+    
+    Возврат Результат;
+    
+КонецФункции
Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between ee78729 and 5b27b49.
Files selected for processing (1)
  • docs/ПредопределенныеЗначения.md (2 hunks)
Additional comments not posted (3)
docs/ПредопределенныеЗначения.md (3)

24-27: The new parameters ИдентификаторНастройки, Родитель, and УровеньИерархии are well-documented and enhance the functionality.


Line range hint 28-49: The example for adding a new element is clear and effectively demonstrates the usage of the new parameters.


Line range hint 51-74: The example for adding a new group is clear and effectively demonstrates hierarchical structuring.

@ivanmolodec ivanmolodec merged commit 37fbb59 into develop May 14, 2024
4 checks passed
@ivanmolodec ivanmolodec deleted the PredefinedValues branch May 14, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Доработать программное создание предопределенных значений
2 participants