chore: refactor datastore samples

[bshaffer]

You (probably): Why are you doing this?
Me: because samples should be able to be executed on the command line (e.g. php datastore/src/delete.php 'key-id-to-delete')
You: Then why do some of the sample functions still contain objects (e.g. datastore/src/run_query.php)?
Me: Well, I didn't really know how to make those accept scalar arguments, and I didn't think it was THAT important of a rule to follow
You: Why did you change the order of arguments around in some cases?
Me: Well, because I'd like to put the most important arguments first, and the $namespaceId (which we needed to add in order to get rid of passing in the whole DatastoreClient object) is not really that interesting, and basically only used in testing, so I wanted it to be last.
You (hopefully): Hey that makes a lot of sense, great job!
Me: Thanks a lot, you too!

/scene

[snippet-bot]

Here is the summary of changes.

You are about to delete 1 region tag.

• datastore/api/src/inequality_invalid.php:31, tag datastore_inequality_invalid

---

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

• Refresh this comment