Author Topic: Event api delete when participants exist (Read 925 times)

Eileen · July 03, 2011, 01:58:28 pm

Hi all,
Does it make sense for the event api to delete an event if participants exists (this will delete the participants) or should it give a warning & only delete if something link

'option.deleteParticipants' is set.

If the later then this should be set to 0 as the default in the api layer & handled in the BAO layer (much like the deleteactivityassignee is in activity). Form layer apparently handles it so we'd make form layer set it to true.

http://issues.civicrm.org/jira/browse/CRM-8406?page=com.atlassian.jira.ext.fisheye%3Afisheye-issuepanel

Erik Hommel · July 04, 2011, 12:07:49 am

I would expect the default behaviour to be that the event is only deleted if there are no participants left (so the latter) but I can imagine it would be useful to have a param to delete the event AND all related participants?

xavier · July 04, 2011, 12:13:56 am

Probably because I'm not too keen on introducing more magic params, but can't we achieve the same with some chaining ?

Code: [Select]

api.participant.delete ({event_id:42,
api.event.delete});

?

Yeap, deleting all the participants is a bit brutal as the default behaviour.

X+

Erik Hommel · July 04, 2011, 12:18:01 am

Agree, chaining would be best......magic params only when no other option available.

Eileen · July 04, 2011, 01:31:36 am

well it turns out the BAO default is to delete away so we are battling with that - unless we want to do the check in the API & give an error there - which is also a crap option & not in-line with what we are trying to do with the api

Erik Hommel · July 04, 2011, 01:40:17 am

so really we should raise an issue and change the BAO?

Eileen · July 04, 2011, 01:42:50 am

tried that - see issue above

Erik Hommel · July 04, 2011, 01:52:03 am

Let me get this right: it was not fixed or the fix did not work?

Eileen · July 04, 2011, 01:53:39 am

It was fixed in that there was another error with event delete that was fixed.

However, test was fixed to say that the correct outcome is success when participants exist - i.e. blow em away without a second thought

Erik Hommel · July 04, 2011, 01:57:03 am

Hmmm so really we need a new issue :-)

CiviCRM Community Forums (archive)

News:

Author Topic: Event api delete when participants exist (Read 925 times)

Eileen

Event api delete when participants exist

Erik Hommel

Re: Event api delete when participants exist

xavier

Re: Event api delete when participants exist

Erik Hommel

Re: Event api delete when participants exist

Eileen

Re: Event api delete when participants exist

Erik Hommel

Re: Event api delete when participants exist

Eileen

Re: Event api delete when participants exist

Erik Hommel

Re: Event api delete when participants exist

Eileen

Re: Event api delete when participants exist

Erik Hommel

Re: Event api delete when participants exist