[INP-1312] Mark themes as deleted instead of actually deleting them - In-Portal Issue Tracker

XML

Word

Printable

Change Log Group:
Fixed
Change Log Message:
Changing theme name in UI or deleting and restoring it on disk no longer results in content block data loss.
Story Points:
1
Sprint:
In-Portal's First Sprint, In-Portal's 2nd Sprint, In-Portal's 3rd Sprint, In-Portal's 4th Sprint
Copy Issue Key:
Patch Instructions:
Patches must be submitted through Phabricator.
- To submit patch via Command Line use Patches Workflow (via Arcanist) tutorial
- To submit patch via Web Interface use Patches Workflow (via Web Interface) tutorial

Plan:

Add "AddedOn" and DeletedOn" columns to theme and display them on grid/edit page.
Add "Deleted" status (new option in existing Status field) to theme.
If during theme scanning we've discovered theme theme record in DB no longer have corresponding folder on disk, then set "Status=Deleted" + "DeletedOn=NOW()" to theme DB record instead of deleting it & associated theme files.
If during theme scanning we've discovered that folder on disk is matching a theme with "Deleted" status, then set "Status=Disabled" + "DeletedOn=NULL" + "AddedOn=NOW()".
Remove "Delete" toolbar button on "Themes" grid.
Remove "in-portal:configure_themes.delete" permission (from db & unit config).
Make "Name" field of the theme read-only.

implemented in

D12 INP-1312 - Mark themes as deleted instead of actually deleting them

Estimated:

Not Specified

Remaining:

Logged:

3.4h