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

Remove qtranxf_postsFilter param reference #545

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

Conversation

mikaelz
Copy link

@mikaelz mikaelz commented Sep 19, 2017

Should fix #544

@k9t9
Copy link

k9t9 commented Jan 4, 2018

yes, that should fix it.
Annoying problem...
there is another one on line line 597 .

@d-packs
Copy link

d-packs commented Feb 9, 2018

The fix is 5 months old. is the project no longer maintained?

@k9t9
Copy link

k9t9 commented Feb 10, 2018

@d-packs
Not sure .
I might be wrong but the last commit seems like 2 years ago ..

@d-packs
Copy link

d-packs commented Feb 10, 2018

Yeah I realised only after I wrote that my go-to translation plugin is no longer maintained for a long time now.
Even if I manage to make the site work there is no guarantee the site won't break for the customer in the next year.

I already uninstalled qtranslate and moved to polylang.

I guess this is what happens when a plugin gets a lot of traction (hence a lot of need for support) but there is no compensation for the author. Both Polylang and Wpml rely on their premium up-sale or are outright commercial, so this ensures a highly motivated team behind them.

@grihenrik
Copy link

Thanks a lot guys, this saved my day. I got translations working again!

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.

Parameter 2 to qtranxf_postsFilter() expected to be a reference
4 participants