From 01abb91160e10dc2fa99043fc3e8e0796c53c3cc Mon Sep 17 00:00:00 2001 From: Andrew Thornton Date: Sun, 19 Dec 2021 18:11:55 +0000 Subject: [PATCH 1/4] Prevent off-by-one error on comments on newly appended lines There was a bug in CutDiffAroundLine whereby if a file without a terminal new line has a patch which appends lines to it and a comment is placed on one of those lines the comment diff will be a line out of place. This fixes CutDiffAroundLine to simply ignore the missing terminal newline - however, we should really improve this rendering to add a marker to say that there was a previously missing terminal newline. Fix #17875 Signed-off-by: Andrew Thornton --- modules/git/diff.go | 3 +++ modules/git/diff_test.go | 52 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 55 insertions(+) diff --git a/modules/git/diff.go b/modules/git/diff.go index 3a82cda1ce037..a78783d484b63 100644 --- a/modules/git/diff.go +++ b/modules/git/diff.go @@ -152,6 +152,7 @@ func CutDiffAroundLine(originalDiff io.Reader, line int64, old bool, numbersOfLi for scanner.Scan() { lof := scanner.Text() + fmt.Printf("%03d: %s\n", currentLine, lof) // Add header to enable parsing if isHeader(lof, inHunk) { @@ -217,6 +218,8 @@ func CutDiffAroundLine(originalDiff io.Reader, line int64, old bool, numbersOfLi } else { otherLine++ } + case '\\': + // FIXME: handle `\ No newline at end of file` default: currentLine++ otherLine++ diff --git a/modules/git/diff_test.go b/modules/git/diff_test.go index 363ff0b970382..cf76399b00c45 100644 --- a/modules/git/diff_test.go +++ b/modules/git/diff_test.go @@ -42,6 +42,58 @@ index d8e4c92..19dc8ad 100644 / ` +var issue_17875_diff = `diff --git a/Geschäftsordnung.md b/Geschäftsordnung.md +index d46c152..a7d2d55 100644 +--- a/Geschäftsordnung.md ++++ b/Geschäftsordnung.md +@@ -1,5 +1,5 @@ + --- +-date: "23.01.2021" ++date: "30.11.2021" + ... + ` + ` + # Geschäftsordnung +@@ -16,4 +16,22 @@ Diese Geschäftsordnung regelt alle Prozesse des Vereins, solange diese nicht du + ` + ` + ## § 3 Datenschutzverantwortlichkeit + ` + ` +-1. Der Verein bestellt eine datenschutzverantwortliche Person mit den Aufgaben nach Artikel 39 DSGVO. +\ No newline at end of file ++1. Der Verein bestellt eine datenschutzverantwortliche Person mit den Aufgaben nach Artikel 39 DSGVO. ++ ++## §4 Umgang mit der SARS-Cov-2-Pandemie ++ ++1. Der Vorstand hat die Befugnis, in Rücksprache mit den Vereinsmitgliedern, verschiedene Hygienemaßnahmen für Präsenzveranstaltungen zu beschließen. ++ ++2. Die Einführung, Änderung und Abschaffung dieser Maßnahmen sind nur zum Zweck der Eindämmung der SARS-Cov-2-Pandemie zulässig. ++ ++3. Die Einführung, Änderung und Abschaffung von Maßnahmen nach Abs. 2 bedarf einer wissenschaftlichen Grundlage. ++ ++4. Die Maßnahmen nach Abs. 2 setzen sich aus den folgenden Bausteinen inklusive einer ihrer Ausprägungen zusammen. ++ ++ 1. Maskenpflicht: Keine; Maskenpflicht, außer am Platz, oder wo Abstände nicht eingehalten werden können; Maskenpflicht, wenn Abstände nicht eingehalten werden können; Maskenpflicht ++ ++ 2. Geimpft-, Genesen- oder Testnachweis: Kein Nachweis notwendig; Nachweis, dass Person geimpft, genesen oder tagesaktuell getestet ist (3G); Nachweis, dass Person geimpft oder genesen ist (2G); Nachweis, dass Person geimpft bzw. genesen und tagesaktuell getestet ist (2G+) ++ ++ 3. Online-Veranstaltung: Keine, parallele Online-Veranstaltung, ausschließlich Online-Veranstaltung ++ ++5. Bei Präsenzveranstungen gelten außerdem die Hygienevorschriften des Veranstaltungsorts. Bei Regelkollision greift die restriktivere Regel. +\ No newline at end of file` + +func TestCutDiffAroundLine_17875(t *testing.T) { + result, err := CutDiffAroundLine(strings.NewReader(issue_17875_diff), 23, false, 3) + assert.NoError(t, err) + // resultByLine := strings.Split(result, "\n") + expected := `diff --git a/Geschäftsordnung.md b/Geschäftsordnung.md +--- a/Geschäftsordnung.md ++++ b/Geschäftsordnung.md +@@ -20,0 +21,3 @@ ++## §4 Umgang mit der SARS-Cov-2-Pandemie ++ ++1. Der Vorstand hat die Befugnis, in Rücksprache mit den Vereinsmitgliedern, verschiedene Hygienemaßnahmen für Präsenzveranstaltungen zu beschließen.` + assert.Equal(t, expected, result) +} + func TestCutDiffAroundLine(t *testing.T) { result, err := CutDiffAroundLine(strings.NewReader(exampleDiff), 4, false, 3) assert.NoError(t, err) From 15e49499a006b42c9ee41f5fdda5da3633091aa6 Mon Sep 17 00:00:00 2001 From: Andrew Thornton Date: Sun, 19 Dec 2021 20:56:20 +0000 Subject: [PATCH 2/4] placate lint Signed-off-by: Andrew Thornton --- modules/git/diff_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/git/diff_test.go b/modules/git/diff_test.go index cf76399b00c45..6715284dd01db 100644 --- a/modules/git/diff_test.go +++ b/modules/git/diff_test.go @@ -42,7 +42,7 @@ index d8e4c92..19dc8ad 100644 / ` -var issue_17875_diff = `diff --git a/Geschäftsordnung.md b/Geschäftsordnung.md +var issue17875Diff = `diff --git a/Geschäftsordnung.md b/Geschäftsordnung.md index d46c152..a7d2d55 100644 --- a/Geschäftsordnung.md +++ b/Geschäftsordnung.md @@ -80,8 +80,8 @@ index d46c152..a7d2d55 100644 +5. Bei Präsenzveranstungen gelten außerdem die Hygienevorschriften des Veranstaltungsorts. Bei Regelkollision greift die restriktivere Regel. \ No newline at end of file` -func TestCutDiffAroundLine_17875(t *testing.T) { - result, err := CutDiffAroundLine(strings.NewReader(issue_17875_diff), 23, false, 3) +func TestCutDiffAroundLineIssue17875(t *testing.T) { + result, err := CutDiffAroundLine(strings.NewReader(issue17875Diff), 23, false, 3) assert.NoError(t, err) // resultByLine := strings.Split(result, "\n") expected := `diff --git a/Geschäftsordnung.md b/Geschäftsordnung.md From 8a6079d1698b09b3d13f09f42bb8e03089e5919a Mon Sep 17 00:00:00 2001 From: zeripath Date: Sun, 19 Dec 2021 21:04:10 +0000 Subject: [PATCH 3/4] Update modules/git/diff.go Co-authored-by: KN4CK3R --- modules/git/diff.go | 1 - 1 file changed, 1 deletion(-) diff --git a/modules/git/diff.go b/modules/git/diff.go index a78783d484b63..f90f911be0725 100644 --- a/modules/git/diff.go +++ b/modules/git/diff.go @@ -152,7 +152,6 @@ func CutDiffAroundLine(originalDiff io.Reader, line int64, old bool, numbersOfLi for scanner.Scan() { lof := scanner.Text() - fmt.Printf("%03d: %s\n", currentLine, lof) // Add header to enable parsing if isHeader(lof, inHunk) { From 48eb2d6bb33c1016be7b7bde17a711ca8df6995c Mon Sep 17 00:00:00 2001 From: zeripath Date: Sun, 19 Dec 2021 21:14:17 +0000 Subject: [PATCH 4/4] Update modules/git/diff_test.go Co-authored-by: KN4CK3R --- modules/git/diff_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/modules/git/diff_test.go b/modules/git/diff_test.go index 6715284dd01db..de71248508f6d 100644 --- a/modules/git/diff_test.go +++ b/modules/git/diff_test.go @@ -83,7 +83,6 @@ index d46c152..a7d2d55 100644 func TestCutDiffAroundLineIssue17875(t *testing.T) { result, err := CutDiffAroundLine(strings.NewReader(issue17875Diff), 23, false, 3) assert.NoError(t, err) - // resultByLine := strings.Split(result, "\n") expected := `diff --git a/Geschäftsordnung.md b/Geschäftsordnung.md --- a/Geschäftsordnung.md +++ b/Geschäftsordnung.md