WordPress.org

Make WordPress Core

Ticket #25468: 25468-2.diff

File 25468-2.diff, 2.9 KB (added by ShinichiN, 8 years ago)

Fixes the #25468 patch according to the review.

  • src/wp-includes/bookmark.php

     
    136136        $key = md5( serialize( $r ) );
    137137        if ( $cache = wp_cache_get( 'get_bookmarks', 'bookmark' ) ) {
    138138                if ( is_array($cache) && isset( $cache[ $key ] ) )
    139                         return apply_filters('get_bookmarks', $cache[ $key ], $r );
     139                        $bookmarks = $cache[ $key ];
     140                        /**
     141                        * Filter the returned list of bookmarks.
     142                        *
     143                        * @since 2.1.0
     144                        *
     145                        * @param array $bookmarks List of the cached bookmarks.
     146                        * @param array $r         An array of bookmark query arguments. @see get_bookmarks()
     147                        */
     148                        return apply_filters( 'get_bookmarks', $bookmarks, $r );
    140149        }
    141150
    142151        if ( !is_array($cache) )
     
    181190                } else {
    182191                        $cache[ $key ] = array();
    183192                        wp_cache_set( 'get_bookmarks', $cache, 'bookmark' );
     193                        /**
     194                        * Filter the returned list of bookmarks.
     195                        *
     196                        * Returns a cached bookmarks empty list if the link category is passed but doesn't exist.
     197                        *
     198                        * @since 2.1.0
     199                        *
     200                        * @param array $r An array of bookmark query arguments. @see get_bookmarks()
     201                        */
    184202                        return apply_filters( 'get_bookmarks', array(), $r );
    185203                }
    186204        }
     
    263281        $cache[ $key ] = $results;
    264282        wp_cache_set( 'get_bookmarks', $cache, 'bookmark' );
    265283
     284        /**
     285        * Filter the returned list of bookmarks.
     286        *
     287        * @since 2.1.0
     288        *
     289        * @param array $results The result of the query to retrieve the bookmarks from the database.
     290        * @param array $r       An array of bookmark query arguments. @see get_bookmarks()
     291        */
    266292        return apply_filters('get_bookmarks', $results, $r);
    267293}
    268294
     
    352378                return $value;
    353379
    354380        if ( 'edit' == $context ) {
     381                /**
     382                 * Filter bookmark field value based on $field of bookmark.
     383                 *
     384                 * @since 2.3.0
     385                 *
     386                 * @param mixed $value       The bookmark field value.
     387                 * @param int   $bookmark_id Bookmark ID.
     388                 */
    355389                $value = apply_filters("edit_$field", $value, $bookmark_id);
    356390
    357391                if ( 'link_notes' == $field ) {
     
    360394                        $value = esc_attr($value);
    361395                }
    362396        } else if ( 'db' == $context ) {
     397                /**
     398                 * Filter bookmark field value based on $field of bookmark.
     399                 *
     400                 * @since 2.3.0
     401                 *
     402                 * @param mixed $value The bookmark field value.
     403                 */
    363404                $value = apply_filters("pre_$field", $value);
    364405        } else {
    365                 // Use display filters by default.
     406                /**
     407                 * Filter bookmark field value.
     408                 *
     409                 * Use display filters by default.
     410                 *
     411                 * @since 2.3.0
     412                 *
     413                 * @param string $field       The bookmark field.
     414                 * @param mixed  $value       The bookmark field value.
     415                 * @param int    $bookmark_id The bookmark ID.
     416                 * @param string $context     How to filter the field value. Either 'attribute', 'js' or 'display'.
     417                 */             
    366418                $value = apply_filters($field, $value, $bookmark_id, $context);
    367419
    368420                if ( 'attribute' == $context )