[REF] remove obsolete code.
authoreileen <emcnaughton@wikimedia.org>
Tue, 15 Oct 2019 08:50:26 +0000 (21:50 +1300)
committereileen <emcnaughton@wikimedia.org>
Tue, 15 Oct 2019 08:50:46 +0000 (21:50 +1300)
On trying to look into a bug around this code I realised that there is quite a bit of code that does nothing.

These lines exist to build the  variable - which is never used. Hence they can go.

I also spotted scriptArray is never used either - this stuff came from svn before everyone was using IDEs that
highlighted unused lines

CRM/Event/Form/Registration/AdditionalParticipant.php

index 25f273cbaf718775a44b6091d1fd217a3bc51ddd..b09b13ff3b806a40f6a52bc1ad2c7ab5ca05c09c 100644 (file)
@@ -180,32 +180,6 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
     if ($this->_values['event']['is_monetary']) {
       CRM_Event_Form_Registration_Register::buildAmount($this);
     }
-    $first_name = $last_name = NULL;
-    $pre = $post = [];
-    foreach ([
-      'pre',
-      'post',
-    ] as $keys) {
-      if (isset($this->_values['additional_custom_' . $keys . '_id'])) {
-        $this->buildCustom($this->_values['additional_custom_' . $keys . '_id'], 'additionalCustom' . ucfirst($keys));
-        $$keys = CRM_Core_BAO_UFGroup::getFields($this->_values['additional_custom_' . $keys . '_id']);
-      }
-      foreach ([
-        'first_name',
-        'last_name',
-      ] as $name) {
-        if (array_key_exists($name, $$keys) &&
-          CRM_Utils_Array::value('is_required', CRM_Utils_Array::value($name, $$keys))
-        ) {
-          $$name = 1;
-        }
-      }
-    }
-
-    $required = ($button == 'skip' ||
-      $this->_values['event']['allow_same_participant_emails'] == 1 &&
-      ($first_name && $last_name)
-    ) ? FALSE : TRUE;
 
     //add buttons
     $js = NULL;