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

Publisher imple's isDocumentExists() check includes document source type and that cause issues. #1463

Closed
rmsamitha opened this issue Jul 24, 2017 · 0 comments · Fixed by wso2/carbon-apimgt#4338
Assignees
Labels
4.0.0 DEPRECATED Label; Use Milestone to track version Type/Bug
Milestone

Comments

@rmsamitha
Copy link
Member

There is an issue in API documentation update publisher impl. When trying to update the document source type it checks whether the API exists with the updating doc info. This brings an issue as the isDocumentExists function compares also the document source type (INLINE/URL/FILE). This returns false always as the document source type is changed now. So whenever we try to update a document source type, it throws an exception saying the document doesn't exist. isDocumentExist() method obviously doesn't need to check the document source type. Checking for other unique values is sufficient.

@rmsamitha rmsamitha added 4.0.0 DEPRECATED Label; Use Milestone to track version Type/Bug labels Jul 24, 2017
@rmsamitha rmsamitha added this to the 3.0.0-m5 milestone Jul 24, 2017
@rmsamitha rmsamitha self-assigned this Jul 24, 2017
@rmsamitha rmsamitha added the fixed label Aug 4, 2017
prasa7 pushed a commit to prasa7/product-apim that referenced this issue Jun 27, 2023
New shareable test report template for integration test result
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.0.0 DEPRECATED Label; Use Milestone to track version Type/Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant