Wordpress: отображение сообщения об ошибке - ловушка admin_notices завершается ошибкой на wp_insert_post_data или publish_post - PullRequest
Новая
       14

Wordpress: отображение сообщения об ошибке - ловушка admin_notices завершается ошибкой на wp_insert_post_data или publish_post

5 голосов
/ 07 августа 2009

Я добавляю проверку, поэтому, если сообщение относится к определенной категории, для него необходимо установить определенные пользовательские поля.

Это должно быть легко перехватывать wp_insert_post_data и admin_notices, но есть перенаправление, которое вызывает обратный вызов admin_notices.

ОК. Итак, я создал хак, который использует Session для хранения моего сообщения об ошибке при перенаправлении: 

function set_post_pending($data, $postarr) {
    // If it's not valid...
        $error = "You are missing some Custom Fields.";
        $_SESSION['admin_notices'] = $error;
        $data['post_status'] = 'pending';
    return $data;
}
add_filter('wp_insert_post_data', 'set_post_pending',1,2);

function session_admin_notice() {
    if($out = $_SESSION['admin_notices']) {
        $_SESSION["admin_notices"] = "";
        echo $out;
    }
    return false;
}
add_action('admin_notices', "session_admin_notice");

Проблема с этим решением заключается в том, что некоторые недоступны для сеанса при вызове session_admin_notice, что имеет простое (но сумасшедшее ) решение: 

public static function fix_session_bs() {
    // TODO: Why do I have to do this?
    if(!session_id() && $_COOKIE["PHPSESSID"]) {
        session_start($_COOKIE["PHPSESSID"]);
    }
}
add_action('admin_init', 'fix_session_bs');

Вопрос в том, почему я должен пройти через все это безумие, чтобы выдать сообщение об ошибке?

Что я делаю не так?

Ответы [ 3 ]

7 голосов
/ 07 ноября 2013

Вы можете просто сделать как WordPress: используя переходные процессы , например: 

function set_post_pending($data, $postarr) {
    // If it's not valid...
        $error = "You are missing some Custom Fields.";
        set_transient( get_current_user_id().'missingfield', $error );
        $data['post_status'] = 'pending';
    return $data;
}
add_filter('wp_insert_post_data', 'set_post_pending',1,2);

function show_admin_notice() {
    if($out = get_transient( get_current_user_id().'missingfield' ) ) {
        delete_transient( get_current_user_id().'missingfield' );
        echo "<div class=\"error\"><p>$out</p></div>";
    }
    // return false; // nothing to return here
}
add_action('admin_notices', "session_admin_notice");

ps: избегайте $ _SESSION в WordPress, спасибо

3 голосов
/ 07 августа 2009

Wordpress не использует сеансы, и, если register_globals включен, он очистит массив $_SESSION.

Wordpress передает свои сообщения, используя целое число message в URL, затем массив сообщений определяется в соответствующем файле edit-[type]-form.php в папке wp-admin. Я думаю, что вы, вероятно, могли бы добавить свою собственную переменную в редирект, а затем получить ее в своей функции admin_notices hook. Посмотрите на файлы edit-[type]-form.php, чтобы понять, как это может работать.

1 голос
/ 07 июля 2011 
if($out = $_SESSION['admin_notices']) { 
      $_SESSION["admin_notices"] = ""; 
      echo $out; 
}

Это условие всегда ИСТИНА, поэтому оно всегда сбрасывает ваш $ _SESSION ['admin_notices'] var

...