Page MenuHomePhabricator

Error message when organizer tries to remove participant [HIGH PRIORITY]
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Enable registration on event page
  • Have a participant register
  • Go to EventDetails for page
  • Select participant to remove
  • You will notice that there is an error message
  • Note that it seems like 'Remove all participants' seems to work, but then the participant is still displayed as registered on the event page

What happens?:

Error message displayed and organizer unable to remove participant

What should have happened instead?:

The participant is removed from the event list

Other information (browser name/version, screenshots, etc.):

Removing one participant:
{F35419443}

Removing all participants:

Screen Shot 2022-08-10 at 9.49.09 AM.png (1ร—2 px, 227 KB)

Participants still displayed on event page, even though they were supposedly removed via 'remove all':

Screen Shot 2022-08-10 at 9.49.18 AM.png (1ร—2 px, 249 KB)

Event Timeline

Restricted Application added a subscriber: Aklapper. ยท View Herald TranscriptAug 10 2022, 1:46 PM
ifried renamed this task from Error message when organizer tries to remove participant to Error message when organizer tries to remove participant [HIGH PRIORITY].Aug 10 2022, 1:47 PM
ifried updated the task description. (Show Details)

Change 822088 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] Fix TypeError in RemoveParticipantsFromEventHandler

https://gerrit.wikimedia.org/r/822088

Change 822088 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Fix TypeError in RemoveParticipantsFromEventHandler

https://gerrit.wikimedia.org/r/822088

(@ifried: If this is "High Priority" when why not set the existing Priority field, instead of cluttering the task title?)

@Aklapper I added it to the title to make it very clear for the team members who may be looking at tickets on our sprint board. Thanks for adding 'High' in the priority field.

vaughnwalters subscribed.

users are correctly deleted now and there are no errors thrown:


removing one participant:

remove one participant.mov-D5F84CF3-EEC0-4C82-AD86-4489301624C5.gif (536ร—720 px, 1 MB)


removing multiple participants:

Screen Recording 2022-08-10 at 2.55.00 PM.mov-18612911-0DD5-4333-A67B-D5A1C5B72E17.gif (472ร—720 px, 2 MB)


removing all participants using select all:

delete all users.mov-2F9E9E78-3B47-42D2-ADDD-A59B5365BD95.gif (536ร—720 px, 1 MB)


โœ… participants are successfully removed from lists

I have also tested this on the beta cluster, and it looks good. I am marking it as Done.