From 1dc7d421bc87e3cf027161b08fbb44875c778f49 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 20 Sep 2013 13:06:48 +0200 Subject: [PATCH] Bug 7813: (follow-up) improvements for deleting local cover images This patch: - displays a js alert if a problem occurred on deleting the image on the server. - adds a class for the new span (in order to avoid future problem, if someone adds a new span). - adds a title for the [X] link. - deal with the upload_local_cover_images permission (don't display the delete link if the logged in user does not have it). - changes the license version. Signed-off-by: Tomas Cohen Arazi I like the ehancements to the original patch and work as expected. Signed-off-by: Jonathan Druart Signed-off-by: Galen Charlton --- .../prog/en/modules/catalogue/detail.tt | 11 +++++--- svc/cover_images | 25 +++++++++---------- 2 files changed, 20 insertions(+), 16 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt index 903f1ad156..684151a86b 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -62,6 +62,9 @@ function verify_images() { $('#imagenumber-' + this.imagenumber).remove(); } }); + }, + error: function(data) { + alert(_("An error occurred on deleting this image")); } }); @@ -70,7 +73,7 @@ function verify_images() { $(document).ready(function() { $('#bibliodetails').tabs(); $('#search-form').focus(); - $('.thumbnails > li > a > span').click(function() { + $('.thumbnails > li > a > span.remove').click(function() { var result = confirm(_("Are you sure you want to delete this cover image?")); if ( result == true ) { @@ -728,8 +731,10 @@ function verify_images() { [% IF image %]
  • - - × + remove this image + [% IF CAN_user_tools_upload_local_cover_images %] + × + [% END %]
  • [% END %] diff --git a/svc/cover_images b/svc/cover_images index d3a3c45983..ff29503eae 100755 --- a/svc/cover_images +++ b/svc/cover_images @@ -1,23 +1,22 @@ #!/usr/bin/perl -# Copyright 2013 Universidad Nacional de Cordoba -# Tomas Cohen Arazi -# # This file is part of Koha. # -# Koha is free software; you can redistribute it and/or modify it under the -# terms of the GNU General Public License as published by the Free Software -# Foundation; either version 2 of the License, or (at your option) any later -# version. +# Copyright 2013 Universidad Nacional de Cordoba +# Tomas Cohen Arazi # -# Koha is distributed in the hope that it will be useful, but WITHOUT ANY -# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR -# A PARTICULAR PURPOSE. See the GNU General Public License for more details. +# Koha is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. # -# You should have received a copy of the GNU General Public License along -# with Koha; if not, write to the Free Software Foundation, Inc., -# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. +# Koha is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. # +# You should have received a copy of the GNU General Public License +# along with Koha; if not, see . use Modern::Perl; -- 2.39.5