From 5e2808ee1f833dd7d2a79c03c7791278a3eaf80d Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <dietrich@adlares.com>
Date: Sun, 19 Nov 2017 23:22:12 +0100
Subject: [PATCH 01/14] Extend the database schema to save the repository for
 comments

The respository name is currently saved in the 'path' column for comments. For subscriptions however it is stored in a separate column.
---
 code_comments/db.py | 31 ++++++++++++++++++++++++++++++-
 1 file changed, 30 insertions(+), 1 deletion(-)

diff --git a/code_comments/db.py b/code_comments/db.py
index 0fb56a2..e2f6356 100644
--- a/code_comments/db.py
+++ b/code_comments/db.py
@@ -4,9 +4,10 @@
 from trac.db.schema import Table, Column, Index
 from trac.env import IEnvironmentSetupParticipant
 from trac.db.api import DatabaseManager
+from trac.versioncontrol.api import RepositoryManager
 
 # Database version identifier for upgrades.
-db_version = 3
+db_version = 4
 db_version_key = 'code_comments_schema_version'
 
 # Database schema
@@ -21,8 +22,11 @@
         Column('author'),
         Column('time', type='int'),
         Column('type'),
+        Column('reponame'),
         Index(['path']),
         Index(['author']),
+        Index(['reponame', 'path']),
+        Index(['revision', 'reponame']),
     ],
     'code_comments_subscriptions': Table('code_comments_subscriptions',
                                          key=('id', 'user', 'type', 'path',
@@ -76,9 +80,34 @@ def upgrade_from_2_to_3(env):
     dbm.create_tables((schema['code_comments_subscriptions'],))
 
 
+def upgrade_from_3_to_4(env):
+    with env.db_transaction as db:
+        # Add the new column "reponame" and indexes.
+        db('ALTER TABLE code_comments ADD COLUMN reponame text')
+        db('CREATE INDEX code_comments_reponame_path_idx ON code_comments (reponame, path)')
+        db('CREATE INDEX code_comments_revision_reponame_idx ON code_comments (revision, reponame)')
+
+        # Comments on attachments need to have the empty string as the reponame instead of NULL.
+        db("UPDATE code_comments SET reponame = '' WHERE type = 'attachment'")
+
+        # Comments on changesets have the reponame in the 'path' column.
+        db("UPDATE code_comments SET reponame = path, path = '' WHERE type = 'changeset'")
+
+        # Comments on files have the reponame as the first component of the 'path' column.
+        db("""
+            UPDATE code_comments
+            SET
+                reponame = substr(path, 1, instr(path, '/') - 1),
+                path = substr(path, instr(path, '/') + 1)
+            WHERE
+                type = 'browser'
+            """)
+
+
 upgrade_map = {
     2: upgrade_from_1_to_2,
     3: upgrade_from_2_to_3,
+    4: upgrade_from_3_to_4,
 }
 
 

From 15bf0597e384597a9aa2ae7414c11d430322a129 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <dietrich@adlares.com>
Date: Fri, 17 Nov 2017 01:00:37 +0100
Subject: [PATCH 02/14] Add reponame to data for JavaScript

---
 code_comments/web.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/code_comments/web.py b/code_comments/web.py
index 83342c7..ad45746 100644
--- a/code_comments/web.py
+++ b/code_comments/web.py
@@ -121,16 +121,18 @@ def templates_js_data(self):
     def changeset_js_data(self, req, data):
         return {
             'page': 'changeset',
+            'reponame': data['reponame'],
             'revision': data['new_rev'],
-            'path': data['reponame'],
+            'path': '',
             'selectorToInsertAfter': 'div.diff div.diff:last'
         }
 
     def browser_js_data(self, req, data):
         return {
             'page': 'browser',
+            'reponame': data['reponame'],
             'revision': data['rev'],
-            'path': (data['reponame'] or '') + '/' + data['path'],
+            'path': data['path'],
             'selectorToInsertAfter': 'table.code'
         }
 
@@ -138,6 +140,7 @@ def attachment_js_data(self, req, data):
         path = req.path_info.replace('/attachment/', 'attachment:/')
         return {
             'page': 'attachment',
+            'reponame': '',
             'revision': 0,
             'path': path,
             'selectorToInsertAfter': 'div#preview'

From bde9868a336f8640e396cd18ef5039118a0e96f2 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Mon, 28 Aug 2023 00:38:38 +0200
Subject: [PATCH 03/14] Use reponame when fetching comments for a page

---
 code_comments/htdocs/code-comments.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/code_comments/htdocs/code-comments.js b/code_comments/htdocs/code-comments.js
index fe1cda6..4e43f2a 100644
--- a/code_comments/htdocs/code-comments.js
+++ b/code_comments/htdocs/code-comments.js
@@ -28,6 +28,7 @@ var underscore = _.noConflict();
 		},
 		defaultFetchParams: {
 			path: CodeComments.path || undefined,
+			reponame: CodeComments.reponame,
 			revision: CodeComments.revision,
 			type: CodeComments.page,
 		},

From bb1acf6371e6b73791f9ee9b59ee9d74f24d911d Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Mon, 28 Aug 2023 00:03:35 +0200
Subject: [PATCH 04/14] Send reponame when creating a comment

---
 code_comments/htdocs/code-comments.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/code_comments/htdocs/code-comments.js b/code_comments/htdocs/code-comments.js
index 4e43f2a..e9177dd 100644
--- a/code_comments/htdocs/code-comments.js
+++ b/code_comments/htdocs/code-comments.js
@@ -218,6 +218,7 @@ var underscore = _.noConflict();
 				text: text,
 				author: CodeComments.username,
 				path: this.path,
+				reponame: CodeComments.reponame,
 				revision: CodeComments.revision,
 				line: line,
 				type: CodeComments.page

From 47d52a962f46ca06069dac194a8c9991a29ff4ea Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <dietrich@adlares.com>
Date: Mon, 20 Nov 2017 00:04:38 +0100
Subject: [PATCH 05/14] Use reponame when constructing links to comments

---
 code_comments/comment.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/code_comments/comment.py b/code_comments/comment.py
index 8850db9..428e380 100644
--- a/code_comments/comment.py
+++ b/code_comments/comment.py
@@ -71,10 +71,10 @@ def validate(self):
 
     def href(self):
         if self.is_comment_to_file:
-            href = self.req.href.browser(self.path, rev=self.revision,
+            href = self.req.href.browser(self.reponame + '/' + self.path, rev=self.revision,
                                          codecomment=self.id)
         elif self.is_comment_to_changeset:
-            href = self.req.href.changeset(self.revision, codecomment=self.id)
+            href = self.req.href.changeset(self.revision + '/' + self.reponame, codecomment=self.id)
         elif self.is_comment_to_attachment:
             href = self.req.href('/attachment/ticket/%d/%s'
                                  % (self.attachment_ticket,

From f4acbaaff09fe871c50830f54433c92a370610d4 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <dietrich@adlares.com>
Date: Mon, 20 Nov 2017 00:28:02 +0100
Subject: [PATCH 06/14] Make subscription from comment use the reponame from
 the comment

Until now it used the path to determine the repository.
---
 code_comments/subscription.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/code_comments/subscription.py b/code_comments/subscription.py
index cbe14dc..fe7a707 100644
--- a/code_comments/subscription.py
+++ b/code_comments/subscription.py
@@ -236,13 +236,13 @@ def from_comment(cls, env, comment, user=None, notify=True):
 
         # Munge changesets and browser
         if comment.type in ('changeset', 'browser'):
-            rm = RepositoryManager(env)
-            reponame, repos, path = rm.get_repository_by_path(comment.path)
             if comment.type == 'browser':
-                sub['path'] = path
+                sub['path'] = comment.path
             else:
                 sub['path'] = ''
-            sub['repos'] = reponame or '(default)'
+            sub['repos'] = comment.reponame
+            rm = RepositoryManager(env)
+            repos = rm.get_repository(comment.reponame)
             try:
                 _cs = repos.get_changeset(comment.revision)
             except NoSuchChangeset:

From f35cf284d8b2e3ac34ec95974affc247435be5d1 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Wed, 30 Aug 2023 23:10:52 +0200
Subject: [PATCH 07/14] Add new column 'repository' to the list of code
 comments

---
 code_comments/comments.py             | 2 +-
 code_comments/templates/comments.html | 1 +
 code_comments/web.py                  | 4 ++--
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/code_comments/comments.py b/code_comments/comments.py
index b8282e9..40ce5e1 100644
--- a/code_comments/comments.py
+++ b/code_comments/comments.py
@@ -13,7 +13,7 @@ class Comments(object):
 
     def __init__(self, req, env):
         self.req, self.env = req, env
-        self.valid_sorting_methods = ('id', 'author', 'time', 'path', 'text')
+        self.valid_sorting_methods = ('id', 'author', 'time', 'reponame', 'path', 'text')
 
     def comment_from_row(self, row):
         return Comment(self.req, self.env, row)
diff --git a/code_comments/templates/comments.html b/code_comments/templates/comments.html
index 2870244..494bc9a 100644
--- a/code_comments/templates/comments.html
+++ b/code_comments/templates/comments.html
@@ -49,6 +49,7 @@ <h1>Code Comments</h1>
                     <td>$comment.id</td>
                     <th>$comment.author</th>
                     <th>${comment.formatted_date()}</th>
+                    <td>$comment.reponame</td>
                     <td>${comment.path_link_tag()}</td>
                     <td>$comment.html</td>
                     <td>${comment.get_ticket_links()}</td>
diff --git a/code_comments/web.py b/code_comments/web.py
index ad45746..4689948 100644
--- a/code_comments/web.py
+++ b/code_comments/web.py
@@ -243,9 +243,9 @@ def href_with_page(page):
 
     def prepare_sortable_headers(self):
         displayed_sorting_methods = \
-            ('id', 'author', 'time', 'path', 'text')
+            ('id', 'author', 'time', 'reponame', 'path', 'text')
         displayed_sorting_method_names = \
-            ('ID', 'Author', 'Date', 'Path', 'Text')
+            ('ID', 'Author', 'Date', 'Repository', 'Path', 'Text')
         query_args = self.req.args
         if 'page' in query_args:
             del query_args['page']

From fdaff49857f31d28411375f5cf11de5486b5be0f Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Wed, 30 Aug 2023 22:57:15 +0200
Subject: [PATCH 08/14] Allow filtering the list of code comments by repository

---
 code_comments/comments.py             | 5 +++++
 code_comments/templates/comments.html | 6 ++++++
 code_comments/web.py                  | 6 ++++++
 3 files changed, 17 insertions(+)

diff --git a/code_comments/comments.py b/code_comments/comments.py
index 40ce5e1..6f7c7a1 100644
--- a/code_comments/comments.py
+++ b/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('/')
diff --git a/code_comments/templates/comments.html b/code_comments/templates/comments.html
index 494bc9a..848a1d6 100644
--- a/code_comments/templates/comments.html
+++ b/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"
diff --git a/code_comments/web.py b/code_comments/web.py
index 4689948..ccc46ae 100644
--- a/code_comments/web.py
+++ b/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']

From 8b51c171109450934ea5b1ee28e485f8094ee610 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Tue, 24 Sep 2024 23:04:46 +0200
Subject: [PATCH 09/14] Use the reponame from the comment when searching for
 subscriptions for a comment

---
 code_comments/subscription.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/code_comments/subscription.py b/code_comments/subscription.py
index fe7a707..62342ff 100644
--- a/code_comments/subscription.py
+++ b/code_comments/subscription.py
@@ -296,11 +296,9 @@ def for_comment(cls, env, comment, notify=None):
             args['rev'] = str(comment.revision)
 
         if comment.type == 'browser':
-            rm = RepositoryManager(env)
-            reponame, _, path = rm.get_repository_by_path(comment.path)
             args['type'] = ('browser', 'changeset')
-            args['path'] = (path, '')
-            args['repos'] = reponame
+            args['path'] = (comment.path, '')
+            args['repos'] = comment.reponame
             args['rev'] = (str(comment.revision), '')
 
         return cls.select(env, args, notify)

From af17e64b438faa8f636859cd38cbeadda6629386 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Wed, 30 Aug 2023 00:07:32 +0200
Subject: [PATCH 10/14] Migrate existing changeset comments

Remove the bogus line numbers and leave an explanation in the text of the comment.
---
 code_comments/db.py | 29 ++++++++++++++++++++++++++++-
 1 file changed, 28 insertions(+), 1 deletion(-)

diff --git a/code_comments/db.py b/code_comments/db.py
index e2f6356..1b1bf77 100644
--- a/code_comments/db.py
+++ b/code_comments/db.py
@@ -7,7 +7,7 @@
 from trac.versioncontrol.api import RepositoryManager
 
 # Database version identifier for upgrades.
-db_version = 4
+db_version = 5
 db_version_key = 'code_comments_schema_version'
 
 # Database schema
@@ -104,10 +104,37 @@ def upgrade_from_3_to_4(env):
             """)
 
 
+def upgrade_from_4_to_5(env):
+    with env.db_transaction as db:
+        # The line numbers of all present comments on changesets are bogus,
+        # see https://github.com/trac-hacks/trac-code-comments-plugin/issues/67
+        # We therefore set them to 0 detaching the comment from the line. We leave a note in
+        # the text of the comment explaining this.
+
+        notice = '\n\nThis comment was created by a previous version of the '\
+                 "code-comments plugin and '''is not properly attached to a line of code'''. "\
+                 'See [https://github.com/trac-hacks/trac-code-comments-plugin/issues/67 '\
+                 'issue #67].\n\nThe comment was originally placed on line $oldLineNumber$ of '\
+                 'the diff as it was displayed when the comment was created.'
+        notice = notice.replace("'", "''")
+        sql = """
+            UPDATE code_comments
+            SET
+                line = 0,
+                text = text || REPLACE('{0}', '$oldLineNumber$', line)
+            WHERE
+                type = 'changeset'
+                AND
+                line != 0
+            """
+        db(sql.format(notice))
+
+
 upgrade_map = {
     2: upgrade_from_1_to_2,
     3: upgrade_from_2_to_3,
     4: upgrade_from_3_to_4,
+    5: upgrade_from_4_to_5
 }
 
 

From 1d1c92634859ff1c50058dcca532e1f7e2ed7a90 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Mon, 28 Aug 2023 23:17:21 +0200
Subject: [PATCH 11/14] Rename functions working with line numbers

The term "line number" is ambiguous because there is a difference between a line number "within the diff" and a line number "within a file".
---
 code_comments/htdocs/code-comments.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/code_comments/htdocs/code-comments.js b/code_comments/htdocs/code-comments.js
index e9177dd..f41f991 100644
--- a/code_comments/htdocs/code-comments.js
+++ b/code_comments/htdocs/code-comments.js
@@ -115,7 +115,7 @@ var underscore = _.noConflict();
 			if (!this.viewPerLine[line]) {
 				this.viewPerLine[line] = new CommentsForALineView( { line: line } );
 
-				var $tr = $( Rows.getTrByLineNumber( line ) );
+				var $tr = $( Rows.getTrByLineNumberInDiff( line ) );
 				$tr.after(this.viewPerLine[line].render().el).addClass('with-comments');
 			}
 			this.viewPerLine[line].addOne(comment);
@@ -239,7 +239,7 @@ var underscore = _.noConflict();
 			var callbackMouseover = function( event ) {
 				var row = new RowView( { el: this } ),
 					file = row.getFile(),
-					line = row.getLineNumber(),
+					line = row.getLineNumberInDiff(),
 					displayLine = row.getDisplayLine();
 				row.replaceLineNumberCellContent( '<a title="Comment on this line" href="#L' + line + '" class="bubble"><span class="ui-icon ui-icon-comment"></span></a>' );
 
@@ -268,10 +268,10 @@ var underscore = _.noConflict();
 			// wrap TH content in spans so we can hide/show them
 			this.wrapTHsInSpans();
 		},
-		getLineByTR: function( tr ) {
+		getLineNumberInDiffByTr: function( tr ) {
 			return $.inArray( tr, this.$rows ) + 1;
 		},
-		getTrByLineNumber: function( line ) {
+		getTrByLineNumberInDiff: function( line ) {
 			return this.$rows[line - 1];
 		},
 		wrapTHsInSpans: function() {
@@ -303,8 +303,8 @@ var underscore = _.noConflict();
 		getFile: function() {
 			return this.$el.parents( 'li' ).find( 'h2>a:first' ).text();
 		},
-		getLineNumber: function() {
-			return Rows.getLineByTR( this.el );
+		getLineNumberInDiff: function() {
+			return Rows.getLineNumberInDiffByTr( this.el );
 		},
 		getDisplayLine: function() {
 			return this.$lineNumberCell.text().trim() || this.$th.first().text() + ' (deleted)';

From 1819ed1814ac5e458e7048bbf0e832ac4407b018 Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Tue, 28 Nov 2017 14:22:14 +0100
Subject: [PATCH 12/14] Split a function into two separate functions

---
 code_comments/htdocs/code-comments.js | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/code_comments/htdocs/code-comments.js b/code_comments/htdocs/code-comments.js
index f41f991..2d5a680 100644
--- a/code_comments/htdocs/code-comments.js
+++ b/code_comments/htdocs/code-comments.js
@@ -303,11 +303,22 @@ var underscore = _.noConflict();
 		getFile: function() {
 			return this.$el.parents( 'li' ).find( 'h2>a:first' ).text();
 		},
+		getLineNumberInFile: function() {
+			var lineNumber = this.$lineNumberCell.text().trim();
+			if (lineNumber)
+				return lineNumber;
+			else
+				return this.$th.first().text().trim() * -1;
+		},
 		getLineNumberInDiff: function() {
 			return Rows.getLineNumberInDiffByTr( this.el );
 		},
 		getDisplayLine: function() {
-			return this.$lineNumberCell.text().trim() || this.$th.first().text() + ' (deleted)';
+			var lineNumber = this.getLineNumberInFile()
+			if (lineNumber > 0)
+				return lineNumber;
+			else
+				return lineNumber * -1 + ' (deleted)';
 		}
 	} );
 

From 0eaf63a227988f4f8e7615ffafc8932dbcec6f4d Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Wed, 6 Dec 2017 17:01:13 +0100
Subject: [PATCH 13/14] Comment is placed on line of file instead of line of
 diff.

Negative line numbers are used to indicate a comment that was placed on a line of the old file.
---
 code_comments/comments.py             |  3 ++
 code_comments/htdocs/code-comments.js | 45 ++++++++++++++++++++++-----
 2 files changed, 41 insertions(+), 7 deletions(-)

diff --git a/code_comments/comments.py b/code_comments/comments.py
index 6f7c7a1..574202f 100644
--- a/code_comments/comments.py
+++ b/code_comments/comments.py
@@ -101,6 +101,9 @@ def get_condition_str_and_corresponding_values(self, args):
             elif name.endswith('__lt'):
                 name = name.replace('__lt', '')
                 conditions.append(name + ' < %s')
+            elif name.endswith('__ne'):
+                name = name.replace('__ne', '')
+                conditions.append(name + ' != %s')
             elif name.endswith('__prefix'):
                 values.append(
                     args[name].replace('%', '\\%').replace('_', '\\_') + '%')
diff --git a/code_comments/htdocs/code-comments.js b/code_comments/htdocs/code-comments.js
index 2d5a680..8542cfd 100644
--- a/code_comments/htdocs/code-comments.js
+++ b/code_comments/htdocs/code-comments.js
@@ -36,7 +36,7 @@ var underscore = _.noConflict();
 			return this.fetch( { data: _.extend( { line: 0 }, this.defaultFetchParams ) } );
 		},
 		fetchLineComments: function() {
-			return this.fetch( { data: _.extend( { line__gt: 0 }, this.defaultFetchParams ) } );
+			return this.fetch( { data: _.extend( { line__ne: 0 }, this.defaultFetchParams ) } );
 		}
 	});
 
@@ -112,13 +112,15 @@ var underscore = _.noConflict();
 		},
 		addOne: function(comment) {
 			var line = comment.get('line');
-			if (!this.viewPerLine[line]) {
-				this.viewPerLine[line] = new CommentsForALineView( { line: line } );
+			var file = comment.get('path');
+			var key = 'file_' + file + ':' + line;
+			if (!this.viewPerLine[key]) {
+				this.viewPerLine[key] = new CommentsForALineView( { file: file, line: line } );
 
-				var $tr = $( Rows.getTrByLineNumberInDiff( line ) );
-				$tr.after(this.viewPerLine[line].render().el).addClass('with-comments');
+				var $tr = $( Rows.getTrByFileAndLineNumberInFile( file, line ) );
+				$tr.after(this.viewPerLine[key].render().el).addClass('with-comments');
 			}
-			this.viewPerLine[line].addOne(comment);
+			this.viewPerLine[key].addOne(comment);
 		},
 		addAll: function() {
 			var view = this;
@@ -134,6 +136,7 @@ var underscore = _.noConflict();
 		template: _.template(CodeComments.templates.comments_for_a_line),
 		initialize: function(attrs) {
 			this.line = attrs.line;
+			this.file = attrs.file;
 		},
 		events: {
 			'click button': 'showAddCommentDialog'
@@ -239,7 +242,7 @@ var underscore = _.noConflict();
 			var callbackMouseover = function( event ) {
 				var row = new RowView( { el: this } ),
 					file = row.getFile(),
-					line = row.getLineNumberInDiff(),
+					line = row.getLineNumberInFile(),
 					displayLine = row.getDisplayLine();
 				row.replaceLineNumberCellContent( '<a title="Comment on this line" href="#L' + line + '" class="bubble"><span class="ui-icon ui-icon-comment"></span></a>' );
 
@@ -274,6 +277,34 @@ var underscore = _.noConflict();
 		getTrByLineNumberInDiff: function( line ) {
 			return this.$rows[line - 1];
 		},
+		getTrByFileAndLineNumberInFile: function ( file, line) {
+			var col;
+			var container;
+			if (CodeComments.page == "browser" && CodeComments.path == file) {
+				container = $( 'thead', 'table.code' );
+				col = 0;
+			} else {
+				if (line < 0) {
+					line = -line;
+					col = 0;
+				} else {
+					col = 1
+				}
+				var containers = $( 'thead', 'table.code, table.trac-diff' );
+				for ( var i = 0; (container = containers[i]) != null; i++ ) {
+					if ($(container).parents( 'li' ).find( 'h2>a:first' ).text() == file) {
+						break;
+					}
+				}
+			}
+			var trs = $(container).parents( 'table' ).find( 'tbody>tr' ).not('.comments');
+			for ( var i = 0, tr; (tr = trs[i]) != null; i++ ) {
+				if ($(tr).find( 'th>span' )[col].textContent == line) {
+					return tr;
+				}
+			}
+			return null;
+		},
 		wrapTHsInSpans: function() {
 			$( 'th', this.$rows ).each( function( i, elem ) {
 				elem.innerHTML = '<span>' + elem.innerHTML + '</span>';

From a1231144341fe9a7aebe61093f2c1da8ba5ddf7b Mon Sep 17 00:00:00 2001
From: Sebastian Dietrich <SebDieBln@users.noreply.github.com>
Date: Sun, 10 Dec 2017 21:39:10 +0100
Subject: [PATCH 14/14] Remove functions that are not used any more

---
 code_comments/htdocs/code-comments.js | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/code_comments/htdocs/code-comments.js b/code_comments/htdocs/code-comments.js
index 8542cfd..8b2b734 100644
--- a/code_comments/htdocs/code-comments.js
+++ b/code_comments/htdocs/code-comments.js
@@ -271,12 +271,6 @@ var underscore = _.noConflict();
 			// wrap TH content in spans so we can hide/show them
 			this.wrapTHsInSpans();
 		},
-		getLineNumberInDiffByTr: function( tr ) {
-			return $.inArray( tr, this.$rows ) + 1;
-		},
-		getTrByLineNumberInDiff: function( line ) {
-			return this.$rows[line - 1];
-		},
 		getTrByFileAndLineNumberInFile: function ( file, line) {
 			var col;
 			var container;
@@ -341,9 +335,6 @@ var underscore = _.noConflict();
 			else
 				return this.$th.first().text().trim() * -1;
 		},
-		getLineNumberInDiff: function() {
-			return Rows.getLineNumberInDiffByTr( this.el );
-		},
 		getDisplayLine: function() {
 			var lineNumber = this.getLineNumberInFile()
 			if (lineNumber > 0)