After #399746: Refactor dependencies AX - Persona I'd propose to do the same for SREG. SREG could invoke a hook_openid_provider_sreg (after the model of hook_openid_provider_ax) that OpenID Provider Persona is an implementer of.

Would this make sense?

CommentFileSizeAuthor
#2 430332_decouple_from_persona.patch1.53 KBaron novak

Comments

darren.ferguson’s picture

darren.ferguson commented

Yes definitely makes sense so we can perform sreg functionality

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Active » Needs review
StatusFileSize
new 430332_decouple_from_persona.patch1.53 KB
aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

Also take care of #434378: Move mapping functionality to openid_cp_field
and i need to prepare a patch for openid_provider_persona.

darren.ferguson’s picture

darren.ferguson commented

Aron the above patch is now in CVS for the module

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Needs review » Fixed

Awesome, thank you Darren!
This means i mark this fixed. I'm sure for SReg at this module that patch contained everything that we need.

darren.ferguson’s picture

darren.ferguson commented
Status: Fixed » Closed (fixed)