BigW Consortium Gitlab

Commit c78f83b5 by Douwe Maan

Merge branch 'avoid-name-shadowing' into 'master'

Avoid using the same name between methods and variables @DouweM This is the one with renaming. See merge request !3359
parents 6c4d6dd6 9374b7eb
...@@ -45,12 +45,12 @@ module Gitlab ...@@ -45,12 +45,12 @@ module Gitlab
note = create_note(reply) note = create_note(reply)
unless note.persisted? unless note.persisted?
message = "The comment could not be created for the following reasons:" msg = "The comment could not be created for the following reasons:"
note.errors.full_messages.each do |error| note.errors.full_messages.each do |error|
message << "\n\n- #{error}" msg << "\n\n- #{error}"
end end
raise InvalidNoteError, message raise InvalidNoteError, msg
end end
end end
...@@ -63,13 +63,13 @@ module Gitlab ...@@ -63,13 +63,13 @@ module Gitlab
end end
def reply_key def reply_key
reply_key = nil key = nil
message.to.each do |address| message.to.each do |address|
reply_key = Gitlab::IncomingEmail.key_from_address(address) key = Gitlab::IncomingEmail.key_from_address(address)
break if reply_key break if key
end end
reply_key key
end end
def sent_notification def sent_notification
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment