changeset 17322:7124f984dc8d stable

help: use the first topic name from helptable, not the longest alias This makes the 'additional help topics' list consistent with the output from keyword search (for instance subrepo/subrepos). The sorting by longest name was introduced in 3f6a4f1de43f. There might have been a good reason for it back then, but now it seems like a better idea to place the preferred name first in the list in helptable.
author Mads Kiilerich <mads@kiilerich.com>
date Wed, 01 Aug 2012 14:59:15 +0200
parents f3fd9d6802b7
children 2be2a070f294
files mercurial/commands.py mercurial/help.py mercurial/hgweb/webcommands.py tests/test-help.t
diffstat 4 files changed, 8 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/commands.py	Wed Aug 01 00:18:23 2012 +0200
+++ b/mercurial/commands.py	Wed Aug 01 14:59:15 2012 +0200
@@ -3297,7 +3297,7 @@
             rst.append(_("\nadditional help topics:\n\n"))
             topics = []
             for names, header, doc in help.helptable:
-                topics.append((sorted(names, key=len, reverse=True)[0], header))
+                topics.append((names[0], header))
             for t, desc in topics:
                 rst.append(" :%s: %s\n" % (t, desc))
 
--- a/mercurial/help.py	Wed Aug 01 00:18:23 2012 +0200
+++ b/mercurial/help.py	Wed Aug 01 14:59:15 2012 +0200
@@ -143,19 +143,19 @@
     (["patterns"], _("File Name Patterns"), loaddoc('patterns')),
     (['environment', 'env'], _('Environment Variables'),
      loaddoc('environment')),
-    (['revs', 'revisions'], _('Specifying Single Revisions'),
+    (['revisions', 'revs'], _('Specifying Single Revisions'),
      loaddoc('revisions')),
-    (['mrevs', 'multirevs'], _('Specifying Multiple Revisions'),
+    (['multirevs', 'mrevs'], _('Specifying Multiple Revisions'),
      loaddoc('multirevs')),
-    (['revset', 'revsets'], _("Specifying Revision Sets"), loaddoc('revsets')),
-    (['fileset', 'filesets'], _("Specifying File Sets"), loaddoc('filesets')),
+    (['revsets', 'revset'], _("Specifying Revision Sets"), loaddoc('revsets')),
+    (['filesets', 'fileset'], _("Specifying File Sets"), loaddoc('filesets')),
     (['diffs'], _('Diff Formats'), loaddoc('diffs')),
     (['merge-tools'], _('Merge Tools'), loaddoc('merge-tools')),
     (['templating', 'templates', 'template', 'style'], _('Template Usage'),
      loaddoc('templates')),
     (['urls'], _('URL Paths'), loaddoc('urls')),
     (["extensions"], _("Using Additional Features"), extshelp),
-    (["subrepo", "subrepos"], _("Subrepositories"), loaddoc('subrepos')),
+    (["subrepos", "subrepo"], _("Subrepositories"), loaddoc('subrepos')),
     (["hgweb"], _("Configuring hgweb"), loaddoc('hgweb')),
     (["glossary"], _("Glossary"), loaddoc('glossary')),
     (["hgignore", "ignore"], _("Syntax for Mercurial Ignore Files"),
--- a/mercurial/hgweb/webcommands.py	Wed Aug 01 00:18:23 2012 +0200
+++ b/mercurial/hgweb/webcommands.py	Wed Aug 01 14:59:15 2012 +0200
@@ -931,8 +931,7 @@
     if not topicname:
         def topics(**map):
             for entries, summary, _ in helpmod.helptable:
-                entries = sorted(entries, key=len)
-                yield {'topic': entries[-1], 'summary': summary}
+                yield {'topic': entries[0], 'summary': summary}
 
         early, other = [], []
         primary = lambda s: s.split('|')[0]
--- a/tests/test-help.t	Wed Aug 01 00:18:23 2012 +0200
+++ b/tests/test-help.t	Wed Aug 01 14:59:15 2012 +0200
@@ -785,7 +785,7 @@
    extensions Using Additional Features
    glossary   Glossary
    phases     Working with Phases
-   subrepo    Subrepositories
+   subrepos   Subrepositories
    urls       URL Paths
   
   Commands: