summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorFrédéric Guillot <fred@kanboard.net>2014-11-25 22:07:24 -0500
committerFrédéric Guillot <fred@kanboard.net>2014-11-25 22:07:24 -0500
commit38070aec79416e529ff85205ee7def06216dd8ff (patch)
tree3b06705bce2cf6ad38f93840045db35ce881d4cf /app
parent93df72bd4daa14c44e30f0a22553d3445506506b (diff)
Remove private projects when a user is deleted
Diffstat (limited to 'app')
-rw-r--r--app/Model/User.php26
1 files changed, 20 insertions, 6 deletions
diff --git a/app/Model/User.php b/app/Model/User.php
index 59015087..0d5b346b 100644
--- a/app/Model/User.php
+++ b/app/Model/User.php
@@ -292,15 +292,29 @@ class User extends Base
*/
public function remove($user_id)
{
- $this->db->startTransaction();
+ return $this->db->transaction(function ($db) use ($user_id) {
- // All tasks assigned to this user will be unassigned
- $this->db->table(Task::TABLE)->eq('owner_id', $user_id)->update(array('owner_id' => 0));
- $result = $this->db->table(self::TABLE)->eq('id', $user_id)->remove();
+ // All assigned tasks are now unassigned
+ if (! $db->table(Task::TABLE)->eq('owner_id', $user_id)->update(array('owner_id' => 0))) {
+ return false;
+ }
- $this->db->closeTransaction();
+ // All private projects are removed
+ $project_ids = $db->table(Project::TABLE)
+ ->eq('is_private', 1)
+ ->eq(ProjectPermission::TABLE.'.user_id', $user_id)
+ ->join(ProjectPermission::TABLE, 'project_id', 'id')
+ ->findAllByColumn(Project::TABLE.'.id');
- return $result;
+ if (! empty($project_ids)) {
+ $db->table(Project::TABLE)->in('id', $project_ids)->remove();
+ }
+
+ // Finally remove the user
+ if (! $db->table(self::TABLE)->eq('id', $user_id)->remove()) {
+ return false;
+ }
+ });
}
/**