From 18820b6c8fc30fb25c99e0e0710d08eda2bc745c Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 11 Feb 2016 12:06:39 -0500 Subject: [PATCH] Bug 15790 - Don't delete a MARC framework if existing records use that framework If a MARC framework is in use by any records it should not be possible to delete it. Deleting a framework which is in use will leave the deleted framework code in those records. This patch modifies the deletion confirmation dialog so that it will not let the user confirm the deletion if there are records using it. To test, apply the patch and go to Administration -> MARC frameworks 1. Try to delete a framework which is in use by 1 or more records. The resulting confirmation dialog should prevent you from proceeding. 2. Try to delete a framework which is not in use. Test both canceling and confirming that deletion to make both function correctly. Signed-off-by: Julian Maurice --- .../prog/en/modules/admin/biblio_framework.tt | 21 ++++++++++++++++----- 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt index 8b90023..ddcb406 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt @@ -135,13 +135,24 @@ [% IF ( delete_confirm ) %]
-

Delete framework for [% frameworktext %] ([% frameworkcode %])?

[% IF ( total ) %] -

This framework is used [% total %] times.

+

This framework cannot be deleted

+

[% frameworktext %] ([% frameworkcode %])

+

This framework is used [% total %] times.

+
+ +
+ [% ELSE %] +

Delete framework for [% frameworktext %] ([% frameworkcode %])?

+
+ + + +
+
+ +
[% END %] -
-
-
[% END %] -- 2.1.4