Make WordPress Core

Opened 15 years ago

Closed 14 years ago

#13211 closed defect (bug) (duplicate)

MultiSite and wp-signup.php no redirect with IE browser

Reported by: frumph's profile Frumph Owned by:
Milestone: Priority: normal
Severity: critical Version: 3.0.1
Component: Multisite Keywords: IE
Focuses: Cc:

Description

wp-signup.php line 30-33

if ( !is_main_site() ) {
	wp_redirect( "http://" . $current_site->domain . $current_site->path . "wp-signup.php" );
	die();
}

Should the above code be redirecting to the same site and not the main blog site?

Currently on subsites if using the donncha domain mapper or the one from ron wpmugur its redirectiong to $current and should be redirectiong to main, because sub sites do not allow registering on.

Change History (15)

#1 @ocean90
15 years ago

  • Component changed from General to Multisite

We should use network_home_url( 'wp-signup.php' ). See #13087.

#2 @Frumph
15 years ago

Tracing my 404 error for why mapped domains are not finding the wp-signup.php there is more to this then just that above. It possibly could either be an IIS7 rewrite rule or something more sinister. Still tracing it, but hardcoding above to the proper url did not fix the mappeddomain.tld/wp-signup.php from coming up as a 404.

mappeddomain.tld/wp-register.php is redirecting to mappeddomain.tld/wp-signup.php as well, wp-register.php is being found on mapped domains however, wp-signup.php is not.

#3 @Frumph
15 years ago

... Does not redirect on IE, redirects with FF, with IE returns a 404 error. I thought I saw a previous ticket for this but cannot currently find it.

#4 @Frumph
15 years ago

  • Summary changed from MultiSite and wp-signup.php redirect from sub sites. to MultiSite and wp-signup.php no redirect with IE browser

#5 @Frumph
15 years ago

If I comment out those lines for the if not mainsite then everything directs fine, tested creating accounts, on domain mapped sites, works. tested on non-mapped sub sites (subdomain style) works.

So basically those 3 lines do not need to be there at all.

#6 @Frumph
15 years ago

I take that back, it's not adding the user to the subsite they registered on, just on the main site.

Although that fixed the 404, it's not adding the user to the site they registered with. If i'm hosting on my multisite other sites that I want to be distinct, it has to add the user to the site they registered on.

#7 @ocean90
15 years ago

  • Keywords reporter-feedback added
  • Version set to 3.0

Still exists?

#8 @Frumph
15 years ago

  • Resolution set to duplicate
  • Status changed from new to closed

12758 This is fixed as a side effect of this ticket.

#9 @Frumph
15 years ago

  • Keywords reporter-feedback removed
  • Version 3.0 deleted

#10 @otuyelu
14 years ago

  • Cc otuyelu added
  • Keywords IE added
  • Resolution duplicate deleted
  • Severity changed from normal to critical
  • Status changed from closed to reopened
  • Version set to 3.0.1

Please reopen and fix!
Just converted a site to multisite with Wordpress 3.0 and the redirect for wp-signup.php fails in IE only with a 404. Redirect to wp-signup.php works when signed-in on IE, but if user can sign-in, they do not need register. Need register to work for users not logged-in.

Redirect for wp-signup.php works on other browsers I tested.

#11 @otuyelu
14 years ago

by the way I checked ticket #12758 It does not resolve this issue, as the issue of wp-signup.php not redirecting for IE browser only, is not addressed in that ticket

#12 @Frumph
14 years ago

Are you using NOBLOGREDIRECT by chance?

Subdomain or subdirectory MS?

#13 @otuyelu
14 years ago

  • Cc otuyelu removed

subdirectory multisite, and NOBLOGREDIRECT not set in wp-config.php

#14 @otuyelu
14 years ago

Frumph thanks for the replies. I just tested it and it all works now, which may mean the system that generates the redirect URL that is compatible with IE is not being triggered during the course of conversion to multi-site.
Continued regular use of the site over several hours, somehow triggered the recreation of the URLs.

#15 @nacin
14 years ago

  • Resolution set to duplicate
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.