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

Name conflict with FilteringTerm object definitions #156

Open
datsirul opened this issue Jul 25, 2024 · 0 comments
Open

Name conflict with FilteringTerm object definitions #156

datsirul opened this issue Jul 25, 2024 · 0 comments

Comments

@datsirul
Copy link

We have encountered an issue with the OpenAPI specifications related to the definition of the FilteringTerm objects.

Problem Details:

The issue arises because the FilteringTerm object is defined in two different locations within the OpenAPI documents, which is causing name conflicts during code generation. Each definition is intended to represent a different object, but they share the same name, as shown below:

framework/src/requests/filteringTerms.yaml

definitions:
  FilteringTerm:
    anyOf:
      - $ref: '#/definitions/OntologyFilter'
      - $ref: '#/definitions/AlphanumericFilter'
      - $ref: '#/definitions/CustomFilter'  

framework/src/responses/sections/beaconFilteringTermsResults.yaml

definitions:
  FilteringTerm:
    type: object
    ...

Due to this duplicate naming, when generating code from the OpenAPI documents (e.g., in Python), the models are generated with conflicting names such as filtering_term.py and filtering_term1.py. This naming conflict can lead to confusion and errors in the generated code, making it difficult to maintain and use.

Possible Solution:

A possible solution to this issue would be to rename one or both of the FilteringTerm definitions to ensure unique and descriptive names that reflect their distinct purposes.

For example, the FilteringTerm definition in beaconFilteringTermsResults.yaml could be renamed to ResponseFilteringTerm while the FilteringTerm definition in filteringTerms.yaml could keep it's short name due to it's frequent use in user requests.

Note:

I understand the current move away from "OpenAPI first" (#40), but I believe this is important as even generated OpenAPI docs may contains this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants