Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#21138 closed defect (bug) (fixed)

Comments rewrite contains bogus rule

Reported by: nacin Owned by: nacin
Priority: normal Milestone: 3.6
Component: Rewrite Rules Version:
Severity: normal Keywords: has-patch commit 3.6-early
Cc: Focuses:

Description (last modified by nacin)

[comments/feed/(feed|rdf|rss|rss2|atom)/?$] =>
    index.php?&feed=$matches[1]&withcomments=1

[comments/(feed|rdf|rss|rss2|atom)/?$] =>
    index.php?&feed=$matches[1]&withcomments=1

[comments/page/?([0-9]{1,})/?$] =>
    index.php?&paged=$matches[1]

That last rule doesn't make any sense. We should call generate_rewrite_rules() with $paged = false here.

Attachments (1)

comments-rewrite.diff​ (652 bytes ) - added by wonderboymusic 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @nacin
14 years ago

  • Description modified (diff)

I found this when porting over the test_query.php unit tests:

// 'comments/page/?([0-9]{1,})/?$' => 'index.php?&paged=$matches[1]',
function test_comments_page() {
       $this->http('/comments/page/2/');
       $this->assertQueryTrue('is_home', 'is_paged');
}

#2 @SergeyBiryukov
14 years ago

Introduced in [1910].

#3 @wonderboymusic
14 years ago

  • Keywords has-patch added

#4 @nacin
14 years ago

  • Keywords commit 3.6-early added
  • Milestone 3.5 → Future Release

#5 @helen
14 years ago

  • Milestone Future Release → 3.6

#6 @nacin
14 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 23524:

Remove bogus pagination rules from the comments permastruct. (As in, comments/page/?([0-9]{1,})/?$.) props wonderboymusic, fixes #21138.

Note: See TracTickets for help on using tickets.