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

pods_group_add sharing a meta group with multiple pods returns a memory error #4385

Closed
wpstudio opened this issue Aug 2, 2017 · 6 comments
Closed
Assignees
Labels
Priority: Blocker Issue is causing a problem that is holding a release or is a major bug that needs to be fixed ASAP Type: Bug
Milestone

Comments

@wpstudio
Copy link
Contributor

wpstudio commented Aug 2, 2017

When attempting to share meta groups creating with pods_group_add() with multiple pods, a memory error (not enough memory) triggers a failure.

pods_group_add( array('books', 'other_books'), 'Book Meta Title', 'field1,field2');

You can see the actual issue during this live training event. https://youtu.be/fXJbXiTnqBk?t=43m34s

I even attempted to allocate over 2GB of memory and this array() sucked it all up. :) Greedy little bastard.

@sc0ttkclark
Copy link
Member

Well of course it would happen while you're doing it live! Thanks for the heads up, will check into it

@sc0ttkclark sc0ttkclark added the Type: Enhancement Enhancements to features that already exist, but are not major additions label Aug 2, 2017
@sc0ttkclark sc0ttkclark added this to the Pods 2.7 milestone Aug 2, 2017
@sc0ttkclark sc0ttkclark self-assigned this Aug 2, 2017
@sc0ttkclark sc0ttkclark added Priority: Blocker Issue is causing a problem that is holding a release or is a major bug that needs to be fixed ASAP Type: Bug and removed Type: Enhancement Enhancements to features that already exist, but are not major additions labels Aug 2, 2017
@sc0ttkclark
Copy link
Member

Found it:

https://github.com/pods-framework/pods/blob/2.x/classes/PodsMeta.php#L596

We should be using $p there, not $pod

@wpstudio
Copy link
Contributor Author

wpstudio commented Aug 2, 2017

Ha... maybe I should have just looked at the code myself :)

sc0ttkclark added a commit that referenced this issue Aug 2, 2017
…adding to multiple pods forever when adding multiple pods forever

In Pods, Pods recurse YOU in Pods, Pods recurse YOU in Pods

Fixes #4385 Fixes #4385 Fixes #4385 Fixes #4385 Fixes #4385
@sc0ttkclark
Copy link
Member

Fix is a one-liner via #4386

@sc0ttkclark
Copy link
Member

This has actually been around since August 5th, 2013 -- which coincidentally we're coming up on the 4 year anniversary for :)

@sc0ttkclark
Copy link
Member

The commit we can blame for this:
72803d1

That committer is a dummy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Blocker Issue is causing a problem that is holding a release or is a major bug that needs to be fixed ASAP Type: Bug
Projects
None yet
Development

No branches or pull requests

2 participants