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 'PlantUMLHTTPError' object has no attribute 'message' #9

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

Conversation

jean
Copy link

@jean jean commented Nov 13, 2019

Also, pass 'message' to super, so that it's displayed.
(I suspect this has to do with Python 3 changes?)

Also, pass 'message' to super, so that it's displayed.
(I suspect this has to do with Python 3 changes?)
`super` should get the current class as parameter
@jean
Copy link
Author

jean commented Dec 9, 2019

@dougn good thing you didn't notice this PR yet 😬
It had a bug. Should be better now.

@yuriikonovaliuk
Copy link

@jean It seems that the new version also has a potential issue. The message is generated regardless of whether there is one already. In case if there is one it will be ignored and the generated one will be passed into the supers __init__.

@anb0s
Copy link

anb0s commented Aug 3, 2021

I'm facing this errror via mkdocs if there are some syntax issues in the plantuml block. Is the PR somehow blocked?

@wonderchang
Copy link

I'm facing this errror via mkdocs if there are some syntax issues in the plantuml block. Is the PR somehow blocked?

+1

@fuhrmanator
Copy link

I'm trying to use PlantUML in colab.research.google.com and can't get error handling to work properly because of this. Hoping you can merge this.

@sharif509
Copy link

At times it appears if the input UML Code has syntax errors. Please double-check the input UML Code. @jean @wonderchang @fuhrmanator

@fuhrmanator
Copy link

@Shareef509 if you could see the rendered image (that's returned despite the syntax error) you could see exactly what the error in the plantuml syntax and fix it. "Double checking" is hard if you can't get the error information.

@sharif509
Copy link

I had the same experience, that's why shared if it helps anyone else. Thank you.

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.

6 participants