From 95452c13139b82c2dc6bb7d4927117319de05681 Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Wed, 9 Sep 2020 01:36:44 -0700 Subject: [PATCH] doc: wikiparser: Fix issue with some URL containing dup. lang part When getting context in included images, the original context might already have a language component. Don't add it again if it does. Signed-off-by: Sunil Mohan Adapa --- doc/scripts/wikiparser.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/doc/scripts/wikiparser.py b/doc/scripts/wikiparser.py index 17e6c2fd0..9201929f1 100644 --- a/doc/scripts/wikiparser.py +++ b/doc/scripts/wikiparser.py @@ -1986,6 +1986,12 @@ def get_context(file_path, file_title=None): {'path': PosixPath('manual/es/some-page'), \ 'name': 'some-page', \ 'language': 'es', \ +'title': 'es/FreedomBox/some-page'} + + >>> get_context(Path('manual/es/some-page'), 'es/FreedomBox/some-page') + {'path': PosixPath('manual/es/some-page'), \ +'name': 'some-page', \ +'language': 'es', \ 'title': 'es/FreedomBox/some-page'} """ page_name = Path(file_path.stem).stem @@ -2001,7 +2007,8 @@ def get_context(file_path, file_title=None): break title = file_title or f'FreedomBox/Manual/{page_name}' - if language != DEFAULT_LANGUAGE: + if title.partition('/')[0] not in LANGUAGES and \ + language != DEFAULT_LANGUAGE: title = f'{language}/{title}' context = {