Bugzilla – Attachment 66117 Details for
Bug 14385
Extend OpacHiddenItems to allow specifying exempt borrower categories
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14385: Trigger GetMarcBiblio's EmbedItemsInMarcBiblio call.
Bug-14385-Trigger-GetMarcBiblios-EmbedItemsInMarcB.patch (text/plain), 3.29 KB, created by
Mark Tompsett
on 2017-08-16 15:11:12 UTC
(
hide
)
Description:
Bug 14385: Trigger GetMarcBiblio's EmbedItemsInMarcBiblio call.
Filename:
MIME Type:
Creator:
Mark Tompsett
Created:
2017-08-16 15:11:12 UTC
Size:
3.29 KB
patch
obsolete
>From 864dd76af919614602dbfb4ed5c973de2a1d848f Mon Sep 17 00:00:00 2001 >From: Mark Tompsett <mtompset@hotmail.com> >Date: Tue, 1 Aug 2017 15:07:11 -0400 >Subject: [PATCH] Bug 14385: Trigger GetMarcBiblio's EmbedItemsInMarcBiblio > call. > >prove t/db_dependent/Items.t >--- > t/db_dependent/Items.t | 31 +++++++++++++++++++++++-------- > 1 file changed, 23 insertions(+), 8 deletions(-) > >diff --git a/t/db_dependent/Items.t b/t/db_dependent/Items.t >index 1852225..f0c6bf6 100755 >--- a/t/db_dependent/Items.t >+++ b/t/db_dependent/Items.t >@@ -567,18 +567,25 @@ subtest 'C4::Biblio::EmbedItemsInMarcBiblio' => sub { > my $record = C4::Biblio::GetMarcBiblio($biblionumber); > warning_is { C4::Biblio::EmbedItemsInMarcBiblio() } > { carped => 'EmbedItemsInMarcBiblio: No MARC record passed' }, >- 'Should crap is no record passed.'; >+ 'Should carp is no record passed.'; > >- C4::Biblio::EmbedItemsInMarcBiblio( $record, $biblionumber ); >+ C4::Biblio::EmbedItemsInMarcBiblio({ >+ marc_record => $record, >+ biblionumber => $biblionumber }); > my @items = $record->field($itemfield); > is( scalar @items, $number_of_items, 'Should return all items' ); > >- C4::Biblio::EmbedItemsInMarcBiblio( $record, $biblionumber, >- [ $itemnumbers[1], $itemnumbers[3] ] ); >+ C4::Biblio::EmbedItemsInMarcBiblio({ >+ marc_record => $record, >+ biblionumber => $biblionumber, >+ item_numbers => [ $itemnumbers[1], $itemnumbers[3] ] }); > @items = $record->field($itemfield); > is( scalar @items, 2, 'Should return all items present in the list' ); > >- C4::Biblio::EmbedItemsInMarcBiblio( $record, $biblionumber, undef, 1 ); >+ C4::Biblio::EmbedItemsInMarcBiblio({ >+ marc_record => $record, >+ biblionumber => $biblionumber, >+ opac => 1 }); > @items = $record->field($itemfield); > is( scalar @items, $number_of_items, 'Should return all items for opac' ); > >@@ -586,13 +593,18 @@ subtest 'C4::Biblio::EmbedItemsInMarcBiblio' => sub { > homebranch: ['$library1->{branchcode}']"; > t::lib::Mocks::mock_preference( 'OpacHiddenItems', $opachiddenitems ); > >- C4::Biblio::EmbedItemsInMarcBiblio( $record, $biblionumber ); >+ C4::Biblio::EmbedItemsInMarcBiblio({ >+ marc_record => $record, >+ biblionumber => $biblionumber }); > @items = $record->field($itemfield); > is( scalar @items, > $number_of_items, > 'Even with OpacHiddenItems set, all items should have been embedded' ); > >- C4::Biblio::EmbedItemsInMarcBiblio( $record, $biblionumber, undef, 1 ); >+ C4::Biblio::EmbedItemsInMarcBiblio({ >+ marc_record => $record, >+ biblionumber => $biblionumber, >+ opac => 1 ); > @items = $record->field($itemfield); > is( > scalar @items, >@@ -603,7 +615,10 @@ subtest 'C4::Biblio::EmbedItemsInMarcBiblio' => sub { > $opachiddenitems = " > homebranch: ['$library1->{branchcode}', '$library2->{branchcode}']"; > t::lib::Mocks::mock_preference( 'OpacHiddenItems', $opachiddenitems ); >- C4::Biblio::EmbedItemsInMarcBiblio( $record, $biblionumber, undef, 1 ); >+ C4::Biblio::EmbedItemsInMarcBiblio({ >+ marc_record => $record, >+ biblionumber => $biblionumber, >+ opac => 1 }); > @items = $record->field($itemfield); > is( > scalar @items, >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 14385
:
40158
|
53483
|
53494
|
53505
|
53506
|
53507
|
55593
|
64688
|
64689
|
64690
|
64691
|
64692
|
64693
|
65322
|
65323
|
65324
|
65325
|
65376
|
65377
|
65378
|
65379
|
65380
|
65381
|
65382
|
65383
|
65384
|
65385
|
65386
|
65387
|
65388
|
65389
|
65408
|
65409
|
65410
|
65411
|
66103
|
66104
|
66105
|
66106
|
66107
|
66108
|
66109
|
66110
|
66111
|
66112
|
66113
|
66114
|
66115
|
66116
|
66117
|
66118
|
66125
|
66126
|
66127
|
66128
|
66129
|
66130
|
66131
|
66132
|
66192
|
66193
|
66194
|
66195
|
66196
|
66197
|
66198
|
66199
|
66200
|
66201
|
66202
|
66203
|
66204
|
66205
|
66206
|
66207
|
66208
|
66209
|
66210
|
66302
|
66303
|
66347
|
66361
|
66372
|
66401
|
66403
|
66520
|
66521
|
66522
|
66523
|
66524
|
66525
|
66526
|
66527
|
66528
|
66529
|
66530
|
66531
|
66532
|
66533
|
66534
|
66535
|
66536
|
66537
|
66538
|
66539
|
66540
|
66541
|
66542
|
66543
|
66544
|
66545
|
70256
|
70257
|
70258
|
70259
|
72445
|
72446
|
72447
|
72448
|
72449
|
72450
|
72451
|
72452
|
72453
|
72454
|
74001
|
78654
|
78655
|
78656
|
78657
|
78825
|
78826
|
78827
|
78828
|
80411
|
80463
|
80464
|
80465
|
80466
|
80467
|
89963
|
90038