Avoid trying to open meeting documents with empty .uploaded_filename.

levkowetz · levkowetz · commit 60e302ab5a56 · 2020-03-10T17:59:56.000Z
- Legacy-Id: 17417
diff --git a/ietf/meeting/helpers.py b/ietf/meeting/helpers.py
@@ -223,7 +223,7 @@ def read_session_file(type, num, doc):
     #
     # FIXME: uploaded_filename should be replaced with a function call that computes names that are fixed
     path = os.path.join(settings.AGENDA_PATH, "%s/%s/%s" % (num, type, doc.uploaded_filename))
-    if os.path.exists(path):
+    if doc.uploaded_filename and os.path.exists(path):
         with io.open(path, 'rb') as f:
             return f.read(), path
     else:

Original file line number	Diff line number	Diff line change
`@@ -223,7 +223,7 @@ def read_session_file(type, num, doc):`
`223`	`223`	`#`
`224`	`224`	`# FIXME: uploaded_filename should be replaced with a function call that computes names that are fixed`
`225`	`225`	`path = os.path.join(settings.AGENDA_PATH, "%s/%s/%s" % (num, type, doc.uploaded_filename))`
`226`		`- if os.path.exists(path):`
	`226`	`+ if doc.uploaded_filename and os.path.exists(path):`
`227`	`227`	`with io.open(path, 'rb') as f:`
`228`	`228`	`return f.read(), path`
`229`	`229`	`else:`