Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(legacy): Introduce public version class and drop version methods from OC_Util #48152

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Sep 17, 2024

  • Introduce a ServerVersion class with properly typed methods for getting different version parts and strings
  • Remove and migrate any usages of OC_Util to the new API

Next step could be to deprecated related OCP\Util methods.

core/Controller/OCSController.php Fixed Show fixed Hide fixed
core/Controller/OCSController.php Fixed Show fixed Hide fixed
@juliusknorr juliusknorr force-pushed the clean/version-ocp branch 5 times, most recently from 6c038da to 2e206ca Compare September 18, 2024 13:06
@juliusknorr juliusknorr requested review from nickvergessen, skjnldsv, a team, icewind1991 and come-nc and removed request for a team September 18, 2024 18:22
lib/public/Util.php Show resolved Hide resolved
lib/public/Util.php Show resolved Hide resolved
lib/public/ServerVersion.php Show resolved Hide resolved
lib/public/ServerVersion.php Show resolved Hide resolved
lib/public/ServerVersion.php Show resolved Hide resolved
lib/public/ServerVersion.php Fixed Show fixed Hide fixed
lib/public/ServerVersion.php Fixed Show fixed Hide fixed
@juliusknorr juliusknorr force-pushed the clean/version-ocp branch 2 times, most recently from 880cd95 to 31f4e27 Compare September 19, 2024 18:27
@juliusknorr juliusknorr merged commit fd576ce into master Sep 20, 2024
174 checks passed
@juliusknorr juliusknorr deleted the clean/version-ocp branch September 20, 2024 16:14
@nickvergessen
Copy link
Member

Since this was defacto-standard, please document the new API and mention the removal in the docs

@nickvergessen nickvergessen added the pending documentation This pull request needs an associated documentation update label Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants