Skip to content
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

ADORecordSet contructor takes only one parameter #278

Closed
wants to merge 1 commit into from

Conversation

mezaof
Copy link
Contributor

@mezaof mezaof commented Sep 20, 2016

ADORecordSet class contructor only takes $queryID parameter, but ADORecordSet_ado class sends two paremeter on constructor.

@dregad
Copy link
Member

dregad commented Sep 20, 2016

There are several occurences of this behavior; I'd suggest you fix all of them while you're at it.

@dregad dregad closed this in a552585 Nov 27, 2017
dregad added a commit that referenced this pull request Nov 27, 2017
@dregad dregad added this to the v5.21 milestone Nov 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants