dev/report#77 - Unused variable in contribute/history civireport

[demeritcowboy]

Overview

E_NOTICE for $entryFound.

This is the contribution aggregrate by relationship report (called History.php of course).

There are a lot of smarty notices on this page - this is a regular old E_NOTICE.

Before

E_NOTICE

After

Unused variable removed

Technical Details

This variable is used in other reports to skip looping over all the rows when the first row doesn't contain any of the columns you're altering. That's not desirable here anyway because of the inner loop higher up that potentially could run on every row. In any case this is no change from current since the variable is never checked.

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[demeritcowboy]

Thanks!