From d1434b8d0565a4b7bd19ec3f7cdf80ec938c23f9 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Thu, 28 Mar 2019 07:12:37 -0400
Subject: [PATCH] Bug 20256: Update groups editor

---
 admin/library_groups.pl                       |  4 +++
 .../prog/en/modules/admin/library_groups.tt   | 25 ++++++++++++++++---
 2 files changed, 26 insertions(+), 3 deletions(-)

diff --git a/admin/library_groups.pl b/admin/library_groups.pl
index 4c5ce25b4e..a235e95b10 100755
--- a/admin/library_groups.pl
+++ b/admin/library_groups.pl
@@ -48,6 +48,7 @@ if ( $action eq 'add' ) {
     my $description = $cgi->param('description') || undef;
     my $branchcode  = $cgi->param('branchcode')  || undef;
     my $ft_hide_patron_info    = $cgi->param('ft_hide_patron_info')    || 0;
+    my $ft_limit_item_editing  = $cgi->param('ft_limit_item_editing')  || 0;
     my $ft_search_groups_opac  = $cgi->param('ft_search_groups_opac')  || 0;
     my $ft_search_groups_staff = $cgi->param('ft_search_groups_staff') || 0;
 
@@ -61,6 +62,7 @@ if ( $action eq 'add' ) {
                 title       => $title,
                 description => $description,
                 ft_hide_patron_info    => $ft_hide_patron_info,
+                ft_limit_item_editing  => $ft_limit_item_editing,
                 ft_search_groups_opac  => $ft_search_groups_opac,
                 ft_search_groups_staff => $ft_search_groups_staff,
                 branchcode  => $branchcode,
@@ -75,6 +77,7 @@ elsif ( $action eq 'edit' ) {
     my $title       = $cgi->param('title')       || undef;
     my $description = $cgi->param('description') || undef;
     my $ft_hide_patron_info    = $cgi->param('ft_hide_patron_info')    || 0;
+    my $ft_limit_item_editing  = $cgi->param('ft_limit_item_editing')  || 0;
     my $ft_search_groups_opac  = $cgi->param('ft_search_groups_opac')  || 0;
     my $ft_search_groups_staff = $cgi->param('ft_search_groups_staff') || 0;
 
@@ -86,6 +89,7 @@ elsif ( $action eq 'edit' ) {
                 title       => $title,
                 description => $description,
                 ft_hide_patron_info      => $ft_hide_patron_info,
+                ft_limit_item_editing    => $ft_limit_item_editing,
                 ft_search_groups_opac    => $ft_search_groups_opac,
                 ft_search_groups_staff   => $ft_search_groups_staff,
             }
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/library_groups.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/library_groups.tt
index d8981cf1fe..3c16e2ccb3 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/library_groups.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/library_groups.tt
@@ -109,6 +109,12 @@
                                         Limit patron data access by group
                                     </label>
                                 </p>
+                                <p>
+                                    <label>
+                                        <input type="checkbox" name="ft_limit_item_editing" id="add-group-modal-ft_limit_item_editing" value="1" />
+                                        Limit item editing by group
+                                    </label>
+                                </p>
                                 <p>
                                     <label>
                                         <input type="checkbox" name="ft_search_groups_opac" id="add-group-modal-ft_search_groups_opac" value="1" />
@@ -166,6 +172,12 @@
                                         Limit patron data access by group
                                     </label>
                                 </p>
+                                <p>
+                                    <label>
+                                        <input type="checkbox" id="edit-group-modal-ft_limit_item_editing" name="ft_limit_item_editing" value="1" />
+                                        Limit item editing by group
+                                    </label>
+                                </p>
                                 <p>
                                     <label>
                                         <input type="checkbox" id="edit-group-modal-ft_search_groups_opac" name="ft_search_groups_opac" value="1" />
@@ -255,9 +267,10 @@
                 var title = $(this).data('groupTitle');
                 var description = $(this).data('groupDescription');
                 var ft_hide_patron_info = $(this).data('groupFt_hide_patron_info');
+                var ft_limit_item_editing = $(this).data('groupFt_limit_item_editing');
                 var ft_search_groups_opac = $(this).data('groupFt_search_groups_opac');
                 var ft_search_groups_staff = $(this).data('groupFt_search_groups_staff');
-                edit_group( id, parent_id, title, description, ft_hide_patron_info, ft_search_groups_opac, ft_search_groups_staff );
+                edit_group( id, parent_id, title, description, ft_hide_patron_info, ft_limit_item_editing, ft_search_groups_opac, ft_search_groups_staff );
             });
 
             $('.delete-group').on('click', function(e) {
@@ -290,6 +303,7 @@
 
 
             $('#add-group-modal-ft_hide_patron_info').prop('checked', false);
+            $('#add-group-modal-ft_limit_item_editing').prop('checked', false);
             $('#add-group-modal-ft_search_groups_opac').prop('checked', false);
             $('#add-group-modal-ft_search_groups_staff').prop('checked', false);
             if ( parent_id ) {
@@ -301,18 +315,20 @@
 
         }
 
-        function edit_group( id, parent_id, title, description, ft_hide_patron_info, ft_search_groups_opac, ft_search_groups_staff ) {
+        function edit_group( id, parent_id, title, description, ft_hide_patron_info, ft_limit_item_editing, ft_search_groups_opac, ft_search_groups_staff ) {
             $('#edit-group-modal-id').val( id );
             $('#edit-group-modal-title').val( title );
             $('#edit-group-modal-description').val( description );
 
             if ( parent_id ) {
                 $('#edit-group-modal-ft_hide_patron_info').prop('checked', false);
+                $('#edit-group-modal-ft_limit_item_editing').prop('checked', false);
                 $('#edit-group-modal-ft_search_groups_opac').prop('checked', false);
                 $('#edit-group-modal-ft_search_groups_staff').prop('checked', false);
                 $('#root-group-features-edit').hide();
             } else {
                 $('#edit-group-modal-ft_hide_patron_info').prop('checked', ft_hide_patron_info ? true : false );
+                $('#edit-group-modal-ft_limit_item_editing').prop('checked', ft_limit_item_editing ? true : false );
                 $('#edit-group-modal-ft_search_groups_opac').prop('checked', ft_search_groups_opac ? true : false );
                 $('#edit-group-modal-ft_search_groups_staff').prop('checked', ft_search_groups_staff ? true : false );
                 $('#root-group-features-edit').show();
@@ -361,6 +377,9 @@
                 [% IF group.ft_hide_patron_info %]
                     <li>Hide patron's info for librarians outside of this group.</li>
                 [% END %]
+                [% IF group.ft_limit_item_editing %]
+                    <li>Limit item editing for librarians outside of this group.</li>
+                [% END %]
                 [% IF group.ft_search_groups_opac %]
                     <li>Use for OPAC search groups</li>
                 [% END %]
@@ -384,7 +403,7 @@
                        </li>
 
                         <li>
-                            <a class="edit-group" id="edit-group-[% group.id | html %]" href="#" data-group-id="[% group.id | html %]" data-group-parent-id="[% group.parent_id | html %]" data-group-title="[% group.title | html %]" data-group-description="[% group.description | html %]" data-group-ft_hide_patron_info="[% group.ft_hide_patron_info | html %]" data-group-ft_search_groups_opac="[% group.ft_search_groups_opac | html %]" data-group-ft_search_groups_staff="[% group.ft_search_groups_staff | html %]" >
+                            <a class="edit-group" id="edit-group-[% group.id | html %]" href="#" data-group-id="[% group.id | html %]" data-group-parent-id="[% group.parent_id | html %]" data-group-title="[% group.title | html %]" data-group-description="[% group.description | html %]" data-group-ft_hide_patron_info="[% group.ft_hide_patron_info | html %]" data-group-ft_limit_item_editing="[% group.ft_limit_item_editing | html %]" data-group-ft_search_groups_opac="[% group.ft_search_groups_opac | html %]" data-group-ft_search_groups_staff="[% group.ft_search_groups_staff | html %]" >
                                 <i class="fa fa-pencil"></i> Edit
                             </a>
                        </li>
-- 
2.20.1 (Apple Git-117)