+editpgpkeys can have an action that is not a string

[Curtis Hovey]

OOPS-1502N4471 shows that getattr(self, action) was called and the action was not a string, it is None.

PersonGPGView.form_action looks like:
    def form_action(self):
        permitted_actions = ['claim_gpg', 'deactivate_gpg',
                             'remove_gpgtoken', 'reactivate_gpg']
        if self.request.method != "POST":
            return ''
        action = self.request.form.get('action')
        if action and (action not in permitted_actions):
            raise UnexpectedFormData("Action was not defined")
        getattr(self, action)()

I cannot see how this happened, but for the view to work as we intend, it should do:
        if action not in permitted_actions:
            raise UnexpectedFormData("Action was not defined")

[Ursula Junque (ursinha)]

Fixed in stable r10904 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/10904>