Project

General

Profile

Actions

Bug #1109

closed

can not unhide an image from web viewer in standard installation

Added by Anchi Cheng over 13 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Web interface
Target version:
Start date:
12/22/2010
Due date:
% Done:

0%

Estimated time:
Affected Version:
Appion/Leginon 2.1.0
Show in known bugs:
No
Workaround:

Description

Since the standard database privilege is set to:

GRANT ALTER, CREATE, INSERT, SELECT, UPDATE ON leginondb.* TO usr_object@'localhost';

while the unhide button in while viewing hidden images and the undo-exemplar button while viewing exemplar images works by deleting the data row in DB_LEGINON.ViewerImageStatus table in updateimagelist.php, it needs more privilege for that particular table.

Testing:
Do this on a fresh installation that uses the above privilege.
1. hide any image from the viewer.
2. select to view hidden images
3. choose the same image and click unhide.
4. refresh the page
(Bug) The unhiden image appears in the hidden list again because it was never deleted.

Proposed change:
Either add instruction and add in the wizard and auto-installation the special privilege for this table, or change the way the image list is queried for hidden and exemplar by most recent entry. I think the first is more efficient. Whatever it is decided, we should warn the 2.0 and 2.1 users out there to adopt the first fix.

Actions #1

Updated by Eric Hou over 13 years ago

For this release we should change the installation guide to:

GRANT ALTER, CREATE, INSERT, SELECT, UPDATE, DELETE ON leginondb.* TO usr_object@'localhost';
GRANT ALTER, CREATE, INSERT, SELECT, UPDATE, DELETE ON projectdb.* TO usr_object@'localhost';

In this page: http://emg.nysbc.org/projects/appion/wiki/Database_Server_Installation

Thanks.

Eric

Actions #2

Updated by Eric Hou over 13 years ago

  • Assignee changed from Eric Hou to Amber Herold

By the way, this this page, section #10, there are two places need to change.

thanks.

Eric

Actions #3

Updated by Anchi Cheng over 13 years ago

  • Assignee changed from Amber Herold to Eric Hou

Since no one has changed the documentation. I gave it a go. I checked the query, processing pages do not have delete query, leginon pages need only on ViewerImageStatus. project pages have most but we have removed most administrative delete buttons, so only need processingdb and tables for grid stuff.

Eric, please see if my changes in Database_Server_Installation_Shared are correct I am having trouble making my test databases to take on the more restricted behavior.

Actions #4

Updated by Eric Hou over 13 years ago

  • Status changed from Assigned to In Test
  • Assignee changed from Eric Hou to Anchi Cheng

I updated the documentation. You missed one table 'ImageStatusData'.

give it a try now.

Thanks.

Eric

Actions #5

Updated by Anchi Cheng about 13 years ago

  • Status changed from In Test to Closed
Actions

Also available in: Atom PDF