Change Request: Even newer python-fedora

[Toshio Kuratomi <a.badger@xxxxxxxxx>]

In testing the bugzilla sync scripts on Fedora Servers I found a few 
bugs in python-fedora's bugzilla handling (one's an update of the user 
to bugzilla email mapping.  The other is returning bugzilla email 
addresses from one piece of the API.)

I need to get a new python-fedora on the app servers to address these 
issues.  Patch with the proposed code changes is attached.
-Toshio

=== modified file 'fedora/accounts/fas2.py'
--- fedora/accounts/fas2.py	2008-04-12 20:10:55 +0000
+++ fedora/accounts/fas2.py	2008-04-24 00:08:20 +0000
@@ -73,7 +73,11 @@
                 # William Jon McCann: mccann@xxxxxxx
                 102489: 'jmccann@xxxxxxxxxx',
                 # Matt Domsch's rebuild script -- bz email goes to /dev/null
-                103590: ' ftbfs@xxxxxxxxxxxxxxxxx',
+                103590: 'ftbfs@xxxxxxxxxxxxxxxxx',
+                # Sindre Pedersen Bjørdal: foolish@xxxxxxxxx
+                100460 : 'sindrepb@xxxxxxxxxxxxxxxxx',
+                # Jesus M. Rodriguez: jmrodri@xxxxxxxxx
+                102180: 'jesusr@xxxxxxxxxx',
                 }
         # A few people have an email account that is used in owners.list but
         # have setup a bugzilla account for their primary account system email
@@ -174,9 +178,14 @@
         # Retrieve further useful information about the users
         request = self.send_request('/group/dump', auth=True)
         for user in request['people']:
-            username = userToId[user[0]]
-            people[username]['email'] = user[1]
-            people[username]['human_name'] = user[2]
+            userid = userToId[user[0]]
+            person = people[userid]
+            person['email'] = user[1]
+            person['human_name'] = user[2]
+            if userid in self.__bugzilla_email:
+                person['bugzilla_email'] = self.__bugzilla_email[userid]
+            else:
+                person['bugzilla_email'] = person['email']
 
         return people

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Fedora-infrastructure-list mailing list
Fedora-infrastructure-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list