クエリ出力が正しい場合でも、$ wpdb-> show_errors（）およびprint_error（）が出力を表示するのはなぜですか？

Question

クエリ出力が正しい場合でも、$ wpdb-> show_errors（）およびprint_error（）が出力を表示するのはなぜですか？

9

71761 2015-02-23

次の問題を解決するには、 https://wordpress.stackexchange.com/questions/178995/sanitize-a-working-query-string-by-using-wpdb-prepare-fails-with-mysql-db -er 私はかなり奇妙な振る舞いに出くわしました.私が使用したクエリが正しく、正しい出力を示していたとしても.

 global $wpdb;
$wpdb->show_errors(); 
$pageposts = $wpdb->get_results( 
        $wpdb->prepare(
          "
          SELECT skposts.*
          FROM  $wpdb->posts skposts,
                $wpdb->postmeta skpostmeta1,
                $wpdb->postmeta skpostmeta2
          WHERE skposts.ID = skpostmeta1.post_id
            AND skposts.ID = skpostmeta2.post_id
            AND skpostmeta1.meta_key = %s
            AND skpostmeta2.meta_key = %s
            AND skposts.post_type = %s
            AND skposts.post_status = %s
          ORDER BY skpostmeta1.meta_value ASC,
                   skpostmeta2.meta_value ASC
          ", 
          'class_day',
          'class_start',
          'courses',
          'publish'
        )
, OBJECT
);
$wpdb->print_error();

show_errorsとprint_errorがアクティブである限り、次のようにWPデータベースエラーが出力されます.

 WordPress database error: []
SELECT skposts.* FROM hmjtZ_posts skposts, hmjtZ_postmeta skpostmeta1, hmjtZ_postmeta skpostmeta2 WHERE skposts.ID = skpostmeta1.post_id AND skposts.ID = skpostmeta2.post_id AND skpostmeta1.meta_key = 'class_day' AND skpostmeta2.meta_key = 'class_start' AND skposts.post_type = 'courses' AND skposts.post_status = 'publish' ORDER BY skpostmeta1.meta_value ASC, skpostmeta2.meta_value ASC

しかし、なぜですか?エラーや警告などの問題が発生した場合にのみ出力が表示されることを期待していましたが、この方法では常に何かが表示されます.

In order of figuring out the following issue, see https://wordpress.stackexchange.com/questions/178995/sanitize-a-working-query-string-by-using-wpdb-prepare-fails-with-mysql-db-er i ran into an rather odd behaviour. Even that my used query was correct and showing the right output.

global $wpdb;
$wpdb->show_errors(); 
$pageposts = $wpdb->get_results( 
        $wpdb->prepare(
          "
          SELECT skposts.*
          FROM  $wpdb->posts skposts,
                $wpdb->postmeta skpostmeta1,
                $wpdb->postmeta skpostmeta2
          WHERE skposts.ID = skpostmeta1.post_id
            AND skposts.ID = skpostmeta2.post_id
            AND skpostmeta1.meta_key = %s
            AND skpostmeta2.meta_key = %s
            AND skposts.post_type = %s
            AND skposts.post_status = %s
          ORDER BY skpostmeta1.meta_value ASC,
                   skpostmeta2.meta_value ASC
          ", 
          'class_day',
          'class_start',
          'courses',
          'publish'
        )
, OBJECT
);
$wpdb->print_error();

Never the less as long as i have show_errors and print_error active i get an WP database error output alongside:

WordPress database error: []
SELECT skposts.* FROM hmjtZ_posts skposts, hmjtZ_postmeta skpostmeta1, hmjtZ_postmeta skpostmeta2 WHERE skposts.ID = skpostmeta1.post_id AND skposts.ID = skpostmeta2.post_id AND skpostmeta1.meta_key = 'class_day' AND skpostmeta2.meta_key = 'class_start' AND skposts.post_type = 'courses' AND skposts.post_status = 'publish' ORDER BY skpostmeta1.meta_value ASC, skpostmeta2.meta_value ASC

But why? I would have expected that an output is shown only if something goes wrong like an error or warning but this way something is shown all the time.

1 回答

投票

David Gard · Accepted Answer · 2015-02-23

20

2015-02-23

上記で投稿した出力は、次の場合に$wpdb->print_error()の予想される動作です-

マルチサイトではなく、単一のサイトを実行しています
$wpdb->suppress_errorsがfalseに設定されています
$wpdb->show_errorsがfalseに設定されています

コードの外観から、これらすべての条件を満たすことができます.

以前にオフにしていない限り、$wpdb->show_errorsはデフォルトでtrueに設定されているため、呼び出す必要はありません. $wpdb->show_errors().

DBエラーが発生した場合にのみ何かを出力するには、次の2つのいずれかを実行できます-

1-エラーを出力し、ログにエラーを追加します

画面に出力するだけでなく、$wpdb->print_error()メソッドはエラーをログに記録します.これが望ましい動作(推奨)である場合は、これを行うことができます-

 if($wpdb->last_error !== '') :
    $wpdb->print_error();
endif;

2-エラーを出力しますが、ログには記録しません

エラーのログ記録に興味がない場合は、独自のmy_print_error()関数を追加して、$wpdb->print_error()の代わりに使用できます-

 function my_print_error(){

    global $wpdb;

    if($wpdb->last_error !== '') :

        $str   = htmlspecialchars( $wpdb->last_result, ENT_QUOTES );
        $query = htmlspecialchars( $wpdb->last_query, ENT_QUOTES );

        print "<div id='error'>
        <p class='wpdberror'><strong>WordPress database error:</strong> [$str]<br />
        <code>$query</code></p>
        </div>";

    endif;

}

最終編集:構文の間違い

The output that you posted above is expected behaviour for $wpdb->print_error() if the following is true -

You are running a single site, not multisite
$wpdb->suppress_errors is set to false
$wpdb->show_errors is set to false

From the looks of your code, you meet all those conditions.

Note also that, unless you have turned them off previously, $wpdb->show_errors is set to true by default, so you don't need to call $wpdb->show_errors().

To output something only when there is a DB error you can do one of these two things -

1 - Output the error and add the error to the log

As well as outputting on the screen, the $wpdb->print_error() method will log your error. If this is desirable behaviour (recommended), you can do this -

if($wpdb->last_error !== '') :
    $wpdb->print_error();
endif;

2 - Output the error but do not log it

If you are not interested in logging the error, you can add your own my_print_error() funciton and use that instead of $wpdb->print_error() -

function my_print_error(){

    global $wpdb;

    if($wpdb->last_error !== '') :

        $str   = htmlspecialchars( $wpdb->last_result, ENT_QUOTES );
        $query = htmlspecialchars( $wpdb->last_query, ENT_QUOTES );

        print "<div id='error'>
        <p class='wpdberror'><strong>WordPress database error:</strong> [$str]<br />
        <code>$query</code></p>
        </div>";

    endif;

}

Last Edit: Syntax Mistake

$ stringと$ queryで空白になります

Gives me blank in $str and $query
- 0
- 2018-07-15
エラーメッセージは、 `$ wpdb-> last_result`ではなく` $ wpdb-> last_error`にあります.

The error message is in `$wpdb->last_error`, not in `$wpdb->last_result`.
- 2
- 2018-12-12
- Martin_W