Project

General

Profile

Actions

Feature #2466

closed

viewer status for removing images

Added by Anchi Cheng over 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Amber Herold
Category:
-
Start date:
08/06/2013
Due date:
% Done:

0%

Estimated time:
Deliverable:

Description

Requested by Rui Zhang and second by others

Users may hide an image but not wanting to doom it for deletion. The extra level of image status lower than hidden gives an extra choice when it comes to deleting data.


Related issues 1 (0 open1 closed)

Related to Appion - Bug #2529: Trash does not Hide images from processingClosedKuang-Lei Tsai09/23/2013

Actions
Actions #1

Updated by Anchi Cheng over 11 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Anchi Cheng to Amber Herold

r17797 added this. Added some comments in the code but would like to clean these some day.
inc/viewer.inc is especially rediculous.

The big change is that, in order to make updateimagelist.php responds to trash and hide button differently, values are assigned to the buttons when clicked and update_imagelist function in js/viewer.js need to check these. After the action is performed, the values are reset. These were not necessary before since the code was just duplicated in different functions.

The behavior:

You may trash an image when the image status is not set (i.e., all presets are viewed) or hidden.
When viewing the trash images, the "hide" button puts the image to hide status rather than trash, effectively taking it out of the trash bin. You can not take the trash back out to the general unmarked status directly (for sanitary reason :)

Actions #2

Updated by Anchi Cheng over 11 years ago

r17882 adds the database schema update and initialization required for this. Had to call the schema update file schema-r17813 so that it would not have problem with schema-r17812 already updated.

Actions #3

Updated by Amber Herold over 10 years ago

  • Status changed from In Code Review to Closed
Actions

Also available in: Atom PDF