Page MenuHomeDevCentral

Split user management between User and UserRepository
ClosedPublic

Authored by dereckson on Sun, Nov 9, 14:31.
Tags
None
Referenced Files
F12589480: D3857.diff
Tue, Nov 11, 17:20
Unknown Object (File)
Sun, Nov 9, 22:41
Subscribers
None

Details

Summary

Move static methods for user ID lookup, loading, and resolution
into UserRepository class.

Add an instance of the UserRepository class to the context.

Authentication methods and workspace-related code are updated to call
these methods via the context or pass the database directly.

[ Next steps ]

From dependency injection point of view, this change is "part 1":
it replaces global variables with context-based usage,
but $context still acts a service locator.

From separation of concern standpoint, the plan is to move
all database-related methods into the UserRepository class.

Diff Detail

Repository
rOBSIDIAN Obsidian Workspaces
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson created this revision.
dereckson edited the summary of this revision. (Show Details)
dereckson edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Sun, Nov 9, 14:38