intuitive-custom-post-order
intuitive-custom-post-order copied to clipboard
Update intuitive-custom-post-order.php
Using the Yoast plugin with Intuitive CPO I noticed the Yoast-snippets wouldn't update anymore in my multisite environment. I fixed this by replacing line 741 to restore_current_blog(); as described here: https://codex.wordpress.org/Function_Reference/switch_to_blog
I also added an extra check (is_plugin_active_for_network('intuitive-custom-post-order/intuitive-custom-post-order.php') ) because apparently there is no use calling the multisite actions if the plugin isn't network activated.
This needs to be addressed. Also causes issues with Broadcast.
restore_current_blog() needs to be used when switching please.
Had to deactivate plugin.
This issue also causes problems with MultilingualPress. I’d appreciate it very much if this could be addressed soon.
@hijiriworld Looks good so far. Please merge.
Meanwhile... the patch. https://gist.github.com/mircobabini/99abd60a15212ecc5d47255b492f6790
@hijiriworld Could you please merge this pull request?
Any word on why this hasn't been merged? This fix would really come in handy.
Please merge this. It's really anoying not to have Yoast SEO working.
Any word on this? I can still reproduce the issue.
I can confirm this works and fixes the Yoast issue. +1 to merge
This also effects the ability to set a primary category with Yoast. Rolling back to v3.0.8 removes the issue.
Just sent an email to @hijiriworld from his website.
Finder crossed 🤞
I confirm that this fix is important and needs to be done to prevent conflict with the Yoast SEO plugin. I have done it manually in our project but still hope for merging pull-request
I've just spent too much time trying to figure it out. ICPO + Multisite = Not Yoast SEO Friend. Please make the change.
@emilio-pro does the problem still exists? If yes it would be great if you can update the PR ✌️
@timohubois I don't know. I haven't used the plugin for a couple of years now. You code has changed a lot since then, so I wouldn't know. But it is pretty safe condition to add, so you could test it.
@emilio-pro I think it should be fixed at the current master branch. Can you give them a try?