more specific fetchers

[quix0rs-gnu-social.git] / actions / password.php

diff --git a/actions/password.php b/actions/password.php
index 4174b6d013c3a6968ef4a2718c7840a304215ff8..8f770a969d57c75ff4220724dd63e6dfaa4c49c6 100644 (file)
--- a/actions/password.php
+++ b/actions/password.php
@@ -34,10 +34,7 @@ class PasswordAction extends SettingsAction {
                common_password('oldpassword', _t('Old password'));
                common_password('newpassword', _t('New password'));
                common_password('confirm', _t('Confirm'));
-               common_element('input', array('name' => 'submit',
-                                                                         'type' => 'submit',
-                                                                         'id' => 'submit',
-                                                                         'value' => _t('Change')));
+               common_submit('submit', _t('Change'));
                common_element_end('form');
                common_show_footer();
        }
@@ -67,6 +64,12 @@ class PasswordAction extends SettingsAction {
 
                $user->password = common_munge_password($newpassword, $user->id);
 
+               $val = $user->validate();
+               if ($val !== TRUE) {
+                       $this->show_form(_t('Error saving user; invalid.'));
+                       return;
+               }
+
                if (!$user->update($original)) {
                        common_server_error(_t('Can\'t save new password.'));
                        return;