From 8835e3f13376a288abded427ddfa78d06d2af653 Mon Sep 17 00:00:00 2001
From: Paul Franklin <pf98052@gmail.com>
Date: Thu, 21 Mar 2013 20:45:59 +0000
Subject: [PATCH] slight tweaks to booklist format

svn: r21718
---
 gramps/gen/plug/report/_book.py | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/gramps/gen/plug/report/_book.py b/gramps/gen/plug/report/_book.py
index 47d4ba38c..bb37ead68 100644
--- a/gramps/gen/plug/report/_book.py
+++ b/gramps/gen/plug/report/_book.py
@@ -329,9 +329,10 @@ class BookList(object):
                 f.write('  <item name="%s" trans_name="%s">\n' % 
                             (item.get_name(),item.get_translated_name() ) )
                 options = item.option_class.handler.options_dict
-                for option_name, option_value in options.items():
+                for option_name in sorted(options.keys()): # enable a diff
+                    option_value = options[option_name]
                     if isinstance(option_value, (list, tuple)):
-                        f.write('  <option name="%s" value="" '
+                        f.write('    <option name="%s" value="" '
                                 'length="%d">\n' % (
                                     escape(option_name),
                                     len(options[option_name]) ) )
@@ -339,21 +340,21 @@ class BookList(object):
                             option_type = type_name(option_value[list_index])
                             value = escape(cuni(option_value[list_index]))
                             value = value.replace('"', '&quot;')
-                            f.write('    <listitem number="%d" type="%s" '
+                            f.write('      <listitem number="%d" type="%s" '
                                     'value="%s"/>\n' % (
-                                    list_index,
-                                    option_type,
-                                    value ) )
-                        f.write('  </option>\n')
+                                        list_index,
+                                        option_type,
+                                        value ) )
+                        f.write('    </option>\n')
                     else:
                         option_type = type_name(option_value)
                         value = escape(cuni(option_value))
                         value = value.replace('"', '&quot;')
-                        f.write('  <option name="%s" type="%s" '
+                        f.write('    <option name="%s" type="%s" '
                                 'value="%s"/>\n' % (
-                                escape(option_name),
-                                option_type,
-                                value) )
+                                    escape(option_name),
+                                    option_type,
+                                    value) )
 
                 f.write('    <style name="%s"/>\n' % item.get_style_name() )
                 f.write('  </item>\n')