From f715a419196c65ee0cc3b7d9e346aa2512c0cd21 Mon Sep 17 00:00:00 2001 From: Saku Ytti Date: Sun, 21 Apr 2013 17:08:37 +0300 Subject: rename 'update' to 'store' 'store' is more logical, as we cannot know if output method guarantees any version history which 'update' implies. --- README.md | 2 +- lib/oxidized/output/file.rb | 2 +- lib/oxidized/output/git.rb | 2 +- lib/oxidized/worker.rb | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/README.md b/README.md index ca7df43..c42b42c 100644 --- a/README.md +++ b/README.md @@ -17,7 +17,7 @@ ## Output * stores config - * must implement 'update' + * must implement 'store' * 'git' and 'file' (store as flat ascii) implemented ## Source diff --git a/lib/oxidized/output/file.rb b/lib/oxidized/output/file.rb index b988c1a..66ad178 100644 --- a/lib/oxidized/output/file.rb +++ b/lib/oxidized/output/file.rb @@ -15,7 +15,7 @@ class OxFile < Output end end - def update node, data, opt={} + def store node, data, opt={} file = @cfg[:directory] if opt[:group] file = File.join File.dirname(file), opt[:group] diff --git a/lib/oxidized/output/git.rb b/lib/oxidized/output/git.rb index 019d83b..202893c 100644 --- a/lib/oxidized/output/git.rb +++ b/lib/oxidized/output/git.rb @@ -19,7 +19,7 @@ class Git < Output end end - def update file, data, opt={} + def store file, data, opt={} msg = opt[:msg] user = (opt[:user] or @cfg[:user]) email = (opt[:email] or @cfg[:email]) diff --git a/lib/oxidized/worker.rb b/lib/oxidized/worker.rb index 6966322..14cf2a5 100644 --- a/lib/oxidized/worker.rb +++ b/lib/oxidized/worker.rb @@ -26,8 +26,8 @@ module Oxidized msg = "update #{node.name}" msg += " from #{node.from}" if node.from msg += " with message '#{node.msg}'" if node.msg - node.output.new.update node.name, job.config, - :msg => msg, :user => node.user, :group => node.group + node.output.new.store node.name, job.config, + :msg => msg, :user => node.user, :group => node.group node.reset else Log.warn "#{node.name} status #{job.status}" -- cgit v1.2.1