Dead code in feed_links_extra()

[dmenard]

Probably a detail, but in general-template.php, function feed_links_extra() contains two elseif statements with the same condition if ( is_post_type_archive() ).

The second one can't be reached and can be considered as dead code.

[desrosj]

Thanks for the ticket, @dmenard! And welcome to Trac.

Good find. Looks like the first conditional was introduced in r25291. I think we can remove the second conditional, but we need to make sure something was not left behind in the new conditional.

Are you interested in creating a patch for this?

[dmenard]

Remove duplicate elseif clause in feed_links_extra()

[dmenard]

Hello @desrosj,

Thanks for the kind words!

we need to make sure something was not left behind in the new conditional

There are some differences between the old clause and the new one:

• post type object: the old clause was using get_queried_object() to get the post type object. The new clause uses get_query_var( 'post_type' ) and keeps only the first value if there are many ones.
• feed title: the old clause was using post_type_archive_title() (which can be filtered) to construct the feed title. The new clause calls directly $post_type_obj->labels->name.
• return value: the old clause was creating a feed link only if we got a valid queried object. There are no checks in the new clause: we use directly the query var, call get_post_type_object() (which can return null) and always generate a feed by calling get_post_type_archive_feed_link( $post_type_obj->name ) (will generate a warning and an empty href if the post type is not valid). But perhaps the post_type query var is validated elsewhere during the query parsing?

So, it's not the same...

However, if the modification was introduced in r25291 as you found, it means that the old code has not been executed for more than five years. So I presume the new code is ok?..

Are you interested in creating a patch for this?

For now, I've added a diff attachment wich removes the duplicate elseif clause in feed_links_extra() and does nothing else.

As I haven't used svn for years, this is a git diff (I followed the guidelines of ​the handbook).

I hope it's ok, please let me know.

[desrosj]

#44071 was marked as a duplicate.

[Ruxton]

There's an issue with having it so far up the if/else stack as the new code (and therefore the proposed patches) has put it.

If you're in a post type archive & are filtering by taxon, it'll fire the post type archive first. I'd expect taxon to come first in that situation.

Though rather than moving them around, perhaps making the the result filterable would be a better solution?

[pento]

This patch needs further investigation and review.