Anonymize BP Core Dependency functions file.

[tw2113]

When JJJ first wrote up the bp-core-dependency.php file, he left a note at the top regarding utilizing PHP 5.3's anonymous function capabilities for the file.

This ticket is meant to help move that to a reality. We will need to keep in mind anyone who may be calling the functions directly, and help them out the best we can.

[tw2113]

Add deprecated function notices to functions in the BP Core Dependency declarations.

[tw2113]

Done all the way to right before bp_ready() with the attached diff file.

[tw2113]

The bp_allowed_themes() function is an odd one, in that it's present, and not hooked into anything, and doesn't appear to be used in core anywhere.

Updated diff pending, once I have a chance to review changes.

[tw2113]

Added a more complete anonymized actions/filters patch, as well as a patch that deprecates all the old functions.

Left bp_allowed_themes() alone like noted earlier.

[DJPaul]

Looks good but it makes these files much longer and adds functions in a file that was originally designed to not really have (m)any functions in. The priorities are also a little harder to read this way. Would be interested in opinions from other people.

Maybe assign anonymous functions to variables and use those to replace the add_action lines, keeping them short and tidy. Maybe this is just me over-reacting to a niggle.

[tw2113]

More than willing to get some second opinions on this one as well.

[tw2113]

For an example highlighting one of Paul's thoughts above, the variable storage would look like this:

<?php
$bp_ready = function() {
        /**
         * Fires on the 'wp' hook, which runs after BP is set up and the page is about to render.
         *
         * @since 1.6.0
         */
        do_action( 'bp_ready' );
};

...

add_action( 'wp', $bp_ready, 10 );

[boonebgorges]

What is the benefit of switching to anonymous functions, especially if we're keeping the old ones around?

[r-a-y]

Yeah, I do not see much of a point to moving to anonymous functions here. Unless there is a gigantic performance boost!

Also, it's harder to backtrace anonymous functions.

[tw2113]

It was pretty much just a response to the note from JJJ in the file. If you'd prefer not proceed with this one and wontfix, no hard feelings.

[tw2113]

After discussion in the dev chat on 2016-12-21, we've decided to mark as wontfix for the time being.

[boonebgorges]

In 11364:

Remove @todo related to migrating bp-core-dependency functions to closures.

See #7299. See #7335.