See http://groups.drupal.org/node/21357
hook_openid_provider_ax($op, $attributes, $request) and
hook_openid_provider_sreg($op, $sreg_fields, $request)
is too much.
Almost the same signature, almost the same purpose. Let's unify them.

CommentFileSizeAuthor
openid_provider_ax_union_provider_hooks.patch1016 bytesaron novak

Comments

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

Also in the patchset (depending patches): #488640: Simplify provider-side hooks, #488636: Simplify provider-side hooks in D7 port

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

Also OpenID Provider Persona needs a patch: #489776: Simplify provider-side hooks at OpenID stack

xamanu’s picture

xamanu commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Fixed

I think it is fine to do this for the 7.x.-1.x version. So committed to that. Let's keep the 6.x-1.x branch as it is.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.