[wp-trac] [WordPress Trac] #13058: Feed link not working with non-category taxonomy in wp_list_categories()
WordPress Trac
wp-trac at lists.automattic.com
Sun Apr 25 08:55:43 UTC 2010
#13058: Feed link not working with non-category taxonomy in wp_list_categories()
--------------------------+-------------------------------------------------
Reporter: greenshady | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.0
Component: Feeds | Version: 3.0
Severity: normal | Keywords: needs-patch
--------------------------+-------------------------------------------------
Changes (by dd32):
* keywords: has-patch needs-testing => needs-patch
Comment:
Patch review:
* xmlrpc.php change is not needed, get_category_link() doesnt have a 3rd
arg.
* Should check if the taxonomy has rewrite enabled, ie. get the taxonomy
permastruct, not the general permastruct, Its possible to register a
taxonomy which does not use the inbuilt rules, those taxonomies will need
to filter this function if they have a special structure.
* @package isnt needed for function-level documentation, leave that for
the file-level documentation
* Coding standards:
{{{
if ( Something )
// do something
else
// do something else
}}}
* Spaces around language structs (if)
* Spaces around conditionals
* Skip the {'s if its only a single line
* Spaces around concatenation ( {{{something".$var."=}}} vs
{{{something" . $var . "=}}}
Patch looks good otherwise, most of those are things i'd change upon
commit, but thought i'd give a review since the rewrite code will need
updating.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13058#comment:3>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list