[Koha-bugs] [Bug 7813] Add ability to delete local cover images
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Sep 20 15:57:33 CEST 2013
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7813
Jonathan Druart <jonathan.druart at biblibre.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #20763|0 |1
is obsolete| |
--- Comment #14 from Jonathan Druart <jonathan.druart at biblibre.com> ---
Created attachment 21276
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21276&action=edit
Bug 7813: Ability to delete local cover images
This patch adds the ability to delete local cover images from the detail page.
It adds a 'x' button to trigger the deletion. It occurs using a new SVC script
called cover_images that accepts the following parameters:
- action
- biblionumber (used for checking imagenumber validity)
- imagenumber (repeatable)
It then deletes all valid images passed and returns a JSON object containing
the imagenumbers and the deletion status (1/0). The operation is triggered
using jQuery.ajax, and with the response the corresponding images get deleted.
Note: currently one image is deleted at a time, but the code is there to be
used.
To test:
- Open the detail page fr a biblionumber containing one or more cover images,
go to its images tab.
- Apply the patch, reload the page, go to the images tab.
- Thumbnails should look better than before ;-)
- An 'X' button should show below each cover.
- Clicking the 'X' button should trigger an alert message asking for
confirmation.
a) Cancel: nothing happens, reload the page and check nothing got deleted.
b) Accept: the deleted image should be gone, reload the page to check it
doesn't exist anymore and is not a browser/DOM manipulation trick.
- Signoff
Note: check having more cover images than those that fit the width and see it
wraps fine.
Any comments are welcome, this is WIP.
Thanks
To+
P.S. I chose not to implement every possible action on the svc script, but
another bug could be filled for more improvements.
Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: David Cook <dcook at prosentient.com.au>
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list