Ticket #17902: 17902-plugin-details-link.diff

wp-admin/includes/class-wp-plugins-list-table.php

@@ -108 +108 @@
                                         unset( $recently_activated[$key] );
                         update_option( 'recently_activated', $recently_activated );
                 }
+                
+                $plugin_info = get_site_transient( 'update_plugins' );
 
                 foreach ( (array) $plugins['all'] as $plugin_file => $plugin_data ) {
+                        // Extra info if known. array_merge() ensures $plugin_data has precedence if keys collide.
+                        if ( isset( $plugin_info->response[ $plugin_file ] ) ) {
+                                $plugins['all'][$plugin_file] = $plugin_data = array_merge( (array)$plugin_info->response[ $plugin_file ], $plugin_data );
+                        } elseif ( isset( $plugin_info->no_update[ $plugin_file ] ) ) {
+                                $plugins['all'][$plugin_file] = $plugin_data = array_merge( (array)$plugin_info->no_update[ $plugin_file ], $plugin_data );
+                        }
+
                         // Filter into individual sections
                         if ( is_multisite() && ! $screen->in_admin( 'network' ) && is_network_only_plugin( $plugin_file ) && ! is_plugin_active( $plugin_file ) ) {
                                 // On the non-network screen, filter out network-only plugins as long as they're not individually activated
@@ -344 +353 @@
                 $actions = array(
                         'deactivate' => '',
                         'activate' => '',
+                        'details' => '',
                         'edit' => '',
                         'delete' => '',
                 );
@@ -392 +402 @@
                                         if ( ! is_multisite() && current_user_can('delete_plugins') )
                                                 $actions['delete'] = '<a href="' . wp_nonce_url('plugins.php?action=delete-selected&amp;checked[]=' . $plugin_file . '&amp;plugin_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s, 'bulk-plugins') . '" title="' . esc_attr__('Delete this plugin') . '" class="delete">' . __('Delete') . '</a>';
                                 } // end if $is_active
+                                
+                                // Extra data from the updates API, if available
+                                if ( isset( $plugin_data['slug'] ) ) {
+                                        $actions['details'] = '<a href="' . self_admin_url( 'plugin-install.php?tab=plugin-information&amp;plugin=' . $plugin_data['slug'] .
+                                                                '&amp;TB_iframe=true&amp;width=600&amp;height=550' ) . '" class="thickbox" title="' .
+                                                                esc_attr( sprintf( __( 'More information about %s' ), $plugin_data['Name'] ) ) . '">' . __( 'Details' ) . '</a>';
+                                }
+                                
                          } // end if $screen->in_admin( 'network' )
 
                         if ( ( ! is_multisite() || $screen->in_admin( 'network' ) ) && current_user_can('edit_plugins') && is_writable(WP_PLUGIN_DIR . '/' . $plugin_file) )

wp-includes/update.php

@@ -277 +277 @@
                         'plugins'      => json_encode( $to_send ),
                         'translations' => json_encode( $translations ),
                         'locale'       => json_encode( $locales ),
+                        'all'          => json_encode( true ),
                 ),
                 'user-agent' => 'WordPress/' . $wp_version . '; ' . get_bloginfo( 'url' )
         );
@@ -286 +287 @@
         }
 
         $url = $http_url = 'http://api.wordpress.org/plugins/update-check/1.1/';
+
         if ( $ssl = wp_http_supports( array( 'ssl' ) ) )
                 $url = set_url_scheme( $url, 'https' );
 
@@ -303 +305 @@
                 $plugin = (object) $plugin;
         }
         unset( $plugin );
+        foreach ( $response['no_update'] as &$plugin ) {
+                $plugin = (object) $plugin;
+        }
+        unset( $plugin );
 
         if ( is_array( $response ) ) {
                 $new_option->response = $response['plugins'];
                 $new_option->translations = $response['translations'];
+                // TODO: Perhaps better to store no_update in a separate transient with an expiry?
+                $new_option->no_update = $response['no_update'];
         } else {
                 $new_option->response = array();
                 $new_option->translations = array();
+                $new_option->no_update = array();
         }
 
         set_site_transient( 'update_plugins', $new_option );