Notify uploader when a new comment is made on their package (Closes #74)

debexpo/controllers/package.py

@@ -255,7 +255,7 @@ class PackageController(BaseController):
             session.save()
             redirect(url('login'))
 
-        self._get_package(packagename)
+        package = self._get_package(packagename)
 
         status = constants.PACKAGE_COMMENT_STATUS_NOT_UPLOADED
 
@@ -281,10 +281,13 @@ class PackageController(BaseController):
 
         user = meta.session.query(User).filter_by(id=session['user_id']).one()
         if len(subscribers) > 0:
-
             email = Email('comment_posted')
             email.send([s.user.email for s in subscribers], package=packagename,
-                       comment=self.form_result['text'], user=user)
+                       comment=self.form_result['text'], user=user, owner=False)
+
+        email = Email('comment_posted')
+        email.send([package.user.email], package=packagename,
+                   comment=self.form_result['text'], user=user, owner=True)
 
         meta.session.query(PackageVersion) \
             .filter_by(id=self.form_result['package_version'])\

debexpo/templates/email/comment_posted.mako

@@ -2,9 +2,11 @@
 <%inherit file="/base.mako"/>To: ${ c.to }
 Subject: ${ _('Comment posted on %s' % c.package) }
 
-${ _('''A comment has been posted on a package that you are subscribed to.
+%if not c.owner:
+${ _('A comment has been posted on a package that you are subscribed to.') }
 
-%s made the following comment about the %s package:''' % (c.user.name, c.package)) }
+%endif
+${ _('''%s made the following comment about the %s package:''' % (c.user.name, c.package)) }
 
 ${ c.comment }
 
@@ -12,6 +14,8 @@ ${ _('You can view information on the package by visiting:') }
 
 ${ c.config['debexpo.server'] }${ h.url('package', packagename=c.package) }
 
+%if not c.owner:
 ${ _('You can change your subscription by visiting your user account settings.') }
 
+%endif
 ${ _('Thanks,') }

debexpo/templates/package/index.mako

@@ -39,11 +39,13 @@
 
 % if 'user_id' in c.session:
 
+% if c.session['user_id'] != c.package.user_id:
   <tr>
     <th>${ _('Subscribe') }:</th>
     <td><a href="${h.url('subscribe', packagename=c.package.name)}">${ _('Edit your subscription') }</a></td>
   </tr>
 
+%endif
 
   <tr>
   <th>${ _('Needs a sponsor') }:</th>