Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1483

PropertyMapStep returns Map<String,E> but puts non String keys in it!

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.2
    • Fix Version/s: 3.3.0
    • Component/s: process
    • Labels:
      None

      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?

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                dkuppitz Daniel Kuppitz
                Reporter:
                JPMoresmau JP Moresmau
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: