Take the 2-minute tour ×
WordPress Development Stack Exchange is a question and answer site for WordPress developers and administrators. It's 100% free, no registration required.

I'd like this code to only run inside the admin area as it is resorting the items on the public side admin bar too.

  /* Reorder Admin Menu to put "Pages" at the top */
  function menu_order_filter($menu) {
  $content_menu = array('edit.php?post_type=page');
  array_splice($menu, 2, 0, $content_menu);
  return array_unique($menu);
  }
  add_filter('custom_menu_order', create_function('', 'return true;'));
  add_filter('menu_order', 'menu_order_filter');
share|improve this question

2 Answers 2

up vote 3 down vote accepted

There is very little overhead to assigning couple of filters on hooks that simply won't fire on front end.

In general it would be something like this:

add_action('init', 'admin_only');

function admin_only() {

    if( !is_admin() )
        return;

    // filter assignemnts and such go here
}

Also create_function() is not recommended for performance and some other reasons. It is better to use more modern Anonymous Functions, but for cases like this WordPress provides ready-made __return_true() function.

share|improve this answer
    
Sorry i'm more of a copy/paste hacker at the moment. How would i modify my above code to use __return_true()? Thanks. –  agileapricot Dec 21 '10 at 13:32
    
@firefusion simply replace create_function('', 'return true;') with '__return_true' –  Rarst Dec 21 '10 at 13:38
2  
You could just hook onto admin_init instead of init and you'll not need that is_admin() check. –  t31os Dec 21 '10 at 13:47
    
@t31os yep - that will work too. I just usually have giant wrapper running on init and it is more convenient to keep both admin and non-admin stuff in there. –  Rarst Dec 21 '10 at 13:53
    
anonymous functions only work in PHP 5.3+. I'd be curious to know more about the disadvantages of create_function –  onetrickpony Dec 21 '10 at 14:09

use the Hook admin_init and later hooks, the init-Hook comes realy ealier as only the admin. But it is important, when you use li18n-functions or AJAX, than it is better you use init.

share|improve this answer

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.