CiviCRM Community Forums (archive)

Hi,

I am new to the Forum and have a question:

I found a bug in the CiviCRM-Export and know how to fix it. What are the next steps?
I searched in the IssueTracker and did not find a reported bug.
Should I create a new Issue?


Here is the bug and the fix:

When exporting a contact with a relationship and using an export-sheme, for Instance: the postal-code of an employer.

You choose which fields to export:

THIS WORKS:
 primary address: postal code
 private address: postal code
 billing address: postal code
 related employer: primary address: postal code
 related employer: private address: postal code
 related employer: billing address: postal code

The Result is a XLS with 6 Fields

DOES NOT WORK:
 related employer: primary address: postal code
 related employer: private address: postal code
 related employer: billing address: postal code
 primary address: postal code
 private address: postal code
 billing address: postal code

The Result is a XLS with only 4 Rows.

REASON:
CRM/Export/BAO/Export.php has a bug.
The variable $relationField is not cleared in the foreach-loop and therefore still contains a value from the last iteration. This bug is encouraged by the fact, that this variable is set with a scope for the whole function, instead of only the loop.

FIX:
limit the scope:
change line 87
 $queryMode  = $relationField = null;
to
 $queryMode  = null;

clear the variable at start of the loop:
after line 145
  $phoneTypeId = $imProviderId = null;
insert:
 $relationField = null;