advice on pagination approach

// scroll way down this post to see the new version.
// this I have taken out for the sake of brevity, 
// and so as not to be a bad influence on novic-er novices than me.
// seriously, if you copied any of that, start over - it shouldn't have been posted :(

[[!paginate]]

$resources = $modx->getCollection('modResource', array('parent' => $parentId));

$parent = $modx->resource->getOne('Parent');
if ($parent) {
    $resources = $parent->getMany('Children');    
}

<?php
// need the parent id
$parent = $modx->resource->get('parent');
// make an array to fill with kids
$ids=array();
// need the current id
$id = $modx->resource->get('id');
// make sure that offset is getted, or make it 0 
isset($_GET['oft'])?$oft = $_GET['oft']:$oft=0;

$children = $modx->getChildIds($parent);
foreach($children as $key=>$val)
	array_push($ids,$val);
// get the current array position
$cur = array_search($id,$ids);
// give me a clue (temporary)
echo '<h4>'.$oft.'</h4>';
// $_GET nasty with big ugly links + query strings
/* unless Bob Ray has another pearl of wisdom... :)
*/
return '<ul><li><a href="[[~'.$ids[$cur-1].']]&oft='.($cur-1).'">PREV</a><a href="[[~'.$ids[$cur+1].']]&oft='.($cur+1).'">NEXT</a></li></ul>';

$oft = isset($_GET['oft']) ? $_GET['oft'] : 0;

<?php
// need the parent id
$parent = $modx->resource->get('parent');

// set the template
$tpl = isset($tpl)?$tpl:'Tpl.siblingNav';

// make an array to fill with kids
$ids=array();

// init the output
$output = '';

// need the current id
$id = $modx->resource->get('id');

// load up the kids
$children = $modx->getChildIds($parent);

// get the current array position
$current = array_search($id,$children);

// make $output array
$output_array=array(
	'count'	=>	count($children),
	'current'	=>	$current,
	'prev'		=>	$children[$current-1],
	'next'		=>	$children[$current+1],
); 

// output = chunk
if(count($children)>1){
	$output .= $modx->getChunk($tpl,$output_array);
};

// return output
return $output;

<div class="pageNav">
<!-- class 'pageNav' for consistency with GetPage template -->
	<ul>
		[[+current:gt=`0`:then=`
		<li>
			<a href="[[~[[+prev]]]]">Previous</a>
		</li>
		`:default=``]]
		<li>
			<small>
				[[+current:increment]] of [[+count]]
			</small>
		</li>
		[[+current:lt=`[[+count:decrement]]`:then=`
		<li>
			<a href="[[~[[+next]]]]">Next</a>
		</li>
		`:default=``]]
</ul>