[TINKERPOP-2332] JavaScript GLV: structure element toString() should internally call toString()

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.3.9, 3.4.4
• Fix Version/s: 3.4.5, 3.3.10
• Component/s: javascript
• Assignee: Jorge Bay
• Reporter: Jorge Bay
• Created: 24/Jan/20 15:02
• Updated: 30/Jan/20 10:20
• Resolved: 30/Jan/20 10:20

Description

Structure classes use String instance methods on properties that might not be strings.

For example, current implementation:

class Property {
  // ...

  toString() {
    return 'p[' + this.key + '->' + this.value.substr(0, 20) + ']';
  }
}

Should be:

class Property {
  // ...

  toString() {
    return 'p[' + this.key + '->' + this.value.toString().substr(0, 20) + ']';
  }
}

Plus checking on null/undefined value.

Issue Links

• links to: GitHub Pull Request #1242