##// END OF EJS Templates
fix missing email_kwargs
marcink -
r2805:2454ed7f beta
parent child Browse files
Show More
@@ -123,6 +123,9 b' class ChangesetCommentsModel(BaseModel):'
123 recipients = ChangesetComment.get_users(revision=revision)
123 recipients = ChangesetComment.get_users(revision=revision)
124 # add changeset author if it's in rhodecode system
124 # add changeset author if it's in rhodecode system
125 recipients += [User.get_by_email(author_email)]
125 recipients += [User.get_by_email(author_email)]
126 email_kwargs = {
127 'status_change': status_change,
128 }
126 #pull request
129 #pull request
127 elif pull_request:
130 elif pull_request:
128 _url = h.url('pullrequest_show',
131 _url = h.url('pullrequest_show',
@@ -147,7 +150,7 b' class ChangesetCommentsModel(BaseModel):'
147 recipients += [x.user for x in pull_request.reviewers]
150 recipients += [x.user for x in pull_request.reviewers]
148
151
149 #set some variables for email notification
152 #set some variables for email notification
150 kwargs = {
153 email_kwargs = {
151 'pr_id': pull_request.pull_request_id,
154 'pr_id': pull_request.pull_request_id,
152 'status_change': status_change,
155 'status_change': status_change,
153 'pr_comment_url': _url,
156 'pr_comment_url': _url,
@@ -160,7 +163,7 b' class ChangesetCommentsModel(BaseModel):'
160 NotificationModel().create(
163 NotificationModel().create(
161 created_by=user, subject=subj, body=body,
164 created_by=user, subject=subj, body=body,
162 recipients=recipients, type_=notification_type,
165 recipients=recipients, type_=notification_type,
163 email_kwargs=kwargs
166 email_kwargs=email_kwargs
164 )
167 )
165
168
166 mention_recipients = set(self._extract_mentions(body))\
169 mention_recipients = set(self._extract_mentions(body))\
General Comments 0
You need to be logged in to leave comments. Login now