Patchwork D6441: help: check if a subtopic exists and raise an error if it doesn't (issue6145)

login
register
mail settings
Submitter phabricator
Date May 26, 2019, 9:06 p.m.
Message ID <6509e2d5c0f408df5c10f33bf675089b@localhost.localdomain>
Download mbox | patch
Permalink /patch/40243/
State Not Applicable
Headers show

Comments

phabricator - May 26, 2019, 9:06 p.m.
ngoldbaum updated this revision to Diff 15268.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6441?vs=15258&id=15268

REVISION DETAIL
  https://phab.mercurial-scm.org/D6441

AFFECTED FILES
  mercurial/help.py

CHANGE DETAILS




To: ngoldbaum, #hg-reviewers, martinvonz
Cc: pulkit, martinvonz, av6, mercurial-devel

Patch

diff --git a/mercurial/help.py b/mercurial/help.py
--- a/mercurial/help.py
+++ b/mercurial/help.py
@@ -689,6 +689,8 @@ 
             for names, header, doc in subtopics[name]:
                 if subtopic in names:
                     break
+            if not any(subtopic in s[0] for s in subtopics[name]):
+                raise error.UnknownCommand(name)
 
         if not header:
             for topic in helptable: