Skip to content

[stable21] Replace OCSController with OCP\API - #27346

Merged
skjnldsv merged 1 commit into
stable21from
debt/noid/missing-import
Jun 2, 2021
Merged

[stable21] Replace OCSController with OCP\API#27346
skjnldsv merged 1 commit into
stable21from
debt/noid/missing-import

Conversation

@kesselb

@kesselbkesselb commented Jun 2, 2021

Copy link
Copy Markdown
Contributor

Psalm is unhappy about the missing import for OCSController. OCSController on stable21 does not have a constant RESPOND_NOT_FOUND. OCP\API seems the right one.

@kesselbkesselb added bug 3. to review Waiting for reviews labels Jun 2, 2021
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselbforce-pushed the debt/noid/missing-import branch from c61492d to 5740a6bCompareJune 2, 2021 13:18
@kesselbkesselb changed the title Add import for OCSControllerReplace OCSController with OCP\APIJun 2, 2021
@kesselbkesselb added this to the Nextcloud 21.0.3 milestone Jun 2, 2021
@kesselbkesselb changed the title Replace OCSController with OCP\API[stable21] Replace OCSController with OCP\APIJun 2, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@kesselb@LukasReschke@skjnldsv