[pLog-svn] r3198 - plog/trunk

[Jon Daley]

 	I don't understand why this change was needed.  The $checks object 
is a separate object than the $checkGroups, so if you modify $checks, 
$checkGroups isn't changed?
 	If that is the case, I want to keep that in my head, because that 
looks like an easy way for me to screw up something.


On Thu, 6 Apr 2006, oscar at devel.lifetype.net wrote:
>
> this was a tricky one... All checks were being unsuccessful in the 
> wizard template because the objects that was being passed in the array 
> were not the same objects that were being instantiated and executed.
>
> Modified: plog/trunk/wizard.php
> ===================================================================
> --- plog/trunk/wizard.php	2006-04-06 11:49:20 UTC (rev 3197)
> +++ plog/trunk/wizard.php	2006-04-06 20:26:24 UTC (rev 3198)
> -            foreach( $checkGroups as $checks ) {
> +            foreach( $checkGroups as $checkGroup => $checks ) {
> 	            foreach( $checks as $id => $check ) {
> -	                $valid = $checks["$id"]->validate();
> +	                $valid = $checkGroups[$checkGroup][$id]->validate();
> 	                // if it doesn't validate but it's not critical, then we can proced too
> -	                if( !$checks["$id"]->isCritical())
> -	                    $valid = true;
> +	                if( !$checkGroups[$checkGroup][$id]->isCritical())
> +	                    $valid = true;
> 	                $ok = ($ok && $valid);