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

fix: Move OC_API into \OC\ApiHelper in standard namespace #47685

Merged
merged 8 commits into from
Sep 12, 2024

Conversation

come-nc
Copy link
Contributor

@come-nc come-nc commented Sep 2, 2024

Summary

It’s only used by ocs/v1.php

Checklist

@come-nc come-nc added the 2. developing Work in progress label Sep 2, 2024
@come-nc come-nc added this to the Nextcloud 31 milestone Sep 2, 2024
@come-nc come-nc self-assigned this Sep 2, 2024
@come-nc

This comment was marked as outdated.

lib/private/OCS/ApiHelper.php Dismissed Show dismissed Hide dismissed
lib/private/OCS/ApiHelper.php Dismissed Show dismissed Hide dismissed
@come-nc

This comment was marked as outdated.

@come-nc come-nc force-pushed the fix/move-apihelper-to-oc-namespace branch from e59e773 to a9e1971 Compare September 5, 2024 15:39
@come-nc come-nc marked this pull request as ready for review September 5, 2024 16:08
@come-nc come-nc added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 5, 2024
@come-nc come-nc requested review from nickvergessen, a team, Altahrim, artonge and Fenn-CS and removed request for a team September 5, 2024 16:08
@come-nc come-nc added technical debt pending documentation This pull request needs an associated documentation update labels Sep 5, 2024
It’s only used by ocs/v1.php

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…s instead

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc force-pushed the fix/move-apihelper-to-oc-namespace branch from df9f4e8 to 90a9485 Compare September 9, 2024 08:46
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<file src="lib/private/legacy/OC_API.php">
<InvalidNullableReturnType>
<code><![CDATA[int]]></code>
</InvalidNullableReturnType>
<NullableReturnStatement>
<code><![CDATA[null]]></code>
</NullableReturnStatement>
</file>
will be removed automatically?

@come-nc
Copy link
Contributor Author

come-nc commented Sep 12, 2024

<file src="lib/private/legacy/OC_API.php">
<InvalidNullableReturnType>
<code><![CDATA[int]]></code>
</InvalidNullableReturnType>
<NullableReturnStatement>
<code><![CDATA[null]]></code>
</NullableReturnStatement>
</file>

will be removed automatically?

Yes there’s a job for updating the baseline.

@come-nc come-nc merged commit b6c2866 into master Sep 12, 2024
174 checks passed
@come-nc come-nc deleted the fix/move-apihelper-to-oc-namespace branch September 12, 2024 12:34
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.

3 participants