Skip to content

增强LdapUser对账户控制标志位的判断能力#754

Merged
Qinyouzeng merged 3 commits intomainfrom
feat/update_LdapUser
Jan 5, 2026
Merged

增强LdapUser对账户控制标志位的判断能力#754
Qinyouzeng merged 3 commits intomainfrom
feat/update_LdapUser

Conversation

@MayueCif
Copy link
Contributor

新增IsAccountDisabled、IsNormalAccount属性,以及HasUserAccountControlFlag方法,便于判断用户账户控制(UserAccountControl)相关的标志位,提高代码可读性和易用性。

新增IsAccountDisabled、IsNormalAccount属性,以及HasUserAccountControlFlag方法,便于判断用户账户控制(UserAccountControl)相关的标志位,提高代码可读性和易用性。
Copilot AI review requested due to automatic review settings December 31, 2025 07:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the LdapUser class with convenient helper properties and a method to check UserAccountControl flag states. This improves code readability when working with LDAP user account control flags.

Key Changes:

  • Added two read-only computed properties (IsAccountDisabled, IsNormalAccount) for common flag checks
  • Added a flexible helper method (HasUserAccountControlFlag) for checking any UserAccountControl flag

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Qinyouzeng Qinyouzeng self-requested a review December 31, 2025 07:56
使UserAccountControl支持按位组合多个枚举值,便于表示和处理用户账户控制的多种状态。
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 5, 2026

@Qinyouzeng Qinyouzeng merged commit 68701c3 into main Jan 5, 2026
3 of 4 checks passed
@Qinyouzeng Qinyouzeng deleted the feat/update_LdapUser branch January 5, 2026 05:44
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.

3 participants