Merge lp:~fjalvingh/bzr-xmloutput/expose-bugs-property into lp:bzr-xmloutput

[Frits Jalvingh]

[Frits Jalvingh (fjalvingh)]

This exposes the Bazaar 'bugs' property in the xmllog command. This property contains a list of bugtracker URLs for issues that were marked as fixed during commits using the --fixes option.

[Guillermo Gonzalez (verterok)]

Hi, sorry for the delay.

The code looks ok, but I'ld like to have an option to show the bugs property (or to not show them ;-))

Do you think you can add a --show-bugs option to xmllog and leave it on by default for log --xml?

Also, what do you think about not adding the <status> tag if there is no status (status == '')?

Thanks.

review: Needs Information

[Jelmer Vernooij (jelmer)]

review: Needs Fixing (code)

Unmerged revisions

140. By Frits Jalvingh on 2009-12-18

Expose the 'bugs' property in bzr xmllog

=== modified file 'logxml.py'
--- logxml.py	2009-07-02 16:12:44 +0000
+++ logxml.py	2009-12-18 00:58:11 +0000
@@ -139,6 +139,26 @@
             for tag in revision.tags:
                 self.to_file.write('<tag>%s</tag>' % tag)
             self.to_file.write('</tags>')
+        # [e-mail] expose 'bugs' property
+        if revision.rev.properties and 'bugs' in revision.rev.properties:
+            bugs = revision.rev.properties['bugs']
+            if len(bugs) > 0:
+                self.to_file.write('<bugs>')
+                bugar = bugs.split('\n')
+                for bug in bugar:
+                    if len(bug) > 0:
+                        self.to_file.write('<bug>')
+                        lsp = bug.rfind(' ')
+                        if lsp == -1:
+                            url = bug
+                            status = ''
+                        else:
+                            url = bug[0:lsp].rstrip()
+                            status = bug[lsp+1:].strip()
+                        self.to_file.write('<url>%s</url>' % url)
+                        self.to_file.write('<status>%s</status>' % status)
+                        self.to_file.write('</bug>')
+                self.to_file.write('</bugs>')
         if self.show_ids:
             self.to_file.write('<revisionid>%s</revisionid>' %
                                 revision.rev.revision_id)