Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

fix: wrong number of arguments #3462

Merged
merged 1 commit into from

3 participants

@tsl0922

maybe a typo?

ArgumentError (wrong number of arguments (1 for 2)):
app/services/notification_service.rb:62:in `reassigned_merge_request'
@coveralls

Coverage decreased (-11.91%) when pulling 45c4804 on tsl0922:fix-reassign-error into 71b0f8e on gitlabhq:master.

View Details

@randx randx merged commit 273741f into from
@randx
Owner

thanks

@tsl0922 tsl0922 deleted the branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 1, 2013
  1. @tsl0922

    fix: wrong argument number

    tsl0922 authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  app/observers/merge_request_observer.rb
View
2  app/observers/merge_request_observer.rb
@@ -20,6 +20,6 @@ def after_reopen(merge_request, transition)
end
def after_update(merge_request)
- notification.reassigned_merge_request(merge_request) if merge_request.is_being_reassigned?
+ notification.reassigned_merge_request(merge_request, current_user) if merge_request.is_being_reassigned?
end
end
Something went wrong with that request. Please try again.