Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#1304 closed Bug (fixed)

Delete dashboard settings when deleting user

Reported by: pebosi Owned by:
Priority: normal Milestone: Piwik 0.6
Component: Core Keywords:
Cc: Sensitive: no

Description

see attached patch

Attachments (1)

userdelete.patch (458 bytes) - added by pebosi 4 years ago.

Download all attachments as: .zip

Change History (4)

Changed 4 years ago by pebosi

comment:1 Changed 4 years ago by vipsoft (robocoder)

  • Milestone set to 1 - Piwik 0.6
  • Priority changed from major to normal

My gut feeling is that this is the wrong place to do it since the user_dashboard table belongs to the Dashboard plugin.

Theoretically, the user_dashboard table may not exist if the Dashboard plugin is uninstalled. (#546)

I think the Piwik convention would be to add a hook, i.e., PostEvent('UsersManager.deleteUser'); and add a listener to Dashboard.php.

comment:2 Changed 4 years ago by vipsoft (robocoder)

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

(In [2110]) fixes #1304 - delete dashboard layout when deleting user

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

(In [2129]) Fixes #1304 Piwik_Query accepts bind parameters but Piwik_Exec doesn't, was causing deleting a user to issue an error

Note: See TracTickets for help on using tickets.