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

httpRequest Plugin encodiert Daten #336

Closed
grinsekatze003 opened this issue Dec 18, 2017 · 7 comments
Closed

httpRequest Plugin encodiert Daten #336

grinsekatze003 opened this issue Dec 18, 2017 · 7 comments

Comments

@grinsekatze003
Copy link

grinsekatze003 commented Dec 18, 2017

Das Plugin httpRequest encodiert alle Daten im Array data. Das Problem dabei ist, dass das Array zuvor nicht kopiert wird, sodass die Originaldaten im Array verändert werden. Alle danach laufenden Plugins bekommen dann beispielsweise nicht mehr die originale ZVEI Beschreibung, sondern die encodierte.

Vielleicht könnte man das, auch wenn die Arbeiten an Boswatch 3 im vollen Gange sind, kurz zwischendurch fixen! :-)

@Schrolli91
Copy link
Owner

Schrolli91 commented Dec 18, 2017

Oh, das ist natürlich ganz schlecht und darf so auf keinen Fall sein, da hast du Recht!

Hättest du die Möglichkeit das zu machen?

(siehe: c77f2a9)

@Schrolli91
Copy link
Owner

@Schrolli91
Copy link
Owner

Bitte mal kurz testen #337

@grinsekatze003
Copy link
Author

grinsekatze003 commented Dec 19, 2017

Das kann nicht funktionieren, da dataCopy = data nur die Referenz speichert, also eine flache Kopie. dataCopy = dict(data) oder auch dataCopy = data.copy() sollte funktionieren! :-) Ich teste das und melde mich heute Nachmittag.

@Schrolli91
Copy link
Owner

Schrolli91 commented Dec 19, 2017

Du hast natürlich Recht

data = {"test":1,"abc":2}
dataCopy1 = data
dataCopy2 = dict(data)

print(data)
print(dataCopy1)
print(hex(id(data)))
print(hex(id(dataCopy1)))
print(hex(id(dataCopy2)))
{'test': 1, 'abc': 2}
{'test': 1, 'abc': 2}
0x1f3f0217af8
0x1f3f0217af8
0x1f3f06a9288

passe ich noch an

EDIT: Anpassung ist durch, würdest du nochmal kurz drüber schauen?

@Schrolli91 Schrolli91 self-assigned this Dec 19, 2017
@grinsekatze003
Copy link
Author

Funktioniert! Vielen Dank für den schnellen Fix. 👍

@Schrolli91
Copy link
Owner

Super - Danke für deine mithilfe

@ghost ghost removed the In Progress label Dec 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants