Skip to content

Conversation

@Dressi
Copy link
Collaborator

@Dressi Dressi commented Dec 8, 2025

No description provided.

@Dressi Dressi requested a review from vsouz4 December 8, 2025 11:09
@Dressi Dressi force-pushed the fix-content-type-construction-parameter-issue-128 branch from 676f319 to 7259326 Compare December 8, 2025 11:49
class AddPetRequest implements RequestInterface
{
private Pet $pet;
public const CONTENT_TYPE_APPLICATION_JSON = 'application/json';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are these constants used anywhere?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not from the Client. I extended the example with ContentType header enum and test run re-generated this example.
As I see these constants consumers. From PHP 8.1 it's not generated, becasue the enum class is used.

@vsouz4 vsouz4 merged commit 468540a into master Dec 9, 2025
2 checks passed
@vsouz4 vsouz4 deleted the fix-content-type-construction-parameter-issue-128 branch December 9, 2025 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants