Skip to content

TINKERPOP-2383 Fixed bug in has(T,Traversal) - #1295

Merged
spmallette merged 1 commit into
3.4-devfrom
TINKERPOP-2383
Jun 19, 2020
Merged

TINKERPOP-2383 Fixed bug in has(T,Traversal)#1295
spmallette merged 1 commit into
3.4-devfrom
TINKERPOP-2383

Conversation

@spmallette

@spmallette spmallette commented Jun 12, 2020

Copy link
Copy Markdown
Contributor

https://issues.apache.org/jira/browse/TINKERPOP-2383

That has() was using PropertiesStep, but properties() doesn't process T values.

gremlin> g.V().has(label, is('software'))
==>v[3]
==>v[5]
gremlin> g.V().has(id, is(6))
==>v[6]

All tests pass with docker/build.sh -t -n -i

VOTE +1

@spmallette
TINKERPOP-2383 Fixed bug in has(T,Traversal)
ba44f73 
That has() was using PropertiesStep, but properties() doesn't process T values.
@divijvaidya

divijvaidya commented Jun 12, 2020

Copy link
Copy Markdown
Member

VOTE +1 LGTM!

@spmallette
spmallette merged commit bd4bd43 into 3.4-dev Jun 19, 2020
@spmallette
spmallette deleted the TINKERPOP-2383 branch June 19, 2020 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@spmallette @divijvaidya