The latest, and recent class methods were not actually used anywhere, nor was the interval local variable in by_interval.
Paired-with: Nick Lewis <n...@puppetlabs.com> Signed-off-by: Jacob Helwig <ja...@puppetlabs.com> --- Local-branch: tickets/next/4403-fix-dst-handling app/models/status.rb | 9 --------- 1 files changed, 0 insertions(+), 9 deletions(-) diff --git a/app/models/status.rb b/app/models/status.rb index f109569..9a8d3c9 100644 --- a/app/models/status.rb +++ b/app/models/status.rb @@ -8,14 +8,6 @@ class Status @start = Time.zone.parse(datum["start"]) end - def self.latest(options={}) - by_interval(options.merge(:limit => 1)).first - end - - def self.recent(options={}) - by_interval options.merge(:start => 1.hour.ago) - end - # Returns an array of Statuses by date for either a :node, or :nodes or all nodes in the system. # # Options: @@ -25,7 +17,6 @@ class Status # * :limit => Limit the number of records to return. def self.by_interval(options={}) return [] if options[:nodes] && options[:nodes].empty? - interval = 1.day # WARNING: This uses the local server time, regardless of what is set in the Rails config. # This should be changed once we have a user-friendly settings file, or can get the browser -- 1.7.4.1 -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to puppet-dev@googlegroups.com. To unsubscribe from this group, send email to puppet-dev+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.