I've found that the setting of the "Require administrator approval for invitees" checkbox on the admin/config/people/invite page is ignored when users register. I've added a bit of logic to the hook_user_presave() implementation to check the value of this variable, and set the new user's status accordingly.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

spiderman created an issue. See original summary.

spiderman’s picture

ckng’s picture

Assigned: spiderman » Unassigned
Status: Active » Needs review
jacob.embree’s picture

Status: Needs review » Needs work

This option may not be useful at all. The option to require approval is at admin/config/people/accounts. If the user configuration page requires users to get admin approval then the checkbox at admin/config/people/invite is redundant. When only administrators can create accounts the accept link gets you a 403. When visitors may register there's no reason to make invitations a special case, requiring approval for invited users only. When only invited users may register admin approval is already required.

If a maintainer agrees with me I'll submit a patch to remove the option.

suffering drupal’s picture

What do you mean "not useful"? VERY USEFUL if it worked.

We do NOT want just anyone to sign up (no sign up form for anonymous users)
But we DO want to to make it as fluently as possible for invited people. So without needing approval!
That's the whole reason of being invited.
In fact, without this option, it's the module in itself that makes no sense.

"The option to require approval is at admin/config/people/accounts."
and the only option that gives sense to "invite by email" would in fact be
New user registration by invitation only, free from administrator approval.
Absolutely not redundant.

So after this bit of user-logic and friendliness...

I didn't try "Invite" by itself. But if the Require approval works there, isn't it simply "Invite by mail" that overlooks it?
Or else add the option I mentioned above to make invite by mail useful.

I am scared of patches, but will have a look at this one.

codingleechallenged’s picture

patch works for me.
thank you.

renatog’s picture

Category: Bug report » Feature request
Status: Needs work » Reviewed & tested by the community

It really makes sense and works good (but as Feature request)

  • RenatoG committed 609ecdc on 7.x-4.x authored by spiderman
    Issue #2600164 by spiderman, ckng, jacob.embree, suffering drupal,...
renatog’s picture

Status: Reviewed & tested by the community » Fixed

Committed to the dev branch

@jacob.embree, if you have a better idea about it please attach one patch with your suggestion (usually your solutions are veery good)

Thanks for contribution guys.

Best,

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.