2011年1月的WordPress管理员菜单挑战赛(又名如何在修改WordPress管理员菜单系统时解决一些挑战?)


14

这个问题有点独特。

这部分是我向WordPress团队或其他任何人发出的与Trac票相关的“挑战”#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月的大WordPress管理菜单挑战
(来源:mikeschinkel.com

不只是我的用例

这三(3)个问题是针对我的用例,但它们是与在WordPress中配置管理菜单有关的问题的象征。

WordPress团队中的一些人说这很容易,因此暗示我缺少一些东西(可能是正确的),但是我已经研究了这个问题了好几个星期,却没有弄清楚如何解决它,因此我创建了您在下面看到的插件(也来自Gist下载的),作为问题的最简单的使用情况的例子。中的代码admin_menu2()有点黑,但这几乎是修改WordPress中的“管理菜单”所需要的。

请注意,我并未尝试使用3.1中remove_menu_page()的新remove_submenu_page()功能或新功能,因为创建该插件将花费更长的时间-我已经admin_menu2()从现有项目中获得了代码-而且我认为它们不会解决该问题。反正问题。

我需要什么?

我需要两(2)件事之一:

  1. 解决该插件问题的解决方案,并在此问题和屏幕截图中说明(顺便说一句,如果您使用PHP Output Buffering解决此问题的任何部分,我将取消您的解决方案的资格,或者

  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最终版本。我的本地测试堆栈不适用于多个内核版本,因此我坚持使用当前稳定器。
罗斯特2011年

我确切地知道您提到迈克的问题,我认为我无法比您描述的问题更好,但是我在为管理员编写下拉菜单(有趣)时看到了同样的问题,只是添加了我的+1。
t31os 2011年

@ t310s-感谢您添加+1。我大概花了2个星期的研究时间才能描述问题,因此您甚至可以认识到它是相关的(并且没有花我2个星期的时间),这意味着您在这方面领先于大多数人,包括我!
MikeSchinkel 2011年

Answers:


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,因为我做不到。


相信您的分析是正确的。有问题的布局是我为满足其目标简化了用例菜单结构而提出的多个客户请求。我已经建议使用子菜单,但是他们不喜欢它。他们觉得这对于他们的用户来说太混乱了。我可能没有提到的一件事是,我正在为他们开发基于WordPress的产品,而不是为他们开发WordPress网站,在那里我可以培训他们知道如何使事情正常进行。他们的另一个选择是放弃WordPress。不是我想要他们做的。
MikeSchinkel 2011年

2

嗨,迈克,您的问题3是由于您指定的($microsite, 'the_microsite_editor')位置(__CLASS__, 'the_microsite_editor')

更新:花了太多时间尝试解决我自己的插件的类似问题后,发现有些东西可能对您的Challenge有所帮助(请注意,函数是类下面的方法):

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 2011年

我修好了它。:)
fuxia

哦,太好了;谢谢!我怎么会错过第一点?!h!
MikeSchinkel 2011年

谢谢,迈克。回到您的原始主题... WP在内部为某些菜单项生成一个ID,并将其存储在菜单数组的值4中。如plugin-page-hooks。但是,对于自定义帖子类型,它存储的ID与plugin-page-hook格式不一致。我想了一大堆可以帮助如果WP将使这是一致的(即创建页面挂钩的一切,并用它们来关联菜单项彼此,而不是由“父塞/文件”的方式连接子菜单。
wyrfel
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.