[PATCH V4] templater: support using templates with non-standard names from map file

[Alexander Plavin]

# HG changeset patch
# User Alexander Plavin <alexander at plav.in>
# Date 1379843538 -14400
#      Sun Sep 22 13:52:18 2013 +0400
# Node ID 439f5648c4de4a4f344dec45847eb9afdc9687c5
# Parent  4074e05bd30287df5d39d0b36a3bb42dbb8701ad
templater: support using templates with non-standard names from map file

Allow to add arbitrarily-named entries to a template map file and then reference
them, to make it possible to deduplicate and simplify templates code.

diff -r 4074e05bd302 -r 439f5648c4de mercurial/templater.py
--- a/mercurial/templater.py	Sun Sep 01 10:40:11 2013 +0400
+++ b/mercurial/templater.py	Sun Sep 22 13:52:18 2013 +0400
@@ -139,7 +139,12 @@
 def runsymbol(context, mapping, key):
     v = mapping.get(key)
     if v is None:
-        v = context._defaults.get(key, '')
+        v = context._defaults.get(key)
+    if v is None:
+        try:
+            v = context.process(key, mapping)
+        except TemplateNotFound:
+            v = ''
     if util.safehasattr(v, '__call__'):
         return v(**mapping)
     if isinstance(v, types.GeneratorType):
@@ -461,6 +466,9 @@
             stylelist.append(split[1])
     return ", ".join(sorted(stylelist))
 
+class TemplateNotFound(util.Abort):
+    pass
+
 class templater(object):
 
     def __init__(self, mapfile, filters={}, defaults={}, cache={},
@@ -512,7 +520,8 @@
             try:
                 self.cache[t] = util.readfile(self.map[t][1])
             except KeyError, inst:
-                raise util.Abort(_('"%s" not in template map') % inst.args[0])
+                raise TemplateNotFound(_('"%s" not in template map') %
+                                       inst.args[0])
             except IOError, inst:
                 raise IOError(inst.args[0], _('template file %s: %s') %
                               (self.map[t][1], inst.args[1]))
diff -r 4074e05bd302 -r 439f5648c4de tests/test-command-template.t
--- a/tests/test-command-template.t	Sun Sep 01 10:40:11 2013 +0400
+++ b/tests/test-command-template.t	Sun Sep 22 13:52:18 2013 +0400
@@ -500,6 +500,28 @@
   1
   0
 
+Missing non-standard names give no error (backward compatibility):
+
+  $ echo "changeset = '{c}'" > t
+  $ hg log --style ./t
+
+Defining non-standard name works:
+
+  $ cat <<EOF > t
+  > changeset = '{c}'
+  > c = q
+  > EOF
+  $ hg log --style ./t
+  8
+  7
+  6
+  5
+  4
+  3
+  2
+  1
+  0
+
 ui.style works:
 
   $ echo '[ui]' > .hg/hgrc