Release Manager · Release Manager · commit a5c4b4850742 · 2025-01-03T00:42:20.000+01:00
sagemathgh-39193: Some improvements to findstat._submit * for some reason the proper header is needed to open the HTML file on my machine. * It's reported that `webbrowser.open()` doesn't work when called on a file path, rather an URI is needed (sagemath#38946) ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [x] I have updated the documentation and checked the documentation preview. (not needed) ### ⌛ Dependencies    URL: sagemath#39193 Reported by: user202729 Reviewer(s): Martin Rubey
diff --git a/src/sage/databases/findstat.py b/src/sage/databases/findstat.py
@@ -213,6 +213,7 @@ def mapping(sigma):
 
 from ast import literal_eval
 from copy import deepcopy
+from pathlib import Path
 import re
 import webbrowser
 import tempfile
@@ -491,8 +492,9 @@ def _submit(args, url):
         ....:         "CurrentEmail": ""}
         sage: _submit(args, url)                                                # optional -- webbrowser
     """
-    f = tempfile.NamedTemporaryFile(mode='w', suffix='.html', delete=False)
+    f = tempfile.NamedTemporaryFile(mode='w', suffix='.html', encoding='utf-8', delete=False)
     verbose("Created temporary file %s" % f.name, caller_name='FindStat')
+    f.write('<!doctype html>\n<html lang="en">\n<meta charset="utf-8">\n')
     f.write(FINDSTAT_POST_HEADER)
     f.write(url)
     for key, value in args.items():
@@ -506,7 +508,7 @@ def _submit(args, url):
     f.write(FINDSTAT_FORM_FOOTER)
     f.close()
     verbose("Opening file with webbrowser", caller_name='FindStat')
-    webbrowser.open(f.name)
+    webbrowser.open(Path(f.name).as_uri())
 
 
 def _data_to_str(data, domain, codomain=None):
