CiviCRM Community Forums (archive)

*

News:

Have a question about CiviCRM?
Get it answered quickly at the new
CiviCRM Stack Exchange Q+A site

This forum was archived on 25 November 2017. Learn more.
How to get involved.
What to do if you think you've found a bug.



  • CiviCRM Community Forums (archive) »
  • Old sections (read-only, deprecated) »
  • Discussion (deprecated) »
  • Alpha and Beta Release Testing »
  • 4.6 Release Testing »
  • CRM-5039 Concurrent event bookings [FIXED]
Pages: [1]

Author Topic: CRM-5039 Concurrent event bookings [FIXED]  (Read 639 times)

rocxa

  • I post occasionally
  • **
  • Posts: 40
  • Karma: 4
  • CiviCRM version: 4.5.5
  • CMS version: Drupal 7.34
  • MySQL version: 5.1.71
  • PHP version: 5.3.3
CRM-5039 Concurrent event bookings [FIXED]
January 08, 2015, 09:50:28 am
The issue described here:

https://issues.civicrm.org/jira/browse/CRM-5039

has been marked as fixed but unfortunately it is not.

I have just tested the 4.6alpha release by booking from two different browsers trying to book 2 different groups of 3 people on to this event.

http://dmaster.demo.civicrm.org/civicrm/event/info?reset=1&id=7

radio1 means 1 space..
radio2 means 2 spaces etc..

If you get both bookers to the confirm page with options that will oversell.. and then click confirm on both.. Only one will get the spaces.. (great so far).. the other booker will get returned to the start of the booking process but won't be able to continue as the options that are sold out are still checked but disabled (see screenshots).

A secondary issue to this is also when someone is paying offsite on a transfer type payment gateway, someone could pay faster than them and the slow person would end up paying for spaces that have sold out while they were offsite.  The way around that is to include 'pending from incomplete transaction' status as counted towards max participants.  This requires clearing pending bookings frequently for busy events.



« Last Edit: March 12, 2015, 05:02:00 am by rocxa »

Donald Lobo

  • Administrator
  • I’m (like) Lobo ;)
  • *****
  • Posts: 15963
  • Karma: 470
    • CiviCRM site
  • CiviCRM version: 4.2+
  • CMS version: Drupal 7, Joomla 2.5+
  • MySQL version: 5.5.x
  • PHP version: 5.4.x
Re: CRM-5039 Concurrent event bookings
January 08, 2015, 10:55:05 am

would really help if your org can contribute code to help investigate and fix these issues

lobo
A new CiviCRM Q&A resource needs YOUR help to get started. Visit our StackExchange proposed site, sign up and vote on 5 questions

rocxa

  • I post occasionally
  • **
  • Posts: 40
  • Karma: 4
  • CiviCRM version: 4.5.5
  • CMS version: Drupal 7.34
  • MySQL version: 5.1.71
  • PHP version: 5.3.3
Re: CRM-5039 Concurrent event bookings
January 09, 2015, 08:28:49 am
I did actually provide the original fix as a pull request (see early comments on the issue) but it seems some of the complexity of the fix was missed when it was taken on by someone else.

The main stumbling block we had trying to incorporate the fix into civicrm properly was the part that  isn't working with this new fix.  The need to remove the sold out options from the each of the participants in the bookers session so that they can't be in the situation where the options are sold out but still selected.

The only way we found to do it was to remove the options directly from the session.. but there is almost certainly a less flaky way to do it as the session layout seemed to change depending on whether at the beginning of registration, on a 2nd/3rd participant or on the confirm stage.

John.K

  • I post occasionally
  • **
  • Posts: 75
  • Karma: 5
  • CiviCRM version: 4.x
  • CMS version: Drupal 7
  • MySQL version: 5.x
  • PHP version: 5
[FIXED] CRM-5039 Concurrent event bookings
March 12, 2015, 04:59:58 am
Marking this as fixed

Pages: [1]
  • CiviCRM Community Forums (archive) »
  • Old sections (read-only, deprecated) »
  • Discussion (deprecated) »
  • Alpha and Beta Release Testing »
  • 4.6 Release Testing »
  • CRM-5039 Concurrent event bookings [FIXED]

This forum was archived on 2017-11-26.