I have been looking at both versions and think that there is not much difference as far as functionality is concerned. Yon´s version looks a little bit cleaner to me. (it does not try to accept the old_password as a parameter, which seems unneccessary to me and makes things more complicated)