These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | /** |
||
3 | * Uninstall page for Email Log Plugin to clean up db. |
||
4 | * |
||
5 | * This file is named uninstall.php since WordPress requires that name. |
||
6 | */ |
||
7 | |||
8 | // exit if WordPress is not uninstalling the plugin. |
||
9 | if ( ! defined( 'ABSPATH' ) && ! defined( 'WP_UNINSTALL_PLUGIN' ) ) { |
||
10 | exit(); |
||
11 | } |
||
12 | |||
13 | if ( is_multisite() ) { |
||
14 | // Note: if there are more than 10,000 blogs or |
||
15 | // if `wp_is_large_network` filter is set, then this may fail. |
||
16 | $sites = wp_get_sites(); |
||
17 | |||
18 | foreach ( $sites as $site ) { |
||
19 | switch_to_blog( $site['blog_id'] ); |
||
20 | email_log_delete_table(); |
||
21 | restore_current_blog(); |
||
22 | } |
||
23 | } else { |
||
24 | email_log_delete_table(); |
||
25 | } |
||
26 | |||
27 | /** |
||
28 | * Delete email log table and db option |
||
29 | * |
||
30 | * @since 1.7 |
||
31 | * |
||
32 | * @global object $wpdb |
||
33 | */ |
||
34 | function email_log_delete_table() { |
||
35 | global $wpdb; |
||
36 | |||
37 | // This is hardcoded on purpose, since the entire plugin is not loaded during uninstall. |
||
38 | $table_name = $wpdb->prefix . 'email_log'; |
||
39 | |||
40 | if ( $wpdb->get_var( "SHOW TABLES LIKE '{$table_name}'" ) == $table_name ) { |
||
1 ignored issue
–
show
|
|||
41 | // If table is present, drop it |
||
42 | $wpdb->query( "DROP TABLE $table_name" ); |
||
1 ignored issue
–
show
As per coding-style, please use concatenation or
sprintf for the variable $table_name instead of interpolation.
It is generally a best practice as it is often more readable to use concatenation instead of interpolation for variables inside strings. // Instead of
$x = "foo $bar $baz";
// Better use either
$x = "foo " . $bar . " " . $baz;
$x = sprintf("foo %s %s", $bar, $baz);
![]() |
|||
43 | } |
||
44 | |||
45 | // Delete the option |
||
46 | delete_option( 'email-log-db' ); |
||
47 | } |
||
48 |
It is generally a best practice as it is often more readable to use concatenation instead of interpolation for variables inside strings.