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

add alias to subquery in doCalcRowCount() #233

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

garak
Copy link
Contributor

@garak garak commented Jul 1, 2015

Without such alias, I get this error: "SQLSTATE[42000]: Syntax error or access violation: 1248 Every derived table must have its own alias"

Without such alias, I get this error: "SQLSTATE[42000]: Syntax error or access violation: 1248 Every derived table must have its own alias"
@Baachi
Copy link
Collaborator

Baachi commented Jul 2, 2015

Hey Massimiliano, thank you for your contribution.
Can you submit the PR to the new repo https://github.com/portphp/dbal-adapter?

@garak
Copy link
Contributor Author

garak commented Jul 2, 2015

Of course I can, but what about this repo?
I'm using this in some projects of mine and I'd love to see this PR merged.
Is this repo still supported? Is there a migration path from this to portphp?

@Baachi
Copy link
Collaborator

Baachi commented Jul 2, 2015

We just moved the repo to an organization, but kept this repo to don't break any project.
So the new repo is the same as this, but with an new name.

@garak
Copy link
Contributor Author

garak commented Jul 3, 2015

@Baachi it looks to me that this is not "just moved". I mean, where is the new "data-import"? Also, I can't find any info in this repo's README about moving, nor I can find docs on the new one.

@ptorti
Copy link

ptorti commented Sep 17, 2015

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants