[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Change Request: Even newer python-fedora



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 jhu edu
                 102489: 'jmccann redhat com',
                 # Matt Domsch's rebuild script -- bz email goes to /dev/null
-                103590: ' ftbfs fedoraproject org',
+                103590: 'ftbfs fedoraproject org',
+                # Sindre Pedersen Bjørdal: foolish guezz net
+                100460 : 'sindrepb fedoraproject org',
+                # Jesus M. Rodriguez: jmrodri gmail com
+                102180: 'jesusr redhat com',
                 }
         # 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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]