Mailinglist Archive: yast-commit (1535 mails)

[Jiri Suchomel <jsuchome@xxxxxxx>]

ref: refs/heads/master
commit 365258a364eb9fedfab2925bcfbc4487f3aba00b
Author: Jiri Suchomel <jsuchome@xxxxxxx>
Date:   Wed Oct 14 11:33:53 2009 +0200

    some validation in controller as well
---
 .../app/controllers/mail_settings_controller.rb    |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/plugins/mail_settings/app/controllers/mail_settings_controller.rb 
b/plugins/mail_settings/app/controllers/mail_settings_controller.rb
index 43f3bae..4623330 100644
--- a/plugins/mail_settings/app/controllers/mail_settings_controller.rb
+++ b/plugins/mail_settings/app/controllers/mail_settings_controller.rb
@@ -14,7 +14,7 @@ class MailSettingsController < ApplicationController
   def index
     @mail_settings     = load_proxy 
'org.opensuse.yast.modules.yapi.mailsettings'
     return unless @mail_settings
-    @mail_settings.confirm_password    = ""
+    @mail_settings.confirm_password    = @mail_settings.password
   end
 
   # PUT
@@ -24,6 +24,13 @@ class MailSettingsController < ApplicationController
 
     @mail_settings.load params["mail_settings"]
 
+    # validate data also here, if javascript in view is off
+    if @mail_settings.password != @mail_settings.confirm_password
+      flash[:error] = _("Passwords do not match.")
+      redirect_to :action => "index"
+      return 
+    end
+
     begin
       response = @mail_settings.save
       rescue ActiveResource::ClientError => e
-- 
To unsubscribe, e-mail: yast-commit+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: yast-commit+help@xxxxxxxxxxxx