Deleting an image used for a skin should give a popup warning

Bug #1240244 reported by Robert Lyon on 2013-10-15
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Son Nguyen

Bug Description

If you have a skin with a background image you should get a warning if you try to delete this image from Content -> files.

Currently there is a warning that exist if you try to delete a file that is used in a view.

so what we will need to do is mimic that functionality to check if the file is used by a skin.

the tricky bit comes in with skins using a serialized blob for storing the artefact id used, so it's not as simple as checking against a table column as it is for the current checks.

Robert Lyon (robertl-9) on 2013-11-21
Changed in mahara:
status: New → Triaged
Aaron Wells (u-aaronw) on 2014-02-25
Changed in mahara:
importance: Low → Medium
Son Nguyen (ngson2000) wrote :

After delete a background image of a skin, we also need to update the skin thumbnail

Changed in mahara:
assignee: nobody → Son Nguyen (ngson2000)
status: Triaged → In Progress
Son Nguyen (ngson2000) wrote :

The artefact file id is also stored in the column bodybgimg or viewbgimg of the table skin.

Son Nguyen (ngson2000) wrote :

A popup warning should also be displayed when deleting a profile picture which is used in a skin

Mahara Bot (dev-mahara) wrote :

Patch for "master" branch:

Robert Lyon (robertl-9) on 2014-03-24
Changed in mahara:
status: In Progress → Fix Committed
Robert Lyon (robertl-9) on 2014-04-22
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers