CiviEvent Import - Remove broken call to parent constructor (dev/core#3932) #24790
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Resolves Issue 3932 on lab.civicrm, in which the Event Participant Import attempts to call a parent constructor that does not exist
Before
Before this patch, when a user goes to Events > Import Participants, fills out the first screen and then clicks Continue they receive a critical error.
After
With this patch, when a user follows the above steps, they move on to the second screen of the import process (field mapping).
Technical Details
Removed line 68 within the
__construct
function (aka constructor) within Participant.php that attempted to call a constructor inCRM_Import_Parser
, though one does not exist.