Checking if a user is logged in, if so redirect to a different page

Question

I'm making a member-only site that requires users to be logged in to use the site.

The homepage is publicly viewable. On the homepage, I'd like to put in a function to redirect users to a page inside the site if they are logged in.

The code I tried

<?php if (is_user_logged_in() ) {
 wp_redirect ('dashboard' );
 exit;
 }
?>

Gives me a "headers already sent" error. The code is placed at the top of my php file.

What am I doing wrong?

Rarst · Answer 1 · 2011-07-24 17:02:16Z

up vote 1 down vote

I am not sure if you just simplified for example, but make sure you use full URL in wp_redirect() call.

Also rather than having it in template, I think it would be more appropriate to run this at template_redirect hook.

answered Jul 24 '11 at 17:02

Rarst♦
41.5k230116

	My solution is `wp_redirect ( home_url("/dashboard") );`, and I've put this code on line 1 of the template. Although I'm not familiar with hooks so you probably have a valid point. Why? – javy Jul 24 '11 at 18:36
	@javy `template_redirect` is semantically place in code like "WP knows what it needs to start loading and outputting, but you can change its mind". Start of template is "we are already loading this thing". If you don't want what's in template there is no reason to get that far in the process. – Rarst♦ Jul 24 '11 at 18:53
	Makes sense, sounds like it saves some server utilization? – javy Jul 25 '11 at 0:14
	@javy I don't think difference in performance would be meaningful between too. – Rarst♦ Jul 25 '11 at 7:28

javy · Answer 2 · 2011-07-25 00:13:41Z

<?php if (is_user_logged_in() ) {
wp_redirect ( home_url("/dashboard") );
exit;
}
?>

Is what I've come up with, and works. Also, I think I also messed up by putting this below

<?php
/*Template Name: Home*/
?>

The new code is on line 1 and works.

2 Answers