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

Empty postgres array loaded as slice with one item #788

Closed
jakeactually opened this issue Sep 10, 2020 · 4 comments · Fixed by #791
Closed

Empty postgres array loaded as slice with one item #788

jakeactually opened this issue Sep 10, 2020 · 4 comments · Fixed by #791
Assignees
Labels
bug Something isn't working s: fixed was fixed or solution offered
Milestone

Comments

@jakeactually
Copy link

the postgres database contains an empty int array, it looks like this

{}

but connection.Find(model, id) loads the model with a slice that looks like this

[0]

it should leave the slice empty

@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment. Otherwise, this will be closed in 7 days.

@sio4 sio4 transferred this issue from gobuffalo/buffalo-pop Sep 27, 2022
@sio4 sio4 added the s: triage Some tests need to be run to confirm the issue label Sep 27, 2022
@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment. Otherwise, this will be closed in 7 days.

@github-actions github-actions bot added the stale label Oct 28, 2022
@sio4 sio4 removed the stale label Nov 1, 2022
@sio4 sio4 linked a pull request Nov 1, 2022 that will close this issue
@sio4 sio4 added this to the v6.1.0 milestone Nov 1, 2022
@sio4 sio4 added bug Something isn't working and removed s: triage Some tests need to be run to confirm the issue labels Nov 1, 2022
@sio4 sio4 self-assigned this Nov 1, 2022
@sio4 sio4 closed this as completed in #791 Nov 1, 2022
@sio4
Copy link
Member

sio4 commented Nov 1, 2022

Thanks for reporting this issue. I think the issue is just fixed by #791. Could you please confirm it if you are still available to reproduce this issue?

@jakeactually
Copy link
Author

Thanks for reporting this issue. I think the issue is just fixed by #791. Could you please confirm it if you are still available to reproduce this issue?

Yes I think that would solve that. Thanks!

@sio4 sio4 added the s: fixed was fixed or solution offered label Nov 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working s: fixed was fixed or solution offered
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants