View Options

	diff --git a/lib/puppet/provider/file_line/ruby.rb b/lib/puppet/provider/file_line/ruby.rb
	index e21eaa8..a3219d3 100644
	--- a/lib/puppet/provider/file_line/ruby.rb
	+++ b/lib/puppet/provider/file_line/ruby.rb
	@@ -1,59 +1,59 @@

	Puppet::Type.type(:file_line).provide(:ruby) do

	def exists?
	lines.find do \|line\|
	line.chomp == resource[:line].chomp
	end
	end

	def create
	if resource[:match]
	handle_create_with_match()
	else
	handle_create_without_match()
	end
	end

	def destroy
	local_lines = lines
	File.open(resource[:path],'w') do \|fh\|
	fh.write(local_lines.reject{\|l\| l.chomp == resource[:line] }.join(''))
	end
	end

	private
	def lines
	# If this type is ever used with very large files, we should
	# write this in a different way, using a temp
	# file; for now assuming that this type is only used on
	# small-ish config files that can fit into memory without
	# too much trouble.
	@lines \|\|= File.readlines(resource[:path])
	end

	def handle_create_with_match()
	regex = resource[:match] ? Regexp.new(resource[:match]) : nil
	- match_count = lines.select { \|l\| regex.match(l) }.count
	+ match_count = lines.select { \|l\| regex.match(l) }.size
	if match_count > 1
	raise Puppet::Error, "More than one line in file '#{resource[:path]}' matches pattern '#{resource[:match]}'"
	end
	File.open(resource[:path], 'w') do \|fh\|
	lines.each do \|l\|
	fh.puts(regex.match(l) ? resource[:line] : l)
	end

	if (match_count == 0)
	fh.puts(resource[:line])
	end
	end
	end

	def handle_create_without_match
	File.open(resource[:path], 'a') do \|fh\|
	fh.puts resource[:line]
	end
	end


	end

File Metadata