From 92dd742e75e10d804cf1f0010c3b18e39902cdb5 Mon Sep 17 00:00:00 2001 From: Eric Price Date: Thu, 24 Jan 2008 21:32:42 -0500 Subject: [PATCH] Only update the differences (and actually work). svn path=/trunk/web/; revision=260 --- cache_acls.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/cache_acls.py b/cache_acls.py index e0094e8..ca0b7c9 100644 --- a/cache_acls.py +++ b/cache_acls.py @@ -35,11 +35,6 @@ if __name__ == '__main__': transaction = ctx.current.create_transaction() - print repr(ctx), repr(ctx.current), repr(transaction) - # Remove existing machine access entries - machine_access_table.delete() - ctx.current.flush() - try: machines = Machine.select() for m in machines: @@ -47,7 +42,11 @@ if __name__ == '__main__': people.update(expandLocker(m.owner)) people.update(expandName(m.administrator)) print '%s: %s' % (m.name, ' '.join(people)) - for p in people: + old_people = set(a.user for a in m.acl) + for removed in old_people - people: + ma = [x for x in m.acl if x.user == removed][0] + ctx.current.delete(ma) + for p in people - old_people: ma = MachineAccess(machine_id=m.machine_id, user=p) ctx.current.save(ma) ctx.current.flush() -- 1.7.9.5