Bugzilla – Attachment 6881 Details for
Bug 7278
In the items table, make items.materials of type text, and show its contents at circulation
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 7728 : Materials Specified field made larger and displayed
SIGNED-OFF-Bug-7728--Materials-Specified-field-mad.patch (text/plain), 6.18 KB, created by
Katrin Fischer
on 2011-12-20 07:45:25 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 7728 : Materials Specified field made larger and displayed
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2011-12-20 07:45:25 UTC
Size:
6.18 KB
patch
obsolete
>From 9f2c56d963272af7c0d7e3be0180138cb432ef50 Mon Sep 17 00:00:00 2001 >From: Chris Cormack <chrisc@catalyst.net.nz> >Date: Thu, 1 Dec 2011 10:46:52 +1300 >Subject: [PATCH] [SIGNED-OFF] Bug 7728 : Materials Specified field made > larger and displayed > >This first page makes the database change, and displays the field on >the detail.pl and moredetail.pl pages in the staff client > >How to Test: > >* Add a note to the materials specified page in cataloguing (952 >subfield 3 for MARC21, but it shouldn't matter as long as you have a >field mapped to items.materials >* Check it displays at catalogue/detail.pl >* Check it displays at catalogue/moredetail.pl > >Patch to follow to have the note display at checkin/checkout > >http://bugs.koha-community.org/show_bug.cgi?id=7278 > >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Fixed conflicts in updatedatabase.pl and kohastructure.sql >--- > catalogue/detail.pl | 6 +++++- > installer/data/mysql/kohastructure.sql | 2 +- > installer/data/mysql/updatedatabase.pl | 8 ++++++++ > .../prog/en/modules/catalogue/detail.tt | 4 ++++ > .../prog/en/modules/catalogue/moredetail.tt | 1 + > 5 files changed, 19 insertions(+), 2 deletions(-) > >diff --git a/catalogue/detail.pl b/catalogue/detail.pl >index 580949d..a700b95 100755 >--- a/catalogue/detail.pl >+++ b/catalogue/detail.pl >@@ -174,6 +174,7 @@ my $authvalcode_items_itemlost = GetAuthValCode('items.itemlost',$fw); > my $authvalcode_items_damaged = GetAuthValCode('items.damaged', $fw); > > my $analytics_flag; >+my $materials_flag; # set this if the items have anything in the materials field > foreach my $item (@items) { > > $item->{homebranch} = GetBranchName($item->{homebranch}); >@@ -251,7 +252,9 @@ foreach my $item (@items) { > $analytics_flag=1; > $item->{countanalytics} = $countanalytics; > } >- >+ if ($item->{'materials'} ne ''){ >+ $materials_flag = 1; >+ } > push @itemloop, $item; > } > >@@ -276,6 +279,7 @@ $template->param( > hostrecords => $hostrecords, > analytics_flag => $analytics_flag, > C4::Search::enabled_staff_search_views, >+ materials => $materials_flag, > ); > > if (C4::Context->preference("AlternateHoldingsField") && scalar @items == 0) { >diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql >index a03bd23..682df4f 100644 >--- a/installer/data/mysql/kohastructure.sql >+++ b/installer/data/mysql/kohastructure.sql >@@ -1031,7 +1031,7 @@ CREATE TABLE `items` ( -- holdings/item information > `cn_source` varchar(10) default NULL, -- classification source used on this item (MARC21 952$2) > `cn_sort` varchar(30) default NULL, -- normalized form of the call number (MARC21 952$o) used for sorting > `ccode` varchar(10) default NULL, -- authorized value for the collection code associated with this item (MARC21 952$8) >- `materials` varchar(10) default NULL, -- materials specified (MARC21 952$3) >+ `materials` text default NULL, -- materials specified (MARC21 952$3) > `uri` varchar(255) default NULL, -- URL for the item (MARC21 952$u) > `itype` varchar(10) default NULL, -- foreign key from the itemtypes table defining the type for this item (MARC21 952$y) > `more_subfields_xml` longtext default NULL, -- additional 952 subfields in XML format >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index 0063a75..8ce3924 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -4578,6 +4578,14 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { > SetVersion($DBversion); > } > >+$DBversion = "3.07.00.XXX"; >+if (C4::Context->preference("Version") < TransformToNum($DBversion)) { >+ $dbh->do("ALTER TABLE items MODIFY materials text;"); >+ print "Upgrade to $DBversion done alter items.material from varchar(10) to text \n"; >+ SetVersion ($DBversion); >+} >+ >+ > =head1 FUNCTIONS > > =head2 DropAllForeignKeys($table) >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >index 019fa10..2a4a975 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >@@ -243,6 +243,7 @@ function verify_images() { > <th>Status</th> > <th>Last seen</th> > <th>Barcode</th> >+ [% IF materials %]<th>Materials Specified</th>[% END %] > [% IF ( volinfo ) %]<th>Publication Details</th>[% END %] > [% IF ( itemdata_uri ) %]<th>url</th>[% END %] > [% IF ( itemdata_copynumber ) %]<th>Copy No.</th>[% END %] >@@ -372,6 +373,9 @@ function verify_images() { > [% IF ( itemdata_copynumber ) %] > <td class="copynumber">[% itemloo.copynumber %]</td> > [% END %] >+ [% IF itemloo.materials %] >+ <td> [% itemloo.materials %] </td> >+ [% END %] > [% IF ( itemdata_itemnotes ) %]<td><div class="itemnotes">[% itemloo.itemnotes %]</div></td>[% END %] > [% IF ( SpineLabelShowPrintOnBibDetails ) %] > <td><a href="/cgi-bin/koha/labels/spinelabel-print.pl?barcode=[% itemloo.barcode %]" >Print Label</a></td> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >index e79e9e8..57dac9a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >@@ -58,6 +58,7 @@ > <li><span class="label">Item Callnumber:</span> [% ITEM_DAT.itemcallnumber %] </li> > [% IF ( ITEM_DAT.copyvol ) %]<li><span class="label">Copy / Vol :</span> [% ITEM_DAT.copyvol %] </li> [% END %] > [% IF ( ITEM_DAT.replacementprice ) %]<li><span class="label">Replacement Price:</span> [% ITEM_DAT.replacementprice %] </li> [% END %] >+ [% IF ITEM_DAT.materials %]<li><span class="label">Materials Specified:</span> [% ITEM_DAT.materials %] </li> [% END %] > </ol></div> > <div class="listgroup"><h4>Statuses [% IF ( ITEM_DAT.status_advisory ) %]( > [% IF ( ITEM_DAT.notforloantext ) %][% ITEM_DAT.notforloantext %] [% END %] >-- >1.7.5.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 7278
:
6480
|
6481
|
6568
| 6881 |
6882
|
6883
|
6885