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

Sooo... #13

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

Conversation

JustinSainton
Copy link

not removing the the_post action sort of stomps on any other post on a page....essentially causing them to not show up at all.

not removing the `the_post` action sort of stomps on any other post on a page....essentially causing them to not show up at all.
@jmslbam
Copy link
Collaborator

jmslbam commented Mar 2, 2015

I'm not having this problem here, could you test it with the latest master?
That filter is quite essential because without it the loop won't work and because of that it won't be remove from the code. I think the problem you got here it originated somewhere else.

P.s. If you are going to remove the filter, the you should also remove the function that was being called.

@JustinSainton
Copy link
Author

Hi @jmslbam,

Yeah, it may have been resolved in some of the more recent commits, I'll give them a go. That being said, the proposed PR only removes the action that was added for the query already, following the pattern of the other two callbacks added, and works as tested.

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.

2 participants