withdrawing players from roster [#1093432]

When you try to "withdrawal" a player (who has a user account) from a roster no email is sent. However, the player is successfully removed from the roster.

The error displayed at the top of the page is
"Unable to send e-mail. Please contact the site administrator if the problem persists."

-----
When trying to "withdrawal" a player who doesn't have a user account from a roster a "page not found" error is displayed.

the URL of the page not found is ....
"http://domainname.com/ult/team/461/roster/invites/withdrawemailinvite/TT"

----

These aren't huge issues just thought I'd mention them.

Comment	File	Size	Author
#2	1093432-FixWithdrawEmailInvite-4290230-D6.patch	732 bytes	jthorson

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

jthorson CreditAttribution: jthorson commented 1 April 2011 at 04:08

Assigned:	Unassigned	» jthorson
Issue tags:		+ult_rosters

Missing a condition in the _ult_rosters_op_load($op) function to handle the 'withdraw email invite' scenario.

Fix: Modify line 223/224 of ult_rosters.module

Before:

if ($op=="acceptinvite" || $op=="rejectinvite" || $op=="withdrawinvite" || $op=="deleteinvite" || $op=="removeinvite" 
  || $op=="acceptapp" || $op=="rejectapp" || $op=="withdrawapp" || $op=="deleteapp" || $op=="removeapp") {

After:

if ($op=="acceptinvite" || $op=="rejectinvite" || $op=="withdrawinvite" || $op=="deleteinvite" || $op=="removeinvite" 
  || $op=="acceptapp" || $op=="rejectapp" || $op=="withdrawapp" || $op=="deleteapp" || $op=="removeapp" || $op=="withdrawemailinvite") {