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

Filamentmanager + äöüß in Filament-Name = Error #201

Closed
Setcover opened this issue Aug 14, 2019 · 2 comments · Fixed by #202
Closed

Filamentmanager + äöüß in Filament-Name = Error #201

Setcover opened this issue Aug 14, 2019 · 2 comments · Fixed by #202
Assignees
Labels

Comments

@Setcover
Copy link

TL;DR: Plugin has problems handling "ä ö ü ß" and similar symbols in a filament-name.

-------- Long version -----
i got: 😖 Error getting spools. Are you sure, you have installed the Filament Manager Plugin?

debug log: (i removed most spools from the log)

2019-08-14 18:32:22,721 - octoprint.plugins.telegram.listener - INFO - Got a command: '/filament' with parameter: 'spools' in chat *****
2019-08-14 18:32:22,727 - octoprint.plugins.telegram.TCMD - INFO - Parameter received for filament: spools
2019-08-14 18:32:27,174 - octoprint.plugins.telegram.TCMD - INFO - Spools: [{u'profile': {u'diameter': 1.72, u'material': u'PLA', u'vendor': u'Motedis', u'id': 3, u'density': 1.25}, u'temp_offset': 0, u'name': u'Gr\xfcn', u'weight': 330.0, u'used': 11.471861346984554, u'cost': 5.4, u'id': 6},{u'profile': {u'diameter': 1.74, u'material': u'PETG', u'vendor': u'Motedis', u'id': 6, u'density': 1.38}, u'temp_offset': 0, u'name': u'Wei\xdf', u'weight': 330.0, u'used': 0.0, u'cost': 5.4, u'id': 21}]
2019-08-14 18:32:27,188 - octoprint.plugins.telegram - DEBUG - Sending a message UPDATE: 😖 Error getting spools. Are you sure, you have installed the Filament Manager Plugin? chatID= *****

After Changing "Grün" to "Gruen" and "Weiß" zu "Weiss" --> No error, works as intended.

@rlogiacco
Copy link
Collaborator

Sadly it makes a lot of sense: the whole plugin has an issue with character encoding.

@rlogiacco rlogiacco added the bug label Aug 14, 2019
@Setcover
Copy link
Author

I think one could forgo a fix (for the time being) and instead have a note somewhere saying something like "Avoid äöüß and other special characters - Applys to connected plugins too."

IMHO: poeple who use a 3D-Printer and are willing to setup octopi + telegram-plugin, are also willing to live without special-characters.

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

Successfully merging a pull request may close this issue.

2 participants