/srv/irclogs.ubuntu.com/2015/05/23/#launchpad-dev.txt

cjwatsonwgrant: Is it intentional that AccessArtifact has FKs for bug and specification, but not branch?12:09
cjwatsonwgrant: Just tracking down the IntegrityError I found when QAing https://bugs.launchpad.net/launchpad/+bug/1456583, and wondering why Branch.destroySelf apparently doesn't need to clean up its AAs12:10
mupBug #1456583: Can't delete Git repositories <feature> <git> <qa-needstesting> <Launchpad itself:Fix Committed by cjwatson> <https://launchpad.net/bugs/1456583>12:10
cjwatsonI wonder if there are leftover AA rows for now-nonexistent branches as a result.12:12
wgrant    bug integer REFERENCES bug,12:20
wgrant    branch integer, -- FK to be added later.12:20
wgrantSomething must have been checking for branch FKs.12:22
wgrantAPGF.grantee is the only other one from that patch that wasn't fixed later.12:24
wgrantOh, no12:36
wgrantAPGF.grantee is deliberately not FKed, I think, because it shouldn't be touched by person merge.12:36
wgrantIt's purely maintained by triggers.12:36
cjwatsonSo do I need something like getUtility(IAccessArtifactSource).delete([self]) ?13:10
cjwatsonOne of these days I will properly understand the access model :)13:11

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!