Quick fix to exporting
Permalink Browser Info Environment
Hi Macareux Team,
I just picked up a little bug with this add-on in terms of exporting, that I thought I should pass on.
When I tried exporting on a client site, it was complaining about not being able to call getUserName() as part of the export. I think this is because some users have been deleted from the site.
In C5jFileDownloadStatistics\Export\DownloadStatisticsExporter, I've just changed in the function formatRows(), to these two lines:
There's probably a more efficient way to do that, but it was a quick fix that worked.
Cheers
-Ryan
I just picked up a little bug with this add-on in terms of exporting, that I thought I should pass on.
When I tried exporting on a client site, it was complaining about not being able to call getUserName() as part of the export. I think this is because some users have been deleted from the site.
In C5jFileDownloadStatistics\Export\DownloadStatisticsExporter, I've just changed in the function formatRows(), to these two lines:
$row->getFile() ? $row->getFile()->getFileName() : '', $row->getUser() ? $row->getUser()->getUserName() : '',
There's probably a more efficient way to do that, but it was a quick fix that worked.
Cheers
-Ryan
| Type: | Ticket |
|---|---|
| Status: | Resolved |
Thanks for letting us know.
We're sorry for replying late.
We released a new version 1.0.3 including your fix.
Thanks.