Расширения информации о правах #465
Conversation
Для MDO\MDC добавлена возможность понять можно ли управлять правами доступа и какими
Добавлены методы определения наличия нужного права и списка ролей с ныжным правом
ОбзорВнесены изменения в несколько классов для поддержки возможности получения и проверки прав доступа ролей. Добавлены методы для возврата возможных прав и проверки прав для определенных объектов. Также были внесены различные импорты и внесены небольшие изменения в порядок методов. Изменения
Поэма
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 Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 28
Outside diff range and nitpick comments (8)
src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTableField.java (1)
24-30: Добавьте пустую строку после блока импортов.Для улучшения читаемости кода добавьте пустую строку после блока импортов.
src/test/java/com/github/_1c_syntax/bsl/smoke/RightTest.java (1)
1-21: Добавьте пустую строку после блока комментариев.Для улучшения читаемости кода добавьте пустую строку после блока комментариев.
src/main/java/com/github/_1c_syntax/bsl/mdo/children/IntegrationServiceChannel.java (1)
24-29: Добавьте пустую строку после блока импортов.Для улучшения читаемости кода добавьте пустую строку после блока импортов.
src/main/java/com/github/_1c_syntax/bsl/mdo/children/Dimension.java (1)
24-30: Добавьте пустую строку после блока импортов.Для улучшения читаемости кода добавьте пустую строку после блока импортов.
src/main/java/com/github/_1c_syntax/bsl/mdo/Sequence.java (1)
Line range hint
44-109:
Исправьте опечатку в имени списка.Переменная
POSIBLE_RIGHTSдолжна быть переименована вPOSSIBLE_RIGHTSдля соответствия стандартам написания.- private static final List<RoleRight> POSIBLE_RIGHTS = List.of(RoleRight.READ, RoleRight.EDIT); + private static final List<RoleRight> POSSIBLE_RIGHTS = List.of(RoleRight.READ, RoleRight.EDIT);- return POSIBLE_RIGHTS; + return POSSIBLE_RIGHTS;src/main/java/com/github/_1c_syntax/bsl/mdo/Constant.java (1)
Line range hint
42-109:
Исправьте опечатку в имени списка.Переменная
POSIBLE_RIGHTSдолжна быть переименована вPOSSIBLE_RIGHTSдля соответствия стандартам написания.- private static final List<RoleRight> POSIBLE_RIGHTS = computePosibleRighs(); + private static final List<RoleRight> POSSIBLE_RIGHTS = computePossibleRights();- return POSIBLE_RIGHTS; + return POSSIBLE_RIGHTS;src/main/java/com/github/_1c_syntax/bsl/mdo/children/WebServiceOperation.java (1)
Line range hint
48-119:
Исправьте опечатку в имени списка.Переменная
POSIBLE_RIGHTSдолжна быть переименована вPOSSIBLE_RIGHTSдля соответствия стандартам написания.- private static final List<RoleRight> POSIBLE_RIGHTS = List.of(RoleRight.USE); + private static final List<RoleRight> POSSIBLE_RIGHTS = List.of(RoleRight.USE);- return POSIBLE_RIGHTS; + return POSSIBLE_RIGHTS;src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectAttribute.java (1)
Line range hint
45-122:
Исправьте опечатку в имени списка.Переменная
POSIBLE_RIGHTSдолжна быть переименована вPOSSIBLE_RIGHTSдля соответствия стандартам написания.- private static final List<RoleRight> POSIBLE_RIGHTS = List.of(RoleRight.VIEW, RoleRight.EDIT); + private static final List<RoleRight> POSSIBLE_RIGHTS = List.of(RoleRight.VIEW, RoleRight.EDIT);- return POSIBLE_RIGHTS; + return POSSIBLE_RIGHTS;
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (50)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/CF.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/CFAccess.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/Configuration.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/ConfigurationExtension.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/Rights.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/package-info.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccountingRegister.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccumulationRegister.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/BusinessProcess.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CalculationRegister.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Catalog.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfAccounts.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCalculationTypes.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCharacteristicTypes.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Command.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonAttribute.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonCommand.java (2 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonForm.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Constant.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/DataProcessor.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Document.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/DocumentJournal.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ExchangePlan.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ExternalDataSource.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/FilterCriterion.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/InformationRegister.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Report.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Sequence.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/SessionParameter.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Subsystem.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Task.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/AccountingFlag.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Dimension.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExtDimensionAccountingFlag.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTable.java (5 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTableField.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/HTTPServiceMethod.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/IntegrationServiceChannel.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectAttribute.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectCommand.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectTabularSection.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Recalculation.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Resource.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/TaskAddressingAttribute.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/WebServiceOperation.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/reader/designer/DesignerReader.java (1 hunks)
- src/test/java/com/github/_1c_syntax/bsl/mdclasses/helpers/RightsTest.java (1 hunks)
- src/test/java/com/github/_1c_syntax/bsl/smoke/RightTest.java (1 hunks)
- src/test/resources/ext/edt/mdclasses_ext/configuration/src/Roles/Роль2/Rights.rights (1 hunks)
Files skipped from review due to trivial changes (2)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/package-info.java
- src/main/java/com/github/_1c_syntax/bsl/reader/designer/DesignerReader.java
Additional comments not posted (96)
src/main/java/com/github/_1c_syntax/bsl/mdo/Command.java (1)
27-29: Изменение интерфейса CommandИнтерфейс
Commandтеперь расширяетAccessRightsOwner. Убедитесь, что все классы, реализующиеCommand, также корректно реализуют методы изAccessRightsOwner.src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java (1)
56-61: Метод isValidRightМетод
isValidRightпроверяет, допустимо ли переданное право для объекта, используя методgetPosibleRights. Это логично и соответствует назначению интерфейса.src/main/java/com/github/_1c_syntax/bsl/mdo/SessionParameter.java (3)
41-43: Реализация интерфейса AccessRightsOwnerКласс
SessionParameterтеперь реализует интерфейсAccessRightsOwner. Это позволяет управлять правами доступа для параметров сессии.
43-43: Статический список POSIBLE_RIGHTSДобавлен статический список
POSIBLE_RIGHTS, содержащий возможные права доступа (GET,SET). Это обеспечивает централизованное управление правами.
71-73: Метод posibleRightsМетод
posibleRightsвозвращает статический списокPOSIBLE_RIGHTS. Это соответствует требованиям интерфейсаAccessRightsOwner.src/main/java/com/github/_1c_syntax/bsl/mdo/CommonCommand.java (2)
Line range hint
41-41:
Реализация интерфейса CommandКласс
CommonCommandтеперь реализует интерфейсCommand. Это позволяет управлять правами доступа для общих команд.
69-71: Метод posibleRightsМетод
posibleRightsвозвращает возможные права доступа, используя методposibleRightsизObjectCommand. Это обеспечивает централизованное управление правами.src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExtDimensionAccountingFlag.java (1)
45-45: Реализация интерфейса AccessRightsOwner.Реализация интерфейса
AccessRightsOwnerвыглядит корректной.src/main/java/com/github/_1c_syntax/bsl/mdo/children/HTTPServiceMethod.java (1)
43-43: Реализация интерфейса AccessRightsOwner.Реализация интерфейса
AccessRightsOwnerвыглядит корректной.src/main/java/com/github/_1c_syntax/bsl/mdo/children/Resource.java (1)
45-45: Реализация интерфейса AccessRightsOwner.Реализация интерфейса
AccessRightsOwnerвыглядит корректной.src/main/java/com/github/_1c_syntax/bsl/mdo/CommonForm.java (1)
45-45: Реализация интерфейса AccessRightsOwner.Реализация интерфейса
AccessRightsOwnerвыглядит корректной.src/main/java/com/github/_1c_syntax/bsl/mdo/Sequence.java (1)
27-27: Импорт утвержден.Импорт
RoleRightнеобходим для новой функциональности, связанной с правами доступа.src/main/java/com/github/_1c_syntax/bsl/mdo/Constant.java (1)
26-26: Импорт утвержден.Импорт
RoleRightнеобходим для новой функциональности, связанной с правами доступа.src/main/java/com/github/_1c_syntax/bsl/mdo/children/WebServiceOperation.java (1)
31-31: Импорт утвержден.Импорт
RoleRightнеобходим для новой функциональности, связанной с правами доступа.src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectAttribute.java (1)
30-30: Импорт утвержден.Импорт
RoleRightнеобходим для новой функциональности, связанной с правами доступа.src/main/java/com/github/_1c_syntax/bsl/mdo/ExternalDataSource.java (2)
27-27: Импорт утверждён.Импорт
RoleRightнеобходим для новых методов, связанных с правами доступа.
46-46: Изменение утверждено.Класс теперь реализует интерфейс
AccessRightsOwner, что согласуется с новой функциональностью управления правами доступа.src/main/java/com/github/_1c_syntax/bsl/mdo/children/TaskAddressingAttribute.java (1)
24-24: Импорт утверждён.Импорт
RoleRightнеобходим для новых методов, связанных с правами доступа.Also applies to: 30-30, 39-40
src/main/java/com/github/_1c_syntax/bsl/mdo/FilterCriterion.java (1)
28-28: Импорт утверждён.Импорт
RoleRightнеобходим для новых методов, связанных с правами доступа.src/main/java/com/github/_1c_syntax/bsl/mdclasses/CFAccess.java (6)
45-47: Метод утверждён.Метод проверяет наличие указанного права хотя бы у одной роли для конфигурации/расширения, что необходимо для новой функциональности управления правами доступа.
56-58: Метод утверждён.Метод проверяет наличие указанного права хотя бы у одной роли для MD объектов, что необходимо для новой функциональности управления правами доступа.
67-69: Метод утверждён.Метод проверяет наличие указанного права хотя бы у одной роли для ссылок, что необходимо для новой функциональности управления правами доступа.
77-79: Метод утверждён.Метод возвращает список ролей, имеющих указанное право для конфигурации/расширения, что необходимо для новой функциональности управления правами доступа.
88-90: Метод утверждён.Метод возвращает список ролей, имеющих указанное право для MD объектов, что необходимо для новой функциональности управления правами доступа.
99-101: Метод утверждён.Метод возвращает список ролей, имеющих указанное право для ссылок, что необходимо для новой функциональности управления правами доступа.
src/main/java/com/github/_1c_syntax/bsl/mdo/AccountingRegister.java (3)
31-31: Добавлен новый импорт.Импорт
RoleRightдобавлен для поддержки новых методов, связанных с правами доступа.
50-50: Класс теперь реализует интерфейс AccessRightsOwner.Это изменение позволяет классу
AccountingRegisterуправлять правами доступа.
119-124: Добавлен метод для возврата возможных прав доступа.Метод
posibleRightsвозвращает возможные права доступа, используя метод из классаAccumulationRegister.src/main/java/com/github/_1c_syntax/bsl/mdo/DocumentJournal.java (4)
30-30: Добавлен новый импорт.Импорт
RoleRightдобавлен для поддержки новых методов, связанных с правами доступа.
50-50: Класс теперь реализует интерфейс AccessRightsOwner.Это изменение позволяет классу
DocumentJournalуправлять правами доступа.
52-52: Добавлен статический список возможных прав доступа.Список
POSIBLE_RIGHTSсодержит праваREADиVIEW. Это позволяет классуDocumentJournalвозвращать фиксированный набор прав доступа.
142-147: Добавлен метод для возврата возможных прав доступа.Метод
posibleRightsвозвращает статический список возможных прав доступа.src/main/java/com/github/_1c_syntax/bsl/mdo/AccumulationRegister.java (5)
31-31: Добавлен новый импорт.Импорт
RoleRightдобавлен для поддержки новых методов, связанных с правами доступа.
50-50: Класс теперь реализует интерфейс AccessRightsOwner.Это изменение позволяет классу
AccumulationRegisterуправлять правами доступа.
52-52: Добавлен статический список возможных прав доступа.Список
POSIBLE_RIGHTSинициализируется методомcomputePosibleRighs, что позволяет динамически вычислять возможные права доступа.
121-126: Добавлен метод для возврата возможных прав доступа.Метод
posibleRightsвозвращает статический список возможных прав доступа.
139-148: Добавлен метод для вычисления возможных прав доступа.Метод
computePosibleRighsвозвращает список возможных прав доступа для классаAccumulationRegister.src/main/java/com/github/_1c_syntax/bsl/mdo/CalculationRegister.java (5)
32-32: Добавлен новый импорт.Импорт
RoleRightдобавлен для поддержки новых методов, связанных с правами доступа.
51-51: Класс теперь реализует интерфейс AccessRightsOwner.Это изменение позволяет классу
CalculationRegisterуправлять правами доступа.
53-53: Добавлен статический список возможных прав доступа.Список
POSIBLE_RIGHTSинициализируется методомcomputePosibleRighs, что позволяет динамически вычислять возможные права доступа.
125-130: Добавлен метод для возврата возможных прав доступа.Метод
posibleRightsвозвращает статический список возможных прав доступа.
144-151: Добавлен метод для вычисления возможных прав доступа.Метод
computePosibleRighsвозвращает список возможных прав доступа для классаCalculationRegister.src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCalculationTypes.java (3)
29-29: Импорт добавлен корректно.Импорт
RoleRightнеобходим для работы с правами доступа.
48-48: Реализация интерфейса корректна.Класс
ChartOfCalculationTypesреализует интерфейсAccessRightsOwner, что соответствует целям PR.
132-134: Метод реализован корректно.Метод
posibleRightsвозвращает перечень возможных прав доступа, вызываяCatalog.posibleRights().src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCharacteristicTypes.java (3)
29-29: Импорт добавлен корректно.Импорт
RoleRightнеобходим для работы с правами доступа.
48-48: Реализация интерфейса корректна.Класс
ChartOfCharacteristicTypesреализует интерфейсAccessRightsOwner, что соответствует целям PR.
133-135: Метод реализован корректно.Метод
posibleRightsвозвращает перечень возможных прав доступа, вызываяCatalog.posibleRights().src/main/java/com/github/_1c_syntax/bsl/mdo/Subsystem.java (3)
26-26: Импорт добавлен корректно.Импорт
RoleRightнеобходим для работы с правами доступа.
46-46: Реализация интерфейса корректна.Класс
Subsystemреализует интерфейсAccessRightsOwner, что соответствует целям PR.
149-151: Метод реализован корректно.Метод
posibleRightsвозвращает перечень возможных прав доступа, используя константуPOSIBLE_RIGHTS.src/main/java/com/github/_1c_syntax/bsl/mdo/InformationRegister.java (4)
31-31: Импорт добавлен корректно.Импорт
RoleRightнеобходим для работы с правами доступа.
50-50: Реализация интерфейса корректна.Класс
InformationRegisterреализует интерфейсAccessRightsOwner, что соответствует целям PR.
125-127: Метод реализован корректно.Метод
posibleRightsвозвращает перечень возможных прав доступа, используя константуPOSIBLE_RIGHTS.
141-158: Метод реализован корректно.Метод
computePosibleRighsвозвращает предопределенный список значенийRoleRight.src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTable.java (4)
24-24: Импорты подтверждены.Импорты
RoleRightиAccessRightsOwnerнеобходимы для реализации новых методов и интерфейсов.Also applies to: 37-37
57-59: Подтверждение реализации интерфейса и статического списка.Класс
ExternalDataSourceTableкорректно реализует интерфейсAccessRightsOwner. Статический списокPOSIBLE_RIGHTSопределен правильно.
148-153: Метод подтвержден.Метод
posibleRightsкорректно возвращает статический списокPOSIBLE_RIGHTS.
163-175: Метод подтвержден.Метод
computePosibleRighsкорректно возвращает список значенийRoleRight.src/main/java/com/github/_1c_syntax/bsl/mdo/DataProcessor.java (3)
29-29: Импорт подтвержден.Импорт
RoleRightнеобходим для реализации новых методов и интерфейсов.
49-51: Подтверждение реализации интерфейса и статического списка.Класс
DataProcessorкорректно реализует интерфейсAccessRightsOwner. Статический списокPOSIBLE_RIGHTSопределен правильно.
161-166: Метод подтвержден.Метод
posibleRightsкорректно возвращает статический списокPOSIBLE_RIGHTS.src/main/java/com/github/_1c_syntax/bsl/mdo/Report.java (3)
29-29: Импорт подтвержден.Импорт
RoleRightнеобходим для реализации новых методов и интерфейсов.
50-52: Подтверждение реализации интерфейса и статического списка.Класс
Reportкорректно реализует интерфейсAccessRightsOwner. Статический списокPOSIBLE_RIGHTSопределен правильно.
163-168: Метод подтвержден.Метод
posibleRightsкорректно возвращает статический списокPOSIBLE_RIGHTS.src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfAccounts.java (3)
31-31: Импорт подтвержден.Импорт
RoleRightнеобходим для реализации новых методов и интерфейсов.
50-50: Подтверждение реализации интерфейса и статического списка.Класс
ChartOfAccountsкорректно реализует интерфейсAccessRightsOwner. Статический списокPOSIBLE_RIGHTSопределен правильно.
149-153: Метод подтвержден.Метод
posibleRightsкорректно возвращает статический списокPOSIBLE_RIGHTSизCatalog.src/main/java/com/github/_1c_syntax/bsl/mdo/CommonAttribute.java (1)
167-169: Изменения одобрены.Метод
posibleRightsвозвращает список возможных прав доступа, используя методObjectAttribute.posibleRights().src/main/java/com/github/_1c_syntax/bsl/mdclasses/CF.java (1)
41-41: Изменения одобрены.Интерфейс
CFтеперь расширяетCFAccess, что обеспечивает наследование методов для управления правами доступа.src/test/resources/ext/edt/mdclasses_ext/configuration/src/Roles/Роль2/Rights.rights (1)
117-117: Изменения одобрены.XML конфигурация обновлена для включения новых прав доступа.
src/main/java/com/github/_1c_syntax/bsl/mdo/BusinessProcess.java (2)
140-142: Изменения одобрены.Метод
posibleRightsвозвращает список возможных прав доступа, используя статический списокPOSIBLE_RIGHTS.
164-191: Изменения одобрены.Метод
computePosibleRighsвозвращает предопределенный список возможных прав доступа. Список прав выглядит полным и правильно определенным.src/main/java/com/github/_1c_syntax/bsl/mdo/Catalog.java (4)
29-29: Импорт утвержден.Импорт
RoleRightнеобходим для управления правами доступа.
48-48: Реализация класса утверждена.Класс
Catalogтеперь реализуетAccessRightsOwner, что соответствует целям PR для управления правами доступа.
50-50: Добавление статического списка и метода утверждено.Статический список
POSIBLE_RIGHTSи методposibleRightsнеобходимы для возврата возможных прав ролей.Also applies to: 132-137
159-187: Реализация метода утверждена.Метод
computePosibleRighsправильно перечисляет все возможные права ролей.src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/Rights.java (5)
24-30: Импорт утвержден.Импорты
CF,Configuration,MD,Role,RoleDataиRoleRightнеобходимы для управления правами доступа.
51-53: Реализация метода утверждена.Метод
rightAccess(CF cf, RoleRight roleRight)проверяет наличие указанного разрешения хотя бы у одной роли для конфигурации/расширения.
63-65: Реализация метода утверждена.Метод
rightAccess(CF cf, RoleRight roleRight, MD md)проверяет наличие указанного разрешения хотя бы у одной роли для объекта MD.
75-77: Реализация метода утверждена.Метод
rightAccess(CF cf, RoleRight roleRight, MdoReference mdoReference)проверяет наличие указанного разрешения хотя бы у одной роли для ссылки MdoReference.
110-112: Реализация метода утверждена.Метод
rolesAccess(CF cf, RoleRight roleRight, MdoReference mdoReference)возвращает список ролей, имеющих указанное разрешение для ссылки MdoReference.src/main/java/com/github/_1c_syntax/bsl/mdo/Task.java (4)
30-30: Импорт утвержден.Импорт
RoleRightнеобходим для управления правами доступа.
49-49: Реализация класса утверждена.Класс
Taskтеперь реализуетAccessRightsOwner, что соответствует целям PR для управления правами доступа.
51-51: Добавление статического списка и метода утверждено.Статический список
POSIBLE_RIGHTSи методposibleRightsнеобходимы для возврата возможных прав ролей.Also applies to: 138-143
165-191: Реализация метода утверждена.Метод
computePosibleRighsправильно перечисляет все возможные права ролей.src/main/java/com/github/_1c_syntax/bsl/mdo/Document.java (4)
29-29: Импорт утвержден.Импорт
RoleRightнеобходим для управления правами доступа.
48-48: Реализация класса утверждена.Класс
Documentтеперь реализуетAccessRightsOwner, что соответствует целям PR для управления правами доступа.
50-50: Добавление статического списка и метода утверждено.Статический список
POSIBLE_RIGHTSи методposibleRightsнеобходимы для возврата возможных прав ролей.Also applies to: 137-142
164-192: Реализация метода утверждена.Метод
computePosibleRighsправильно перечисляет все возможные права ролей.src/main/java/com/github/_1c_syntax/bsl/mdo/ExchangePlan.java (1)
179-181: МетодposibleRightsМетод возвращает статический список прав доступа. Всё выглядит корректно.
src/test/java/com/github/_1c_syntax/bsl/mdclasses/helpers/RightsTest.java (4)
38-68: МетодrightAccessCfМетод тестирует различные права доступа для конфигурации. Всё выглядит корректно.
70-91: МетодrightAccessCfeМетод тестирует различные права доступа для расширения конфигурации. Всё выглядит корректно.
93-130: МетодrolesAccessCfМетод тестирует права доступа ролей для конфигурации. Всё выглядит корректно.
132-158: МетодrolesAccessCfeМетод тестирует права доступа ролей для расширения конфигурации. Всё выглядит корректно.
src/main/java/com/github/_1c_syntax/bsl/mdclasses/Configuration.java (1)
351-353: МетодposibleRightsМетод возвращает статический список прав доступа. Всё выглядит корректно.
src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTableField.java
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTableField.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExtDimensionAccountingFlag.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdclasses/Configuration.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdo/ExternalDataSource.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdo/ExternalDataSource.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/_1c_syntax/bsl/mdo/children/TaskAddressingAttribute.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/package-info.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java (1 hunks)
- src/test/java/com/github/_1c_syntax/bsl/smoke/RightTest.java (1 hunks)
Files skipped from review due to trivial changes (1)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/package-info.java
Files skipped from review as they are similar to previous changes (2)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java
- src/test/java/com/github/_1c_syntax/bsl/smoke/RightTest.java
|
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (43)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/Configuration.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/ConfigurationExtension.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/Rights.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java (1 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccountingRegister.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccumulationRegister.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/BusinessProcess.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CalculationRegister.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Catalog.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfAccounts.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCalculationTypes.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCharacteristicTypes.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonAttribute.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonCommand.java (2 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonForm.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Constant.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/DataProcessor.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Document.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/DocumentJournal.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ExchangePlan.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/ExternalDataSource.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/FilterCriterion.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/InformationRegister.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Report.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Sequence.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/SessionParameter.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Subsystem.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/Task.java (4 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/AccountingFlag.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Dimension.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExtDimensionAccountingFlag.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTable.java (5 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTableField.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/HTTPServiceMethod.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/IntegrationServiceChannel.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectAttribute.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectCommand.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectTabularSection.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Recalculation.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Resource.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/TaskAddressingAttribute.java (3 hunks)
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/WebServiceOperation.java (3 hunks)
- src/test/java/com/github/_1c_syntax/bsl/smoke/RightTest.java (1 hunks)
Files skipped from review as they are similar to previous changes (42)
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/Configuration.java
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/ConfigurationExtension.java
- src/main/java/com/github/_1c_syntax/bsl/mdclasses/helpers/Rights.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccessRightsOwner.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccountingRegister.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/AccumulationRegister.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/BusinessProcess.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/CalculationRegister.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Catalog.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCalculationTypes.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfCharacteristicTypes.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonAttribute.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonCommand.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/CommonForm.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Constant.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/DataProcessor.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Document.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/DocumentJournal.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/ExchangePlan.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/ExternalDataSource.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/FilterCriterion.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/InformationRegister.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Report.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Sequence.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/SessionParameter.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Subsystem.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/Task.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/AccountingFlag.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Dimension.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExtDimensionAccountingFlag.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTable.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ExternalDataSourceTableField.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/HTTPServiceMethod.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/IntegrationServiceChannel.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectAttribute.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectCommand.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/ObjectTabularSection.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Recalculation.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/Resource.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/TaskAddressingAttribute.java
- src/main/java/com/github/_1c_syntax/bsl/mdo/children/WebServiceOperation.java
- src/test/java/com/github/_1c_syntax/bsl/smoke/RightTest.java
Additional comments not posted (2)
src/main/java/com/github/_1c_syntax/bsl/mdo/ChartOfAccounts.java (2)
31-31: Импорт утвержденИмпорт
RoleRightнеобходим для реализации методаpossibleRightsи интерфейсаAccessRightsOwner.
149-154: Метод утвержденМетод
possibleRightsкорректно реализован и возвращает список возможных прав доступа, вызывая методCatalog.possibleRights().
Описание
Связанные задачи
Closes
Чеклист
Общие
gradlew precommit)Дополнительно