2011年1月の素晴らしいWordPress管理メニューチャレンジ(別名、WordPress管理メニューシステムの変更時にいくつかの課題を解決する方法?)


14

この質問は少しユニークです。

一部は、Tracチケットに関連するWordPressチームまたは他の誰か)に発行している「チャレンジ」です:#16048#16050および#16204

目標

目標は、WordPress管理メニューセクションを変更しようとするときに、以下のスクリーンショットに示す3つの問題を回避することです。

  1. ゲット「マイクロサイト」弁護士を編集するときに強調表示されるサブメニューのページを(このために、我々は何とかサブメニュー項目に「現在」を適用できるようにする必要があり、_と_wp_menu_output()関数内のいくつかのフックが、ここで必要なものを提供します)

  2. 弁護士を/wp-admin/edit.php?post_type=attorney編集するときにリンクする弁護士メニューページリンクを取得します(そして、_wp_menu_output()関数で必要な同じフックがこれを処理できます)

  3. このページにアクセスするための十分な権限がありません」エラーをトリガーしないように「マイクロサイト」リンクを取得します(これは解決するのが最も厄介なものであり、戻り値のフックuser_can_access_admin_page()がこの問題をうまく処理できます)。

2011年1月のThe Great WordPress Admin Menu Challengeのスクリーンショット
(ソース:mikeschinkel.com

私のユースケース以上のもの

これらの3つの問題は、私のユースケース向けですが、WordPressの管理メニューの設定に関連する問題を象徴しています。

WordPressチームの何人かは簡単だと言ったので、何かが欠けていることを暗示しています(正しいかもしれません)が、この問題を何週間も見てきました。 (Gistからもダウンロード可能)問題の最も単純な使用例として。のコードadmin_menu2()はかなりハックですが、WordPressの管理メニューを変更するために必要なことはほとんどあります。

プラグインの作成に時間がかかっていたため、3.1でremove_menu_page()新しいremove_submenu_page()関数も新しい関数も使用しようとしなかったことに注意してくださいadmin_menu2()-既存のプロジェクトからコードを既に持っていた-そして、彼らが対処するとは思わないとにかく問題。

私が必要なものは何?

次の2つのいずれかが必要です。

  1. このプラグインで公開し、この質問とスクリーンショットで説明する問題の解決策(ところで、PHP出力バッファリングを使用してこの部分を解決する場合、ソリューションを失格させますまたは

  2. WordPressチームにこれらのフックが実際に必要であることを認識させ、チケットの位置を再考させるようにするため。

どのように挑戦しますか?

  1. WordPress 3.1の初期状態の新しいコピーを ダウンロードしてインストールします(おそらくどのリビジョンでも可能です)

  2. 以下の「2011年1月の素晴らしいWordPress管理者メニューチャレンジ」プラグインをダウンロード、インストール、アクティブ化(またはGistからプラグインをダウンロードしてから、

  3. このプラグインのプラグインページにある指示に従います(次のスクリーンショットを参照が、基本的には上記のスクリーンショットを読み込んでから、説明されている3つの問題を把握してください。

「2011年1月の素晴らしいWordPress管理メニューチャレンジ」プラグインの説明のスクリーンショット
(ソース:mikeschinkel.com

ワン・レイ・オブ・ホープ

幸いなことに、WordPressチームのAndrew Nacinがコードを作成したらこれを見るように申し出たので、主に彼がレビューとコメントのためにここに投稿し、他の人がコメントできるようにします。彼が忙しいことはわかっていますが、(またはあなたも)がこのプラグインをv3.1の初期インストールにインストールして、問題を解決できるかどうかを確認してください。

チャレンジが不可能であることに同意する場合

この挑戦をしようとした後、あなたは私と同じ結論に来て、あなたはWordPressの管理メニューを見たい場合は、これらのTracのチケットの複数の構成下さいコメント(可能ならば#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();
}

これを読んでいるすべての人に、あなたが助けてくれることを本当に望んでいます。

前もって感謝します。


私は興味があります(そして、管理者側での経験を本当にレベルアップする必要があります)が、おそらく3.1最終リリースを待つでしょう。私のローカルテストスタックは複数のコアバージョンにはあまり適していないため、現在の安定版を使用しています。
11年

あなたがマイクについて話す問題を正確に知っています、あなたがそこにいるよりもうまく説明できるとは思いませんが、管理者用のドロップダウンメニューを書くとき(楽しみのために)、ちょうど同じ問題を見て、私の+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されることを思い出しました。実際の編集はで行われます。もしあなたが本当に#2に縛られているなら...他の2つの基準は機能しません。/wp-admin/post.php?post=10&action=edit

これが、実装で#3が失敗する理由です...そして、私はそこまで到達できなかったので#1を試みることさえできませんでした。


分析が正しいと信じてください。問題のレイアウトは、ユースケースのメニュー構造を簡素化するという目標で、複数のクライアントリクエストがあったレイアウトです。私はサブメニューを使用することを提案しましたが、彼らはそれが好きではありませんでした。ユーザーにとっては混乱しすぎると感じました。私が言及しなかったかもしれないことの1つは、彼らが配布するために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()呼び出し中に自動作成されたメニューにサブメニュー項目が適切に追加されます。


おっと... CLASSを囲む二重アンダースコアが太字の書式に変更されました;-)
wyrfel

それを私が直した。:)
fuxia

ああ、それは最高です。ありがとう!どうしてその最初のポイントを見逃したのでしょうか?!?ドッ!
MikeSchinkel

ありがとう、マイク。元のトピックに戻る... WPはいくつかのメニューエントリのIDを内部的に生成し、メニュー配列の値4に格納します。plugin-page-hooksなど。ただし、カスタムポストタイプの場合、plugin-page-hook形式と矛盾するIDが保存されます。私は、すなわち、すべてのページのフックを作成して、かなりの親スラグ/ファイル'によってサブメニューを取り付けるよりも、相互に関連付けるメニュー項目にそれらを使用する(WPが、これは一貫性を持たせるならば全体の多くを助けられると思う。
wyrfel
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.