Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#3017 closed Bug (fixed)

PDF via cronjob creates summary reports for all websites

Reported by: networker Owned by:
Priority: major Milestone: 1.12.x - Piwik 1.12.x
Component: Core Keywords:
Cc: Sensitive: no

Description

Sorry, can´t reopen ticket #3006

"[...] an user created pdf report includes stats for all tracked websites, not only for the sites the user has the right to view the stats."

I reactivate the report plugin again and it doesn´t work for me. Just get the full report for all websites when I reactivate the plugin. Created the report request as simple user, not as Super User.

Sorry, in my bug report yesterday I forgot to write, that the manual requested ist fine, only the Email by cronjob report includes all sites. Runs the Cron without additional parameter as Super User?

Change History (3)

comment:1 Changed 2 years ago by matt (mattab)

  • Milestone changed from 1.x - Piwik 1.x to 1.7.x - Piwik 1.7.2
  • Priority changed from normal to critical

Thank you for the report. I could verify, I will try to fix asap!

comment:2 Changed 2 years ago by matt (mattab)

  • Resolution set to fixed
  • Status changed from new to closed

(In [6085]) Fixes #3017

BUG explanation: During scheduled tasks, Super User status is enforced. This was not a problem until recently, because the Scheduled report was always about one website in particular. since we added All websites Refs #2600 now the report was generated as "Super User"....
so I force the getSitesWithAtLeastViewAccess in the SitesManager API to be restricted to the user who created the report..
This solution is unfortunately a bit hacky but I couldn't find a better way and it took me 4 hours to get it right... pff

comment:3 Changed 2 years ago by matt (mattab)

  • Keywords pdf automatic e-mail report stats removed
  • Priority changed from critical to major
  • Summary changed from PDF >cronjob< report creates summary reports for all websites to PDF via cronjob creates summary reports for all websites
Note: See TracTickets for help on using tickets.