Enable pretty permalinks for comment paging

[Viper007Bond]

Suggested format: .../comment-page-1/ (matches plugin and is nice looking IMO)

If someone doesn't beat me to it, I can probably handle it.

[markjaquith]

Go for it. Agree on using that format to match the plugin.

[DD32]

It'd be nice to support comment paging on attachments too(Just incase someone else does it, and only does it for Posts|Pages), I went to implement this ticket.. However, Got confused quickly in WP_Rewrite::generate_rewrite_rules() as for how its doing its black magic :)

[DD32]

attachment 7492.diff added.

• ../comment-page-xx/ support for pages/posts/attachments (anything with comments)
• canonical redirection for ?cpage=xx to comment-page-xx
• comment pagination links use pretty links if enabled
• Yes, This patch is for this ticket, even though i managed to muddle the ticket number up in the filename..

[ryan]

(In [9300]) pretty permalinks for comment paging. Props Viper007Bind. fixes #7942

[ryan]

Duh, propped the wrong person and even spelt that wrong.

[ryan]

previous_comments_link() and next_comments_link() also need to be updated.

What do we do if both the post and comments are paged? Paging comments while on page 2 of the post should preserve the post page 2 in the links?

[DD32]

What do we do if both the post and comments are paged? Paging comments while on page 2 of the post should preserve the post page 2 in the links?

Another Rewrite rule will need to be added to the Paged rules for that, And some funky coding will need to happen inside the function to get the right orderings of the page/xx/comment-page-xx/ i believe

[DD32]

attachment 7942.diff added.

• (previous|next)_comments_link() converted to permalinks
• It'd be nice to condense that If branch together, But i'm not 100% on the logic behind it, So i've left it as 2 branches.

[Viper007Bond]

Bug fix of DD32's patch for when newest page is showed by default

[markjaquith]

(In [9318]) Enable pretty permalinks for previous_ and next_comments_link(). props DD32, Viper007Bond. see #7942

[DD32]

redirect post-name/comment-page-xx/ => post-name/ when paging disabled

[markjaquith]

(In [9322]) Redirect away from paged comments permalinks when comment paging has been turned off. props DD32. see #7942