Changeset 6320

Timestamp:

11/07/2007 04:30:11 AM (17 years ago)

Author:

ryan

Message:

has_action and has_filter. see #5231

Location:

trunk

Files:

• wp-admin/includes/plugin.php (modified) (1 diff)
• wp-includes/classes.php (modified) (1 diff)
• wp-includes/plugin.php (modified) (8 diffs)
• wp-settings.php (modified) (1 diff)

trunk/wp-admin/includes/plugin.php

@@ -330 +330 @@
 
 function get_plugin_page_hook( $plugin_page, $parent_page ) {
-    global $wp_filter;
-
     $hook = get_plugin_page_hookname( $plugin_page, $parent_page );
-    if ( isset( $wp_filter[$hook] ))
+    if ( has_action($hook) )
         return $hook;
     else

trunk/wp-includes/classes.php

@@ -239 +239 @@
 
         // query_string filter deprecated.  Use request filter instead.
-        global $wp_filter;
-        if ( isset($wp_filter['query_string']) ) {  // Don't bother filtering and parsing if no plugins are hooked in.
+        if ( has_filter('query_string') ) {  // Don't bother filtering and parsing if no plugins are hooked in.
             $this->query_string = apply_filters('query_string', $this->query_string);
             parse_str($this->query_string, $this->query_vars);

trunk/wp-includes/plugin.php

@@ -80 +80 @@
 
 /**
+ * Check if any filter has been registered for a hook.  Optionally returns the priority on that hook for the specified function.
+ * @package WordPress
+ * @subpackage Plugin
+ * @since 2.4
+ * @global array $wp_filter Stores all of the filters
+ *
+ * @param string $tag The name of the filter hook.
+ * @param callback $function_to_check optional.  If specified, return the priority of that function on this hook or false if not attached.
+ * @return int|boolean
+ */
+function has_filter($tag, $function_to_check = false) {
+    global $wp_filter;
+
+    $has = !empty($wp_filter[$tag]);
+    if ( false === $function_to_check || false == $has )
+        return $has;
+
+    if ( !$idx = _wp_filter_build_unique_id($tag, $function_to_check, false, 'filter') )
+        return false;
+
+    foreach ( array_keys($wp_filter[$tag]) as $priority ) {
+        if ( isset($wp_filter[$tag][$priority][$idx]) )
+            return $priority;
+    }
+
+    return false;
+}
+
+/**
  * Call the functions added to a filter hook.
  *
@@ -181 +210 @@
     if ( true === $r) {
         unset($GLOBALS['wp_filter'][$tag][$priority][$function_to_remove]);
+        if ( empty($GLOBALS['wp_filter'][$tag][$priority]) )
+            unset($GLOBALS['wp_filter'][$tag][$priority]);
         unset($GLOBALS['merged_filters'][$tag]);
     }
@@ -326 +357 @@
  * the functions hooked to <tt>$tag</tt> are supplied using an array.
  *
- * @uses merge_filters()
- *
  * @package WordPress
  * @subpackage Plugin
@@ -378 +407 @@
 
 /**
+ * Check if any action has been registered for a hook.  Optionally returns the priority on that hook for the specified function.
+ * @package WordPress
+ * @subpackage Plugin
+ * @since 2.4
+ * @global array $wp_action Stores all of the actions
+ *
+ * @param string $tag The name of the action hook.
+ * @param callback $function_to_check optional.  If specified, return the priority of that function on this hook or false if not attached.
+ * @return int|boolean
+ */
+function has_action($tag, $function_to_check = false) {
+    global $wp_action;
+
+    $has = !empty($wp_action[$tag]);
+    if ( false === $function_to_check || false == $has )
+        return $has;
+
+    if ( !$idx = _wp_filter_build_unique_id($tag, $function_to_check, false, 'action') )
+        return false;
+
+    foreach ( array_keys($wp_action[$tag]) as $priority ) {
+        if ( isset($wp_action[$tag][$priority][$idx]) )
+            return $priority;
+    }
+
+    return false;
+}
+
+/**
  * Removes a function from a specified action hook.
  *
@@ -383 +441 @@
  * method can be used to remove default functions attached to a specific filter
  * hook and possibly replace them with a substitute.
- *
- * @uses remove_filter() Uses remove_filter to remove actions added.
  *
  * @package WordPress
@@ -403 +459 @@
     if ( true === $r) {
         unset($GLOBALS['wp_action'][$tag][$priority][$function_to_remove]);
+        if ( empty($GLOBALS['wp_action'][$tag][$priority]) )
+            unset($GLOBALS['wp_action'][$tag][$priority]);
         unset($GLOBALS['merged_actions'][$tag]);
     }
@@ -514 +572 @@
  * @param string $tag Used in counting how many hooks were applied
  * @param string|array $function Used for creating unique id
- * @param int $priority Used in counting how many hooks were applied
+ * @param int|bool $priority Used in counting how many hooks were applied.  If === false and $function is an object reference, we return the unique id only if it already has one, false otherwise.
  * @param string $type filter or action
  * @return string Unique ID for usage as array key
@@ -529 +587 @@
         $obj_idx = get_class($function[0]).$function[1];
         if ( !isset($function[0]->wp_filter_id) ) {
+            if ( false === $priority )
+                return false;
             if ( 'filter' == $type )
                 $count = count((array)$wp_filter[$tag][$priority]);

trunk/wp-settings.php

@@ -21 +21 @@
 wp_unregister_GLOBALS();
 
-unset( $wp_filter, $cache_lastcommentmodified, $cache_lastpostdate );
+unset( $wp_filter, $wp_action, $cache_lastcommentmodified, $cache_lastpostdate );
 
 if ( ! isset($blog_id) )