IntenseWebs/sphinx
3
0
Fork 0
mirror of https://github.com/sphinx-doc/sphinx.git synced 2025-02-25 18:55:22 -06:00
Code Issues Packages Projects Releases Wiki Activity

refactor: linkcheck: Call write_linkstat() at the top of process_result()

Browse Source
This commit is contained in:
Takeshi KOMIYA 2021-04-29 18:19:36 +09:00
parent a7d3e9684d
commit 29038c9d4c
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
sphinx/builders/linkcheck.py
View File

@ -230,14 +230,15 @@ class CheckExternalLinksBuilder(DummyBuilder):
def process_result(self, result: CheckResult) -> None: def process_result(self, result: CheckResult) -> None:
filename = self.env.doc2path(result.docname, None) filename = self.env.doc2path(result.docname, None)
linkstat = dict(filename=filename, lineno=result.lineno, linkstat = dict(filename=filename, lineno=result.lineno,
status=result.status, code=result.code, uri=result.uri, status=result.status, code=result.code, uri=result.uri,
info=result.message) info=result.message)
if result.status == 'unchecked':
self.write_linkstat(linkstat) self.write_linkstat(linkstat)
if result.status == 'unchecked':
return return
if result.status == 'working' and result.message == 'old': if result.status == 'working' and result.message == 'old':
self.write_linkstat(linkstat)
return return
if result.lineno: if result.lineno:
logger.info('(%16s: line %4d) ', result.docname, result.lineno, nonl=True) logger.info('(%16s: line %4d) ', result.docname, result.lineno, nonl=True)
@ -246,14 +247,11 @@ class CheckExternalLinksBuilder(DummyBuilder):
logger.info(darkgray('-ignored- ') + result.uri + ': ' + result.message) logger.info(darkgray('-ignored- ') + result.uri + ': ' + result.message)
else: else:
logger.info(darkgray('-ignored- ') + result.uri) logger.info(darkgray('-ignored- ') + result.uri)
self.write_linkstat(linkstat)
elif result.status == 'local': elif result.status == 'local':
logger.info(darkgray('-local- ') + result.uri) logger.info(darkgray('-local- ') + result.uri)
self.write_entry('local', result.docname, filename, result.lineno, result.uri) self.write_entry('local', result.docname, filename, result.lineno, result.uri)
self.write_linkstat(linkstat)
elif result.status == 'working': elif result.status == 'working':
logger.info(darkgreen('ok ') + result.uri + result.message) logger.info(darkgreen('ok ') + result.uri + result.message)
self.write_linkstat(linkstat)
elif result.status == 'broken': elif result.status == 'broken':
if self.app.quiet or self.app.warningiserror: if self.app.quiet or self.app.warningiserror:
logger.warning(__('broken link: %s (%s)'), result.uri, result.message, logger.warning(__('broken link: %s (%s)'), result.uri, result.message,
@ -262,7 +260,6 @@ class CheckExternalLinksBuilder(DummyBuilder):
logger.info(red('broken ') + result.uri + red(' - ' + result.message)) logger.info(red('broken ') + result.uri + red(' - ' + result.message))
self.write_entry('broken', result.docname, filename, result.lineno, self.write_entry('broken', result.docname, filename, result.lineno,
result.uri + ': ' + result.message) result.uri + ': ' + result.message)
self.write_linkstat(linkstat)
elif result.status == 'redirected': elif result.status == 'redirected':
try: try:
text, color = { text, color = {
@ -279,7 +276,6 @@ class CheckExternalLinksBuilder(DummyBuilder):
color(' - ' + text + ' to ' + result.message)) color(' - ' + text + ' to ' + result.message))
self.write_entry('redirected ' + text, result.docname, filename, self.write_entry('redirected ' + text, result.docname, filename,
result.lineno, result.uri + ' to ' + result.message) result.lineno, result.uri + ' to ' + result.message)
self.write_linkstat(linkstat)
else: else:
raise ValueError("Unknown status %s." % result.status) raise ValueError("Unknown status %s." % result.status)