CiviCRM Community Forums (archive)

I have an event set to no registration limit (i.e. 0 max participants), but after registering a single user for the event, I get an "event full" error every time I try to register another test user.  What might I be doing wrong?

Nope, can't reproduce on the demo site.  So I must be doing something wrong... or using some odd combination of configurations.

I have no clue how to track this down.  Worst case scenario, I can set the event to a ridiculously large limit, but I fear that may be too complicated to ask the staff who will be managing events to remember to do.

Well, I've tracked down the problem.

I'm using the API to create events.  If passed a NULL value (or no value at all) for max_participants, the API instead inserts 0 into the max_participants column.

Then the following code in CRM_Event_BAO_Participant::eventFull() sees a 0 instead of a NULL and decides that 1 registrant is greater than the limit of 0:

        if ( $dao->fetch( ) ) {
            if( $dao->max_participants == NULL ) {
                return false;
            }
           
            if( $dao->total_participants >= $dao->max_participants ) {
                if( $dao->event_full_text ) {
                    return $dao->event_full_text;
                } else {
                    return ts( "This event is full !!!" );
                }
            }
        }
        return false;
Presumably this could be fixed either in the API or in CRM_Event_BAO_Participant, depending on what other behavior it would break.  For an emergency workaround, I'll be duplicating CRM_Event_BAO_Participant and adding a check for 0 to the check for NULL.

Kurund - I'm wondering if the BAO should check for 0 and handle it the same as NULL (since max = 0 doesn't make sense).

can u apply the following patch, it should work (let us know if not)

svn diff CRM/Event/BAO/Participant.php
Index: CRM/Event/BAO/Participant.php
===================================================================
--- CRM/Event/BAO/Participant.php       (revision 19367)
+++ CRM/Event/BAO/Participant.php       (working copy)
@@ -296,7 +296,8 @@
         $dao =& CRM_Core_DAO::executeQuery( $query, CRM_Core_DAO::$_nullArray );
         
         if ( $dao->fetch( ) ) {
-            if( $dao->max_participants == NULL ) {
+            if( $dao->max_participants == NULL ||
+                $dao->max_participants <= 0 ) {
                 return false;

we'll incorporate this into 2.1 and future releases

lobo

thanx. this is now part of 2.1 (and 2.2 etc). issue filed and fixed here: http://issues.civicrm.org/jira/browse/CRM-4057

lobo