Skip to content

Refactor OC\Server::getEncryptionManager - #40134

Closed
summersab wants to merge 1 commit into
nextcloud:masterfrom
summersab:refactor/OC-Server-getEncryptionManager
Closed

Refactor OC\Server::getEncryptionManager#40134
summersab wants to merge 1 commit into
nextcloud:masterfrom
summersab:refactor/OC-Server-getEncryptionManager

Conversation

@summersab

Copy link
Copy Markdown
Contributor

This PR refactors the deprecated method OC\Server::getEncryptionManager and replaces it with OC\Server::get(\OCP\Encryption\IManager::class) throughout the entire NC codebase (excluding ./apps and ./3rdparty).

Additionally, where necessary, the \OCP\Encryption\IManager class is imported via the use directive.

Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
@summersab
summersabforce-pushed the refactor/OC-Server-getEncryptionManager branch from bc1534f to 87515a3CompareAugust 30, 2023 02:29
@solracsfsolracsf added the 3. to review Waiting for reviews label Aug 30, 2023
@solracsfsolracsf added this to the Nextcloud 28 milestone Aug 30, 2023
@skjnldsvskjnldsv mentioned this pull request Nov 1, 2023
This was referenced Nov 6, 2023
This was referenced Nov 14, 2023
@blizzzblizzz modified the milestones: Nextcloud 28, Nextcloud 29Nov 23, 2023
@skjnldsvskjnldsv added the technical debt 🧱 🤔🚀 label Feb 23, 2024
This was referenced Mar 12, 2024
This was referenced Mar 20, 2024
@skjnldsvskjnldsv mentioned this pull request Mar 28, 2024
81 tasks
@skjnldsv

Copy link
Copy Markdown
Member

Too much conflicts, closing until this is done fresh again

@skjnldsvskjnldsv removed this from the Nextcloud 30 milestone Aug 14, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewstechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@summersab@skjnldsv@blizzz@solracsf