Skip to content

Identifiers array #608

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 14, 2013
Merged

Identifiers array #608

merged 2 commits into from
Oct 14, 2013

Conversation

Loddan
Copy link
Contributor

@Loddan Loddan commented Oct 6, 2013

SqlString.escapeId turns arrays to a list of identifiers as discussed in issue #524.

var userId = 1;
var columns = ['username', 'email'];
var query = connection.query('SELECT ?? FROM ?? WHERE id = ?', [columns, 'users', userId], function(err, results) {
  // ...
});

console.log(query.sql); // SELECT `username`, `email` FROM `users` WHERE id = 1

dresende added a commit that referenced this pull request Oct 14, 2013
@dresende dresende merged commit d75ff3c into mysqljs:master Oct 14, 2013
@Loddan Loddan deleted the identifiers-array branch November 1, 2013 11:17
dveeden pushed a commit to dveeden/mysql that referenced this pull request Jan 31, 2023
* Add supports to context.Context

* add authors related context.Context support

* fix comment of mysqlContext.

- s/from/for/
- and start the comment with a space please

* closed is now just bool flag.

* drop read-only transactions support

* remove unused methods from mysqlContext interface.

* moved checking canceled logic into method of connection.

* add a section about context.Context to the README.

* use atomic variable for closed.

* short circuit for context.Background()

* fix illegal watching state

* set rows.finish directly.

* move namedValueToValue to utils_go18.go

* move static interface implementation checks to the top of the file

* add the new section about `context.Context` to the table of contents, and fix the section.

* mark unsupported features with TODO comments

* rename watcher to starter.

* use mc.error() instead of duplicated logics.
dveeden pushed a commit to dveeden/mysql that referenced this pull request Jan 31, 2023
* Add atomic wrappers for bool and error

Improves mysqljs#608

* Drop Go 1.2 and Go 1.3 support

* "test" noCopy.Lock()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants