[14:46] <EdwinGrubbs> noodles785: are you reviewing today?
[14:48] <noodles785> EdwinGrubbs: I haven't been no (I'm currently working on an ISD project), but if you've something urgent I can take a look.
[14:48] <EdwinGrubbs> no, it's not urgent
[14:48] <EdwinGrubbs> nevermind
[16:02] <EdwinGrubbs> rockstar: are you reviewing today?
[16:13] <rockstar> EdwinGrubbs, indeed I am, as soon as I get my shit together for the morning.
[16:13] <rockstar> EdwinGrubbs, go ahead and throw yourself on queue.
[16:14] <EdwinGrubbs> here's the mp: https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-568390-cyclic-membership-error/+merge/29407
[17:35] <rockstar> EdwinGrubbs, maybe validateOwner's abstract method should return False or raise NotImplementedError.
[17:37] <EdwinGrubbs> rockstar: it's not required that validateOwner() be overridden. The vocabulary normally does a good enough job. This is the only subclass that actually does override it.
[17:37] <EdwinGrubbs> so raising NotImplementedError isn't necessary, and returning false would be meaningless.
[17:38] <rockstar> EdwinGrubbs, pass seems like it's very unopinionated
[17:39] <EdwinGrubbs> rockstar: it's no different than LaunchpadFormView.validate().
[17:44] <rockstar> EdwinGrubbs, okay.