Skip to content

Fix 'PlantUMLHTTPError' object has no attribute 'message' - #9

Open
jean wants to merge 2 commits into
dougn:masterfrom
jean:master
Open

Fix 'PlantUMLHTTPError' object has no attribute 'message'#9
jean wants to merge 2 commits into
dougn:masterfrom
jean:master

Conversation

@jean

@jean jean commented Nov 13, 2019

Copy link
Copy Markdown

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

jean added 2 commits November 13, 2019 11:44
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

jean commented Dec 9, 2019

Copy link
Copy Markdown
Author

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

@yuriikonovaliuk

Copy link
Copy Markdown

@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

anb0s commented Aug 3, 2021

Copy link
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

@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
Copy Markdown

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