Contrib.social

The "Export All" functionality does not work

Open on Drupal.org →
Created on 22 March 2023, over 1 year ago
Updated 23 March 2023, over 1 year ago

Problem

The "Download statistics" page "Export all" functionality doesn't work. The resulting file contains errors and header row only.
Example:

<br />
<b>Warning</b>:  Attempt to read property "filename" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>59</b><br />
<br />
<b>Warning</b>:  Attempt to read property "type" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>60</b><br />
<br />
<b>Warning</b>:  Attempt to read property "id" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>61</b><br />
<br />
<b>Warning</b>:  Attempt to read property "filename" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>127</b><br />
<br />
<b>Warning</b>:  Attempt to read property "type" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>168</b><br />
<br />
<b>Warning</b>:  Attempt to read property "id" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>169</b><br />
<br />
<b>Warning</b>:  Attempt to read property "fid" on bool in <b>/var/www/html/docroot/modules/contrib/download_count/src/Form/DownloadCountExportForm.php</b> on line <b>170</b><br />
"Download count id","File id","File name","File URI","File size","Entity type","Entity id","Downloading user id","Downloading username","Downloading user ip address","HTTP referrer","Date time"
🐛 Bug report
Status

RTBC

Version

2.0

Component

Code

Created by

🇺🇦Ukraine Oleksiy Lutsk 🇺🇦

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @Oleksiy
  • Comment over 1 year ago →
    🇺🇦Ukraine Oleksiy Lutsk 🇺🇦
  • @oleksiy opened merge request.
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • Comment over 1 year ago →
    🇺🇦Ukraine Oleksiy Lutsk 🇺🇦
  • Status changed to RTBC over 1 year ago
  • Comment over 1 year ago →
    🇮🇳India arisen Goa

    Reviewed the MR and tested the patch on 2.0.0 version.

    The patch applies cleanly.

    Checking patch src/Form/DownloadCountExportForm.php...
Applied patch src/Form/DownloadCountExportForm.php cleanly.

    The export all functionality works fine after applying the patch. The records are added as expected.

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024