Bug 14889: QA Follow-up

- Removes unused package from BiblioFramework.pm
- Makes framework table sortable
- Some small changes in wording of error messages

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com

Koha/BiblioFramework.pm

@@ -19,8 +19,6 @@ use Modern::Perl;
 
 use Carp;
 
-use C4::Members::Messaging;
-
 use Koha::Database;
 
 use base qw(Koha::Object);

koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt

@@ -20,7 +20,7 @@
                 { "aTargets": [ -1, -2, -3, -4, -5 ], "bSortable": false, "bSearchable": false },
                 { "aTargets": [ 0, 1 ], "sType": "natural" },
             ],
-            "bSort": false,
+            "bSort": true,
             "sPaginationType": "four_button"
         }));
 
@@ -116,13 +116,13 @@
         [% CASE 'error_on_update' %]
             An error occurred when updating this framework. Perhaps it already exists.
         [% CASE 'error_on_insert' %]
-            An error occurred when inserting this framework. The framework might already exist.
+            An error occurred when adding this framework. The framework might already exist.
         [% CASE 'error_on_delete' %]
             An error occurred when deleting this framework. Check the logs.
         [% CASE 'success_on_update' %]
             Framework updated successfully.
         [% CASE 'success_on_insert' %]
-            Framework inserted successfully.
+            Framework added successfully.
         [% CASE 'success_on_delete' %]
             Framework deleted successfully.
         [% CASE 'already_exists' %]