[TINKERPOP-1483] PropertyMapStep returns Map<String,E> but puts non String keys in it!

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.2.2
• Fix Version/s: 3.3.0
• Component/s: process
• Assignee: Daniel Kuppitz
• Reporter: JP Moresmau
• Created: 29/Sep/16 19:21
• Updated: 30/Nov/16 16:55
• Resolved: 30/Nov/16 16:55

Description

PropertyMapStep.map has return type Map<String,E>, but if includeTokens is true:

if (element instanceof VertexProperty) {
                    map.put(T.id, element.id());
                    map.put(T.key, ((VertexProperty) element).key());
                    map.put(T.value, ((VertexProperty) element).value());
                } else {
                    map.put(T.id, element.id());
                    map.put(T.label, element.label());
                }

T.id, T.key and T.value are NOT strings, so code looping through the keys in Java fails. toString() are missing... But do we rely on having these keys in other operations?

Issue Links

• links to: GitHub Pull Request #446