Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete dashboard settings when deleting user #1304

Closed
pebosi opened this issue Apr 14, 2010 · 4 comments
Closed

Delete dashboard settings when deleting user #1304

pebosi opened this issue Apr 14, 2010 · 4 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@pebosi
Copy link
Contributor

pebosi commented Apr 14, 2010

see attached patch

@pebosi
Copy link
Contributor Author

pebosi commented Apr 14, 2010

Attachment:
userdelete.patch

@robocoder
Copy link
Contributor

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.

@robocoder
Copy link
Contributor

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

@mattab
Copy link
Member

mattab commented May 5, 2010

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

@pebosi pebosi added this to the Piwik 0.6 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants