Remove usage of werkzeug.urls.url_encode

[matrss]

Replaces it with urllib.parse.urlencode. Fixes #1910.

[matrss]

I don't get why this "AttributeError: module 'PyQt5.QtCore' has no attribute 'Slot'" is happening here, but not on develop. I did not change anything remotely related to that and PyQt5.QtCore simply does not have this attribute. See: https://stackoverflow.com/questions/50459672/attributeerror-module-pyqt5-qtcore-has-no-attribute-slot

Both CI runs report the same pyqt version though...

[matrss]

OK, so it looks like the Slot attribute must be coming from

lib/python3.10/site-packages/matplotlib/backends/qt_compat.py
94:        QtCore.Slot = QtCore.pyqtSlot
106:        QtCore.Slot = QtCore.pyqtSlot

in the CI run on develop. I am not sure why that is used in develop but not in this PR pipeline though.

Anyway, I think we should change all occurrences of QtCore.Slot to QtCore.pyqtSlot.

[matrss]

>>> werkzeug.urls.url_encode({"test": None})
''
>>> urllib.parse.urlencode({"test": None})
'test=None'

That's the difference causing our tests to fail.

[ReimarBauer]

see comment.

related to #1614

[ReimarBauer]

>>> werkzeug.urls.url_encode({"test": None})
''
>>> urllib.parse.urlencode({"test": None})
'test=None'

That's the difference causing our tests to fail.

Good catch !

[ReimarBauer]

Thx

[ReimarBauer]

@joernu76 when this is merged we need to patch our server