Skip to content

Refactor OC\Server::getLockingProvider - #40144

Merged
skjnldsv merged 1 commit into
nextcloud:masterfrom
summersab:refactor/OC-Server-getLockingProvider
Mar 15, 2024
Merged

Refactor OC\Server::getLockingProvider#40144
skjnldsv merged 1 commit into
nextcloud:masterfrom
summersab:refactor/OC-Server-getLockingProvider

Conversation

@summersab

Copy link
Copy Markdown
Contributor

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

Additionally, where necessary, the OCP\Lock\ILockingProvider class is imported via the use directive.

@summersab
summersabforce-pushed the refactor/OC-Server-getLockingProvider branch from 22f4cac to bd14931CompareAugust 30, 2023 02:31
@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
@skjnldsv
skjnldsv requested review from a team, ArtificialOwl, nfebe and sorbaugh and removed request for a teamMarch 15, 2024 12:11
Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
@skjnldsv
skjnldsvforce-pushed the refactor/OC-Server-getLockingProvider branch from bd14931 to 45eb4a8CompareMarch 15, 2024 12:11
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 15, 2024
@skjnldsv
skjnldsv merged commit 98ac8ac into nextcloud:masterMar 15, 2024
@AltahrimAltahrim mentioned this pull request Mar 18, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishtechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@summersab@nfebe@skjnldsv@blizzz@solracsf