Merge branch 'guizmaii-fix/formatter'

This commit is contained in:
aserafin 2016-03-15 21:12:37 +01:00
commit 09f9dbc8e9
3 changed files with 6 additions and 11 deletions

View file

@ -12,13 +12,12 @@ module GrapeLogging
private private
def replace_parameters(parameters) def replace_parameters(parameters)
parameters.tap do |parameters| @filter_parameters.each do |parameter_name|
@filter_parameters.each do |parameter_name| if parameters.key?(parameter_name.to_s)
if parameters.key?(parameter_name.to_s) parameters[parameter_name.to_s] = @replacement
parameters[parameter_name.to_s] = @replacement
end
end end
end end
parameters
end end
end end
end end

View file

@ -29,12 +29,8 @@ module GrapeLogging
def after def after
stop_time stop_time
@reporter.perform(collect_parameters)
params = collect_parameters
@reporter.perform(params)
invoke_included_loggers(:after) invoke_included_loggers(:after)
nil nil
end end

View file

@ -2,7 +2,7 @@ module Reporters
class LoggerReporter class LoggerReporter
def initialize(logger, formatter) def initialize(logger, formatter)
@logger = logger || Logger.new(STDOUT) @logger = logger || Logger.new(STDOUT)
@logger.formatter = formatter || GrapeLogging::Formatters::Default.new @logger.formatter = formatter || GrapeLogging::Formatters::Default.new if @logger.respond_to?(:formatter=)
end end
def perform(params) def perform(params)