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

(-)a/Koha/Item.pm (+12 lines)
Lines 1830-1835 that this complicates filtering, but some query trickery could do it in the cont Link Here
1830
1830
1831
sub item_type {
1831
sub item_type {
1832
    return shift->itemtype;
1832
    return shift->itemtype;
1833
=head3 effective_bookable
1834
1835
  my $bookable = $item->effective_bookable;
1836
1837
Returns the effective bookability of the current item, be that item or itemtype level
1838
1839
=cut
1840
1841
sub effective_bookable {
1842
    my ($self) = @_;
1843
1844
    return defined( $self->bookable ) ? $self->bookable : $self->itemtype->bookable;
1833
}
1845
}
1834
1846
1835
=head3 orders
1847
=head3 orders
(-)a/Koha/Items.pm (-5 / +10 lines)
Lines 169-180 Returns a new resultset, containing only those items that are allowed to be book Link Here
169
sub filter_by_bookable {
169
sub filter_by_bookable {
170
    my ($self) = @_;
170
    my ($self) = @_;
171
171
172
    my $params =
172
    my @bookable_itemtypes = Koha::ItemTypes->search( { bookable => 1 } )->get_column('itemtype');
173
        C4::Context->preference('item-level_booking') == 1
174
        ? { bookable => 1 }
175
        : { itype    => { -in => \'SELECT itemtype FROM itemtypes WHERE bookable = 1' } };
176
173
177
    return $self->search($params);
174
    my ( $where, $attr );
175
    if ( C4::Context->preference("item-level_itypes") ) {
176
        $where = [ { bookable => 1 }, { bookable => undef, itype => { -in => \@bookable_itemtypes } } ];
177
    } else {
178
        $where = [ { bookable => 1 }, { bookable => undef, 'biblioitem.itemtype' => { -in => \@bookable_itemtypes } } ];
179
        $attr  = { join => 'biblioitem' };
180
    }
181
182
    return $self->search( $where, $attr );
178
}
183
}
179
184
180
=head3 move_to_biblio
185
=head3 move_to_biblio
(-)a/api/v1/swagger/definitions/item.yaml (-3 / +11 lines)
Lines 24-29 properties: Link Here
24
      - "null"
24
      - "null"
25
    description: Information about the acquisition source (it is not really a vendor id)
25
    description: Information about the acquisition source (it is not really a vendor id)
26
  bookable:
26
  bookable:
27
    type:
28
      - boolean
29
      - "null"
30
    description: Item level bookability override.
31
  effective_bookable:
27
    type: boolean
32
    type: boolean
28
    description: Allow bookings on this item.
33
    description: Allow bookings on this item.
29
  home_library_id:
34
  home_library_id:
Lines 90-96 properties: Link Here
90
      - string
95
      - string
91
      - "null"
96
      - "null"
92
    format: date-time
97
    format: date-time
93
    description: The date and time an item was last marked as withdrawn, NULL if not
98
    description:
99
      The date and time an item was last marked as withdrawn, NULL if not
94
      withdrawn
100
      withdrawn
95
  callnumber:
101
  callnumber:
96
    type:
102
    type:
Lines 150-163 properties: Link Here
150
    type:
156
    type:
151
      - string
157
      - string
152
      - "null"
158
      - "null"
153
    description: Linked to the CART and PROC temporary locations feature, stores the
159
    description:
160
      Linked to the CART and PROC temporary locations feature, stores the
154
      permanent shelving location
161
      permanent shelving location
155
  checked_out_date:
162
  checked_out_date:
156
    type:
163
    type:
157
      - string
164
      - string
158
      - "null"
165
      - "null"
159
    format: date
166
    format: date
160
    description: Defines if item is checked out (NULL for not checked out, and checkout
167
    description:
168
      Defines if item is checked out (NULL for not checked out, and checkout
161
      date for checked out)
169
      date for checked out)
162
  call_number_source:
170
  call_number_source:
163
    type:
171
    type:
(-)a/api/v1/swagger/paths/biblios.yaml (-2 / +4 lines)
Lines 372-378 Link Here
372
        collectionFormat: csv
372
        collectionFormat: csv
373
      - name: checked_in
373
      - name: checked_in
374
        in: query
374
        in: query
375
        description: By default, current checkouts are returned, when this is true then
375
        description:
376
          By default, current checkouts are returned, when this is true then
376
          checked in checkouts are returned as result.
377
          checked in checkouts are returned as result.
377
        type: boolean
378
        type: boolean
378
    produces:
379
    produces:
Lines 447-452 Link Here
447
            - cover_image_ids
448
            - cover_image_ids
448
            - item_group_item.item_group.description
449
            - item_group_item.item_group.description
449
            - serial_item.serial
450
            - serial_item.serial
451
            - effective_bookable
450
        collectionFormat: csv
452
        collectionFormat: csv
451
      - $ref: "../swagger.yaml#/parameters/match"
453
      - $ref: "../swagger.yaml#/parameters/match"
452
      - $ref: "../swagger.yaml#/parameters/order_by"
454
      - $ref: "../swagger.yaml#/parameters/order_by"
Lines 811-817 Link Here
811
                - integer
813
                - integer
812
                - "null"
814
                - "null"
813
          required:
815
          required:
814
              - rating
816
            - rating
815
          additionalProperties: false
817
          additionalProperties: false
816
    produces:
818
    produces:
817
      - application/json
819
      - application/json
(-)a/api/v1/swagger/paths/items.yaml (-1 / +3 lines)
Lines 22-27 Link Here
22
          enum:
22
          enum:
23
            - +strings
23
            - +strings
24
            - biblio
24
            - biblio
25
            - effective_bookable
25
        collectionFormat: csv
26
        collectionFormat: csv
26
      - $ref: "../swagger.yaml#/parameters/match"
27
      - $ref: "../swagger.yaml#/parameters/match"
27
      - $ref: "../swagger.yaml#/parameters/order_by"
28
      - $ref: "../swagger.yaml#/parameters/order_by"
Lines 81-86 Link Here
81
          type: string
82
          type: string
82
          enum:
83
          enum:
83
            - +strings
84
            - +strings
85
            - effective_bookable
84
        collectionFormat: csv
86
        collectionFormat: csv
85
    consumes:
87
    consumes:
86
      - application/json
88
      - application/json
Lines 150-156 Link Here
150
            * linked_analytics: The item has linked analytic records
152
            * linked_analytics: The item has linked analytic records
151
            * not_same_branch: The item is blocked by independent branches
153
            * not_same_branch: The item is blocked by independent branches
152
        schema:
154
        schema:
153
              $ref: "../swagger.yaml#/definitions/error"
155
          $ref: "../swagger.yaml#/definitions/error"
154
      "500":
156
      "500":
155
        description: |
157
        description: |
156
          Internal server error. Possible `error_code` attribute values:
158
          Internal server error. Possible `error_code` attribute values:
(-)a/catalogue/updateitem.pl (-1 / +2 lines)
Lines 39-45 my $itemnotes_nonpublic=$cgi->param('itemnotes_nonpublic'); Link Here
39
my $withdrawn=$cgi->param('withdrawn');
39
my $withdrawn=$cgi->param('withdrawn');
40
my $damaged=$cgi->param('damaged');
40
my $damaged=$cgi->param('damaged');
41
my $exclude_from_local_holds_priority = $cgi->param('exclude_from_local_holds_priority');
41
my $exclude_from_local_holds_priority = $cgi->param('exclude_from_local_holds_priority');
42
my $bookable = $cgi->param('bookable');
42
my $bookable = $cgi->param('bookable') // q{};
43
43
44
my $confirm=$cgi->param('confirm');
44
my $confirm=$cgi->param('confirm');
45
my $dbh = C4::Context->dbh;
45
my $dbh = C4::Context->dbh;
Lines 77-82 elsif ( $op eq "cud-set_public_note" ) { # i.e., itemnotes parameter passed from Link Here
77
    $item->exclude_from_local_holds_priority($exclude_from_local_holds_priority);
77
    $item->exclude_from_local_holds_priority($exclude_from_local_holds_priority);
78
    $messages = "updated_exclude_from_local_holds_priority=$exclude_from_local_holds_priority&";
78
    $messages = "updated_exclude_from_local_holds_priority=$exclude_from_local_holds_priority&";
79
} elsif ( $op eq "cud-set_bookable" && $bookable ne $item_data_hashref->{'bookable'} ) {
79
} elsif ( $op eq "cud-set_bookable" && $bookable ne $item_data_hashref->{'bookable'} ) {
80
    undef($bookable) if $bookable eq "";
80
    $item->bookable($bookable);
81
    $item->bookable($bookable);
81
} elsif ( $op eq "cud-set_damaged" && $damaged ne $item_data_hashref->{'damaged'}) {
82
} elsif ( $op eq "cud-set_damaged" && $damaged ne $item_data_hashref->{'damaged'}) {
82
    $item->damaged($damaged);
83
    $item->damaged($damaged);
(-)a/installer/data/mysql/atomicupdate/Bug_35906_add-column-bookable-itemtype.pl (-7 / +21 lines)
Lines 7-21 return { Link Here
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{
10
        $dbh->do(
11
            INSERT IGNORE INTO systempreferences (`variable`, `value`, `options`, `explanation`, `type`) VALUES  ('item-level_booking', 1, '', 'enables item type level for future booking', 'YesNo');
11
            q{
12
        });
13
14
        $dbh->do(q{
15
            ALTER TABLE itemtypes ADD IF NOT EXISTS bookable INT(1) DEFAULT 0
12
            ALTER TABLE itemtypes ADD IF NOT EXISTS bookable INT(1) DEFAULT 0
16
        });
13
        }
14
        );
17
15
18
        say $out "Added new system preference 'item-level_booking'";
19
        say $out "Added column 'itemtypes.bookable'";
16
        say $out "Added column 'itemtypes.bookable'";
17
18
        $dbh->do(
19
            q{
20
            ALTER TABLE items MODIFY COLUMN bookable tinyint(1) DEFAULT NULL COMMENT 'nullable boolean value defining whether this this item is available for bookings or not'
21
        }
22
        );
23
24
        say $out "Updated column 'items.bookable' allow nullable";
25
26
        $dbh->do(
27
            q{
28
            ALTER TABLE deleteditems MODIFY COLUMN bookable tinyint(1) DEFAULT NULL COMMENT 'nullable boolean value defining whether this this item is available for bookings or not'
29
        }
30
        );
31
32
        say $out "Updated column 'deleteditems.bookable' allow nullable";
33
20
    },
34
    },
21
};
35
};
(-)a/installer/data/mysql/kohastructure.sql (-2 / +2 lines)
Lines 2738-2744 CREATE TABLE `deleteditems` ( Link Here
2738
  `biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from biblio table used to link this item to the right bib record',
2738
  `biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from biblio table used to link this item to the right bib record',
2739
  `biblioitemnumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblioitems table to link to item to additional information',
2739
  `biblioitemnumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblioitems table to link to item to additional information',
2740
  `barcode` varchar(20) DEFAULT NULL COMMENT 'item barcode (MARC21 952$p)',
2740
  `barcode` varchar(20) DEFAULT NULL COMMENT 'item barcode (MARC21 952$p)',
2741
  `bookable` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'boolean value defining whether this this item is available for bookings or not',
2741
  `bookable` tinyint(1) DEFAULT NULL COMMENT 'nullable boolean value defining whether this this item is available for bookings or not',
2742
  `dateaccessioned` date DEFAULT NULL COMMENT 'date the item was acquired or added to Koha (MARC21 952$d)',
2742
  `dateaccessioned` date DEFAULT NULL COMMENT 'date the item was acquired or added to Koha (MARC21 952$d)',
2743
  `booksellerid` longtext DEFAULT NULL COMMENT 'where the item was purchased (MARC21 952$e)',
2743
  `booksellerid` longtext DEFAULT NULL COMMENT 'where the item was purchased (MARC21 952$e)',
2744
  `homebranch` varchar(10) DEFAULT NULL COMMENT 'foreign key from the branches table for the library that owns this item (MARC21 952$a)',
2744
  `homebranch` varchar(10) DEFAULT NULL COMMENT 'foreign key from the branches table for the library that owns this item (MARC21 952$a)',
Lines 3995-4001 CREATE TABLE `items` ( Link Here
3995
  `biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from biblio table used to link this item to the right bib record',
3995
  `biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from biblio table used to link this item to the right bib record',
3996
  `biblioitemnumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblioitems table to link to item to additional information',
3996
  `biblioitemnumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblioitems table to link to item to additional information',
3997
  `barcode` varchar(20) DEFAULT NULL COMMENT 'item barcode (MARC21 952$p)',
3997
  `barcode` varchar(20) DEFAULT NULL COMMENT 'item barcode (MARC21 952$p)',
3998
  `bookable` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'boolean value defining whether this this item is available for bookings or not',
3998
  `bookable` tinyint(1) DEFAULT NULL COMMENT 'nullable boolean value defining whether this this item is available for bookings or not',
3999
  `dateaccessioned` date DEFAULT NULL COMMENT 'date the item was acquired or added to Koha (MARC21 952$d)',
3999
  `dateaccessioned` date DEFAULT NULL COMMENT 'date the item was acquired or added to Koha (MARC21 952$d)',
4000
  `booksellerid` longtext DEFAULT NULL COMMENT 'where the item was purchased (MARC21 952$e)',
4000
  `booksellerid` longtext DEFAULT NULL COMMENT 'where the item was purchased (MARC21 952$e)',
4001
  `homebranch` varchar(10) DEFAULT NULL COMMENT 'foreign key from the branches table for the library that owns this item (MARC21 952$a)',
4001
  `homebranch` varchar(10) DEFAULT NULL COMMENT 'foreign key from the branches table for the library that owns this item (MARC21 952$a)',
(-)a/installer/data/mysql/mandatory/sysprefs.sql (-1 lines)
Lines 347-353 INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` Link Here
347
('IssueLostItem','alert','Defines what should be done when an attempt is made to issue an item that has been marked as lost.','alert|confirm|nothing','Choice'),
347
('IssueLostItem','alert','Defines what should be done when an attempt is made to issue an item that has been marked as lost.','alert|confirm|nothing','Choice'),
348
('IssuingInProcess','0',NULL,'If ON, disables fines if the patron is issuing item that accumulate debt','YesNo'),
348
('IssuingInProcess','0',NULL,'If ON, disables fines if the patron is issuing item that accumulate debt','YesNo'),
349
('item-level_itypes','1','','If ON, enables Item-level Itemtype / Issuing Rules','YesNo'),
349
('item-level_itypes','1','','If ON, enables Item-level Itemtype / Issuing Rules','YesNo'),
350
('item-level_booking','1','','If ON, enables Item-level for future booking feature','YesNo'),
351
('itemBarcodeFallbackSearch','0',NULL,'If set, uses scanned item barcodes as a catalogue search if not found as barcodes','YesNo'),
350
('itemBarcodeFallbackSearch','0',NULL,'If set, uses scanned item barcodes as a catalogue search if not found as barcodes','YesNo'),
352
('itemBarcodeInputFilter','','whitespace|T-prefix|cuecat|libsuite8|EAN13','If set, allows specification of a item barcode input filter','Choice'),
351
('itemBarcodeInputFilter','','whitespace|T-prefix|cuecat|libsuite8|EAN13','If set, allows specification of a item barcode input filter','Choice'),
353
('itemcallnumber','',NULL,'The MARC field/subfield that is used to calculate the itemcallnumber (Dewey would be 082ab or 092ab; LOC would be 050ab or 090ab) could be 852hi from an item record','free'),
352
('itemcallnumber','',NULL,'The MARC field/subfield that is used to calculate the itemcallnumber (Dewey would be 082ab or 092ab; LOC would be 050ab or 090ab) could be 852hi from an item record','free'),
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt (-11 / +9 lines)
Lines 258-274 Link Here
258
                        [% END %]
258
                        [% END %]
259
                        <span class="hint">If checked, items will be automatically checked in once they've reached their due date. This feature requires the <code>misc/cronjobs/automatic_checkin.pl</code> cronjob. Ask your system administrator to schedule it.</span>
259
                        <span class="hint">If checked, items will be automatically checked in once they've reached their due date. This feature requires the <code>misc/cronjobs/automatic_checkin.pl</code> cronjob. Ask your system administrator to schedule it.</span>
260
                </li>
260
                </li>
261
                [% IF Koha.Preference('item-level_booking') == 0 %]
261
                <li>
262
                    <li>
262
                    <label for="bookable">Bookable: </label>
263
                        <label for="bookable">Bookable: </label>
263
                        [% IF itemtype.bookable %]
264
                            [% IF itemtype.bookable %]
264
                            <input type="checkbox" id="bookable" name="bookable" checked="checked" value="1" />
265
                                <input type="checkbox" id="bookable" name="bookable" checked="checked" value="1" />
265
                        [% ELSE %]
266
                            [% ELSE %]
266
                            <input type="checkbox" id="bookable" name="bookable" value="1" />
267
                                <input type="checkbox" id="bookable" name="bookable" value="1" />
267
                        [% END %]
268
                            [% END %]
268
                        <span class="hint">If checked, all items of this type will be enabled for future bookings unless overridden at the item level.</span>
269
                            <span class="hint">If checked, all items of this type will be enabled for future bookings.</span>
269
                </li>
270
                    </li>
271
                [% END %]
272
                <li>
270
                <li>
273
                    <label for="rentalcharge">Rental charge: </label>
271
                    <label for="rentalcharge">Rental charge: </label>
274
                    <input type="text" id="rentalcharge" name="rentalcharge" size="10" value="[% itemtype.rentalcharge | $Price on_editing => 1 %]" min="0" />
272
                    <input type="text" id="rentalcharge" name="rentalcharge" size="10" value="[% itemtype.rentalcharge | $Price on_editing => 1 %]" min="0" />
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref (-7 lines)
Lines 1019-1031 Circulation: Link Here
1019
                  0: "Don't allow"
1019
                  0: "Don't allow"
1020
            - holds to be automatically filled after being automatically checked in.
1020
            - holds to be automatically filled after being automatically checked in.
1021
            - '<br><strong>NOTE:</strong> This system preference requires the <code>misc/cronjobs/automatic_checkin.pl</code> cronjob. Ask your system administrator to schedule it.'
1021
            - '<br><strong>NOTE:</strong> This system preference requires the <code>misc/cronjobs/automatic_checkin.pl</code> cronjob. Ask your system administrator to schedule it.'
1022
        -
1023
            - Use the type of the
1024
            - pref: item-level_booking
1025
              choices:
1026
                  1: item
1027
                  0: itemtype
1028
            - level to defined witch items can be bookable in future.
1029
    Patron restrictions:
1022
    Patron restrictions:
1030
        -
1023
        -
1031
            - pref: PatronRestrictionTypes
1024
            - pref: PatronRestrictionTypes
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt (-5 / +12 lines)
Lines 360-380 Link Here
360
                                                    <input type="hidden" name="biblionumber" value="[% ITEM_DAT.biblionumber | html %]" />
360
                                                    <input type="hidden" name="biblionumber" value="[% ITEM_DAT.biblionumber | html %]" />
361
                                                    <input type="hidden" name="biblioitemnumber" value="[% ITEM_DAT.biblioitemnumber | html %]" />
361
                                                    <input type="hidden" name="biblioitemnumber" value="[% ITEM_DAT.biblioitemnumber | html %]" />
362
                                                    <input type="hidden" name="itemnumber" value="[% ITEM_DAT.itemnumber | html %]" />
362
                                                    <input type="hidden" name="itemnumber" value="[% ITEM_DAT.itemnumber | html %]" />
363
                                                    <select name="bookable" [% IF ITEM_DAT.effective_itemtype.bookable == 1 && Koha.Preference('item-level_booking') == 0 %]disabled[% END %]>
363
                                                    <select name="bookable">
364
                                                    [% IF ITEM_DAT.bookable == 1 || ITEM_DAT.effective_itemtype.bookable == 1 %]
364
                                                    [% IF ITEM_DAT.bookable == 1 %]
365
                                                        <option value="">Follow item type</option>
365
                                                        <option value="1" selected="selected">Yes</option>
366
                                                        <option value="1" selected="selected">Yes</option>
366
                                                        <option value="0">No</option>
367
                                                        <option value="0">No</option>
367
                                                        [% ELSE %]
368
                                                    [% ELSIF ITEM_DAT.bookable == 0 %]
369
                                                        <option value="">Follow item type</option>
368
                                                        <option value="1">Yes</option>
370
                                                        <option value="1">Yes</option>
369
                                                        <option value="0" selected="selected">No</option>
371
                                                        <option value="0" selected="selected">No</option>
370
                                                        [% END %]
372
                                                    [% ELSE %]
373
                                                        <option value="" selected="selected">Follow item type</option>
374
                                                        <option value="1">Yes</option>
375
                                                        <option value="0">No</option>
376
                                                    [% END %]
371
                                                    </select>
377
                                                    </select>
372
                                                    <input type="hidden" name="op" value="cud-set_bookable" />
378
                                                    <input type="hidden" name="op" value="cud-set_bookable" />
373
                                                    <input type="submit" name="submit" class="btn btn-primary btn-xs" value="Update" />
379
                                                    <input type="submit" name="submit" class="btn btn-primary btn-xs" value="Update" />
374
                                                </form>
380
                                                </form>
375
                                            [% ELSE %]
381
                                            [% ELSE %]
376
                                                [% IF ITEM_DAT.bookable == 1 %] Yes [% ELSE %] No [% END %]
382
                                                [% IF ITEM_DAT.bookable == 1 %] Yes [% ELSIF ITEM_DAT.bookable == 0 %] No [% ELSE %] Follow item type [% END %]
377
                                            [% END %]
383
                                            [% END %]
384
                                            <span class="hint"> Item type bookable: [% IF ITEM_DAT.effective_itemtype.bookable == 1 %]Yes[% ELSE %]No[% END %]</span>
378
                                        </li>
385
                                        </li>
379
386
380
                                    </ol> <!-- /.bibliodetails -->
387
                                    </ol> <!-- /.bibliodetails -->
(-)a/t/db_dependent/Koha/Items.t (-23 / +18 lines)
Lines 1993-2033 subtest 'filter_by_for_hold' => sub { Link Here
1993
};
1993
};
1994
1994
1995
subtest 'filter_by_bookable' => sub {
1995
subtest 'filter_by_bookable' => sub {
1996
    plan tests => 3;
1996
    plan tests => 4;
1997
1997
1998
    $schema->storage->txn_begin;
1998
    $schema->storage->txn_begin;
1999
1999
2000
    my $biblio = $builder->build_sample_biblio;
2000
    t::lib::Mocks::mock_preference('item-level_itypes', 0);
2001
2002
    my $bookable_item_type = $builder->build_object( { class => 'Koha::ItemTypes', value => { bookable => 1 } } );
2003
    my $non_bookable_item_type = $builder->build_object( { class => 'Koha::ItemTypes', value => { bookable => 0 } } );
2004
    my $biblio = $builder->build_sample_biblio({ itemtype => $bookable_item_type->itemtype });
2001
2005
2002
    # bookable items
2006
    # bookable items
2003
    my $bookable_item1 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, bookable => 1 } );
2007
    my $bookable_item1 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, itype => $bookable_item_type->itemtype, bookable => 1 } );
2004
2008
2005
    # not bookable items
2009
    # not bookable items
2006
    my $non_bookable_item1 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, bookable => 0 } );
2010
    my $non_bookable_item1 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, itype => $non_bookable_item_type->itemtype, bookable => 0 } );
2007
    my $non_bookable_item2 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, bookable => 0 } );
2011
    my $non_bookable_item2 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, itype => $non_bookable_item_type->itemtype, bookable => 0 } );
2008
2012
2009
    is( $biblio->items->filter_by_bookable->count, 1, "filter_by_bookable returns the correct number of items" );
2013
    is( $biblio->items->filter_by_bookable->count, 1, "filter_by_bookable returns the correct number of items set at item level" );
2010
    is(
2014
    is(
2011
        $biblio->items->filter_by_bookable->next->itemnumber, $bookable_item1->itemnumber,
2015
        $biblio->items->filter_by_bookable->next->itemnumber, $bookable_item1->itemnumber,
2012
        "the correct item is returned from filter_by_bookable"
2016
        "the correct item is returned from filter_by_bookable at the item level"
2013
    );
2017
    );
2014
2018
2015
    # unset level booking on item (for itemtype)
2019
    $bookable_item1->bookable(undef)->store();
2016
    t::lib::Mocks::mock_preference( 'item-level_booking', 0 );
2020
    $non_bookable_item1->bookable(undef)->store();
2017
2021
    $non_bookable_item2->bookable(undef)->store();
2018
    # test with itemtype directly bookable
2022
    $biblio->get_from_storage;
2019
    my $item_type = $builder->build_object( { class => 'Koha::ItemTypes', value => { bookable => 1 } } );
2023
    is( $biblio->items->filter_by_bookable->count, 3, "filter_by_bookable returns the correct number of items when not set at item level and using biblio level itemtypes" );
2020
    my $biblio2   = $builder->build_sample_biblio( { itemtype => $item_type->itemtype } );
2021
2024
2022
    # bookable items
2025
    t::lib::Mocks::mock_preference('item-level_itypes', 1);
2023
    my $bookable_item3 = $builder->build_sample_item(
2026
    is( $biblio->items->filter_by_bookable->count, 1, "filter_by_bookable returns the correct number of items when not set at item level and using item level itemtypes" );
2024
        { biblionumber => $biblio2->biblionumber, itype => $item_type->itemtype, bookable => 1 } );
2025
    my $bookable_item4 = $builder->build_sample_item(
2026
        { biblionumber => $biblio2->biblionumber, itype => $item_type->itemtype, bookable => 0 } );
2027
2028
    # items are bookable even if bookable => 0 on item (due to itemtype bookable => 1)
2029
    is( $biblio2->items->filter_by_bookable->count, 2, "filter_by_bookable returns the correct number of items" );
2030
2027
2031
    $schema->storage->txn_rollback;
2028
    $schema->storage->txn_rollback;
2032
2033
};
2029
};
2034
- 

Return to bug 35906