Merge lp:~zyga/checkbox/fix-exporter-regression into lp:checkbox

[Zygmunt Krynicki]

This fixes a regression introduced with exporter units. Any script that today uses -f json (etc) should work forever. We keep our public APIs small (cli tools and file formats) but precious.

[Maciej Kisielewski (kissiel)]

Tested, looks good.
overall +1

review: Approve

=== modified file 'plainbox/plainbox/impl/session/manager.py'
--- plainbox/plainbox/impl/session/manager.py	2015-06-25 11:52:05 +0000
+++ plainbox/plainbox/impl/session/manager.py	2015-07-02 20:16:55 +0000
@@ -470,4 +470,16 @@
                 support = unit.support
                 if support:
                     exporter_map[unit.id] = support
+        # Patch exporter map to expose short names
+        legacy_mapping = {
+            '2013.com.canonical.plainbox::hexr': 'xml',
+            '2013.com.canonical.plainbox::html': 'html',
+            '2013.com.canonical.plainbox::json': 'json',
+            '2013.com.canonical.plainbox::rfc822': 'rfc822',
+            '2013.com.canonical.plainbox::text': 'text',
+            '2013.com.canonical.plainbox::xlsx': 'xlsx'
+        }
+        for new_id, legacy_id in legacy_mapping.items():
+            if new_id in exporter_map:
+                exporter_map[legacy_id] = exporter_map[new_id]
         return exporter_map