← Back to team overview

mahara-contributors team mailing list archive

[Bug 1940033] A change has been merged

 

Reviewed:  https://reviews.mahara.org/11978
Committed: https://git.mahara.org/mahara/mahara/commit/d8efdf5a6bd6cc74daceb724884041ec3ae96d99
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit d8efdf5a6bd6cc74daceb724884041ec3ae96d99
Author: Dianne Tennent <diannetennent@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Date:   Wed Sep 8 14:30:20 2021 +1200

Bug 1940033: Now able to delete an account when there is a cover image
on a collection

Change-Id: Icc3a6aba4483137f18390b49976a1ffcabd7637a

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1940033

Title:
  Unable to delete account when they have a collection with a cover
  image

Status in Mahara:
  Fix Committed
Status in Mahara 20.04 series:
  Confirmed
Status in Mahara 20.10 series:
  Confirmed
Status in Mahara 21.04 series:
  Confirmed
Status in Mahara 21.10 series:
  Fix Committed

Bug description:
  We get ERROR:  update or delete on table "artefact" violates foreign
  key constraint "coll_cov_fk" on table "collection"

  To test:

  1) New site
  2) Log in as admin and create a new user
  3) Masquerade as new user
  4) add a page
  5) add a collection and add a cover image to the collection
  6) add page to collection
  7) stop masquerading (be admin again)
  8) click on the 'Suspend or delete this account' and chose delete account

  Expected result account is deleted

  Actual outcome get error

  The problem is the coverimage artefact id is still in the collection
  table when we try to delete the artefact.

  In the delete_user() function the deletion of artefacts happens before
  the deletion of collections

  We should maybe try those the other way around

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1940033/+subscriptions



References