Skip to content

Refactor OC\Server::getLazyRootFolder - #40142

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

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

Conversation

@summersab

Copy link
Copy Markdown
Contributor

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

Additionally, where necessary, the OCP\Files\IRootFolder class is imported via the use directive.

@summersab
summersabforce-pushed the refactor/OC-Server-getLazyRootFolder branch from 300a07a to 4e189b7CompareAugust 30, 2023 02:30
@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, icewind1991 and nfebe 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-getLazyRootFolder branch from 4e189b7 to fe5eeadCompareMarch 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 f29fc47 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