View | Details | Raw Unified | Return to bug 31391
Collapse All | Expand All

(-)a/installer/data/mysql/atomicupdate/bug_31391_-_add_HOLD_CANCELLATION-RECALLED_authorised_value.pl (-1 / +3 lines)
Lines 2-12 use Modern::Perl; Link Here
2
2
3
return {
3
return {
4
    bug_number => "31391",
4
    bug_number => "31391",
5
    description => "Add new RECALLED value to HOLD_CANCELLATION authorised value category",
5
    description => "Staff-side recalls",
6
    up => sub {
6
    up => sub {
7
        my ($args) = @_;
7
        my ($args) = @_;
8
        my ($dbh, $out) = @$args{qw(dbh out)};
8
        my ($dbh, $out) = @$args{qw(dbh out)};
9
9
10
        $dbh->do(q{ INSERT IGNORE INTO authorised_values (category, authorised_value, lib) VALUES ('HOLD_CANCELLATION','RECALLED','Hold was converted to a recall') });
10
        $dbh->do(q{ INSERT IGNORE INTO authorised_values (category, authorised_value, lib) VALUES ('HOLD_CANCELLATION','RECALLED','Hold was converted to a recall') });
11
12
        say $out "Added RECALLED value to HOLD_CANCELLATION authorised value category";
11
    },
13
    },
12
};
14
};
(-)a/installer/data/mysql/atomicupdate/bug_31391_-_add_RecallsInterface_syspref.pl (-1 / +3 lines)
Lines 2-12 use Modern::Perl; Link Here
2
2
3
return {
3
return {
4
    bug_number => "31391",
4
    bug_number => "31391",
5
    description => "Add new system preference RecallsInterface",
5
    description => "Staff-side recalls",
6
    up => sub {
6
    up => sub {
7
        my ($args) = @_;
7
        my ($args) = @_;
8
        my ($dbh, $out) = @$args{qw(dbh out)};
8
        my ($dbh, $out) = @$args{qw(dbh out)};
9
9
10
        $dbh->do(q{INSERT IGNORE INTO systempreferences (variable,value,options,explanation,type) VALUES ('RecallsInterface','opac','opac|staff|both','The interface that recalls can be placed through','Choice') });
10
        $dbh->do(q{INSERT IGNORE INTO systempreferences (variable,value,options,explanation,type) VALUES ('RecallsInterface','opac','opac|staff|both','The interface that recalls can be placed through','Choice') });
11
12
        say $out "Added system preference 'RecallsInterface'";
11
    },
13
    },
12
};
14
};
(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc (+6 lines)
Lines 23-28 Link Here
23
            <th>&nbsp;</th>
23
            <th>&nbsp;</th>
24
            <th>&nbsp;</th>
24
            <th>&nbsp;</th>
25
            <th>Print hold/transfer slip</th>
25
            <th>Print hold/transfer slip</th>
26
            <th>&nbsp;</th>
26
        </tr>
27
        </tr>
27
    </thead>
28
    </thead>
28
    [%- SET first_priority = 0 -%]
29
    [%- SET first_priority = 0 -%]
Lines 277-282 Link Here
277
            [% ELSE %]
278
            [% ELSE %]
278
                <td></td>
279
                <td></td>
279
            [% END %]
280
            [% END %]
281
            <td>
282
                [% IF Koha.Preference('UseRecalls') AND Koha.Preference('RecallsInterface') != 'opac' %]
283
                <a class="convert-to-recall btn" title="Convert hold to recall" data-id="[% hold.reserve_id | html %]">Mark for conversion to recall</a>
284
                [% END %]
285
            </td>
280
        </tr>
286
        </tr>
281
    [% END %]
287
    [% END %]
282
</table>
288
</table>
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref (-9 / +9 lines)
Lines 1340-1345 Circulation: Link Here
1340
                  1: Use
1340
                  1: Use
1341
                  0: "Don't use"
1341
                  0: "Don't use"
1342
            - recalls. Make sure you configure <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a> for recalls once enabled.
1342
            - recalls. Make sure you configure <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a> for recalls once enabled.
1343
        -
1344
            - Recalls can be placed through
1345
            - pref: RecallsInterface
1346
              default: opac
1347
              choices:
1348
                  opac: the OPAC only
1349
                  staff: the staff interface only
1350
                  both: both the OPAC and the staff interface
1351
            - . (Requires <a href="/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=UseRecalls">UseRecalls</a>.)
1343
1352
1344
    SIP2:
1353
    SIP2:
1345
        -
1354
        -
Lines 1376-1387 Circulation: Link Here
1376
                  1: Enable
1385
                  1: Enable
1377
                  0: Disable
1386
                  0: Disable
1378
            - "the curbside pickup module."
1387
            - "the curbside pickup module."
1379
        -
1380
            - Recalls can be placed through
1381
            - pref: RecallsInterface
1382
              default: opac
1383
              choices:
1384
                  opac: the OPAC only
1385
                  staff: the staff interface only
1386
                  both: both the OPAC and the staff interface
1387
            - . (Requires <a href="/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=UseRecalls">UseRecalls</a>.)
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/recalls/request.tt (-4 / +17 lines)
Lines 57-62 Link Here
57
            </div>
57
            </div>
58
        [% END %]
58
        [% END %]
59
59
60
        [% IF patron_holds_count %]
61
            <div class="dialog message">
62
                Patron has placed one or more record-level holds. <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% biblio.biblionumber | html %]">Convert a hold to a recall</a>.
63
            </div>
64
        [% END %]
65
60
        <fieldset class="rows">
66
        <fieldset class="rows">
61
            <legend>Recall details</legend>
67
            <legend>Recall details</legend>
62
            <form id="recallform" action="/cgi-bin/koha/recalls/request.pl" method="post">
68
            <form id="recallform" action="/cgi-bin/koha/recalls/request.pl" method="post">
Lines 109-115 Link Here
109
                        <th>Barcode</th>
115
                        <th>Barcode</th>
110
                        [% UNLESS ( single_branch_mode ) %]
116
                        [% UNLESS ( single_branch_mode ) %]
111
                            <th>Home library</th>
117
                            <th>Home library</th>
112
                            <th>Last location</th>
118
                            <th>Holding library</th>
113
                        [% END %]
119
                        [% END %]
114
                        <th>Collection</th>
120
                        <th>Collection</th>
115
                        <th>Call number</th>
121
                        <th>Call number</th>
Lines 146-152 Link Here
146
                                <span class="intransit">In transit from [% Branches.GetName( item.get_transfer.frombranch ) | html %] to [% Branches.GetName( item.get_transfer.tobranch ) | html %] since [% item.get_transfer.datesent | $KohaDates %]</span>
152
                                <span class="intransit">In transit from [% Branches.GetName( item.get_transfer.frombranch ) | html %] to [% Branches.GetName( item.get_transfer.tobranch ) | html %] since [% item.get_transfer.datesent | $KohaDates %]</span>
147
                            [% END %]
153
                            [% END %]
148
                            [% IF ( item.itemlost || item.withdrawn ) %]
154
                            [% IF ( item.itemlost || item.withdrawn ) %]
149
                                <span class="lost">Unavailable (lost or missing)</span>
155
                                <span class="lost">Unavailable (lost or withdrawn)</span>
150
                            [% END %]
156
                            [% END %]
151
                            [% IF ( item.notforloan ) %]
157
                            [% IF ( item.notforloan ) %]
152
                                <span class="notforloan">Not for loan ([% item.notforloan | html %])</span>
158
                                <span class="notforloan">Not for loan ([% item.notforloan | html %])</span>
Lines 166-172 Link Here
166
                                </span>
172
                                </span>
167
                            [% ELSIF ( hold.borrowernumber == logged_in_user.borrowernumber ) %]
173
                            [% ELSIF ( hold.borrowernumber == logged_in_user.borrowernumber ) %]
168
                                <span class="waiting">
174
                                <span class="waiting">
169
                                    Patron has already placed a <a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% patron.borrowernumber | uri %]#reserves">reserve</a> on this item.
175
                                    Patron has already placed a <a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% patron.borrowernumber | uri %]#reserves">hold</a> on this item.
170
                                </span>
176
                                </span>
171
                            [% END # / IF ( item.recall or hold ) %]
177
                            [% END # / IF ( item.recall or hold ) %]
172
                        </td>
178
                        </td>
Lines 213-218 Link Here
213
219
214
        [% END %]
220
        [% END %]
215
221
222
        <div class="page-section">
223
216
        <h2>Existing recalls</h2>
224
        <h2>Existing recalls</h2>
217
225
218
        [% IF Koha.Preference('UseRecalls') %]
226
        [% IF Koha.Preference('UseRecalls') %]
Lines 224-230 Link Here
224
                        <input type="checkbox" id="select_all"> <span id="select_all_text">Select all</span>
232
                        <input type="checkbox" id="select_all"> <span id="select_all_text">Select all</span>
225
                        [% INCLUDE 'recalls.inc' %]
233
                        [% INCLUDE 'recalls.inc' %]
226
                        <fieldset class="action">
234
                        <fieldset class="action">
227
                            <button type="submit" id="cancel_selected" class="btn btn-default btn-sm">Cancel selected recalls</button>
235
                            <button type="submit" id="cancel_selected" class="btn btn-primary btn-sm">Cancel selected recalls</button>
228
                        </fieldset>
236
                        </fieldset>
229
                    </form>
237
                    </form>
230
                </div> <!-- /.page-section -->
238
                </div> <!-- /.page-section -->
Lines 235-240 Link Here
235
            <div class="dialog message">Recalls have not been enabled. Enable the <a href="/cgi-bin/koha/admin/preferences.pl?tab=circulation">UseRecalls</a> system preference to use recalls.</div>
243
            <div class="dialog message">Recalls have not been enabled. Enable the <a href="/cgi-bin/koha/admin/preferences.pl?tab=circulation">UseRecalls</a> system preference to use recalls.</div>
236
        [% END %]
244
        [% END %]
237
245
246
        </div> <!-- page-section Existing recalls -->
247
238
    [% END %]
248
    [% END %]
239
249
240
</main>
250
</main>
Lines 254-263 Link Here
254
    [% INCLUDE 'datatables.inc' %]
264
    [% INCLUDE 'datatables.inc' %]
255
    [% INCLUDE 'columns_settings.inc' %]
265
    [% INCLUDE 'columns_settings.inc' %]
256
    [% INCLUDE 'calendar.inc' %]
266
    [% INCLUDE 'calendar.inc' %]
267
    [% INCLUDE 'select2.inc' %]
257
    [% SET url_biblio_params = "biblionumber=" _ biblio.biblionumber %]
268
    [% SET url_biblio_params = "biblionumber=" _ biblio.biblionumber %]
258
    [% PROCESS patron_search_js table_id => 'table_borrowers', categories => categories, libraries => libraries, extended_attribute_types => attribute_type_codes, columns => columns, open_on_row_click => 1, on_click_url => '/cgi-bin/koha/recalls/request.pl?' _ url_biblio_params, redirect_if_one_result => 1, redirect_url => '/cgi-bin/koha/recalls/request.pl?' _ url_biblio_params, redirect_if_attribute_equal => 'cardnumber' %]
269
    [% PROCESS patron_search_js table_id => 'table_borrowers', categories => categories, libraries => libraries, extended_attribute_types => attribute_type_codes, columns => columns, open_on_row_click => 1, on_click_url => '/cgi-bin/koha/recalls/request.pl?' _ url_biblio_params, redirect_if_one_result => 1, redirect_url => '/cgi-bin/koha/recalls/request.pl?' _ url_biblio_params, redirect_if_attribute_equal => 'cardnumber' %]
259
    <script>
270
    <script>
260
        table_settings = [% TablesSettings.GetColumns( 'circ', 'circulation', 'table_borrowers', 'json' ) | $raw %];
271
        table_settings = [% TablesSettings.GetColumns( 'circ', 'circulation', 'table_borrowers', 'json' ) | $raw %];
272
        $.fn.select2.defaults.set("width", "100%" );
273
        $.fn.select2.defaults.set("dropdownAutoWidth", true );
261
        $(document).ready(function(){
274
        $(document).ready(function(){
262
            $("#recalls_patronsearch").on("submit", filter);
275
            $("#recalls_patronsearch").on("submit", filter);
263
            [% UNLESS patron %]
276
            [% UNLESS patron %]
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt (-1 / +5 lines)
Lines 212-218 Link Here
212
                [% IF ( recallerror ) %]
212
                [% IF ( recallerror ) %]
213
                    <div class="dialog alert">
213
                    <div class="dialog alert">
214
                        <h3>Hold cannot be converted to a recall</h3>
214
                        <h3>Hold cannot be converted to a recall</h3>
215
                        <p>A recall cannot be placed on one or more holds. Check your <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a>.</p>
215
                        <p>A recall cannot be placed for one or more holds. Check your <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a>.</p>
216
                    </div>
216
                    </div>
217
                [% END %]
217
                [% END %]
218
218
Lines 1752-1757 Link Here
1752
                window.open("/cgi-bin/koha/circ/hold-transfer-slip.pl?reserve_id=" + reserve_id);
1752
                window.open("/cgi-bin/koha/circ/hold-transfer-slip.pl?reserve_id=" + reserve_id);
1753
                return false;
1753
                return false;
1754
        })
1754
        })
1755
        $(".convert-to-recall").click(function(){
1756
            var reserve_id = $(this).attr('data-id');
1757
            $("select[class='rank-request'][data-hold-id='"+reserve_id+"']").val('recall');
1758
        });
1755
    </script>
1759
    </script>
1756
[% END %]
1760
[% END %]
1757
1761
(-)a/recalls/request.pl (+2 lines)
Lines 122-130 if ( $borrowernumber ) { Link Here
122
        if ( !$biblio->can_be_recalled({ patron => $patron }) ) {
122
        if ( !$biblio->can_be_recalled({ patron => $patron }) ) {
123
            $error = 1;
123
            $error = 1;
124
        }
124
        }
125
        my $patron_holds_count = Koha::Holds->search({ borrowernumber => $borrowernumber, biblionumber => $biblio->biblionumber, item_level_hold => 0 })->count;
125
126
126
        $template->param(
127
        $template->param(
127
            patron => $patron,
128
            patron => $patron,
129
            patron_holds_count => $patron_holds_count,
128
        );
130
        );
129
    }
131
    }
130
}
132
}
(-)a/t/db_dependent/Holds.t (-1 / +12 lines)
Lines 1827-1832 subtest 'ModReserve to convert a hold to a recall' => sub { Link Here
1827
            surname      => 'whatever surname',
1827
            surname      => 'whatever surname',
1828
            categorycode => $category->{categorycode},
1828
            categorycode => $category->{categorycode},
1829
            branchcode   => $branch,
1829
            branchcode   => $branch,
1830
            email        => 'name@email.com',
1830
        }
1831
        }
1831
    )->store;
1832
    )->store;
1832
    my $borrowernumber = $patron->borrowernumber;
1833
    my $borrowernumber = $patron->borrowernumber;
Lines 1837-1842 subtest 'ModReserve to convert a hold to a recall' => sub { Link Here
1837
            surname      => 'whatever surname',
1838
            surname      => 'whatever surname',
1838
            categorycode => $category->{categorycode},
1839
            categorycode => $category->{categorycode},
1839
            branchcode   => $branch,
1840
            branchcode   => $branch,
1841
            email        => 'name2@email.com',
1840
        }
1842
        }
1841
    )->store;
1843
    )->store;
1842
1844
Lines 1856-1861 subtest 'ModReserve to convert a hold to a recall' => sub { Link Here
1856
        },
1858
        },
1857
    });
1859
    });
1858
1860
1861
    my $notice = Koha::Notice::Template->new({
1862
        name                   => 'Hold cancellation',
1863
        module                 => 'reserves',
1864
        code                   => 'HOLD_CANCELLATION',
1865
        title                  => 'Hold cancelled',
1866
        content                => 'Your hold was cancelled.',
1867
        message_transport_type => 'email',
1868
        branchcode             => q{},
1869
    })->store();
1870
1859
    my $reserve_id = AddReserve(
1871
    my $reserve_id = AddReserve(
1860
        {
1872
        {
1861
            branchcode     => $branch,
1873
            branchcode     => $branch,
1862
- 

Return to bug 31391