Sidebar menu with Child Page (SubPage) active

PS: You could try get_permalink($id) to quick retreive permalinks of pages.
Reference:

http://codex.wordpress.org/Function_Reference/get_permalink

OK Accordo, I misunderstood some things… 🙂
Here’s a new version (it could be correct now):

<?php
	$pages = get_pages( array(
		    	'sort_order' => 'ASC',
		    	'sort_column' => 'post_title',
		    	'parent' => 0 )
				);
	?>

	<?php
		if ($post->post_parent)
			//I am a subpage
			$id = $post->post_parent;
		else
			//I am a page
			$id = $post->ID;

		$subpages = get_pages(array("child_of"=>$id));
	?>
<ul>
<?php foreach ($pages as $page):?>
	<li>

		<a href="<?php echo get_permalink($page->ID); ?> "><?php echo $page->post_title ?></a>
		<?php
		if ( ($page->ID == $post->ID) || ($post->post_parent == $page->ID) ):	

		?>
		<ul>
			<?php
				foreach($subpages as $subpage):
			?>
			<li>
				<a href="<?php echo get_permalink($subpage->ID); ?> "><?php echo $subpage->post_title ?></a>
			</li>
			<?php endforeach; ?>
		</ul>
		<?php endif; ?>
	</li>

<?php endforeach;?>
</ul>

You could see the right use of get_permalink() API (you’ve missed the ID of post and the echo to print it out 😉 )
Check if it works.
Bye.

Here we are!

Replace the first
<li>

with

<li <?php if ($page->ID == $post->ID) echo 'class="current-page-item"'?>>

and the one inside the second loop

<?php foreach($subpages as $subpage): ?>
<!-- this one above -->
<li>

with

<li <?php if ($subpage->ID == $post->ID) echo 'class="current-page-item"'?>>

This solution doesn’t cover current-page-ancestor and current-page-item hierarchy (to higlight parent and child items togheter), but if you start delegate some parts with custom functions in your functions.php you could cover that and let your code more readable.
See you.

A simple way is like this

<li><?php if ( ($page->ID == $post->ID) || ($post->post_parent == $page->ID) ) echo 'class="current-page-item"'?></li>

for the first <li>, I simply add a check for post parent, when it’s found.
With this you can’t choose a different style for “father” and “son” items,
they will be highlighted with “current-page-item” class, both.
It could be ok?

Different ways are possible if you write a function that discover if you are in a page or in a subpage. I advice you to do that just to keep your code clean, it’s not mandatory.

Try this, then let me know.

OK, try this, simply replace the second

<li <?php if ($subpage->ID == $post->ID) echo 'class="current-page-item"'?>>

with

<li <?php if ($subpage->ID == $post->ID) echo 'class="current-subpage-item"'?>>

Now you have the same structure, but I don’t test it so much, so let me know if it’s ok.
Bye

Hey,

I’ve used this at my website too, and it works perfect… except for two things.

1. I would like the second level (sub-menu) to sort by ID and not alphabetically.
2. The third level on the menu is now included in the second level. I don’t want the third to be displayed at all.

Any ideas? Here’s my code.

<?php
	$pages = get_pages( array(
		    	'sort_order' => 'ASC',
		    	'sort_column' => 'menu_order',
		    	'parent' => 0 )
				);
	?>
	<?php
		if ($post->post_parent)
			//I am a subpage
			$id = $post->post_parent;
		else
			//I am a page
			$id = $post->ID;

		$subpages = get_pages(array("child_of"=>$id));
	?>
<ul id="navcatlist">
<?php foreach ($pages as $page):?>
	<li <?php if ( ($page->ID == $post->ID) || ($post->post_parent == $page->ID) ) echo 'class="current-page-item"'?>>

		<a href="<?php echo get_permalink($page->ID); ?>"><?php echo $page->post_title ?></a>
		<?php
		if ( ($page->ID == $post->ID) || ($post->post_parent == $page->ID) ):	

		?>
		<ul>
			<?php
				foreach($subpages as $subpage):
			?>
			<li <?php if ($subpage->ID == $post->ID) echo 'class="current-subpage-item"'?>>
				<a href="<?php echo get_permalink($subpage->ID); ?>"><?php echo $subpage->post_title ?></a>
			</li>
			<?php endforeach; ?>
		</ul>
		<?php endif; ?>
	</li>

<?php endforeach;?>