Fix issue #190: [Claude] User model custom manager #191
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request fixes #190.
The issue has been successfully resolved based on the following implemented changes and their effects:
A custom
NamedUserManager
was correctly implemented that filters users based on non-empty name_first and name_last fields using proper Django query filters (excluding null and empty strings).The manager was properly added to the User model as
named_users
while preserving the defaultobjects
manager, allowing both filtering methods to coexist without interference.Comprehensive tests were added that:
The implementation follows Django best practices by:
The changes directly address all requirements from the issue description and the tests demonstrate that both the new functionality works as intended and existing functionality remains unchanged. The code is properly structured and follows Django conventions for custom managers.
Automatic fix generated by OpenHands 🙌