Merge lp:~brian-murray/launchpad/api-export-person-logo_link into lp:launchpad

[Brian Murray]

Export Person.logo in the API.

[Brian Murray (brian-murray)]

This exports logo for people and team's in the API.

[Jeroen T. Vermeulen (jtv)]

Can't think of anything that could go wrong, except perhaps tests breaking—and our process will catch that.

Thanks for the fix!

review: Approve (code)

[Jeroen T. Vermeulen (jtv)]

I just kicked off an "ec2 land" for this branch.

=== modified file 'lib/lp/registry/interfaces/person.py'
--- lib/lp/registry/interfaces/person.py	2010-02-08 14:37:50 +0000
+++ lib/lp/registry/interfaces/person.py	2010-02-24 17:36:33 +0000
@@ -508,14 +508,15 @@
             "in listings of bugs or on a person's membership table."))
     iconID = Int(title=_('Icon ID'), required=True, readonly=True)
 
-    logo = LogoImageUpload(
-        title=_("Logo"), required=False,
-        default_image_resource='/@@/person-logo',
-        description=_(
-            "An image of exactly 64x64 pixels that will be displayed in "
-            "the heading of all pages related to you. Traditionally this "
-            "is a logo, a small picture or a personal mascot. It should be "
-            "no bigger than 50kb in size."))
+    logo = exported(
+        LogoImageUpload(
+            title=_("Logo"), required=False,
+            default_image_resource='/@@/person-logo',
+            description=_(
+                "An image of exactly 64x64 pixels that will be displayed in "
+                "the heading of all pages related to you. Traditionally this "
+                "is a logo, a small picture or a personal mascot. It should be "
+                "no bigger than 50kb in size.")))
     logoID = Int(title=_('Logo ID'), required=True, readonly=True)
 
     mugshot = exported(MugshotImageUpload(

=== modified file 'lib/lp/registry/stories/webservice/xx-person.txt'
--- lib/lp/registry/stories/webservice/xx-person.txt	2010-02-05 09:02:39 +0000
+++ lib/lp/registry/stories/webservice/xx-person.txt	2010-02-24 17:36:33 +0000
@@ -28,6 +28,7 @@
     karma: 0
     languages_collection_link: u'http://.../~salgado/languages'
     latitude: None
+    logo_link: u'http://.../~salgado/logo'
     longitude: None
     mailing_list_auto_subscribe_policy: u'Ask me when I join a team'
     members_collection_link: u'http://.../~salgado/members'
@@ -74,6 +75,7 @@
     karma: 0
     languages_collection_link: u'http://.../~ubuntu-team/languages'
     latitude: None
+    logo_link: u'http://.../~ubuntu-team/logo'
     longitude: None
     mailing_list_auto_subscribe_policy: u'Ask me when I join a team'
     members_collection_link: u'http://.../~ubuntu-team/members'