Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#13638 closed defect (bug) (fixed)

Do not show a 404 title on wp-activate.php

Reported by: ocean90's profile ocean90 Owned by: wpmuguru's profile wpmuguru
Milestone: 3.0 Priority: low
Severity: normal Version: 3.0
Component: Multisite Keywords: has-patch dev-reviewed
Focuses: Cc:


Reported by mngreg:

I noticed one very small thing while testing: When I clicked the activation link in the e-mail that was sent to me after I signed up, the title of that page it took me too was "Not Found," though everything worked fine and the page content included the confirmation message, etc.

Attachments (4)

13638.patch (362 bytes) - added by ocean90 14 years ago.
13638.2.patch (670 bytes) - added by ocean90 14 years ago.
Set is_home = true to get name+description as title, same for wp-signup.php
13638.3.diff (504 bytes) - added by wpmuguru 14 years ago.
13638.4.diff (2.0 KB) - added by wpmuguru 14 years ago.
teh rewrites for review

Download all attachments as: .zip

Change History (15)

14 years ago

#1 @wpmuguru
14 years ago

  • Owner set to wpmuguru
  • Status changed from new to accepted

#2 @wpmuguru
14 years ago

I'm not able to reproduce the 404 title on activation in either a subdomain or folder install.

#3 @wpmuguru
14 years ago

Occurs when custom permalinks are set.

14 years ago

Set is_home = true to get name+description as title, same for wp-signup.php

#4 @westi
14 years ago

  • Cc westi added
  • Keywords 2nd-opinion added

A better fix for this would be to add some rewrite rules to the internal rewriter so that it matches for the request.

I just tested with the following hacked in in the right place and the direct setting of $wp_query attributes removed.

$robots_rewrite['.*wp-signup.php$'] = $this->index . '?signup=true';
$robots_rewrite['.*wp-activate.php$'] = $this->index . '?activate=true';

That kind of thing does the trick.

We need to decide how we want them to rewrite though.

#5 @wpmuguru
14 years ago

@ocean90: we don't want to be setting $wp_query variables directly like that. Even though we aren't requiring php5, we should observe class abstraction where we can.

14 years ago

#6 @wpmuguru
14 years ago

  • Keywords dev-feedback added; 2nd-opinion removed

13638.3.diff works. Since activation & signup are not looking for posts/pages. I don't see the point of running through the rewrites checking for matches. Are there reasons we wouldn't want to apply that patch?

#7 @nacin
14 years ago

I like westi's approach. 13638.3.diff seems like a patch that could break some very peculiar deep integration use cases and shouldn't go in during an RC.

14 years ago

teh rewrites for review

#8 @westi
14 years ago

  • Keywords dev-reviewed added; dev-feedback removed

That last patch looks great :-)

#9 @wpmuguru
14 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

(In [15081]) add rewrites for registration pages, fixes #13638

#10 @wpmuguru
14 years ago

Quick note for anyone following the ticket - you need to flush your rewrite rules before that change takes effect.

#11 @nacin
14 years ago

I bumped the db version in [15084] to trigger a flush for this.

Note: See TracTickets for help on using tickets.