Bug 11266 - Z39.50 search results: DATE blank - add PUBLISHER
Summary: Z39.50 search results: DATE blank - add PUBLISHER
Status: RESOLVED MOVED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Galen Charlton
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-18 14:55 UTC by Andrea Furin
Modified: 2015-05-25 08:14 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrea Furin 2013-11-18 14:55:47 UTC
1. If you have the field DATE blank when you make a Z39.50 search you have to modify the file:
/usr/share/koha/lib/C4/Breeding.pm
at line 394:
  $row_data{date}         =  $oldbiblio->{copyrightdate};
in
  $row_data{date}         =  $oldbiblio->{publicationyear};
(Make a copy of your modification becaurse probably ad update will change that file).

2. If you want to see the publisher's name in Z39.50 results you have to add a line in:
/usr/share/koha/lib/C4/Breeding.pm
at line 394:
       $row_data{server}       = $servername[$k];
       $row_data{isbn}         = $oldbiblio->{isbn};
       $row_data{lccn}         = $oldbiblio->{lccn};
       $row_data{title}        = $oldbiblio->{title};
       $row_data{author}       = $oldbiblio->{author};
       $row_data{date}         = $oldbiblio->{publicationyear};
****   $row_data{publisher}    = $oldbiblio->{publishercode};
       $row_data{edition}      = $oldbiblio->{editionstatement};

and in:
/usr/share/koha/intranet/htdocs/intranet-tmpl/prog/en/modules/cataloguing/z3950_search.tt
(change ../prog/LANGUAGE/modules/.. if not english)
at line 155 the search results mask:
        <th>Server</th>
        <th>Title</th>
        <th>Author</th>
        <th>Date</th>
****    <th>Publisher</th>
        <th>Edition</th>
        <th>ISBN</th>
        <th>LCCN</th>

[Below the correspondet modified lines:]

            <td>[% breeding_loo.server %] <div class="linktools"><a
href="/cgi-bin/koha/catalogue/showmarc.pl?importid=[% breeding_loo.bree$
            <td>[% breeding_loo.title |html %]</td>
            <td>[% breeding_loo.author %]</td>
            <td>[% breeding_loo.date %]</td>
****        <td>[% breeding_loo.publisher %]</td>
            <td>[% breeding_loo.edition %]</td>
            <td>[% breeding_loo.isbn %]</td>
            <td>[% breeding_loo.lccn %]</td>
Comment 1 Katrin Fischer 2013-12-05 21:28:40 UTC
Hi Andrea, this looks quite good - do you think you could provide a git formatted patch?

There is only one small thing about this change:
  $row_data{date}         =  $oldbiblio->{copyrightdate};
in
  $row_data{date}         =  $oldbiblio->{publicationyear};

The problem here is that for MARC21 copyrightdate has the right date, but for UNIMARC installations it's publicationyear. So it would need to check both fields in the database.
Comment 2 Katrin Fischer 2015-01-06 22:25:25 UTC
Tested on my UNIMARC and my MARC21 installation - the publication year now shows up for both.

Adding the publisher might still be good.
Comment 3 Katrin Fischer 2015-05-25 08:14:39 UTC
I think adding the publisher could be done with 12747 - marking moved.