Skip to content

Commit cbb8da7

Browse files
authored
fix: regenerate correct shepherd template. Fixes ietf-tools#4558 (ietf-tools#4595)
* fix: regenerate correct shepherd template * chore: remove unnecessary extra whitespace * fix: improve readability of code per review comments
1 parent fffe0c1 commit cbb8da7

2 files changed

Lines changed: 14 additions & 6 deletions

File tree

ietf/doc/tests_draft.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1960,6 +1960,10 @@ def test_shepherd_writeup_generation(self):
19601960
login_testing_unauthorized(self, "secretary", url)
19611961
r = self.client.get(url)
19621962
self.assertContains(r, "for Individual Documents", status_code=200)
1963+
r = self.client.post(url,dict(reset_text=''))
1964+
self.assertContains(r, "for Individual Documents", status_code=200)
19631965
url = urlreverse('ietf.doc.views_draft.edit_shepherd_writeup', kwargs=dict(name=wg_draft.name))
19641966
r = self.client.get(url)
19651967
self.assertContains(r, "for Group Documents", status_code=200)
1968+
r = self.client.post(url,dict(reset_text=''))
1969+
self.assertContains(r, "for Group Documents", status_code=200)

ietf/doc/views_draft.py

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -989,14 +989,16 @@ def edit_shepherd_writeup(request, name):
989989
return redirect("ietf.doc.views_doc.document_main", name=doc.name)
990990

991991
elif "reset_text" in request.POST:
992+
if not doc.group.type.slug or doc.group.type.slug != "wg":
993+
generate_type = "individ"
994+
else:
995+
generate_type = "group"
992996
init = {
993997
"content": render_to_string(
994998
"doc/shepherd_writeup.txt",
995999
dict(
9961000
doc=doc,
997-
type="individ"
998-
if not doc.group.type.slug or doc.group.type.slug != "ietf"
999-
else "group",
1001+
type=generate_type,
10001002
stream=doc.stream.slug,
10011003
group=doc.group.type.slug,
10021004
),
@@ -1020,13 +1022,15 @@ def edit_shepherd_writeup(request, name):
10201022
if previous_writeup:
10211023
init["content"] = previous_writeup.text
10221024
else:
1025+
if not doc.group.type.slug or doc.group.type.slug != "wg":
1026+
generate_type = "individ"
1027+
else:
1028+
generate_type = "group"
10231029
init["content"] = render_to_string(
10241030
"doc/shepherd_writeup.txt",
10251031
dict(
10261032
doc=doc,
1027-
type="individ"
1028-
if not doc.group.type.slug or doc.group.type.slug != "wg"
1029-
else "group",
1033+
type=generate_type,
10301034
stream=doc.stream.slug,
10311035
group=doc.group.type.slug,
10321036
),

0 commit comments

Comments
 (0)