Forum OpenACS Development: Response to Providing a patch and the /register/ maze

Collapse
Posted by russ m on
oops -  the "needs approval" thing would be my bad...

up untill not too long ago member_state was set to NULL for
accounts that needed approval, but that broke
[ad_registration_finite_state_machine_admin_links] which
assumed that they'd be "needs approval". Just changing back to
NULLs will re-break the user admin pages. Everything needs to
settle on one or the other, and I'd lean towards "needs approval"
rather than NULL for consistency with how the other member
states are represented.

Where else in the registration process is NULL used to mark
users who need admin approval? I ran through a couple of
registrations before submitting the patch that made things as
they are now and it all semed to work fine...