Changeset 10741

Timestamp:

03/07/2009 09:34:01 PM (12 years ago)

Author:

azaozz

Message:

feed_links_extra() DRY, props scribu, fixes #9291

File:

• trunk/wp-includes/general-template.php (modified) (1 diff)

trunk/wp-includes/general-template.php

@@ -1445 +1445 @@
     if ( is_single() || is_page() ) {
         $post = &get_post( $id = 0 );
-        if ( comments_open() || pings_open() || $post->comment_count > 0 )
-            echo '<link rel="alternate" type="' . feed_content_type() . '" title="' . attribute_escape(sprintf( $args['singletitle'], get_bloginfo('name'), $args['seperator'], wp_specialchars( get_the_title() ) )) . '" href="' . get_post_comments_feed_link( $post->ID ) . "\" />\n";
-    }
-
-    elseif ( is_category() ) {
+
+        if ( comments_open() || pings_open() || $post->comment_count > 0 ) {
+            $title = attribute_escape(sprintf( $args['singletitle'], get_bloginfo('name'), $args['seperator'], wp_specialchars( get_the_title() ) ));
+            $href = get_post_comments_feed_link( $post->ID );
+        }
+    } elseif ( is_category() ) {
         $cat_id = intval( get_query_var('cat') );
-        echo '<link rel="alternate" type="' . feed_content_type() . '" title="' . attribute_escape(sprintf( $args['cattitle'], get_bloginfo('name'), $args['seperator'], get_cat_name( $cat_id ) )) . '" href="' . get_category_feed_link( $cat_id ) . "\" />\n";
-    }
-
-    elseif ( is_tag() ) {
+
+        $title = attribute_escape(sprintf( $args['cattitle'], get_bloginfo('name'), $args['seperator'], get_cat_name( $cat_id ) ));
+        $href = get_category_feed_link( $cat_id );
+    } elseif ( is_tag() ) {
         $tag_id = intval( get_query_var('tag_id') );
         $tag = get_tag( $tag_id );
-        echo '<link rel="alternate" type="' . feed_content_type() . '" title="' . attribute_escape(sprintf( $args['tagtitle'], get_bloginfo('name'), $args['seperator'], $tag->name )) . '" href="' . get_tag_feed_link( $tag_id ) . "\" />\n";
-    }
-
-    elseif ( is_author() ) {
+
+        $title = attribute_escape(sprintf( $args['tagtitle'], get_bloginfo('name'), $args['seperator'], $tag->name ));
+        $href = get_tag_feed_link( $tag_id );
+    } elseif ( is_author() ) {
         $author_id = intval( get_query_var('author') );
-        echo '<link rel="alternate" type="' . feed_content_type() . '" title="' . attribute_escape(sprintf( $args['authortitle'], get_bloginfo('name'), $args['seperator'], get_author_name( $author_id ) )) . '" href="' . get_author_feed_link( $author_id ) . "\" />\n";
-    }
-
-    elseif ( is_search() ) {
-        echo '<link rel="alternate" type="' . feed_content_type() . '" title="' . attribute_escape(sprintf( $args['searchtitle'], get_bloginfo('name'), $args['seperator'], get_search_query() )) . '" href="' . get_search_feed_link() . "\" />\n";
-    }
+
+        $title = attribute_escape(sprintf( $args['authortitle'], get_bloginfo('name'), $args['seperator'], get_author_name( $author_id ) ));
+        $href = get_author_feed_link( $author_id );
+    } elseif ( is_search() ) {
+        $title = attribute_escape(sprintf( $args['searchtitle'], get_bloginfo('name'), $args['seperator'], get_search_query() ));
+        $href = get_search_feed_link();
+    }
+
+    if ( isset($title) && isset($href) )
+        echo '<link rel="alternate" type="' . feed_content_type() . '" title="' . $title . '" href="' . $href . '" />' . "\n";
 }