Fixes bug in linking code and makes it more readable

"vars" should have been "escaped_locals_strings"
snowflakedb · Oct 24, 2018 · 061c572 · 061c572
1 parent 8f21eaa
commit 061c572
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/plugins/create_jira.py b/src/plugins/create_jira.py
@@ -33,13 +33,13 @@
 
 
 def jira_ticket_body(vars):
-    sources = ', '.join(vars['sources'])
-
+    vars['sources'] = ', '.join(vars['sources'])
     escaped_locals_strings = {k: escape_jira_strings(v) for k, v in vars.items()}
-    vars['sources'] = f'[{sources}|{link_search_todos(f"Sources: {sources}")}]'
+    sources = escaped_locals_strings['sources']
+    escaped_locals_strings['sources'] = f'[{sources}|{link_search_todos(f"Sources: {sources}")}]'
 
-    body = JIRA_TICKET_BODY_FMT.format(**escaped_locals_strings)
-    return body[:99000]
+    ticket_body = JIRA_TICKET_BODY_FMT.format(**escaped_locals_strings)
+    return ticket_body[:99000]
 
 
 def escape_jira_strings(v):