anyway: function isValidPassword( $password ) { global $wgMinimalPasswordLength, $wgContLang;
$result = null; if( !wfRunHooks( 'isValidPassword', array( $password, &$result,$this ) ) ) return $result;
should be: function isValidPassword( $password ) { global $wgMinimalPasswordLength, $wgContLang;
$result = null; if( !wfRunHooks( 'isValidPassword', array( $password, &$result,$this ) ) ) return $result;
I don't see any difference...
However, I think it is correct. Hooks should return true if the code should keep going and run other hooks and the rest of the standard code, and false if it should stop there.