CRM-14069 Allow setting event price set ids by api

Joe Murray added a comment - 2014-01-12 14:19 I think our discussions have moved us towards the approach of calling a new higher level, more encompassing api, currently still called contribution.create, that would do the financial stuff as well as create the event participants, using line_items, not price_set_id, as the params: http://forum.civicrm.org/index.php/topic,31242.msg134061.html#msg134061 .

Eileen McNaughton added a comment - 2014-01-12 21:12 this ticket & the commit against it is specific to the creation of an event rather than participants - ie, when you set up an event you should be able to link to a price set

Eileen McNaughton added a comment - 2014-01-31 19:22 https://github.com/eileenmcnaughton/civicrm-core/commit/a9a754ff5324de9d66b36ccb37abc79d4a2716d6

Eileen McNaughton added a comment - 2014-01-31 19:37 https://github.com/civicrm/civicrm-core/pull/2326

Eileen McNaughton added a comment - 2014-05-15 23:39 I've refreshed the PR - I don't think there is any fundamental reason we can't keep this code - the functionality it adds is described here + function testCreatePaidEvent() { + //@todo alter API so that an integer is converted to an array + $priceSetParams = array('price_set_id' => (array) 1, 'is_monetary' => 1); + $result = $this->callAPISuccess('Event', 'Create', array_merge($this->_params[0], $priceSetParams)); + $event = $this->callAPISuccess('Event', 'getsingle', array('id' => $result['id'], 'return' => 'price_set_id')); + $this->assertArrayKeyExists('price_set_id', $event); + }

David Greenberg added a comment - 2014-06-16 20:38 Event api tests are running w/o error. Event create UI is working as expected.