Steve Linabery
2009-Mar-10 16:03 UTC
[Ovirt-devel] [PATCH server] Fix timezone offset problem in graph_controller.rb
Ruby Time.at(seconds_since_epoch) creates a localized Time object. Adding utc method converts to UTC for purposes of querying database. --- src/app/controllers/graph_controller.rb | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/app/controllers/graph_controller.rb b/src/app/controllers/graph_controller.rb index 15fa572..06f1c06 100644 --- a/src/app/controllers/graph_controller.rb +++ b/src/app/controllers/graph_controller.rb @@ -7,8 +7,8 @@ class GraphController < ApplicationController def flexchart_data @id = params[:id] target = params[:target] - startTime = Time.at(params[:startTime].to_i) - endTime = Time.at(params[:endTime].to_i) + startTime = Time.at(params[:startTime].to_i).utc + endTime = Time.at(params[:endTime].to_i).utc duration = endTime.to_i - startTime.to_i dataFunction = DEV_KEY_FUNCTIONS[params[:dataFunction]] ? DEV_KEY_FUNCTIONS[params[:dataFunction]] : DEV_KEY_FUNCTIONS['peak'] #the maximum number of data points we want in any chart @@ -77,7 +77,7 @@ class GraphController < ApplicationController #expect (pool) id, resolution, startTime, target, dataFunction @id = params[:id] resolution = params[:resolution].to_i - startTime = Time.at(params[:startTime].to_i) + startTime = Time.at(params[:startTime].to_i).utc target = params[:target] dataFunction = params[:dataFunction] @@ -95,7 +95,7 @@ class GraphController < ApplicationController eventsInRange = host.membership_audit_events.from_pool(pool, startTime, - Time.at(startTime.to_i + resolution)) + Time.at(startTime.to_i + resolution).utc) excluded = false if eventsInRange.empty? priorAuditEvent = host.membership_audit_events.most_recent_prior_event_from_pool(pool,startTime) -- 1.6.0.6
Scott Seago
2009-Mar-10 16:45 UTC
[Ovirt-devel] [PATCH server] Fix timezone offset problem in graph_controller.rb
Steve Linabery wrote:> Ruby Time.at(seconds_since_epoch) creates a localized Time object. Adding > utc method converts to UTC for purposes of querying database. >Seems to work for me now. ACK Scott