why doesnt this code work?

[insanityonline]

ok im creating a blog on my page...can some1 help me find out why when ever i go to "/?id=dailyblog&archive=july&week=2" why it goes to the "/?id=dailyblog&archive=july" page??

<?php 

// *** months *** // 

if($_GET['id'] == 'dailyblog' && $_GET['year'] == '2004') 
{ include("blog/2004.php"); } 

elseif($_GET['id'] == 'dailyblog' && $_GET['archive'] == 'july') 
{ include("blog/july.php"); } 

elseif($_GET['id'] == 'dailyblog' && $_GET['archive'] == 'august') 
{ include("blog/august.php"); } 

// *** weeks *** // 

elseif($_GET['id'] == 'dailyblog' && $_GET['archive'] == 'july' && $_GET['week'] == '2')   

{ include("blog/july/4_10.php"); } 

elseif($_GET['id'] == 'dailyblog' && $_GET['archive'] == 'july' && $_GET['week'] == '3') 
{ include("blog/july/11_17.php"); } 

elseif($_GET['id'] == 'dailyblog' && $_GET['archive'] == 'july' && $_GET['week'] == '4') 
{ include("blog/july/18_24.php"); } 

elseif($_GET['id'] == 'dailyblog' && $_GET['archive'] == 'july' && $_GET['week'] == '5') 
{ include("blog/july/25-31.php"); } 

// *** default *** // 

else { include("blog/home.php"); } 

?> 

[sumathi]

Your problem may be because month condition is checked before the week checking. so please change the order of your condition checking...

try this...

<?

if($GET['id'] == 'dailyblog' && $GET['year'] == '2004')

{ include("blog/2004.php"); }

// weeks //

elseif($GET['id'] == 'dailyblog' && $GET['archive'] == 'july' && $_GET['week'] == '2')

{ include("blog/july/4_10.php"); }

elseif($GET['id'] == 'dailyblog' && $GET['archive'] == 'july' && $_GET['week'] == '3')

{ include("blog/july/11_17.php"); }

elseif($GET['id'] == 'dailyblog' && $GET['archive'] == 'july' && $_GET['week'] == '4')

{ include("blog/july/18_24.php"); }

elseif($GET['id'] == 'dailyblog' && $GET['archive'] == 'july' && $_GET['week'] == '5')

{ include("blog/july/25-31.php"); }

// months //

if($GET['id'] == 'dailyblog' && $GET['year'] == '2004')

{ include("blog/2004.php"); }

elseif($GET['id'] == 'dailyblog' && $GET['archive'] == 'july')

{ include("blog/july.php"); }

elseif($GET['id'] == 'dailyblog' && $GET['archive'] == 'august')

{ include("blog/august.php"); }

// default //

else { include("blog/home.php"); }

?>

[laserlight]

You might want to put it in a more structured form, e.g.

<?php
if ($_GET['id'] == 'dailyblog') {
	switch ($_GET['archive']) {
	case 'july':
		switch ($_GET['week']) {
		case '2':
			include "blog/july/4_10.php";
			break;
		case '3':
			include "blog/july/11_17.php";
			break;
		case '4':
			include "blog/july/18_24.php";
			break;
		case '5':
			include "blog/july/25-31.php";
			break;
		default:
			include "blog/july.php";
		}
		break;
	case 'august':
		include "blog/august.php";
		break;
	default:
		if ($_GET['year'] == '2004') {
			include "blog/2004.php";
		} else {
			include "blog/home.php";
		}
	}
} else {
	include "blog/home.php";
}
?>