Fixed and improved models.Aspect().getUsers()
authorMarek Marecki <triviuss@gmail.com>
Wed, 3 Jul 2013 19:02:05 +0000 (21:02 +0200)
committerMarek Marecki <triviuss@gmail.com>
Wed, 3 Jul 2013 19:02:05 +0000 (21:02 +0200)
diaspy/models.py

index e568138f2e336ea70ac1a8ddf1b73d339c0210fa..c412d51ef461d7b5ee77194f46d6386d9c610e45 100644 (file)
@@ -59,18 +59,15 @@ class Aspect():
 
         ajax = self._connection.get('aspects/{0}/edit'.format(self.id)).text
 
-        #print(ajax)
-
         begin = ajax.find(start_regexp.search(ajax).group(0))
-        #print(0)
         end = ajax.find('</ul>')
         ajax = ajax[begin:end]
 
-        #print(ajax)
-
         usernames = [(line[17:33], line[35:-4]) for line in userline_regexp.findall(ajax)]
-        for guid, name in usernames:
-            print(name, guid)
+        for i, data in enumerate(usernames):
+            guid, name = data
+            for char in ['(', ')', '.']: name = name.replace(char, '\\'+char)
+            usernames[i] = (guid, name)
         personids = [re.compile(personid_regexp.format(name)).search(ajax).group(0) for guid, name in usernames]
         for n, line in enumerate(personids):
             i, id = -2, ''
@@ -86,8 +83,7 @@ class Aspect():
         users = []
         for i, user in enumerate(usernames):
             guid, name = user
-            if name in users_in_aspect:
-                users.append(guid)
+            if name in users_in_aspect: users.append(guid)
         return users
 
     def addUser(self, user_id):