Skip to content
This repository was archived by the owner on Oct 27, 2022. It is now read-only.

Use a valid filter() for userId Datastore query#70

Open
dbs wants to merge 1 commit into
googlesamples:masterfrom
dbs:fix_69_two_arg_filter
Open

Use a valid filter() for userId Datastore query#70
dbs wants to merge 1 commit into
googlesamples:masterfrom
dbs:fix_69_two_arg_filter

Conversation

@dbs

@dbs dbs commented Mar 22, 2016

Copy link
Copy Markdown

The code for the solution for the Datastore query that is supposed to return
only a specific user's books fails to return any books. The problem is that
the comparison operator and property key were included in a single string.
Break the values out into separate strings so that the filter() method has
a property, comparison operator, and value as expected.

Closes #69.

Signed-off-by: Dan Scott denials@gmail.com

The code for the solution for the Datastore query that is supposed to return
only a specific user's books fails to return any books. The problem is that
the comparison operator and property key were included in a single string.
Break the values out into separate strings so that the filter() method has
a property, comparison operator, and value as expected.

Closes googlesamples#69.

Signed-off-by: Dan Scott <denials@gmail.com>
@googlebot

Copy link
Copy Markdown

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@dbs dbs changed the title Use a 3-arg filter() for a Datastore query Use a valid filter() for userId Datastore query Mar 22, 2016
@dbs

dbs commented Mar 22, 2016

Copy link
Copy Markdown
Author

I'm Dan Scott, and I confirm that I did author this commit.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants