Великий вызов админ-меню WordPress за январь 2011 года (иначе как решить некоторые проблемы при изменении системы админ-меню WordPress?)

14

Этот вопрос немного уникален.

Это частично «вызов», который я выдаю команде WordPress ( или кому-либо еще ), связанный с билетами trac: # 16048 , # 16050 и # 16204 .

Цель

Цель состоит в том, чтобы обойти три (3) проблемы, показанные на снимке экрана ниже, при попытке изменить раздел меню WordPress Admin:

  1. Получить страницу подменю «Microsite», которая будет выделена при редактировании Attorney (для этого нам нужно каким-то образом иметь возможность применить «current» к элементу подменю, _ и некоторые перехваты в функции _wp_menu_output () предоставят то, что здесь необходимо) ,

  2. Получить ссылку на страницу меню Attorney, на которую можно ссылаться /wp-admin/edit.php?post_type=attorneyпри редактировании Attorney (и те же самые необходимые хуки в функции _wp_menu_output () могут справиться с этим) , и

  3. Получите ссылку «Микросайт», чтобы не вызывать ошибку «У вас недостаточно прав для доступа к этой странице» * (это самая неприятная проблема, которую можно решить, и зацепка возвращаемого значенияuser_can_access_admin_page() могла бы решить эту проблему).

Снимок экрана для Великого админ-меню WordPress Challenge за январь 2011
(источник: mikeschinkel.com )

Больше, чем просто мой сценарий использования

Эти три (3) проблемы для моего варианта использования, но они символизируют проблемы, связанные с настройкой меню администратора в WordPress.

Некоторые из команды WordPress сказали, что это легко и, следовательно, подразумевается, что я что-то упустил (что может быть правильным), но я смотрел на эту проблему неделями и не придумал, как ее обойти, поэтому я создал плагин, который вы видите ниже ( также можно загрузить из Gist ) в качестве простейшего примера использования проблем. Код admin_menu2()довольно хакерский, но это почти то, что требуется для изменения меню администратора в WordPress.

Обратите внимание, что я не пытался использовать новые remove_menu_page()или новые remove_submenu_page()функции в 3.1, потому что создание плагина заняло бы больше времени - у меня уже был код admin_menu2()из существующего проекта - и я не верю, что они рассмотрели бы проблема в любом случае.

Что мне нужно?

Мне нужна одна из двух (2) вещей:

  1. Решение проблем, которые я раскрываю с помощью этого плагина и объясняю в этом вопросе и на скриншоте (кстати, я дисквалифицирую ваше решение, если вы используете PHP Output Buffering для решения какой-либо части этого) , или

  2. Чтобы команда WordPress признала, что эти крючки действительно нужны, и заставила их пересмотреть свою позицию в билетах.

Как вы принимаете вызов?

  1. Загрузите и установите новую версию WordPress 3.1 (возможно, подойдет любая ревизия) ,

  2. Загрузите, установите и активируйте плагин «The Great WordPress Admin Menu от января 2011» ниже (или загрузите плагин из Gist ) , а затем

  3. Следуйте инструкциям, приведенным на странице плагина для этого плагина (см. Следующий снимок экрана), но в основном загрузите снимок экрана, который вы видите выше, а затем попытайтесь выяснить три (3) описанные проблемы:

Снимок экрана с инструкциями к плагину "The Great WordPress Admin Menu от января 2011"
(источник: mikeschinkel.com )

Один Луч Надежды

К счастью, Эндрю Нацин из команды WordPress предложил взглянуть на это, как только я закодировал его, поэтому я в первую очередь публикую здесь его, чтобы он мог просмотреть и прокомментировать, а также попросить других прокомментировать. Я знаю, что он занят, но я надеюсь, что он (или даже вы) может потратить время на установку этого плагина в первоначальной версии v3.1 и посмотреть, сможет ли он решить проблему.

Если вы согласны, что вызов невозможен?

Если после выполнения этого задания вы пришли к тому же выводу, что и я, и если вы хотите, чтобы меню администратора WordPress были более настраиваемыми, прокомментируйте эти заявки ( # 16048 - # 16050 - # 16204 ) и проголосуйте за этот вопрос. чтобы поддержать его.

Я с радостью признаю, что что-то упустил, если бы сделал

Конечно, возможно, я мог бы быть полностью мертвым от этого, и кто-то мог бы точно указать, как это сделать. На самом деле, я очень надеюсь, что в конечном итоге это так; Я бы предпочел быть неправым, и это сработало, а не наоборот.

А вот и плагин

Вы также можете скачать, если из Gist :

<?php
/*
Plugin Name: The Great WordPress Admin Menu Challenge of Jan 2011
Description: <em>"The Great WordPress Admin Menu Challenge of Jan 2011"</em> was inspired by the WordPress team's apparent lack of understanding of the problems addressed by trac tickets <a href="http://core.trac.wordpress.org/ticket/16048">#16048</a> and <a href="http://core.trac.wordpress.org/ticket/16050">#16050</a> <em>(See also: <a href="http://core.trac.wordpress.org/ticket/16204">#16204</a>)</em> and suggestion that the <a href="http://wordpress.org/extend/plugins/admin-menu-editor/>Admin Menu Editor</a> plugin handles the use-cases that the tickets address. Debate spilled over onto Twitter with participation from <a href="http://twitter.com/nacin">@nacin</a>, <a href="http://twitter.com/aaronjorbin">@aaronjorbin</a>, <a href="http://twitter.com/petemall">@petemall</a>, <a href="http://twitter.com/westi">@westi</a>, <a href="http://twitter.com/janeforshort">@janeforshort</a>, <a href="http://twitter.com/PatchesWelcome">@PatchesWelcome</a>; supportive comments from <a href="http://twitter.com/ramsey">@ramsey</a>, <a href="http://twitter.com/brianlayman">@brianlayman</a>, <a href="http://twitter.com/TheLeggett">@TheLeggett</a>, a retweeting of @nacin's simple yet <em>(AFAICT)</em> insufficient solution by <a href="http://twitter.com/vbakaitis">@vbakaitis</a>, <a href="http://twitter.com/Viper007Bond">@Viper007Bond</a>, <a href="http://twitter.com/nickopris">@nickopris</a>, <a href="http://twitter.com/Trademark">@Trademark</a>, <a href="http://twitter.com/favstar_pop">@favstar_pop</a>, <a href="http://twitter.com/designsimply">@designsimply</a>, <a href="http://twitter.com/darylkoop">@darylkoop</a>, <a href="http://twitter.com/iamjohnford">@iamjohnford</a>, <a href="http://twitter.com/markjaquith">@markjaquith</a>, <a href="http://twitter.com/JohnJamesJacoby">@JohnJamesJacoby</a> and <a href="http://twitter.com/dd32">@dd32</a>. Also see <a href="http://andrewnacin.com/2010/12/20/better-admin-menu-controls-custom-post-types-wordpress-3-1/#comment-6360">comments</a> on @nacin's blog post entitled "<em>Better admin menu handling for post types in WordPress 3.1</em>." <strong>The desired goal of the <em>"challenge"</em></strong> is to simply either to find a solution that has eluded me or, to get those who are dismissing it as solvable without added hooks in WordPress to have a tangible example to explore in hopes they will recognize that there is indeed a need for at least some of the requested hooks. <strong>There are three (3) steps to the challenge:</strong> 1.) Get the "Microsite" submenu page to be highlighted when editing an Attorney, 2.) Get the Attorney Menu Page link to link <a href="/wordpress//wp-admin/edit.php?post_type=attorney">here</a>  when editing an Attorney, and 3.) Get the "Microsite" link not to trigger a "You do not have sufficient permissions to access this page" error.  Here is <a href="https://mikeschinkel.com/websnaps/skitched-20110114-235302.png" target="_blank"><strong>a screenshot</strong> that attempts to illustrate the callenge</a>. The code can be found on gist <a href="https://gist.github.com/780709"><strong>here</strong></a>. Activate it as a plugin in a WordPress 3.1 install and go <a href="/wordpress//wp-admin/post.php?post=10&action=edit"><strong>here</strong></a> to see what the screenshot illustrates. <strong>Be sure to load the <a href="https://mikeschinkel.com/websnaps/skitched-20110114-235302.png" target="_blank">screenshot</a> in another browser tab or window first</strong>.
Author:      Mike Schinkel
Author URI:  http://about.me/mikeschinkel
Plugin URI:  https://gist.github.com/780709
*/
if (!class_exists('TheGreatWordPressAdminMenuChallenge')) {
  class TheGreatWordPressAdminMenuChallenge {
    static function on_load() {
      add_action('init',array(__CLASS__,'init'));
      add_action('admin_menu',array(__CLASS__,'admin_menu1'));      // Simulates generic "Microsite" plugin
      add_action('admin_menu',array(__CLASS__,'admin_menu2'),100);  // Simulates website-specific plugin
      add_action('post_row_actions',array(__CLASS__,'post_row_actions'),10,2);
    }
    static function post_row_actions($actions,$post) {
      $url = admin_url(self::this_microsite_url($post->ID));
      $actions = array_merge(array('microsite'=>"<a href=\"{$url}\" title=\"Manage this Microsite\">Microsite</a>"),$actions);
      return $actions;
    }
    static function the_microsite_editor() {
      echo "We are in the Microsite Editor for " . self::post_title();
    }
    static function admin_menu1() {
      if (self::this_post_id() && in_array(self::this_post_type(),array('attorney','practice_area'))) {
        add_submenu_page(
          self::this_parent_slug(),
          self::microsite_page_title(),
          self::microsite_page_title(),
          $capability = 'edit_posts',
          'microsite',
          array($microsite,'the_microsite_editor')
        );
        global $wp_post_types;
        $parent_type_meta = $wp_post_types[self::this_post_type()];
        global $menu;
        $slug = false;
        foreach($menu as $index => $menu_page)
          if ($menu_page[0]===$parent_type_meta->label) {
            $slug = $menu_page[2];
            break;
          }
        if ($slug) {
          global $pagenow;
          global $submenu;
          // Setting this makes gives the link to the microsite in the menu the highlight for "current" menu option
          global $submenu_file;
          $submenu_file = self::this_microsite_url();
          $index = end(array_keys($submenu[$slug]));
          $submenu[$slug][$index][12] = $submenu_file;
        }
      }
    }
    static function this_parent_slug() {
      return "edit.php?post_type=" . self::this_post_type();
    }
    static function post_title() {
      $post_id = self::this_post_id();
      return ($post_id ? get_post($post_id)->post_title : false);
    }
    static function microsite_page_title() {
      return 'Microsite for ' . self::post_title();
    }
    static function this_post_type($get_post=true) {
      $post_type = (isset($_GET['post_type']) ? $_GET['post_type'] : false);
      if (!$post_type && $get_post) {
        $post_id = self::this_post_id();
        $post_type = get_post($post_id)->post_type;
      }
      return $post_type;
    }
    static function this_post_id() {
      $post_id = false;
      $post_type = self::this_post_type(false);
      if (isset($_GET[$post_type]))
        $post_id = intval($_GET[$post_type]);
      else if (isset($_GET['post']))
        $post_id = intval($_GET['post']);
      return $post_id;
    }
    static function this_microsite_url($post_id=false) {
      $post_type = self::this_post_type();
      $post_id = $post_id ? intval($post_id) : self::this_post_id();
      return "edit.php?post_type={$post_type}&page=microsite&attorney={$post_id}";
    }
    static function admin_menu2() {
      // The code required for this is super, nasty, ugly and shouldn't be, but at least it *is* doable
      global $menu;
      global $submenu;
      global $microsite;

      $parent_type = self::this_post_type();
      foreach(array('attorney','practice_area') as $post_type) {
        $slug = "edit.php?post_type={$post_type}";
        if ($post_type==$parent_type) {  // If a microsite remove everything except the microsite editor
          $microsite_url = self::this_microsite_url();
          foreach($submenu[$slug] as $submenu_index => $submenu_page) {
            if ($submenu_page[2]!=$microsite_url) {
              unset($submenu[$slug][$submenu_index]);
            }
          }
        } else {
          $submenu[$slug] = array();
        }
      }

       // Remove the Submenus for each menu
      unset($submenu['index.php']);
      unset($submenu['edit.php?post_type=article']);
      unset($submenu['edit.php?post_type=event']);
      unset($submenu['edit.php?post_type=case_study']);
      unset($submenu['edit.php?post_type=news_item']);
      unset($submenu['edit.php?post_type=transaction']);
      unset($submenu['edit.php?post_type=page']);
      unset($submenu['upload.php']);

      unset($submenu['users.php'][13]); // Removed the "Add New"

      $remove = array_flip(array(
        'edit.php',
        'link-manager.php',
        'edit-comments.php',
        'edit.php?post_type=microsite-page',
      ));
      if (!current_user_can('manage_tools'))
        $remove['tools.php'] = count($remove);

      foreach($menu as $index => $menu_page) {
        if (isset($remove[$menu_page[2]])) {
          unset($submenu[$menu_page[2]]);
          unset($menu[$index]);
        }
      }

      $move = array(
        'edit.php?post_type=page' => array( 'move-to' => 35,  0 => 'Other Pages' ),
        'separator2' => array( 'move-to' => 40 ),
        'upload.php' => array( 'move-to' => 50, 0 => 'Media Library' ),
      );
      $add = array();
      foreach($menu as $index => $menu_page) {
        if (isset($move[$menu_page[2]])) {
          foreach($move[$menu_page[2]] as $value_index => $value) {
            if ($value_index==='move-to') {
              $move_to = $value;
            } else {
              $menu_page[$value_index] = $value;
            }
          }
          $add[$move_to] = $menu_page;
          unset($menu[$index]);
        }
      }
      foreach($add as $index => $value)
        $menu[$index] = $value;

      add_menu_page(
        'Attorney Positions',
        'Attorney Positions',
        'edit_posts',
        'edit-tags.php?taxonomy=attorney-position&amp;post_type=attorney',
        false,
        false,
        55);

      ksort($menu); // Need to sort or it doesn't come out right.
    }
    static function init() {
      register_post_type('attorney',array(
        'label'           => 'Attorneys',
        'public'          => true,
      ));
      register_post_type('practice_area',array(
        'label'           => 'Practice Areas',
        'public'          => true,
      ));
      register_taxonomy('attorney-position','attorney',array(
        'label'=>'Attorney Positions',
      ));
      register_post_type('article',array(
        'label'           => 'Articles & Presentations',
        'public'          => true,
      ));
      register_post_type('case_study',array(
        'label'           => 'Case Studies',
        'public'          => true,
      ));
      register_post_type('news_item',array(
        'label'           => 'Firm News',
        'public'          => true,
      ));
      register_post_type('event',array(
        'label'           => 'Events',
        'public'          => true,
      ));
      register_post_type('transaction',array(
        'label'           => 'Transactions',
        'public'          => true,
      ));

      // Install the test data
      $post_id = 10;
      $attorney = get_post($post_id);
      if (!$attorney) {
        global $wpdb;
        $wpdb->insert($wpdb->posts,array(
          'ID' => $post_id,
          'post_title' => 'John Smith',
          'post_type' => 'attorney',
          'post_content' => 'This is a post about the Attorney John Smith.',
          'post_status' => 'publish',
          'post_author' => 1,
        ));
      }
    }
  }
  TheGreatWordPressAdminMenuChallenge::on_load();
}

Всем, кто читает это, я очень надеюсь, что вы можете помочь.

Заранее спасибо.

MikeSchinkel
источник
Я заинтересован (и действительно должен повысить свой опыт работы на стороне администратора), но, вероятно, подожду финальной версии 3.1. Мой локальный тестовый стек не очень подходит для многоядерных версий, поэтому я придерживаюсь текущей стабильной версии.
Первое
Я точно знаю проблему, о которой вы говорите, Майк, я не думаю, что смог бы описать ее лучше, чем вы, но я видел те же самые проблемы при написании выпадающего меню для администратора (для удовольствия), просто добавив мой +1.
t31os
@ t310s - Спасибо за добавление +1. У меня ушло, вероятно, 2 недели исследований, чтобы описать проблему, поэтому тот факт, что вы можете даже признать, что она актуальна (и не потратила 2 недели, которые у меня есть), означает, что вы намного опережаете большинство всех в этом, включая мне!
MikeSchinkel

Ответы:

2

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

  1. Получить страницу подменю «Микросайт», которая будет выделена при редактировании Адвоката
  2. Получить ссылку на страницу меню адвоката, на которую можно перейти /wp-admin/edit.php?post_type=attorneyпри редактировании адвоката
  3. Получите ссылку «Микросайт», чтобы не вызвать ошибку «У вас недостаточно прав для доступа к этой странице»

И ключевой вопрос здесь # 2.

Что я пробовал

Я попытался добавить пользовательский тип сообщения для адвокатов, и мне сразу же напомнили, что /wp-admin/edit.php?post_type=attorneyдаст вам список адвокатов, а не экран редактирования. Фактическое редактирование происходит /wp-admin/post.php?post=10&action=edit. Так что, если вы действительно привязаны к # 2 ... два других критерия не сработают.

Вот почему № 3 терпит неудачу в реализации ... и я даже не смог попробовать # 1, потому что я не мог зайти так далеко.

EAMann
источник
Поверьте, ваш анализ верен. Макет, показанный в вопросе, один, у меня было несколько запросов клиентов в целях упрощения структур меню для их вариантов использования. Я предложил использовать подменю, но им это не понравилось; они чувствовали, что это будет слишком запутанным для их пользователей. Одна вещь, которую я, возможно, не упомянул, это то, что я разрабатываю продукт на основе WordPress для их распространения по сравнению с веб-сайтом WordPress для них, где я могу просто научить их, как заставить вещи работать. Другой вариант - отказаться от WordPress; не то, что я хочу, чтобы они сделали.
MikeSchinkel
2

Привет, Майк, твоя проблема # 3 связана с тем, что ты уточняешь ($microsite, 'the_microsite_editor'), где это должно быть (__CLASS__, 'the_microsite_editor').

Обновление: потратив слишком много времени на решение некоторых подобных проблем для моего собственного плагина, я нашел кое-что, что может помочь с вашей задачей (обратите внимание, что функции - это методы вашего класса):

function add_posttype_submenu_page($mytype, $label, $cap, $slug) {  
    /* we add two submenu pages to work around the 
       edit.php?post_type=...&page=...problem and have 
       our page called as admin.php?page=... instead */
    //first create a 'blind' pseudo-entry to register our page callback
    add_submenu_page($mytype, $label, $label, $cap, $slug, 
                     array( &$this, 'admin_'.$mytype ));
    //then create a real entry that 'calls' our pseudo-entry
    add_submenu_page('edit.php?post_type='.$mytype, $label, 
                     $label, $cap, 'admin.php?page='.$slug);
    /* then lets fix/hack the highlighting */
    global $plugin_page;
    global $submenu_file;
    if ($plugin_page == $slug) {
        // this next line highlights the submenu entry
        $submenu_file = 'admin.php?page='.$slug; 
        add_filter('parent_file', 
                   array(&$this, 'evil_parent_file_hack'));
    }
} 

function evil_parent_file_hack() {
    //we do this to get the parent menu properly highlighted, too
    //it only gets called on the submenu menu page in question
    global $self;
    global $parent_file;
    $self = $parent_file;
    remove_filter('parent_file', array(&$this, 'evil_parent_file_hack'));
}

Тогда вы просто звоните add_posttype_submenu_page()с соответствующими параметрами. Это должно правильно добавить пункт подменю в меню, которое было создано автоматически во время register_post_type()вызова.

wyrfel
источник
упс ... двойные подчеркивания вокруг CLASS были превращены в жирный формат ;-)
wyrfel
Я починил это. :)
fuxia
О, это просто потрясающе; Благодарность! Как я мог пропустить этот первый пункт?!? Doh!
MikeSchinkel
Спасибо, Майк. Вернуться к исходной теме ... WP внутренне генерирует идентификатор для некоторых пунктов меню и сохраняет его в значении 4 массивов меню. Такие как плагин-страница-крючки. Однако для пользовательских типов сообщений хранится идентификатор, который не согласуется с форматом plugin-page-hook. Я думаю, что многое могло бы помочь, если бы WP сделал это непротиворечивым (то есть создавал хуки страниц для всего и использовал их для связывания пунктов меню друг с другом, вместо того, чтобы прикреплять подменю с помощью «родительского слаг / файла».
wyrfel