Closed (fixed)
Project:
OpenID URL
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
14 Sep 2008 at 14:41 UTC
Updated:
25 Mar 2017 at 20:41 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
robloachYou're right, saving this data in the variables table is not good. Could it save the OpenID settings in the $user->data[] column?
Comment #2
Anonymous (not verified) commentedYou would know better than me how this might affect openid delegate queries. The user's profile is the URL where this information needs to be put into the page headers. The user is fully loaded from the database anyway when the profile URL is hit, so the delegate variables would be in easy reach.
Nobody ever needs to query the delegate URLs directly?
The sitewide openid delegate can continue to stay in variables table, yes?
Comment #3
christefano commentedThis is mostly just thinking out loud but what the heck. I'd love to see this solved by using profile fields created by the core Profile module.
It's an extra dependency but I think it would be an acceptable one. The profile fields could be created automatically when OpenID URL is enabled (the same way the vCard module does it).
Comment #4
wim leersStill up for writing a patch, bangpound? :) At first sight, all you'd have to do is implement hook_user().
Comment #5
Anonymous (not verified) commentedHere's the patch to move openidurl data out of the variables table and into the user data field.
No upgrade path until we agree that this patch is acceptable. It's going to be tedious to migrate this data, I worry.
Comment #6
Anonymous (not verified) commentedComment #7
robloachThanks! Provides the update to remove the delegates from the variables table too.