stray closing tags printed by wp_plugin_update_row if current_user_can('update_plugins' ) and !is_network_admin() and is_multisite()

[mitchoyoshitaka]

Seems a little contrived, but if current_user_can('update_plugins' ) and !is_network_admin() and is_multisite(), wp_plugin_update_row may be triggered for a plugin.

wp_plugin_update_row will then skip the block which prints <tr><td><div> and the upgrade message, but will still print the closing </div></td></tr> after calling the in_plugin_update_message-$file hook. Note this hook was intended (#9553) to be fired only when there actually is an upgrade notice being printed, so it can be safely moved into the conditional.

[nacin]

Replying to mitchoyoshitaka:

Note this hook was intended (#9553) to be fired only when there actually is an upgrade notice being printed, so it can be safely moved into the conditional.

Based on my review, I agree.

[nacin]

In [17984]:

Move the closing tags inside the same conditional in wp_plugin_update_row. props mitchoyoshitaka, fixes #17536.