Download & Extend
OpenID Provider SREG » Issues

Make SREG independent of Persona

Posted by alex_b on April 10, 2009 at 8:36pm
Project:OpenID Provider SREG
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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?

Login or register to post comments

Comments

#1

Posted by darren.ferguson on April 10, 2009 at 9:33pm

Yes definitely makes sense so we can perform sreg functionality

#2

Posted by Aron Novak on April 20, 2009 at 1:02pm
Status:active» needs review
AttachmentSize
430332_decouple_from_persona.patch 1.53 KB

#3

Posted by Aron Novak on April 20, 2009 at 1:03pm

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

#4

Posted by darren.ferguson on April 20, 2009 at 2:01pm

Aron the above patch is now in CVS for the module

#5

Posted by Aron Novak on April 20, 2009 at 8:27pm
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.

#6

Posted by darren.ferguson on April 20, 2009 at 8:41pm
Status:fixed» closed (fixed)
nobody click here