[wp-trac] [WordPress Trac] #14921: WP_Dependencies is tagged as external but is core package
WordPress Trac
wp-trac at lists.automattic.com
Tue Sep 21 12:11:09 UTC 2010
#14921: WP_Dependencies is tagged as external but is core package
--------------------------+-------------------------------------------------
Reporter: hakre | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: General | Version: 3.0
Severity: normal | Keywords:
--------------------------+-------------------------------------------------
While creating some runtime and static analyzer tools I ran over a problem
rating files properly as core-files with the following files:
1. wp-includes/class.wp-dependencies.php
2. wp-includes/class.wp-scripts.php
3. wp-includes/class.wp-styles.php
4. wp-includes/functions.wp-scripts.php
5. wp-includes/functions.wp-styles.php
Those files are tagged as being package "!BackPress". I suggest to remove
this external constraint. The foremost reason is this code has been
contributed to and is developed in !WordPress core, so it's not an
external but a core file.
Core files are tagged as package "!WordPress" in the project.
The current @package value mislabels them as being external files
(BackPress backports WordPress and other projects' code into a library of
it's own).
As new subpackage value I suggest "Dependencies". Additionally the @since
tagging of those files / classes / functions can be improved.
Most of the code has been contributed in
[http://core.trac.wordpress.org/attachment/ticket/7011/7011.diff
7011.diff] by mdawaffe.
The current tagging has been contributed in
[http://core.trac.wordpress.org/attachment/ticket/7550/7550.r8991.2.diff
7550.r8991.2.diff] and committed in [9002] (Related: #7550).
--
Ticket URL: <http://core.trac.wordpress.org/ticket/14921>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list