Skip to content

Update RowDataPacket.js #1188

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update RowDataPacket.js #1188

wants to merge 2 commits into from

Conversation

ashwinks
Copy link

When using subqueries in select statements, there's no table which means the data would be omitted if using nestTables. This fix only appends data to table objects if there is a table name, otherwise it's appended to the root object.

…ans the data would be omitted if using nestTables. This fix only appends data to table objects if there is a table name, otherwise it's appended to the root object.
…he result object correctly. Changed table to concrete table instead of temporary table because we can't run subqueries on temporary tables.
@dougwilson dougwilson self-assigned this Oct 25, 2015
@dougwilson
Copy link
Member

Hi @ashwinks , that's for this PR! I'm looking over it, and it doesn't seem to be backwards-compatible, is that correct? It seems like we were storing the columns under the key '' before this patch, which it is likely people have probably written code to work-around after these years. I can certainly mark this PR for the 3.0 milestone, though.

@dougwilson dougwilson added the bug label Nov 30, 2015
@dougwilson dougwilson added this to the 3.0 milestone Nov 30, 2015
@dougwilson dougwilson force-pushed the master branch 4 times, most recently from 5847ec8 to be37e88 Compare May 8, 2016 05:09
@dougwilson dougwilson force-pushed the master branch 3 times, most recently from 65c4c0c to fa96a75 Compare June 8, 2016 02:00
@dougwilson dougwilson force-pushed the master branch 2 times, most recently from 638d79d to 88bade0 Compare June 29, 2017 18:13
@dougwilson dougwilson force-pushed the master branch 2 times, most recently from 946727b to 37fbbdd Compare November 18, 2018 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants