[Koha-bugs] [Bug 7813] Add ability to delete local cover images

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7813

David Cook <dcook at prosentient.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #20761|0                           |1
        is obsolete|                            |

--- Comment #7 from David Cook <dcook at prosentient.com.au> ---
Created attachment 20763
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20763&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>

Works as described. This is a nice addition!

-- 
You are receiving this mail because:
You are watching all bug changes.