Merge lp:~jcsackett/launchpad/too-many-findAP-methods into lp:launchpad

[j.c.sackett]

Removes redundant and unused method for finding accesspolicies.

Summary
=======
There is a redundant and unused method, `AccessPolicy.findByPillarAndGrantee`.
It is exactly the same as `AccessPolicy.findByPillar`

Implementation
==============
The redundant method isn't used anywhere and isn't defined on the interface.
It has been deleted.

Tests
=====
bin/test -vvct access

QA
==
None; this method had no callsites.

LoC
===
This only removes code.

Lint
====

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/registry/model/accesspolicy.py

[Richard Harding (rharding)]

Awesome cleanup.

review: Approve

=== modified file 'lib/lp/registry/model/accesspolicy.py'
--- lib/lp/registry/model/accesspolicy.py	2012-09-11 14:06:21 +0000
+++ lib/lp/registry/model/accesspolicy.py	2012-09-18 18:24:25 +0000
@@ -264,13 +264,6 @@
             Or(*(cls.person == team for team in teams)))
 
     @classmethod
-    def findByPillarAndGrantee(cls, pillars):
-        """See `IAccessPolicySource`."""
-        return IStore(cls).find(
-            cls,
-            Or(*(cls._constraintForPillar(pillar) for pillar in pillars)))
-
-    @classmethod
     def delete(cls, pillars_and_types):
         """See `IAccessPolicySource`."""
         cls.find(pillars_and_types).remove()