Skip to content

Refactor OC\Server::getL10NFactory - #40141

Merged
skjnldsv merged 4 commits into
nextcloud:masterfrom
summersab:refactor/OC-Server-getL10NFactory
May 30, 2024
Merged

Refactor OC\Server::getL10NFactory#40141
skjnldsv merged 4 commits into
nextcloud:masterfrom
summersab:refactor/OC-Server-getL10NFactory

Conversation

@summersab

Copy link
Copy Markdown
Contributor

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

Additionally, where necessary, the \OCP\L10N\IFactory 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-getL10NFactory branch from b2b6d3c to f80282aCompareAugust 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
@Altahrim

Copy link
Copy Markdown
Collaborator

@summersab Can you fix the conflicts please?

This was referenced Mar 20, 2024
@skjnldsvskjnldsv mentioned this pull request Mar 28, 2024
81 tasks
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
@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 May 30, 2024
@skjnldsv
skjnldsv enabled auto-merge May 30, 2024 12:20
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
@skjnldsv
skjnldsv disabled auto-merge May 30, 2024 15:51
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
@skjnldsv
skjnldsv merged commit 5866e49 into nextcloud:masterMay 30, 2024
@blizzzblizzz mentioned this pull request Jul 24, 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.

6 participants

@summersab@Altahrim@skjnldsv@come-nc@blizzz@solracsf