Group Invites Option persists even though disabled friending removes invites tab

[modemlooper]

To reproduce:

deactivate friending component
visit group settings (option to choose who can invite is there)
invites tab is disabled

So you can choose option but no way to access the send invites page. I think we should use a define like define( 'BP_MESSAGES_AUTOCOMPLETE_ALL', true );

To allow the send invites to stay and use all members

[modemlooper]

but the options shouldn't be there if they do nothing

[boonebgorges]

I agree that the current reliance on the Friends component is pretty crummy. But:

a) the Send Invites interface is pretty heavily dependent on autocomplete, and autocomplete performance breaks down quickly when working with lots of members. That's why BP_MESSAGES_AUTOCOMPLETE_ALL is turned off by default. (My plugin Invite Anyone enables autocomplete for all members.

b) Turning on Send Invites and allowing autocomplete for all users is a potential vector for spam. It may be unwise to have it on by default.

If we can find a way to mitigate these issues, or at least explain our way out of them :), I would be glad to change the default behavior to something more sensible.

In the meantime, an off-by-default constant or filter like the one you've suggested would be fine.

[modemlooper]

After playing with this, I think its best to just hide the options if friends is inactive. The UI would need to be reworked for an auto complete. currently its a list of checkboxes. No way to us that UI for all members.

[boonebgorges]

The UI would need to be reworked for an auto complete.

Oh right, I think I wrote the autocomplete as part of my Invite Anyone plugin.

We *should* do a rework of this UI at some point, so let's keep the ticket open. cc @dcavins

[imath]

FWIW here's an example of what could be achieved :) ​https://vimeo.com/145308971

[dcavins]

I was just going to say that @imath worked on that in his template pack. :)