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

Missing parent table in SelectFields for UnionType and InterfaceType #979

Closed
illambo opened this issue Feb 7, 2023 · 3 comments
Closed

Comments

@illambo
Copy link
Contributor

illambo commented Feb 7, 2023

Versions:

  • graphql-laravel Version: 8.5.0
  • Laravel Version: 9.48.0
  • PHP Version: 8.1.14

Description:


Hi, it seems missing to specify the parent table, otherwise it can cause problems with joins (I have a local project on which I had the problem but I can not share more data at the moment).

Better use:
$select = [$parentTable . '.*'];

What do you think?

@illambo illambo added the bug label Feb 7, 2023
@mfn
Copy link
Collaborator

mfn commented Feb 7, 2023

🤷🏼

SelectFields is a bane, I've no idea. Did you try it? Write a test with your situation, change the code, all tests still green?

@illambo
Copy link
Contributor Author

illambo commented Feb 8, 2023

Ok, as soon as I can I elaborate tests and pr.

@illambo
Copy link
Contributor Author

illambo commented Dec 19, 2023

I'm sorry, I only realized now that nothing has been done about it. I'm going to close the issue for now and if necessary I'll reopen it with a complete example. Thank you

@illambo illambo closed this as not planned Won't fix, can't repro, duplicate, stale Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants