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

Plugin install button doesn't fire in thickbox modal #649

Open
NateWr opened this issue Mar 20, 2017 · 4 comments
Open

Plugin install button doesn't fire in thickbox modal #649

NateWr opened this issue Mar 20, 2017 · 4 comments

Comments

@NateWr
Copy link
Contributor

NateWr commented Mar 20, 2017

On the TGMPA page to install recommended/required plugins, clicking on a plugin will open a thickbox modal with an install button. Clicking the install button doesn't seem to have any effect.

The similar modal from the core plugins screen does work. And I feel like this was working in the past so it may be a recent change introduced in core.

I was able to track down one symptom which may be the source of the issue. The /wp-admin/js/plugin-install.js file reacts to an thickbox:iframe:loaded event fired when the thickbox is loaded:

https://github.com/WordPress/WordPress/blob/master/wp-admin/js/plugin-install.js#L60

This event seems to be getting caught on the core plugin installation screens, but not on TGMPA. I just chucked a console.log('event fired'); in there to test.

So that seems to be an issue. But it leaves me a bit confused, because the button is a link with a proper href attribute that should work directly without requiring any JavaScript. 🤷‍♂️

NateWr added a commit to NateWr/totc-theme-setup that referenced this issue Mar 20, 2017
There seem to be issues with installing from the thickbox.
See TGMPA/TGM-Plugin-Activation#649
@jrfnl
Copy link
Contributor

jrfnl commented Mar 20, 2017

@NateWr Thanks for reporting this. If I remember correctly, I've seen issues with this before in the past, but those generally turned out to be caused by a theme/plugin.

  • What version of WP are you using ?
  • What theme are you using and which plugins are active ?
  • Have you / could you test whether this also occurs with one of the WP standard themes and all plugins deactivated ?

@NateWr
Copy link
Contributor Author

NateWr commented Mar 21, 2017

Hi @jrfnl,

WP 4.7.3. It's a theme I've developed. I'm happy to try with a standard theme. Do you have a standard base theme that I can use for testing? Or should I just whip up a child theme of twentyseventeen with the tgmpa integrated?

@jrfnl
Copy link
Contributor

jrfnl commented Mar 21, 2017

should I just whip up a child theme of twentyseventeen with the tgmpa integrated?

Either that or drop it in must-use plugins with a mini-setup.

@jrfnl
Copy link
Contributor

jrfnl commented Jul 2, 2017

OK, finally had some time to properly test this and I can consistently reproduce this issue on WP 4.6 and higher.

Also tested this on WP 4.5 and 4.4 and the install does seem to work there, though it doesn't use the TGMPA installation screen, but the WP native on, so the "Return to plugin install" page link at the end of the page take people to the WP plugin install page, not the TGMPA one. Different issue, but not the way it was intended either.

The issue may well be related to the Shiny updates introduction in WP 4.6 - #580. Needs in-depth investigation to see what can be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants