Allow filtering the list of code comments by repository

Author: SebDieBln

code_comments/comments.py

@@ -21,10 +21,15 @@ def comment_from_row(self, row):
     def get_filter_values(self):
         comments = self.all()
         return {
+            'repos': self.get_all_repos(comments),
             'paths': self.get_all_paths(comments),
             'authors': self.get_all_comment_authors(comments),
         }
 
+    def get_all_repos(self, comments):
+        # Skip the empty string which is the repository for comments on attachments
+        return sorted(list(set([comment.reponame for comment in comments if comment.reponame != ''])))
+
     def get_all_paths(self, comments):
         def get_directory(path):
             parts = os.path.split(path)[0].split('/')

code_comments/templates/comments.html

@@ -15,6 +15,12 @@ <h1>Code Comments</h1>
         <button id="send-to-ticket" type="submit" data-url="${href('code-comments', 'create-ticket')}">Create ticket with selected</button>
         &nbsp;Filter comments:
         <form action="${href('code-comments')}" method="GET" style="display: inline;">
+            <select id="filter-by-repo" name="filter-by-repo">
+                <option value='' selected="selected">-- All repositories --</option>
+                <option py:for="repo in repos"
+                           selected="${current_repo_selection == repo or None}"
+                           value="$repo" py:content="repo"></option>
+            </select>
             <select id="filter-by-path" name="filter-by-path">
                 <option value='' selected="selected">-- All paths --</option>
                 <option py:for="path in paths"

code_comments/web.py

@@ -198,9 +198,15 @@ def post_process_request(self, req, template, data, content_type):
         return template, data, content_type
 
     def add_path_and_author_filters(self):
+        self.data['current_repo_selection'] = ''
         self.data['current_path_selection'] = ''
         self.data['current_author_selection'] = ''
 
+        if self.req.args.get('filter-by-repo'):
+            self.args['reponame'] = \
+                self.req.args['filter-by-repo']
+            self.data['current_repo_selection'] = \
+                self.req.args['filter-by-repo']
         if self.req.args.get('filter-by-path'):
             self.args['path__prefix'] = \
                 self.req.args['filter-by-path']