Skip to content

Commit 8b89e84

Browse files
authored
Merge pull request getsentry#625 from miyachik/fix-format-message
Fix format message if params is empty Close getsentry#624
2 parents e5e6642 + deca9ed commit 8b89e84

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

lib/raven/interfaces/message.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ def initialize(*arguments)
1010
end
1111

1212
def unformatted_message
13-
params.nil? ? message : message % params
13+
Array(params).empty? ? message : message % params
1414
end
1515

1616
def self.sentry_alias

spec/raven/interface_spec.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,4 +30,8 @@ class TestInterface < Raven::Interface
3030
interface = Raven::MessageInterface.new(:params => nil, :message => "test '%'")
3131
expect(interface.unformatted_message).to eq("test '%'")
3232
end
33+
it "supports invalid format string message when params is empty" do
34+
interface = Raven::MessageInterface.new(:message => "test '%'")
35+
expect(interface.unformatted_message).to eq("test '%'")
36+
end
3337
end

0 commit comments

Comments
 (0)