Forum OpenACS Q&A: two suggestions for auth::create_local_account

Collapse
1: two suggestions for auth::create_local_account
Posted by Michael Totschnig on 07/14/10 04:47 PM
Hello,

1) in the method create_local_account, if there is an error while sending the email verification, this information is not included in the result, I think this is by error.
2) the parameters RegistrationRequiresEmailVerificationP and RegistrationRequiresApprovalP are retrieved without specifying a package_id. Since I am creating an account from my own package, this means that the values I have set for these parameters on the subsite are ignored.

I have posted a patch https://openacs.org/bugtracker/openacs/patch?patch_number=912 that addresses these two issues.

Michael

Collapse
2: Re: two suggestions for auth::create_local_account (response to 1)
Posted by Dave Bauer on 10/25/10 03:16 PM
How does this function if a subsite does not specify these settings, but instead should reflect the Main Subsite settings?
Collapse
3: Re: two suggestions for auth::create_local_account (response to 2)
Posted by Michael Totschnig on 01/11/11 03:10 PM
Dave,
sorry for not answering earlier. I saw your post only today. Haven't been subscribed to notifications for this forum.

Currently it would not reflect the Main Subsite settings, but take the package defaults, right? But isn't that the right thing to do, since these are acs-subsite parameters, and not system wide parameters?
If inheritance of parameters were required here, I think this is something that would have to be implemented generically on the level of the parameter API. I do not know if something like this exists, or is planned?

Michael