Change Request: Even newer python-fedora

[Toshio Kuratomi]

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.)

-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: [EMAIL PROTECTED]
                 102489: '[EMAIL PROTECTED]',
                 # Matt Domsch's rebuild script -- bz email goes to /dev/null
-                103590: ' [EMAIL PROTECTED]',
+                103590: '[EMAIL PROTECTED]',
+                # Sindre Pedersen Bjørdal: [EMAIL PROTECTED]
+                100460 : '[EMAIL PROTECTED]',
+                # Jesus M. Rodriguez: [EMAIL PROTECTED]
+                102180: '[EMAIL PROTECTED]',
                 }
         # 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
 

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

signature.asc
Description: OpenPGP digital signature

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