search

thatdot / quine

Quine • a streaming graph • https://quine.io • Discord: https://discord.gg/GMhd8TE4MR
https://quine.io
Other
295 stars 39 forks source link

Don't include labels twice in literal operation #5

Closed harpocrates closed 2 years ago

harpocrates commented 2 years ago

The LiteralMessage.GetRawPropertiesCommand includes labels twice: once in the labels field and a second time as an entry in the properties map. This changes that behaviour to omit the label from the properties.

This fixes #4 since UserDefinedProcedure.getAsCypherNode (which is used by recentNodes and other procedures) now doesn't include the label as a property.

emanb29 commented 2 years ago

Thanks for the contribution! Looks good: I will merge this when CI passes.