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 generated schema name conflicts #4772

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DmitrySboychakov
Copy link

@DmitrySboychakov DmitrySboychakov commented Nov 4, 2024

this PR fixes two issues within swagger
in our project we have multiple member enums with the same name, for example:

class SomeObject{
    public enum Type {
        TypeValue1,
        TypeValue2;
    }
}

the commit TypeNameResolver to handle member class names correctly fixes the override for that enums and classes to generate SomeObjectType instead of Type

The second commit fixes overall schema name conflicts if there different classes with the same name exists in different packages by appending numeric postfix to the schema name

@DmitrySboychakov DmitrySboychakov changed the title Swagger updates Fix generated schema name conflicts Nov 4, 2024
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.

1 participant