Minor invitation template adjustment

[dcavins]

Moves the nonce and submit button out of invites-loop.php (that is the piece that is refreshed if using bp-legacy) to the wrapper file send-invites.php.

[DJPaul]

If someone has modified one template file in their theme, they could end up with two submit button. I am inclined to think we should leave this alone unless @dcavins you have a good reason as to why we should make the change (I acknowledge it's what the template ought to have been like originally).

[dcavins]

Replying to DJPaul:

If someone has modified one template file in their theme, they could end up with two submit button. I am inclined to think we should leave this alone unless @dcavins you have a good reason as to why we should make the change (I acknowledge it's what the template ought to have been like originally).

Yes, there's the possibility of users having modified one or the other template parts and getting no submit buttons or two. Happily, that problem would be somewhat obvious at least.

I have no good reason beyond too much is happening in the AJAX request and recreating the submit button/nonce could cause heartache if a user is trying to bind javascript to those DOM objects. That said, it would be nice to fix but isn't currently breaking anything.

[DJPaul]

Need more dev opinion on whether we should make this change or not, please.

[boonebgorges]

I lean toward going with the fix suggested by dcavins. It's going to be an extremely small number of BP installations that have overridden one but not both of these templates.

[DJPaul]

@dcavins if this patch still applies, I think consensus is that you can go ahead and commit it for 2.3 if you have time?

[dcavins]

In 9854:

Change placement of invite nonce and submit.

Move the nonce input and the submit button out of
invites-loop.php (which is refreshed when new
invitations are created or existing invites are
removed) to the wrapper file send-invites.php.

Fixes #5924.