ref: refs/heads/master
commit 2361e77a782134d3351bfe5599f8b31ee8401da2
Author: Jiří Šrain
Date: Fri Jul 24 14:54:33 2009 +0200
fixed changing time zone region
---
.../app/controllers/system_time_controller.rb | 16 +++++++++++++++-
1 files changed, 15 insertions(+), 1 deletions(-)
diff --git a/plugins/systemtime/app/controllers/system_time_controller.rb b/plugins/systemtime/app/controllers/system_time_controller.rb
index 094c573..d2f215f 100644
--- a/plugins/systemtime/app/controllers/system_time_controller.rb
+++ b/plugins/systemtime/app/controllers/system_time_controller.rb
@@ -140,12 +140,26 @@ class SystemTimeController < ApplicationController
#AJAX function that renders new timezones for selected region. Expected
# initialized values from index call.
def timezones_for_region
- region = ""
+ # since while calling this function there is different instance of the class
+ # than when calling index, @timezones were empty; reinitialize them
+ # possible FIXME: how does it increase the amount of data transferred?
+ systemtime = load_proxy 'org.opensuse.yast.modules.yapi.time'
+
+ unless systemtime
+ return false #possible FIXME: is returnign false for AJAX correct?
+ end
+
+ @timezones = systemtime.timezones
+
+ region = "" #possible FIXME later it gets class, not a string
@timezones.each do |r|
if r.name == params[:value]
region = r
end
end
+ if region == ""
+ return false; #possible FIXME: is returnign false for AJAX correct?
+ end
render(:partial => 'timezones',
:locals => {:region => region, :default => region.central,
:disabled => ! params[:disabled]=="true"})
--
To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org
For additional commands, e-mail: yast-commit+help@opensuse.org
