Conflict with BNS Inline Asides in Single Views

Thanks for the report.

To help us narrowing down the problem, could you let me know if you still experience the issue when deactivating the Sharing module and the Publicize module?

You can find out how to deactivate Jetpack modules here:
https://jetpack.me/support/#turn-off-jetpack-modules

Thanks for the response!

Deactivating both Publicize and Sharing modules appears to sort out the conflict; also to note, if only one of the two modules are deactivated the conflict will return.

I currently use the following conditional:
if ( ( is_single() || is_page() ) && Jetpack::is_active() ) { ... }

… is there a more specific conditional I can use to test if both of these modules are active in its place?

Thanks for the confirmation. We’ll have a look at your plugin and see if there is a better way to address the conflict!

@edward You might want to have a look at this forum thread, since the problem seems similar to yours:
https://www.ads-software.com/support/topic/jetpack-204-upgrade-breaks-several-plugins?replies=9#post-3675712
https://lists.automattic.com/pipermail/wp-hackers/2013-January/045053.html

@jeremy – Thanks for the references. I can see how there may be similarities with these conflicts; and, I can tell from reviewing the page source the JavaScript in the plugin is being stripped out when the Publicize and/or the Sharing modules are activated, which appears to be the root of the conflict.

Although I have not tested some of the suggestions from your references I also do not see reducing (or eliminating) functionality from one plugin to serve another as the best method to go forward long-term.

I am willing to accept they may be an alternative to writing the dynamic JavaScript I am using, but since the plugin’s initial release I have not been able to successfully refactor the code to work any other way.

Any other suggestions?

As I mentioned earlier, I took note of the conflict with your plugin, and we will have a look at it and see if we can help.

Of course, you shouldn’t have to reduce or eliminate functionality from your plugin; in the links I posted earlier, the plugin authors made a change to their plugin, solved the conflict, without removing any features.

Thank you for the offer of assistance, and I would still appreciate more eyes on the plugin, but I also believe I have found a solution.

It required I sort out a couple of points:

1. How to use wp_localize_script to pass a dynamic variable to the script
2. Putting the JavaScript into a separate file (best practice, but point 1 had to be sorted out first)
3. Enqueue the script so it plays nice with everything else

I will be making the commits shortly to Extend (it’s already on GitHub).

I wanted to let you know that we found on a work-around tolsve all these plugin conflicts. Could you try to update to Jetpack 2.2?

Thank you.

Thanks for the update. The “fix” I added last month also appears to have had a beneficial impact.