• Resolved campusboy

    (@campusboy1987)


    Hello! After activating the plugin, errors appear in the admin panel:

    Notice: Undefined index: page in D:\server72\sites\wp-test.ru\wp-content\plugins\slider-matches-results\slider-matches-results.php on line 621
    Notice: Undefined index: page in D:\server72\sites\wp-test.ru\wp-content\plugins\slider-matches-results\slider-matches-results.php on line 815
    Notice: Undefined index: page in D:\server72\sites\wp-test.ru\wp-content\plugins\slider-matches-results\slider-matches-results.php on line 826
    Warning: Cannot modify header information - headers already sent by (output started at D:\server72\sites\wp-test.ru\wp-content\plugins\slider-matches-results\slider-matches-results.php:621) in D:\server72\sites\wp-test.ru\wp-admin\includes\misc.php on line 1116

    Please check the array arguments $ _GET with the isset() construct.

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Cédric Chevillard

    (@cedche99)

    Hi,

    Sorry for not answering you before, I did not see your topic.

    I made a correction, can you tell me if that solves your problem.

    Cédric

    Thread Starter campusboy

    (@campusboy1987)

    Yes, the errors on the first message are eliminated. Thank you!

    But they stayed. There are errors on the admin.php?page=smr_plugin_matches page, they are related to the incorrect use of wpdb::prepare.

    * File slider-matches-results.php, line 232
    * File slider-matches-results.php, line 370
    * And so on

    This is not dynamic getting the value, so the prepare() method can not be used at all. But if you want reliability, it is better to do so:

    
    $table_name_matches = sanitize_key( $this->get_table_name_matches() );
    
    $sql = "SELECT * FROM $table_name_matches";
    
    // Select all matches in DB
    $results_matches = $wpdb->get_results( $sql );
    
    • This reply was modified 6 years, 6 months ago by campusboy.
    Plugin Author Cédric Chevillard

    (@cedche99)

    Hi,

    Thx :).

    A fix has been made.

    Cédric

    Plugin Author Cédric Chevillard

    (@cedche99)

    Hi, i close the topic.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Errors’ is closed to new replies.