Skip to content
This repository has been archived by the owner on Mar 23, 2021. It is now read-only.

entries in trash #1

Open
woistjadefox opened this issue Sep 9, 2015 · 1 comment
Open

entries in trash #1

woistjadefox opened this issue Sep 9, 2015 · 1 comment
Labels

Comments

@woistjadefox
Copy link

Dear Mathieu!

When I export a Contribution Rapport "Constituent Summary", deleted contacts and their contributions are still shown in the excel export. In the preview the data is correct, as-well the csv export, it justs wrong in the excel export.

Have you any idea what that could be?
I also figured out that there is an additional "Internal Contact ID" column in the excel export, is this normal or eventually possible to disable?

Regards
Goran

mlutfy added a commit that referenced this issue Sep 9, 2015
@mlutfy
Copy link
Owner

mlutfy commented Sep 9, 2015

Hi Goran,

Thanks for reporting the issue. I was able to reproduce it. The current implementation of exportexcel is a bit tricky, since CiviCRM doesn't really have a good API to extend export mechanisms in report/search.

  • the deleted contacts probably come from the fact that buildACLClause() is not being called. That can potentially be a big problem... When I developed this extension, I was mostly testing against custom reports.

  • the "internal contact ID" can sort of be fixed by decommenting this line in civiexportexcel.php:

    // format result set.
    // This seems to cause more problems than it fixes.
    $form->formatDisplay($rows);

I haven't found a good "one size fits all" approach, and really need to propose a patch to core so that we can stop hacking around the issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants