Attachment 74641 Details for Bug 20600 – Bug 20600: Add filtering of ILL requests in list

[patch] Bug 20600: Add filtering of ILL requests in list

Bug-20600-Add-filtering-of-ILL-requests-in-list.patch (text/plain), 17.77 KB, created by Andrew Isherwood on 2018-04-20 13:33:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Isherwood

Created: 2018-04-20 13:33:02 UTC

Size: 17.77 KB

patch

obsolete

>From 95fa79b706eba39b429495bc3306ff5a247e2295 Mon Sep 17 00:00:00 2001
>From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
>Date: Fri, 20 Apr 2018 14:25:49 +0100
>Subject: [PATCH] Bug 20600: Add filtering of ILL requests in list
>
>This patch adds the ability for the ILL request list in the staff
>interface to be filtered by a number of criteria:
>
>- Status
>- Date placed
>- Date modified
>- Pickup branch
>- Borrower card number
>
>To test:
>1) Apply patch
>2) Ensure you have a reasonable range of ILL requests created
>3) Navigate to the "View ILL requests" page
>4) Choose one or more filtering criteria
>5) Click "Search"
>6) Observe the results are filtered to match the selected criteria
>7) Click "Clear"
>8) Observe your results are returned to their initial state
>9) Repeat steps 4 -> 6 until you are happy.
>---
> koha-tmpl/intranet-tmpl/prog/css/staff-global.css  |   5 +
> .../prog/en/modules/ill/ill-requests.tt            | 304 +++++++++++++++++++--
> 2 files changed, 282 insertions(+), 27 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css
>index f68945a773..199a944a73 100644
>--- a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css
>+++ b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css
>@@ -3148,6 +3148,11 @@ table#ill-requests {
>     width: 100% !important;
> }
> 
>+#illfilter_dateplaced,
>+#illfilter_datemodified {
>+    width: 80%;
>+}
>+
> #helper span {
>     display: none;
> }
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>index 70f31cad16..f12f9ded20 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>@@ -9,12 +9,30 @@
> [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") %]
> [% Asset.css("css/datatables.css") %]
> [% INCLUDE 'datatables.inc' %]
>+[% INCLUDE 'calendar.inc' %]
> <script type="text/javascript">
>     //<![CDATA[
>     $(document).ready(function() {
> 
>         // Illview Datatable setup
> 
>+        var table;
>+
>+        // Filters that are active
>+        var activeFilters = {};
>+
>+        // Date format for datepickers
>+        var dateMap = {
>+            dmydot: 'dd.mm.yy',
>+            iso: 'yy-mm-dd',
>+            metric: 'dd/mm/yy',
>+            us: 'mm/dd/yy'
>+        };
>+        var dateFormat = dateMap['[% Koha.Preference('dateformat') %]'];
>+        $('#illfilter_dateplaced, #illfilter_datemodified').datepicker(
>+            'option', 'dateFormat', dateFormat
>+        );
>+
>         // Fields we don't want to display
>         var ignore = [
>             'accessurl',
>@@ -26,7 +44,6 @@
>             'medium',
>             'notesopac',
>             'notesstaff',
>-            'placed',
>             'replied'
>         ];
> 
>@@ -45,14 +62,147 @@
>             'metadata_Author',
>             'metadata_Title',
>             'borrowername',
>+            'patron_cardnumber',
>             'biblio_id',
>             'library',
>             'status',
>+            'placed',
>             'updated',
>             'illrequest_id',
>             'action'
>         ];
> 
>+        // Filterable columns
>+        var filterable = {
>+            status: {
>+                prep: function(tableData, oData) {
>+                    var uniques = {};
>+                    tableData.forEach(function(row) {
>+                        var resolvedName = getStatusName(
>+                            oData[0].capabilities[row.status].name
>+                        );
>+                        uniques[resolvedName] = 1
>+                    });
>+                    Object.keys(uniques).sort().forEach(function(unique) {
>+                        $('#illfilter_status').append(
>+                            '<option value="' + unique  +
>+                            '">' + unique +  '</option>'
>+                        );
>+                    });
>+                },
>+                listener: function() {
>+                    var me = 'status';
>+                    $('#illfilter_status').change(function() {
>+                        var sel = $('#illfilter_status option:selected').val();
>+                        if (sel && sel.length > 0) {
>+                            activeFilters[me] = function() {
>+                                table.column(6).search(sel);
>+                            }
>+                        } else {
>+                            if (activeFilters.hasOwnProperty(me)) {
>+                                delete activeFilters[me];
>+                            }
>+                        }
>+                    });
>+                },
>+                clear: function() {
>+                    $('#illfilter_status').val('');
>+                }
>+            },
>+            pickupBranch: {
>+                prep: function(tableData, oData) {
>+                    var uniques = {};
>+                    tableData.forEach(function(row) {
>+                        uniques[row.library.branchname] = 1
>+                    });
>+                    Object.keys(uniques).sort().forEach(function(unique) {
>+                        $('#illfilter_branchname').append(
>+                            '<option value="' + unique  +
>+                            '">' + unique +  '</option>'
>+                        );
>+                    });
>+                },
>+                listener: function() {
>+                    var me = 'pickupBranch';
>+                    $('#illfilter_branchname').change(function() {
>+                        var sel = $('#illfilter_branchname option:selected').val();
>+                        if (sel && sel.length > 0) {
>+                            activeFilters[me] = function() {
>+                                table.column(5).search(sel);
>+                            }
>+                        } else {
>+                            if (activeFilters.hasOwnProperty(me)) {
>+                                delete activeFilters[me];
>+                            }
>+                        }
>+                    });
>+                },
>+                clear: function() {
>+                    $('#illfilter_branchname').val('');
>+                }
>+            },
>+            barcode: {
>+                listener: function() {
>+                    var me = 'barcode';
>+                    $('#illfilter_barcode').change(function() {
>+                        var val = $('#illfilter_barcode').val();
>+                        if (val && val.length > 0) {
>+                            activeFilters[me] = function() {
>+                                table.column(3).search(val);
>+                            }
>+                        } else {
>+                            if (activeFilters.hasOwnProperty(me)) {
>+                                delete activeFilters[me];
>+                            }
>+                        }
>+                    });
>+                },
>+                clear: function() {
>+                    $('#illfilter_barcode').val('');
>+                }
>+            },
>+            dateModified: {
>+                listener: function() {
>+                    var me = 'dateModified';
>+                    $('#illfilter_datemodified').change(function() {
>+                        var val = $('#illfilter_datemodified').val();
>+                        if (val && val.length > 0) {
>+                            activeFilters[me] = function() {
>+                                table.column(8).search(val);
>+                            }
>+                        } else {
>+                            if (activeFilters.hasOwnProperty(me)) {
>+                                delete activeFilters[me];
>+                            }
>+                        }
>+                    });
>+                },
>+                clear: function() {
>+                    $('#illfilter_datemodified').val('');
>+                }
>+            },
>+            datePlaced: {
>+                listener: function() {
>+                    var me = 'datePlaced';
>+                    $('#illfilter_dateplaced').change(function() {
>+                        var val = $('#illfilter_dateplaced').val();
>+                        if (val && val.length > 0) {
>+                            activeFilters[me] = function() {
>+                                table.column(7).search(val);
>+                            }
>+                        } else {
>+                            if (activeFilters.hasOwnProperty(me)) {
>+                                delete activeFilters[me];
>+                            }
>+                        }
>+                    });
>+                },
>+                clear: function() {
>+                    $('#illfilter_dateplaced').val('');
>+                }
>+            }
>+        };
>+
>         // Remove any fields we're ignoring
>         var removeIgnore = function(dataObj) {
>             dataObj.forEach(function(thisRow) {
>@@ -132,31 +282,35 @@
>                 var status_name = meta.settings.oInit.originalData[0].capabilities[
>                     row.status
>                 ].name;
>-                switch( status_name ) {
>-                    case "New request":
>-                        return _("New request");
>-                    case "Requested":
>-                        return _("Requested");
>-                    case "Requested from partners":
>-                        return _("Requested from partners");
>-                    case "Request reverted":
>-                        return _("Request reverted");
>-                    case "Queued request":
>-                        return _("Queued request");
>-                    case "Cancellation requested":
>-                        return _("Cancellation requested");
>-                    case "Completed":
>-                        return _("Completed");
>-                    case "Delete request":
>-                        return _("Delete request");
>-                    default:
>-                        return status_name;
>-                }
>+                return getStatusName(status_name);
>             } else {
>                 return '';
>             }
>         };
> 
>+        var getStatusName = function(origName) {
>+            switch( origName ) {
>+                case "New request":
>+                    return _("New request");
>+                case "Requested":
>+                    return _("Requested");
>+                case "Requested from partners":
>+                    return _("Requested from partners");
>+                case "Request reverted":
>+                    return _("Request reverted");
>+                case "Queued request":
>+                    return _("Queued request");
>+                case "Cancellation requested":
>+                    return _("Cancellation requested");
>+                case "Completed":
>+                    return _("Completed");
>+                case "Delete request":
>+                    return _("Delete request");
>+                default:
>+                    return status_name;
>+            }
>+        };
>+
>         // Render function for creating a row's action link
>         var createActionLink = function(data, type, row) {
>             return '<a class="btn btn-default btn-sm" ' +
>@@ -190,6 +344,12 @@
>             library: {
>                 name: _("Library"),
>                 func: createLibrary
>+            },
>+            updated: {
>+                name: _("Updated on"),
>+            },
>+            patron_cardnumber: {
>+                name: _("Patron barcode")
>             }
>         };
> 
>@@ -240,7 +400,8 @@
>                     // Create the base column object
>                     var colObj = {
>                         name: thisCol,
>-                        className: thisCol
>+                        className: thisCol,
>+                        defaultContent: ''
>                     };
>                     // We may need to process the data going in this
>                     // column, so do it if necessary
>@@ -256,7 +417,7 @@
>                 });
> 
>                 // Initialise the datatable
>-                $('#ill-requests').DataTable($.extend(true, {}, dataTablesDefaults, {
>+                table = $('#ill-requests').DataTable($.extend(true, {}, dataTablesDefaults, {
>                     'aoColumnDefs': [  // Last column shouldn't be sortable or searchable
>                         {
>                             'aTargets': [ 'actions' ],
>@@ -264,19 +425,67 @@
>                             'bSearchable': false
>                         },
>                     ],
>-                    'aaSorting': [[ 6, 'desc' ]], // Default sort, updated descending
>+                    'aaSorting': [[ 7, 'desc' ]], // Default sort, updated descending
>                     'processing': true, // Display a message when manipulating
>                     'iDisplayLength': 10, // 10 results per page
>                     'sPaginationType': "full_numbers", // Pagination display
>                     'deferRender': true, // Improve performance on big datasets
>                     'data': dataCopy,
>                     'columns': colData,
>-                    'originalData': data // Enable render functions to access
>-                                       // our original data
>+                    'originalData': data, // Enable render functions to access
>+                                          // our original data
>+                    'initComplete': function() {
>+
>+                        // Prepare any filter elements that need it
>+                        for (var el in filterable) {
>+                            if (filterable.hasOwnProperty(el)) {
>+                                if (filterable[el].hasOwnProperty('prep')) {
>+                                    filterable[el].prep(dataCopy, data);
>+                                }
>+                                if (filterable[el].hasOwnProperty('listener')) {
>+                                    filterable[el].listener();
>+                                }
>+                            }
>+                        }
>+                        
>+                    }
>                 }));
>+
>             }
>         );
> 
>+        var clearSearch = function() {
>+            table.search('').columns().search('');
>+            activeFilters = {};
>+            for (var filter in filterable) {
>+                if (
>+                    filterable.hasOwnProperty(filter) &&
>+                    filterable[filter].hasOwnProperty('clear')
>+                ) {
>+                    filterable[filter].clear();
>+                }
>+            }
>+            table.draw();
>+        };
>+
>+        // Apply any search filters, or clear any previous
>+        // ones
>+        $('#illfilter_form').submit(function(event) {
>+            event.preventDefault();
>+            table.search('').columns().search('');
>+            for (var active in activeFilters) {
>+                if (activeFilters.hasOwnProperty(active)) {
>+                    activeFilters[active]();
>+                }
>+            }
>+            table.draw();
>+        });
>+
>+        // Clear all filters
>+        $('#clear_search').click(function() {
>+            clearSearch();
>+        });
>+
>     });
>     //]]>
> </script>
>@@ -298,6 +507,45 @@
> 
> <div id="doc3" class="yui-t2">
>     <div id="bd">
>+        [% IF query_type == 'illlist' %]
>+        <div id="illfilter_yui_column" class="yui-b">
>+            <form method="get" id="illfilter_form">
>+                <fieldset class="brief">
>+                    <h3>Filters</h3>
>+                    <ol>
>+                        <li>
>+                            <label for="illfilter_status">Status:</label>
>+                            <select name="illfilter_status" id="illfilter_status">
>+                                <option value=""></option>
>+                            </select>
>+                        </li>
>+                        <li>
>+                            <label for="illfilter_dateplaced">Date placed:</label>
>+                            <input type="text" name="illfilter_dateplaced" id="illfilter_dateplaced" class="datepicker">
>+                        </li>
>+                        <li>
>+                            <label for="illfilter_datemodified">Date modified:</label>
>+                            <input type="text" name="illfilter_datemodified" id="illfilter_datemodified" class="datepicker">
>+                        </li>
>+                        <li>
>+                            <label for="illfilter_branchname">Pickup branch:</label>
>+                            <select name="illfilter_branchname" id="illfilter_branchname">
>+                                <option value=""></option>
>+                            </select>
>+                        </li>
>+                        <li>
>+                            <label for="illfilter_barcode">Borrower card number:</label>
>+                            <input type="text" name="illfilter_barcode" id="illfilter_barcode">
>+                        </li>
>+                    </ol>
>+                    <fieldset class="action">
>+                        <input type="submit" value="Search" />
>+                        <input type="button" value="Clear" id="clear_search" />
>+                    </fieldset>
>+                </fieldset>
>+            </form>
>+        </div>
>+        [% END %]
>         <div id="yui-main">
>             <div id="interlibraryloans" class="yui-b">
>         [% IF !backends_available %]
>@@ -588,10 +836,12 @@
>                                 <tr id="illview-header">
>                                     <th>Author</th>
>                                     <th>Title</th>
>-                                    <th>Patron</th>
>+                                    <th>Patron name</th>
>+                                    <th>Patron barcode</th>
>                                     <th>Biblio ID</th>
>                                     <th>Library</th>
>                                     <th>Status</th>
>+                                    <th>Date placed</th>
>                                     <th>Updated on</th>
>                                     <th>Request number</th>
>                                     <th class="actions"></th>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 20600: 74640 | 74641 | 75202 | 75276 | 75734 | 75735 | 75753 | 75755 | 75844 | 75845 | 75848 | 75849 | 75850 | 75851 | 75852 | 75853 | 75854 | 75871 | 77691 | 77692 | 77693 | 77694 | 77695 | 77696 | 77697 | 78439 | 78440 | 78441 | 78442 | 78443 | 78444 | 78445 | 78446 | 78463 | 78464 | 78465 | 78466 | 78467 | 78468 | 78469 | 78470 | 79183 | 80468 | 81992 | 81993 | 82032 | 82033 | 82034 | 82035 | 82036 | 82037 | 82038 | 82039 | 82062 | 82170 | 82171 | 82432 | 82433 | 82434 | 82435 | 82436 | 82437 | 82438 | 82439 | 82440 | 82441 | 82442 | 83882 | 83887 | 84736 | 84737 | 84738 | 84739 | 84740 | 84741 | 84742 | 84743 | 84744 | 84745 | 84746 | 84747 | 84748 | 84749 | 85037 | 85038 | 85039 | 85040 | 85041 | 85042 | 85043 | 85044 | 85045 | 85046 | 85047 | 85048 | 85049 | 85050