Skip to content

fix: rename ACommonsColumnsController to fix openapi attraction#2323

Merged
enjeck merged 4 commits intomainfrom
jtr/fix-openapi-extraction-ACommonColumnsController
Mar 3, 2026
Merged

fix: rename ACommonsColumnsController to fix openapi attraction#2323
enjeck merged 4 commits intomainfrom
jtr/fix-openapi-extraction-ACommonColumnsController

Conversation

@joshtrichards
Copy link
Member

The new parent class introduced in #2211 broke OpenAPI extraction. The extractor requires a OCSController suffix on parent classes:

https://github.com/nextcloud/openapi-extractor/blob/772b90e1dd1c97989f2aed6abbaa0a513536dccb/generate-spec.php#L394-L401

Fixes warnings like this from the openapi workflow:

Warning: ApiColumns#index: You are extending a custom controller class. Make sure that it ends with "OCSController" if it extends "OCSController" itself.

@joshtrichards joshtrichards added 3. to review Waiting for reviews feature: api documentation regression Regression of a previous working feature labels Feb 12, 2026
@blizzz
Copy link
Member

blizzz commented Feb 12, 2026

Wow, thanks for spotting!

I suppose it needs a run of composer openapi and committing the openapi bits.

Required for openapi-extractor

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
@enjeck enjeck force-pushed the jtr/fix-openapi-extraction-ACommonColumnsController branch from 2492ac2 to 8fa0e8a Compare March 3, 2026 05:27
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
@enjeck enjeck merged commit 38452f3 into main Mar 3, 2026
61 of 62 checks passed
@enjeck enjeck deleted the jtr/fix-openapi-extraction-ACommonColumnsController branch March 3, 2026 06:56
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 documentation feature: api regression Regression of a previous working feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants