[game-data-packager] 04/11: GameData.to_yaml: omit keys with trivial values

[Simon McVittie]

This is an automated email from the git hooks/post-receive script.

smcv pushed a commit to branch master
in repository game-data-packager.

commit a57557e7f5d3ba9e343124b0e58a9285c17e7429
Author: Simon McVittie <smcv at debian.org>
Date:   Tue Nov 3 08:39:16 2015 +0000

    GameData.to_yaml: omit keys with trivial values
---
 game_data_packager/__init__.py | 44 +++++++++++++++++++++++++++++-------------
 1 file changed, 31 insertions(+), 13 deletions(-)

diff --git a/game_data_packager/__init__.py b/game_data_packager/__init__.py
index cb77466..50807e6 100644
--- a/game_data_packager/__init__.py
+++ b/game_data_packager/__init__.py
@@ -667,6 +667,7 @@ class GameData(object):
         files = {}
         groups = {}
         packages = {}
+        ret = {}
 
         def sort_set_values(d):
             ret = {}
@@ -684,19 +685,36 @@ class GameData(object):
         for name, package in self.packages.items():
             packages[name] = package.to_yaml()
 
-        return {
-            'copyright_notice': self.copyright_notice,
-            'help_text': self.help_text,
-            'known_filenames': sort_set_values(self.known_filenames),
-            'known_md5s': sort_set_values(self.known_md5s),
-            'known_sha1s': sort_set_values(self.known_sha1s),
-            'known_sha256s': sort_set_values(self.known_sha256s),
-            'known_sizes': sort_set_values(self.known_sizes),
-            'packages': packages,
-            'providers': sort_set_values(self.providers),
-            'files': files,
-            'groups': groups,
-        }
+        if files:
+            ret['files'] = files
+
+        if groups:
+            ret['groups'] = groups
+
+        if packages:
+            ret['packages'] = packages
+
+        for k in (
+                'known_filenames',
+                'known_md5s',
+                'known_sha1s',
+                'known_sha256s',
+                'known_sizes',
+                'providers',
+                ):
+            v = getattr(self, k)
+            if v:
+                ret[k] = sort_set_values(v)
+
+        for k in (
+                'copyright_notice',
+                'help_text',
+                ):
+            v = getattr(self, k)
+            if v is not None:
+                ret[k] = v
+
+        return ret
 
     def size(self, package):
         size_min = 0

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-games/game-data-packager.git