Does the redirect not stop the parsing of templates?

[shbchk]

Just ran into this.

{if logged_out}
	{redirect="404"}
{/if}

{if !segment_3}
	{redirect="404"}
{/if}

{if logged_in}
	{exp:query sql="SELECT thing FROM table WHERE something = {segment_3}"}
		{thing}
	{/exp:query}
{/if}

So, when there is no {segment_3} or if I am logged out, EE should redirect me to 404, but instead I'm getting SQL syntax error.

[robinsowell]

So it comes down to the parse order, and whether the variable is parsed before the tag. Segments are very early parsed, so that conditional check will run before the tags. The logged in/out aren't as early- but oddly, logged_in_member_id is parsed earlier, so you could use that and check against 0 for non-logged in.

It's not exactly a bug, but if we can parse the logged in globals that early, we could the logged_in/out ones.