Make WordPress Core

Ticket #18974: 35083-18974.patch

File 35083-18974.patch, 834 bytes (added by khag7, 9 years ago)

ditch the id attribute in favor of the data-plugin attribute; also use esc_attr

  • class-wp-plugins-list-table.php

     
    695695                        $plugin_name = $plugin_data['Name'];
    696696                }
    697697
    698                 $id = sanitize_title( $plugin_name );
    699698                if ( ! empty( $totals['upgrade'] ) && ! empty( $plugin_data['update'] ) )
    700699                        $class .= ' update';
    701700
    702701                $plugin_slug = ( isset( $plugin_data['slug'] ) ) ? $plugin_data['slug'] : '';
    703                 printf( "<tr id='%s' class='%s' data-slug='%s'>",
    704                         $id,
    705                         $class,
    706                         $plugin_slug
     702                printf( "<tr class='%s' data-plugin='%s' data-slug='%s'>",
     703                        esc_attr( $class ),
     704                        esc_attr( $plugin_file ),
     705                        esc_attr( $plugin_slug )
    707706                );
    708707
    709708                list( $columns, $hidden, $sortable, $primary ) = $this->get_column_info();