use strict;
require Exporter;
+# use utf8;
use C4::Context;
use MARC::Record;
use MARC::File::USMARC;
use ZOOM;
use C4::Koha;
use C4::Date;
-use utf8;
use C4::Log; # logaction
use vars qw($VERSION @ISA @EXPORT);
&GetBiblioFromItemNumber
&GetMarcItem
+ &GetItem
&GetItemInfosOf
&GetItemStatus
&GetItemLocation
+ &GetLostItems
+ &GetItemsForInventory
+ &GetItemsCount
&GetMarcNotes
&GetMarcSubjects
&GetMarcBiblio
&GetMarcAuthors
&GetMarcSeries
+ &GetUsedMarcStructure
&GetItemsInfo
- &GetItemFromBarcode
+ &GetItemsByBiblioitemnumber
+ &GetItemnumberFromBarcode
&get_itemnumbers_of
&GetXmlBiblio
&GetMarcStructure
&GetMarcFromKohaField
&GetFrameworkCode
+ &GetPublisherNameFromIsbn
&TransformKohaToMarc
);
push @EXPORT, qw(
&ModBiblio
&ModItem
+ &ModItemTransfer
&ModBiblioframework
&ModZebra
&ModItemInMarc
&ModItemInMarconefield
+ &ModDateLastSeen
);
# To delete something
# Others functions
push @EXPORT, qw(
&TransformMarcToKoha
+ &TransformHtmlToMarc2
&TransformHtmlToMarc
&TransformHtmlToXml
&PrepareItemrecordDisplay
&char_decode
+ &GetNoZebraIndexes
);
=head1 NAME
# we build the new field with biblionumber and biblioitemnumber
# we drop the original field
# we add the new builded field.
- ( my $biblio_tag, my $biblio_subfield ) = GetMarcFromKohaField($dbh,"biblio.biblionumber",$frameworkcode);
- ( my $biblioitem_tag, my $biblioitem_subfield ) = GetMarcFromKohaField($dbh,"biblioitems.biblioitemnumber",$frameworkcode);
+ ( my $biblio_tag, my $biblio_subfield ) = GetMarcFromKohaField("biblio.biblionumber",$frameworkcode);
+ ( my $biblioitem_tag, my $biblioitem_subfield ) = GetMarcFromKohaField("biblioitems.biblioitemnumber",$frameworkcode);
my $newfield;
}
# now add the record
- my $biblionumber =
- ModBiblioMarc( $record, $biblionumber, $frameworkcode );
+ $biblionumber = ModBiblioMarc( $record, $biblionumber, $frameworkcode );
&logaction(C4::Context->userenv->{'number'},"CATALOGUING","ADD",$biblionumber,"biblio")
if C4::Context->preference("CataloguingLog");
-
+
return ( $biblionumber, $biblioitemnumber );
}
=head2 AddItem
-=over
+=over 2
-$biblionumber = AddItem( $record, $biblionumber)
-Exported function (core API) for adding a new item to Koha
+ $biblionumber = AddItem( $record, $biblionumber)
+ Exported function (core API) for adding a new item to Koha
=back
&_koha_new_items( $dbh, $item, $item->{barcode} );
# add itemnumber to MARC::Record before adding the item.
- $sth =
- $dbh->prepare(
+ $sth = $dbh->prepare(
"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
);
&TransformKohaToMarcOneField( $sth, $record, "items.itemnumber", $itemnumber,
=head2 ModBiblio
-=over
-
-ModBiblio( $record,$biblionumber,$frameworkcode);
-Exported function (core API) to modify a biblio
-
-=back
+ ModBiblio( $record,$biblionumber,$frameworkcode);
+ Exported function (core API) to modify a biblio
=cut
sub ModBiblio {
my ( $record, $biblionumber, $frameworkcode ) = @_;
-
- if (C4::Context->preference("CataloguingLog")) {
+ if (C4::Context->preference("CataloguingLog")) {
my $newrecord = GetMarcBiblio($biblionumber);
- &logaction(C4::Context->userenv->{'number'},"CATALOGUING","MODIFY",$biblionumber,$newrecord->as_formatted)
+ &logaction(C4::Context->userenv->{'number'},"CATALOGUING","MODIFY",$biblionumber,"BEFORE=>".$newrecord->as_formatted);
}
my $dbh = C4::Context->dbh;
$frameworkcode = "" unless $frameworkcode;
- # update the MARC record with the new record data
- &ModBiblioMarc($record, $biblionumber, $frameworkcode );
-
+ # get the items before and append them to the biblio before updating the record, atm we just have the biblio
+ my ( $itemtag, $itemsubfield ) = GetMarcFromKohaField("items.itemnumber",$frameworkcode);
+ my $oldRecord = GetMarcBiblio( $biblionumber );
+
+ # parse each item, and, for an unknown reason, re-encode each subfield
+ # if you don't do that, the record will have encoding mixed
+ # and the biblio will be re-encoded.
+ # strange, I (Paul P.) searched more than 1 day to understand what happends
+ # but could only solve the problem this way...
+ my @fields = $oldRecord->field( $itemtag );
+ foreach my $fielditem ( @fields ){
+ my $field;
+ foreach ($fielditem->subfields()) {
+ if ($field) {
+ $field->add_subfields(Encode::encode('utf-8',$_->[0]) => Encode::encode('utf-8',$_->[1]));
+ } else {
+ $field = MARC::Field->new("$itemtag",'','',Encode::encode('utf-8',$_->[0]) => Encode::encode('utf-8',$_->[1]));
+ }
+ }
+ $record->append_fields($field);
+ }
+
+ # adding biblionumber
+ my ($tag_biblionumber, $subfield_biblionumber) = GetMarcFromKohaField('biblio.biblionumber',$frameworkcode);
+ $record->append_fields(
+ MARC::Field->new(
+ $tag_biblionumber,'','',$subfield_biblionumber => $biblionumber
+ )
+ ) unless ($record->subfield($tag_biblionumber,$subfield_biblionumber));
+
+ # update the MARC record (that now contains biblio and items) with the new record data
+ &ModBiblioMarc( $record, $biblionumber, $frameworkcode );
+
# load the koha-table data object
my $oldbiblio = TransformMarcToKoha( $dbh, $record, $frameworkcode );
# modify the other koha tables
- my $biblionumber = _koha_modify_biblio( $dbh, $oldbiblio );
+ _koha_modify_biblio( $dbh, $oldbiblio );
_koha_modify_biblioitem( $dbh, $oldbiblio );
-
return 1;
}
=head2 ModItem
-=over
+=over 2
Exported function (core API) for modifying an item in Koha.
}
}
-=head2 ModBiblioframework
-
-=over
+sub ModItemTransfer {
+ my ( $itemnumber, $frombranch, $tobranch ) = @_;
+
+ my $dbh = C4::Context->dbh;
+
+ #new entry in branchtransfers....
+ my $sth = $dbh->prepare(
+ "INSERT INTO branchtransfers (itemnumber, frombranch, datesent, tobranch)
+ VALUES (?, ?, now(), ?)");
+ $sth->execute($itemnumber, $frombranch, $tobranch);
+ #update holdingbranch in items .....
+ $sth= $dbh->prepare(
+ "UPDATE items set holdingbranch = ? WHERE items.itemnumber = ?");
+ $sth->execute($tobranch,$itemnumber);
+ &ModDateLastSeen($itemnumber);
+ $sth = $dbh->prepare(
+ "SELECT biblionumber FROM items WHERE itemnumber=?"
+ );
+ $sth->execute($itemnumber);
+ while ( my ( $biblionumber ) = $sth->fetchrow ) {
+ &ModItemInMarconefield( $biblionumber, $itemnumber,
+ 'items.holdingbranch', $tobranch );
+ }
+ return;
+}
-ModBiblioframework($biblionumber,$frameworkcode);
-Exported function to modify a biblio framework
+=head2 ModBiblioframework
-=back
+ ModBiblioframework($biblionumber,$frameworkcode);
+ Exported function to modify a biblio framework
=cut
sub ModBiblioframework {
my ( $biblionumber, $frameworkcode ) = @_;
my $dbh = C4::Context->dbh;
- my $sth =
- $dbh->prepare(
- "UPDATE biblio SET frameworkcode=? WHERE biblionumber=$biblionumber");
+ my $sth = $dbh->prepare(
+ "UPDATE biblio SET frameworkcode=? WHERE biblionumber=$biblionumber"
+ );
$sth->execute($frameworkcode);
return 1;
}
}
my $record = GetMarcItem( $biblionumber, $itemnumber );
- my ($tagfield, $tagsubfield) = GetMarcFromKohaField($dbh, $itemfield,'');
+ my ($tagfield, $tagsubfield) = GetMarcFromKohaField( $itemfield,'');
if ($tagfield && $tagsubfield) {
my $tag = $record->field($tagfield);
if ($tag) {
# get complete MARC record & replace the item field by the new one
my $completeRecord = GetMarcBiblio($biblionumber);
- my ($itemtag,$itemsubfield) = GetMarcFromKohaField($dbh,"items.itemnumber",$frameworkcode);
+ my ($itemtag,$itemsubfield) = GetMarcFromKohaField("items.itemnumber",$frameworkcode);
my $itemField = $ItemRecord->field($itemtag);
my @items = $completeRecord->field($itemtag);
foreach (@items) {
my $sth = $dbh->prepare("update biblioitems set marc=?,marcxml=? where biblionumber=?");
$sth->execute( $completeRecord->as_usmarc(), $completeRecord->as_xml_record(),$biblionumber );
$sth->finish;
- ModZebra($biblionumber,"specialUpdate","biblioserver");
+ ModZebra($biblionumber,"specialUpdate","biblioserver",$completeRecord);
}
+=head2 ModDateLastSeen
+
+&ModDateLastSeen($itemnum)
+Mark item as seen. Is called when an item is issued, returned or manually marked during inventory/stocktaking
+C<$itemnum> is the item number
+
+=cut
+
+sub ModDateLastSeen {
+ my ($itemnum) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth =
+ $dbh->prepare(
+ "update items set itemlost=0, datelastseen = now() where items.itemnumber = ?"
+ );
+ $sth->execute($itemnum);
+ return;
+}
=head2 DelBiblio
=over
# First make sure there are no items with issues are still attached
my $sth =
$dbh->prepare(
- "SELECT biblioitemnumber FROM biblioitems WHERE biblionumber=?");
+ "SELECT itemnumber FROM items WHERE biblionumber=?");
$sth->execute($biblionumber);
- while ( my $biblioitemnumber = $sth->fetchrow ) {
- my @issues = C4::Circulation::Circ2::itemissues($biblioitemnumber);
- foreach my $issue (@issues) {
+ while ( my $itemnumber = $sth->fetchrow ) {
+ my $issues = GetItemIssues($itemnumber);
+ foreach my $issue (@$issues) {
if ( ( $issue->{date_due} )
&& ( $issue->{date_due} ne "Available" ) )
{
}
return $error if $error;
- # Delete in Zebra
- ModZebra($biblionumber,"delete_record","biblioserver");
+ # Delete in Zebra. Be careful NOT to move this line after _koha_delete_biblio
+ # for at least 2 reasons :
+ # - we need to read the biblio if NoZebra is set (to remove it from the indexes
+ # - if something goes wrong, the biblio may be deleted from Koha but not from zebra
+ # and we would have no way to remove it (except manually in zebra, but I bet it would be very hard to handle the problem)
+ ModZebra($biblionumber, "delete_record", "biblioserver", undef);
# delete biblio from Koha tables and save in deletedbiblio
$error = &_koha_delete_biblio( $dbh, $biblionumber );
$copy2deleted->execute( $record->as_usmarc(), $itemnumber );
#search item field code
- my ( $itemtag, $itemsubfield ) = GetMarcFromKohaField($dbh,"items.itemnumber",$frameworkcode);
+ my ( $itemtag, $itemsubfield ) = GetMarcFromKohaField("items.itemnumber",$frameworkcode);
my @fields = $record->field($itemtag);
# delete the item specified
foreach my $field (@fields) {
my ( $biblionumber, $type ) = @_;
my $dbh = C4::Context->dbh;
my $query = "SELECT *,items.notforloan as itemnotforloan
- FROM items, biblio, biblioitems
+ FROM items
+ LEFT JOIN biblio ON biblio.biblionumber = items.biblionumber
+ LEFT JOIN biblioitems ON biblioitems.biblioitemnumber = items.biblioitemnumber
LEFT JOIN itemtypes on biblioitems.itemtype = itemtypes.itemtype
WHERE items.biblionumber = ?
- AND biblioitems.biblioitemnumber = items.biblioitemnumber
- AND biblio.biblionumber = items.biblionumber
ORDER BY items.dateaccessioned desc
";
my $sth = $dbh->prepare($query);
while ( my $data = $sth->fetchrow_hashref ) {
my $datedue = '';
my $isth = $dbh->prepare(
- "SELECT issues.*,borrowers.cardnumber
- FROM issues, borrowers
+ "SELECT issues.*,borrowers.cardnumber,borrowers.surname,borrowers.firstname
+ FROM issues LEFT JOIN borrowers ON issues.borrowernumber=borrowers.borrowernumber
WHERE itemnumber = ?
- AND returndate IS NULL
- AND issues.borrowernumber=borrowers.borrowernumber"
+ AND returndate IS NULL"
);
$isth->execute( $data->{'itemnumber'} );
if ( my $idata = $isth->fetchrow_hashref ) {
$data->{borrowernumber} = $idata->{borrowernumber};
$data->{cardnumber} = $idata->{cardnumber};
+ $data->{surname} = $idata->{surname};
+ $data->{firstname} = $idata->{firstname};
$datedue = format_date( $idata->{'date_due'} );
}
if ( $datedue eq '' ) {
#$datedue="Available";
my ( $restype, $reserves ) =
- C4::Reserves2::CheckReserves( $data->{'itemnumber'} );
+ C4::Reserves::CheckReserves( $data->{'itemnumber'} );
if ($restype) {
#$datedue=$restype;
my $sth;
$fwk = '' unless ($fwk);
my ( $tag, $subfield ) =
- GetMarcFromKohaField( $dbh, "items.notforloan", $fwk );
+ GetMarcFromKohaField( "items.notforloan", $fwk );
if ( $tag and $subfield ) {
my $sth =
$dbh->prepare(
my $sth;
$fwk = '' unless ($fwk);
my ( $tag, $subfield ) =
- GetMarcFromKohaField( $dbh, "items.location", $fwk );
+ GetMarcFromKohaField( "items.location", $fwk );
if ( $tag and $subfield ) {
my $sth =
$dbh->prepare(
return \%itemlocation;
}
+=head2 GetLostItems
+
+$items = GetLostItems($where,$orderby);
+
+This function get the items lost into C<$items>.
+
+=over 2
+
+=item input:
+C<$where> is a hashref. it containts a field of the items table as key
+and the value to match as value.
+C<$orderby> is a field of the items table.
+
+=item return:
+C<$items> is a reference to an array full of hasref which keys are items' table column.
+
+=item usage in the perl script:
+
+my %where;
+$where{barcode} = 0001548;
+my $items = GetLostItems( \%where, "homebranch" );
+$template->param(itemsloop => $items);
+
+=back
+
+=cut
+
+sub GetLostItems {
+ # Getting input args.
+ my $where = shift;
+ my $orderby = shift;
+ my $dbh = C4::Context->dbh;
+
+ my $query = "
+ SELECT *
+ FROM items
+ WHERE itemlost IS NOT NULL
+ AND itemlost <> 0
+ ";
+ foreach my $key (keys %$where) {
+ $query .= " AND " . $key . " LIKE '%" . $where->{$key} . "%'";
+ }
+ $query .= " ORDER BY ".$orderby if defined $orderby;
+
+ my $sth = $dbh->prepare($query);
+ $sth->execute;
+ my @items;
+ while ( my $row = $sth->fetchrow_hashref ){
+ push @items, $row;
+ }
+ return \@items;
+}
+
+=head2 GetItemsForInventory
+
+$itemlist = GetItemsForInventory($minlocation,$maxlocation,$datelastseen,$offset,$size)
+
+Retrieve a list of title/authors/barcode/callnumber, for biblio inventory.
+
+The sub returns a list of hashes, containing itemnumber, author, title, barcode & item callnumber.
+It is ordered by callnumber,title.
+
+The minlocation & maxlocation parameters are used to specify a range of item callnumbers
+the datelastseen can be used to specify that you want to see items not seen since a past date only.
+offset & size can be used to retrieve only a part of the whole listing (defaut behaviour)
+
+=cut
+
+sub GetItemsForInventory {
+ my ( $minlocation, $maxlocation,$location, $datelastseen, $branch, $offset, $size ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth;
+ if ($datelastseen) {
+ $datelastseen=format_date_in_iso($datelastseen);
+ my $query =
+ "SELECT itemnumber,barcode,itemcallnumber,title,author,biblio.biblionumber,datelastseen
+ FROM items
+ LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber
+ WHERE itemcallnumber>= ?
+ AND itemcallnumber <=?
+ AND (datelastseen< ? OR datelastseen IS NULL)";
+ $query.= " AND items.location=".$dbh->quote($location) if $location;
+ $query.= " AND items.homebranch=".$dbh->quote($branch) if $branch;
+ $query .= " ORDER BY itemcallnumber,title";
+ $sth = $dbh->prepare($query);
+ $sth->execute( $minlocation, $maxlocation, $datelastseen );
+ }
+ else {
+ my $query ="
+ SELECT itemnumber,barcode,itemcallnumber,biblio.biblionumber,title,author,datelastseen
+ FROM items
+ LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber
+ WHERE itemcallnumber>= ?
+ AND itemcallnumber <=?";
+ $query.= " AND items.location=".$dbh->quote($location) if $location;
+ $query.= " AND items.homebranch=".$dbh->quote($branch) if $branch;
+ $query .= " ORDER BY itemcallnumber,title";
+ $sth = $dbh->prepare($query);
+ $sth->execute( $minlocation, $maxlocation );
+ }
+ my @results;
+ while ( my $row = $sth->fetchrow_hashref ) {
+ $offset-- if ($offset);
+ $row->{datelastseen}=format_date($row->{datelastseen});
+ if ( ( !$offset ) && $size ) {
+ push @results, $row;
+ $size--;
+ }
+ }
+ return \@results;
+}
+
=head2 &GetBiblioItemData
=over 4
#'
sub GetBiblioItemData {
- my ($bibitem) = @_;
+ my ($biblioitemnumber) = @_;
my $dbh = C4::Context->dbh;
my $sth =
$dbh->prepare(
-"Select *,biblioitems.notes as bnotes from biblioitems, biblio,itemtypes where biblio.biblionumber = biblioitems.biblionumber and biblioitemnumber = ? and biblioitems.itemtype = itemtypes.itemtype"
+ "SELECT *,biblioitems.notes AS bnotes
+ FROM biblioitems,biblio,itemtypes
+ WHERE biblio.biblionumber = biblioitems.biblionumber
+ AND biblioitemnumber = ? "
);
my $data;
-
- $sth->execute($bibitem);
-
+ $sth->execute($biblioitemnumber);
$data = $sth->fetchrow_hashref;
-
$sth->finish;
return ($data);
} # sub &GetBiblioItemData
-=head2 GetItemFromBarcode
+=head2 GetItemnumberFromBarcode
=over 4
-$result = GetItemFromBarcode($barcode);
+$result = GetItemnumberFromBarcode($barcode);
=back
=cut
-sub GetItemFromBarcode {
+sub GetItemnumberFromBarcode {
my ($barcode) = @_;
my $dbh = C4::Context->dbh;
sub GetBiblioFromItemNumber {
my ( $itemnumber ) = @_;
my $dbh = C4::Context->dbh;
- my $env;
my $sth = $dbh->prepare(
- "SELECT * FROM biblio,items,biblioitems
- WHERE items.itemnumber = ?
- AND biblio.biblionumber = items.biblionumber
- AND biblioitems.biblioitemnumber = items.biblioitemnumber"
+ "SELECT * FROM items
+ LEFT JOIN biblio ON biblio.biblionumber = items.biblionumber
+ LEFT JOIN biblioitems ON biblioitems.biblioitemnumber = items.biblioitemnumber
+ WHERE items.itemnumber = ?"
);
$sth->execute($itemnumber);
return ( $count, @results );
} # sub GetBiblio
+=head2 GetItem
+
+=over 4
+
+$data = &GetItem($itemnumber,$barcode);
+
+return Item information, for a given itemnumber or barcode
+
+=back
+
+=cut
+
+sub GetItem {
+ my ($itemnumber,$barcode) = @_;
+ my $dbh = C4::Context->dbh;
+ if ($itemnumber) {
+ my $sth = $dbh->prepare("
+ SELECT * FROM items
+ WHERE itemnumber = ?");
+ $sth->execute($itemnumber);
+ my $data = $sth->fetchrow_hashref;
+ return $data;
+ } else {
+ my $sth = $dbh->prepare("
+ SELECT * FROM items
+ WHERE barcode = ?"
+ );
+ $sth->execute($barcode);
+ my $data = $sth->fetchrow_hashref;
+ return $data;
+ }
+} # sub GetItem
+
=head2 get_itemnumbers_of
=over 4
return get_infos_of( $query, 'itemnumber' );
}
+=head2 GetItemsByBiblioitemnumber
+
+=over 4
+
+GetItemsByBiblioitemnumber($biblioitemnumber);
+
+Returns an arrayref of hashrefs suitable for use in a TMPL_LOOP
+Called by moredetail.pl
+
+=back
+
+=cut
+
+sub GetItemsByBiblioitemnumber {
+ my ( $bibitem ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sth = $dbh->prepare("SELECT * FROM items WHERE items.biblioitemnumber = ?") || die $dbh->errstr;
+ # Get all items attached to a biblioitem
+ my $i = 0;
+ my @results;
+ $sth->execute($bibitem) || die $sth->errstr;
+ while ( my $data = $sth->fetchrow_hashref ) {
+ # Foreach item, get circulation information
+ my $sth2 = $dbh->prepare( "SELECT * FROM issues,borrowers
+ WHERE itemnumber = ?
+ AND returndate is NULL
+ AND issues.borrowernumber = borrowers.borrowernumber"
+ );
+ $sth2->execute( $data->{'itemnumber'} );
+ if ( my $data2 = $sth2->fetchrow_hashref ) {
+ # if item is out, set the due date and who it is out too
+ $data->{'date_due'} = $data2->{'date_due'};
+ $data->{'cardnumber'} = $data2->{'cardnumber'};
+ $data->{'borrowernumber'} = $data2->{'borrowernumber'};
+ }
+ else {
+ # set date_due to blank, so in the template we check itemlost, and wthdrawn
+ $data->{'date_due'} = '';
+ } # else
+ $sth2->finish;
+ # Find the last 3 people who borrowed this item.
+ my $query2 = "SELECT * FROM issues, borrowers WHERE itemnumber = ?
+ AND issues.borrowernumber = borrowers.borrowernumber
+ AND returndate is not NULL
+ ORDER BY returndate desc,timestamp desc LIMIT 3";
+ $sth2 = $dbh->prepare($query2) || die $dbh->errstr;
+ $sth2->execute( $data->{'itemnumber'} ) || die $sth2->errstr;
+ my $i2 = 0;
+ while ( my $data2 = $sth2->fetchrow_hashref ) {
+ $data->{"timestamp$i2"} = $data2->{'timestamp'};
+ $data->{"card$i2"} = $data2->{'cardnumber'};
+ $data->{"borrower$i2"} = $data2->{'borrowernumber'};
+ $i2++;
+ }
+ $sth2->finish;
+ push(@results,$data);
+ }
+ $sth->finish;
+ return (\@results);
+}
+
+
=head2 GetBiblioItemInfosOf
=over 4
=over 4
-$res = GetMarcStructure($dbh,$forlibrarian,$frameworkcode);
+$res = GetMarcStructure($forlibrarian,$frameworkcode);
-Returns a reference to a big hash of hash, with the Marc structure fro the given frameworkcode
-$dbh : DB handler
+Returns a reference to a big hash of hash, with the Marc structure for the given frameworkcode
$forlibrarian :if set to 1, the MARC descriptions are the librarians ones, otherwise it's the public (OPAC) ones
$frameworkcode : the framework code to read
=back
-=back
-
=cut
sub GetMarcStructure {
- my ( $dbh, $forlibrarian, $frameworkcode ) = @_;
+ my ( $forlibrarian, $frameworkcode ) = @_;
+ my $dbh=C4::Context->dbh;
$frameworkcode = "" unless $frameworkcode;
my $sth;
my $libfield = ( $forlibrarian eq 1 ) ? 'liblibrarian' : 'libopac';
( $forlibrarian or !$libopac ) ? $liblibrarian : $libopac;
# why the hell do we need to explicitly decode utf8 ?
# that's a good question, but we must do it...
- use utf8;
- utf8::decode($res->{$tag}->{lib});
-# warn "$liblibrarian";
$res->{$tab}->{tab} = ""; # XXX
$res->{$tag}->{mandatory} = $mandatory;
$res->{$tag}->{repeatable} = $repeatable;
$sth =
$dbh->prepare(
"select tagfield,tagsubfield,liblibrarian,libopac,tab, mandatory, repeatable,authorised_value,authtypecode,value_builder,kohafield,seealso,hidden,isurl,link,defaultvalue from marc_subfield_structure where frameworkcode=? order by tagfield,tagsubfield"
- );
+ );
+
$sth->execute($frameworkcode);
my $subfield;
$res->{$tag}->{$subfield}->{seealso} = $seealso;
$res->{$tag}->{$subfield}->{hidden} = $hidden;
$res->{$tag}->{$subfield}->{isurl} = $isurl;
- $res->{$tag}->{$subfield}->{link} = $link;
+ $res->{$tag}->{$subfield}->{'link'} = $link;
$res->{$tag}->{$subfield}->{defaultvalue} = $defaultvalue;
}
return $res;
}
+=head2 GetUsedMarcStructure
+
+ the same function as GetMarcStructure expcet it just take field
+ in tab 0-9. (used field)
+
+ my $results = GetUsedMarcStructure($frameworkcode);
+
+ L<$results> is a ref to an array which each case containts a ref
+ to a hash which each keys is the columns from marc_subfield_structure
+
+ L<$frameworkcode> is the framework code.
+
+=cut
+
+sub GetUsedMarcStructure($){
+ my $frameworkcode = shift || '';
+ my $dbh = C4::Context->dbh;
+ my $query = qq/
+ SELECT *
+ FROM marc_subfield_structure
+ WHERE tab > -1
+ AND frameworkcode = ?
+ /;
+ my @results;
+ my $sth = $dbh->prepare($query);
+ $sth->execute($frameworkcode);
+ while (my $row = $sth->fetchrow_hashref){
+ push @results,$row;
+ }
+ return \@results;
+}
+
=head2 GetMarcFromKohaField
=over 4
-($MARCfield,$MARCsubfield)=GetMarcFromKohaField($dbh,$kohafield,$frameworkcode);
+($MARCfield,$MARCsubfield)=GetMarcFromKohaField($kohafield,$frameworkcode);
Returns the MARC fields & subfields mapped to the koha field
for the given frameworkcode
=cut
sub GetMarcFromKohaField {
- my ( $dbh, $kohafield, $frameworkcode ) = @_;
+ my ( $kohafield, $frameworkcode ) = @_;
return 0, 0 unless $kohafield;
my $relations = C4::Context->marcfromkohafield;
return (
my $sth =
$dbh->prepare("select marcxml from biblioitems where biblionumber=? ");
$sth->execute($biblionumber);
- my ($marcxml) = $sth->fetchrow;
-# warn "marcxml : $marcxml";
- MARC::File::XML->default_record_format(C4::Context->preference('marcflavour'));
- $marcxml =~ s/\x1e//g;
- $marcxml =~ s/\x1f//g;
- $marcxml =~ s/\x1d//g;
- $marcxml =~ s/\x0f//g;
- $marcxml =~ s/\x0c//g;
+ my ($marcxml) = $sth->fetchrow;
+ MARC::File::XML->default_record_format(C4::Context->preference('marcflavour'));
+ $marcxml =~ s/\x1e//g;
+ $marcxml =~ s/\x1f//g;
+ $marcxml =~ s/\x1d//g;
+ $marcxml =~ s/\x0f//g;
+ $marcxml =~ s/\x0c//g;
+# warn $marcxml;
my $record = MARC::Record->new();
- $record = MARC::Record::new_from_xml( $marcxml, "utf8",C4::Context->preference('marcflavour')) if $marcxml;
+
+ $record = eval {MARC::Record::new_from_xml( $marcxml, "utf8",C4::Context->preference('marcflavour'))} if ($marcxml);
+ if ($@) {warn $@;}
+# $record = MARC::Record::new_from_usmarc( $marc) if $marc;
return $record;
}
#---- itemtypes
if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq "itemtypes" ) {
- return getitemtypeinfo($value);
+ return getitemtypeinfo($value)->{description};
}
#---- "true" authorized value
my $category = $tagslib->{$tag}->{$subfield}->{'authorised_value'};
-
if ( $category ne "" ) {
my $sth =
$dbh->prepare(
my $marcxml = GetXmlBiblio($biblionumber);
my $record = MARC::Record->new();
-# warn "marcxml :$marcxml";
$record = MARC::Record::new_from_xml( $marcxml, "utf8", $marcflavour );
-# warn "record :".$record->as_formatted;
# now, find where the itemnumber is stored & extract only the item
my ( $itemnumberfield, $itemnumbersubfield ) =
- GetMarcFromKohaField( $dbh, 'items.itemnumber', '' );
+ GetMarcFromKohaField( 'items.itemnumber', '' );
my @fields = $record->field($itemnumberfield);
foreach my $field (@fields) {
if ( $field->subfield($itemnumbersubfield) eq $itemnumber ) {
return $newrecord;
}
+
+
=head2 GetMarcNotes
=over 4
foreach my $field ( $record->fields ) {
next unless $field->tag() >= $mintag && $field->tag() <= $maxtag;
my @subfields = $field->subfields();
- my $link;
- my $label = "su:";
+ my $link = "su:";
+ my $label;
my $flag = 0;
+ my $authoritysep=C4::Context->preference("authoritysep");
for my $subject_subfield ( @subfields ) {
+ if (
+ $marcflavour ne 'MARC21'
+ and (
+ ($subject_subfield->[0] eq '3') or
+ ($subject_subfield->[0] eq '4') or
+ ($subject_subfield->[0] eq '5')
+ )
+ )
+ {
+ next;
+ }
my $code = $subject_subfield->[0];
- $label .= $subject_subfield->[1] . " and su-to:" unless ( $code == 9 );
+ $label .= $subject_subfield->[1].$authoritysep unless ( $code == 9 );
+ $link .= " and su-to:".$subject_subfield->[1] unless ( $code == 9 );
if ( $code == 9 ) {
- $link = "Koha-Auth-Number:".$subject_subfield->[1];
+ $link = "an:".$subject_subfield->[1];
$flag = 1;
}
elsif ( ! $flag ) {
- $link = $label;
$link =~ s/ and\ssu-to:$//;
}
}
- $label =~ s/su/ /g;
- $label =~ s/://g;
- $label =~ s/-to//g;
- $label =~ s/and//g;
+ $label =~ s/$authoritysep$//;
push @marcsubjcts,
{
label => $label,
sub GetMarcAuthors {
my ( $record, $marcflavour ) = @_;
my ( $mintag, $maxtag );
+ # tagslib useful for UNIMARC author reponsabilities
+ my $tagslib = &GetMarcStructure( 1, '' ); # FIXME : we don't have the framework available, we take the default framework. May be bugguy on some setups, will be usually correct.
if ( $marcflavour eq "MARC21" ) {
$mintag = "100";
$maxtag = "111";
my $count_auth = 0;
my $and ;
for my $authors_subfield (@subfields) {
- if ($count_auth ne '0'){
- $and = " and au:";
- }
+ if (
+ $marcflavour ne 'MARC21'
+ and (
+ ($authors_subfield->[0] eq '3') or
+ ($authors_subfield->[0] eq '5')
+ )
+ )
+ {
+ next;
+ }
+ if ($count_auth ne '0'){
+ $and = " and au:";
+ }
$count_auth++;
- my $subfieldcode = $authors_subfield->[0];
- my $value = $authors_subfield->[1];
- $hash{'tag'} = $field->tag;
- $hash{value} .= $value . " " if ($subfieldcode != 9) ;
- $hash{link} .= $value if ($subfieldcode eq 9);
+ my $subfieldcode = $authors_subfield->[0];
+ my $value;
+ # deal with UNIMARC author responsibility
+ if (
+ $marcflavour ne 'MARC21'
+ and ($authors_subfield->[0] eq '4')
+ )
+ {
+ $value = "(".GetAuthorisedValueDesc( $field->tag(), $authors_subfield->[0], $authors_subfield->[1], '', $tagslib ).")";
+ } else {
+ $value = $authors_subfield->[1];
+ }
+ $hash{tag} = $field->tag;
+ $hash{value} .= $value . " " if ($subfieldcode != 9) ;
+ $hash{link} .= $value if ($subfieldcode eq 9);
}
push @marcauthors, \%hash;
}
=over 4
-$frameworkcode = GetFrameworkCode( $biblionumber )
+ $frameworkcode = GetFrameworkCode( $biblionumber )
=back
sub GetFrameworkCode {
my ( $biblionumber ) = @_;
my $dbh = C4::Context->dbh;
- my $sth =
- $dbh->prepare("select frameworkcode from biblio where biblionumber=?");
+ my $sth = $dbh->prepare("select frameworkcode from biblio where biblionumber=?");
$sth->execute($biblionumber);
my ($frameworkcode) = $sth->fetchrow;
return $frameworkcode;
}
+=head2 GetPublisherNameFromIsbn
+
+ $name = GetPublishercodeFromIsbn($isbn);
+ if(defined $name){
+ ...
+ }
+
+=cut
+
+sub GetPublisherNameFromIsbn($){
+ my $isbn = shift;
+ $isbn =~ s/[- _]//g;
+ $isbn =~ s/^0*//;
+ my @codes = (split '-', DisplayISBN($isbn));
+ my $code = $codes[0].$codes[1].$codes[2];
+ my $dbh = C4::Context->dbh;
+ my $query = qq{
+ SELECT distinct publishercode
+ FROM biblioitems
+ WHERE isbn LIKE ?
+ AND publishercode IS NOT NULL
+ LIMIT 1
+ };
+ my $sth = $dbh->prepare($query);
+ $sth->execute("$code%");
+ my $name = $sth->fetchrow;
+ return $name if length $name;
+ return undef;
+}
+
=head2 TransformKohaToMarc
=over 4
-$record = TransformKohaToMarc( $hash )
-This function builds partial MARC::Record from a hash
-Hash entries can be from biblio or biblioitems.
-This function is called in acquisition module, to create a basic catalogue entry from user entry
+ $record = TransformKohaToMarc( $hash )
+ This function builds partial MARC::Record from a hash
+ Hash entries can be from biblio or biblioitems.
+ This function is called in acquisition module, to create a basic catalogue entry from user entry
=back
=over 4
-$record = TransformKohaToMarcOneField( $sth, $record, $kohafieldname, $value, $frameworkcode );
+ $record = TransformKohaToMarcOneField( $sth, $record, $kohafieldname, $value, $frameworkcode );
=back
if ( !defined $sth ) {
my $dbh = C4::Context->dbh;
- $sth =
- $dbh->prepare(
-"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
- );
+ $sth = $dbh->prepare(
+ "select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+ );
}
$sth->execute( $frameworkcode, $kohafieldname );
if ( ( $tagfield, $tagsubfield ) = $sth->fetchrow ) {
=over 4
-$xml = TransformHtmlToXml( $tags, $subfields, $values, $indicator, $ind_tag )
+$xml = TransformHtmlToXml( $tags, $subfields, $values, $indicator, $ind_tag, $auth_type )
+
+$auth_type contains :
+- nothing : rebuild a biblio, un UNIMARC the encoding is in 100$a pos 26/27
+- UNIMARCAUTH : rebuild an authority. In UNIMARC, the encoding is in 100$a pos 13/14
+- ITEM : rebuild an item : in UNIMARC, 100$a, it's in the biblio ! (otherwise, we would get 2 100 fields !)
=back
=cut
sub TransformHtmlToXml {
- my ( $tags, $subfields, $values, $indicator, $ind_tag ) = @_;
+ my ( $tags, $subfields, $values, $indicator, $ind_tag, $auth_type ) = @_;
my $xml = MARC::File::XML::header('UTF-8');
- if ( C4::Context->preference('marcflavour') eq 'UNIMARC' ) {
- MARC::File::XML->default_record_format('UNIMARC');
- use POSIX qw(strftime);
- my $string = strftime( "%Y%m%d", localtime(time) );
- $string = sprintf( "%-*s", 35, $string );
- substr( $string, 22, 6, "frey50" );
- $xml .= "<datafield tag=\"100\" ind1=\"\" ind2=\"\">\n";
- $xml .= "<subfield code=\"a\">$string</subfield>\n";
- $xml .= "</datafield>\n";
- }
+ $auth_type = C4::Context->preference('marcflavour') unless $auth_type;
+ MARC::File::XML->default_record_format($auth_type);
+ # in UNIMARC, field 100 contains the encoding
+ # check that there is one, otherwise the
+ # MARC::Record->new_from_xml will fail (and Koha will die)
+ my $unimarc_and_100_exist=0;
+ $unimarc_and_100_exist=1 if $auth_type eq 'ITEM'; # if we rebuild an item, no need of a 100 field
my $prevvalue;
my $prevtag = -1;
my $first = 1;
my $j = -1;
for ( my $i = 0 ; $i <= @$tags ; $i++ ) {
+ if (C4::Context->preference('marcflavour') eq 'UNIMARC' and @$tags[$i] eq "100" and @$subfields[$i] eq "a") {
+ # if we have a 100 field and it's values are not correct, skip them.
+ # if we don't have any valid 100 field, we will create a default one at the end
+ my $enc = substr( @$values[$i], 26, 2 );
+ if ($enc eq '01' or $enc eq '50' or $enc eq '03') {
+ $unimarc_and_100_exist=1;
+ } else {
+ next;
+ }
+ }
@$values[$i] =~ s/&/&/g;
@$values[$i] =~ s/</</g;
@$values[$i] =~ s/>/>/g;
@$values[$i] =~ s/"/"/g;
@$values[$i] =~ s/'/'/g;
- if ( !utf8::is_utf8( @$values[$i] ) ) {
- utf8::decode( @$values[$i] );
- }
+# if ( !utf8::is_utf8( @$values[$i] ) ) {
+# utf8::decode( @$values[$i] );
+# }
if ( ( @$tags[$i] ne $prevtag ) ) {
$j++ unless ( @$tags[$i] eq "" );
if ( !$first ) {
}
$prevtag = @$tags[$i];
}
+ if (C4::Context->preference('marcflavour') and !$unimarc_and_100_exist) {
+# warn "SETTING 100 for $auth_type";
+ use POSIX qw(strftime);
+ my $string = strftime( "%Y%m%d", localtime(time) );
+ # set 50 to position 26 is biblios, 13 if authorities
+ my $pos=26;
+ $pos=13 if $auth_type eq 'UNIMARCAUTH';
+ $string = sprintf( "%-*s", 35, $string );
+ substr( $string, $pos , 6, "50" );
+ $xml .= "<datafield tag=\"100\" ind1=\"\" ind2=\"\">\n";
+ $xml .= "<subfield code=\"a\">$string</subfield>\n";
+ $xml .= "</datafield>\n";
+ }
$xml .= MARC::File::XML::footer();
-
return $xml;
}
=head2 TransformHtmlToMarc
-=over 4
-
-$record = TransformHtmlToMarc( $dbh, $rtags, $rsubfields, $rvalues, %indicators )
-
-=back
+ L<$record> = TransformHtmlToMarc(L<$params>,L<$cgi>)
+ L<$params> is a ref to an array as below:
+ {
+ 'tag_010_indicator_531951' ,
+ 'tag_010_code_a_531951_145735' ,
+ 'tag_010_subfield_a_531951_145735' ,
+ 'tag_200_indicator_873510' ,
+ 'tag_200_code_a_873510_673465' ,
+ 'tag_200_subfield_a_873510_673465' ,
+ 'tag_200_code_b_873510_704318' ,
+ 'tag_200_subfield_b_873510_704318' ,
+ 'tag_200_code_e_873510_280822' ,
+ 'tag_200_subfield_e_873510_280822' ,
+ 'tag_200_code_f_873510_110730' ,
+ 'tag_200_subfield_f_873510_110730' ,
+ }
+ L<$cgi> is the CGI object which containts the value.
+ L<$record> is the MARC::Record object.
=cut
sub TransformHtmlToMarc {
- my ( $dbh, $rtags, $rsubfields, $rvalues, %indicators ) = @_;
- my $prevtag = -1;
+ my $params = shift;
+ my $cgi = shift;
+
+ # creating a new record
my $record = MARC::Record->new();
-
- # my %subfieldlist=();
- my $prevvalue; # if tag <10
- my $field; # if tag >=10
- for ( my $i = 0 ; $i < @$rtags ; $i++ ) {
- next unless @$rvalues[$i];
-
- # rebuild MARC::Record
- # warn "0=>".@$rtags[$i].@$rsubfields[$i]." = ".@$rvalues[$i].": ";
- if ( @$rtags[$i] ne $prevtag ) {
- if ( $prevtag < 10 ) {
- if ($prevvalue) {
-
- if ( $prevtag ne '000' ) {
- $record->insert_fields_ordered(
- ( sprintf "%03s", $prevtag ), $prevvalue );
- }
- else {
-
- $record->leader($prevvalue);
-
- }
- }
- }
- else {
- if ($field) {
- $record->insert_fields_ordered($field);
- }
- }
- $indicators{ @$rtags[$i] } .= ' ';
- if ( @$rtags[$i] < 10 ) {
- $prevvalue = @$rvalues[$i];
- undef $field;
- }
- else {
- undef $prevvalue;
- $field = MARC::Field->new(
- ( sprintf "%03s", @$rtags[$i] ),
- substr( $indicators{ @$rtags[$i] }, 0, 1 ),
- substr( $indicators{ @$rtags[$i] }, 1, 1 ),
- @$rsubfields[$i] => @$rvalues[$i]
+ my $i=0;
+ my @fields;
+ while ($params->[$i]){ # browse all CGI params
+ my $param = $params->[$i];
+ my $newfield=0;
+ # if we are on biblionumber, store it in the MARC::Record (it may not be in the edited fields)
+ if ($param eq 'biblionumber') {
+ my ( $biblionumbertagfield, $biblionumbertagsubfield ) =
+ &GetMarcFromKohaField( "biblio.biblionumber", '' );
+ if ($biblionumbertagfield < 10) {
+ $newfield = MARC::Field->new(
+ $biblionumbertagfield,
+ $cgi->param($param),
+ );
+ } else {
+ $newfield = MARC::Field->new(
+ $biblionumbertagfield,
+ '',
+ '',
+ "$biblionumbertagsubfield" => $cgi->param($param),
);
}
- $prevtag = @$rtags[$i];
- }
- else {
- if ( @$rtags[$i] < 10 ) {
- $prevvalue = @$rvalues[$i];
- }
- else {
- if ( length( @$rvalues[$i] ) > 0 ) {
- $field->add_subfields( @$rsubfields[$i] => @$rvalues[$i] );
+ push @fields,$newfield if($newfield);
+ }
+ elsif ($param =~ /^tag_(\d*)_indicator_/){ # new field start when having 'input name="..._indicator_..."
+ my $tag = $1;
+
+ my $ind1 = substr($cgi->param($param),0,1);
+ my $ind2 = substr($cgi->param($param),1,1);
+ $newfield=0;
+ my $j=$i+1;
+
+ if($tag < 10){ # no code for theses fields
+ # in MARC editor, 000 contains the leader.
+ if ($tag eq '000' ) {
+ $record->leader($cgi->param($params->[$j+1])) if length($cgi->param($params->[$j+1]))==24;
+ # between 001 and 009 (included)
+ } else {
+ $newfield = MARC::Field->new(
+ $tag,
+ $cgi->param($params->[$j+1]),
+ );
+ }
+ # > 009, deal with subfields
+ } else {
+ while($params->[$j] =~ /_code_/){ # browse all it's subfield
+ my $inner_param = $params->[$j];
+ if ($newfield){
+ if($cgi->param($params->[$j+1])){ # only if there is a value (code => value)
+ $newfield->add_subfields(
+ $cgi->param($inner_param) => $cgi->param($params->[$j+1])
+ );
+ }
+ } else {
+ if ( $cgi->param($params->[$j+1]) ) { # creating only if there is a value (code => value)
+ $newfield = MARC::Field->new(
+ $tag,
+ ''.$ind1,
+ ''.$ind2,
+ $cgi->param($inner_param) => $cgi->param($params->[$j+1]),
+ );
+ }
+ }
+ $j+=2;
}
}
- $prevtag = @$rtags[$i];
+ push @fields,$newfield if($newfield);
}
+ $i++;
}
-
- # the last has not been included inside the loop... do it now !
- $record->insert_fields_ordered($field) if $field;
-
- # warn "HTML2MARC=".$record->as_formatted;
- $record->encoding('UTF-8');
-
- # $record->MARC::File::USMARC::update_leader();
+
+ $record->append_fields(@fields);
return $record;
}
=over 4
-$result = TransformMarcToKoha( $dbh, $record, $frameworkcode )
+ $result = TransformMarcToKoha( $dbh, $record, $frameworkcode )
=back
sub TransformMarcToKoha {
my ( $dbh, $record, $frameworkcode ) = @_;
- my $sth =
- $dbh->prepare(
-"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
- );
+
+ # FIXME :: This query is unused..
+ # my $sth =
+ # $dbh->prepare(
+ #"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+ # );
+
my $result;
my $sth2 = $dbh->prepare("SHOW COLUMNS from biblio");
$sth2->execute;
&TransformMarcToKohaOneField( "biblio", $field, $record, $result,
$frameworkcode );
}
- $sth2 = $dbh->prepare("SHOW COLUMNS from biblioitems");
+ my $sth2 = $dbh->prepare("SHOW COLUMNS from biblioitems");
$sth2->execute;
while ( ($field) = $sth2->fetchrow ) {
if ( $field eq 'notes' ) { $field = 'bnotes'; }
sub TransformMarcToKohaOneField {
-# FIXME ? if a field has a repeatable subfield that is used in old-db, only the 1st will be retrieved...
+ # FIXME ? if a field has a repeatable subfield that is used in old-db,
+ # only the 1st will be retrieved...
my ( $kohatable, $kohafield, $record, $result, $frameworkcode ) = @_;
-
my $res = "";
my ( $tagfield, $subfield ) =
- GetMarcFromKohaField( "", $kohatable . "." . $kohafield,
+ GetMarcFromKohaField( $kohatable . "." . $kohafield,
$frameworkcode );
foreach my $field ( $record->field($tagfield) ) {
if ( $field->tag() < 10 ) {
}
return $result;
}
+
=head1 OTHER FUNCTIONS
=head2 char_decode
my $dbh = C4::Context->dbh;
my $frameworkcode = &GetFrameworkCode( $bibnum );
my ( $itemtagfield, $itemtagsubfield ) =
- &GetMarcFromKohaField( $dbh, "items.itemnumber", $frameworkcode );
- my $tagslib = &GetMarcStructure( $dbh, 1, $frameworkcode );
+ &GetMarcFromKohaField( "items.itemnumber", $frameworkcode );
+ my $tagslib = &GetMarcStructure( 1, $frameworkcode );
my $itemrecord = GetMarcItem( $bibnum, $itemnum) if ($itemnum);
my @loop_data;
my $authorised_values_sth =
=over 4
-ModZebra( $dbh, $biblionumber, $op, $server );
+ModZebra( $biblionumber, $op, $server, $newRecord );
+ $biblionumber is the biblionumber we want to index
+ $op is specialUpdate or delete, and is used to know what we want to do
+ $server is the server that we want to update
+ $newRecord is the MARC::Record containing the new record. It is usefull only when NoZebra=1, and is used to know what to add to the nozebra database. (the record in mySQL being, if it exist, the previous record, the one just before the modif. We need both : the previous and the new one.
+
=back
=cut
sub ModZebra {
###Accepts a $server variable thus we can use it for biblios authorities or other zebra dbs
- my ( $biblionumber, $op, $server ) = @_;
+ my ( $biblionumber, $op, $server, $newRecord ) = @_;
my $dbh=C4::Context->dbh;
- #warn "SERVER:".$server;
-#
-# true ModZebra commented until indexdata fixes zebraDB crashes (it seems they occur on multiple updates
-# at the same time
-# replaced by a zebraqueue table, that is filled with ModZebra to run.
-# the table is emptied by misc/cronjobs/zebraqueue_start.pl script
-my $sth=$dbh->prepare("insert into zebraqueue (biblio_auth_number ,server,operation) values(?,?,?)");
-$sth->execute($biblionumber,$server,$op);
-$sth->finish;
+ # true ModZebra commented until indexdata fixes zebraDB crashes (it seems they occur on multiple updates
+ # at the same time
+ # replaced by a zebraqueue table, that is filled with ModZebra to run.
+ # the table is emptied by misc/cronjobs/zebraqueue_start.pl script
+
+ if (C4::Context->preference("NoZebra")) {
+ # lock the nozebra table : we will read index lines, update them in Perl process
+ # and write everything in 1 transaction.
+ # lock the table to avoid someone else overwriting what we are doing
+ $dbh->do('LOCK TABLES nozebra WRITE,biblio WRITE,biblioitems WRITE, systempreferences WRITE, auth_types WRITE, auth_header WRITE');
+ my %result; # the result hash that will be builded by deletion / add, and written on mySQL at the end, to improve speed
+ my $record;
+ if ($server eq 'biblioserver') {
+ $record= GetMarcBiblio($biblionumber);
+ } else {
+ $record= C4::AuthoritiesMarc::GetAuthority($biblionumber);
+ }
+ if ($op eq 'specialUpdate') {
+ # OK, we have to add or update the record
+ # 1st delete (virtually, in indexes) ...
+ %result = _DelBiblioNoZebra($biblionumber,$record,$server);
+ # ... add the record
+ %result=_AddBiblioNoZebra($biblionumber,$newRecord, $server, %result);
+ } else {
+ # it's a deletion, delete the record...
+ # warn "DELETE the record $biblionumber on $server".$record->as_formatted;
+ %result=_DelBiblioNoZebra($biblionumber,$record,$server);
+ }
+ # ok, now update the database...
+ my $sth = $dbh->prepare("UPDATE nozebra SET biblionumbers=? WHERE server=? AND indexname=? AND value=?");
+ foreach my $key (keys %result) {
+ foreach my $index (keys %{$result{$key}}) {
+ $sth->execute($result{$key}->{$index}, $server, $key, $index);
+ }
+ }
+ $dbh->do('UNLOCK TABLES');
+
+ } else {
+ #
+ # we use zebra, just fill zebraqueue table
+ #
+ my $sth=$dbh->prepare("insert into zebraqueue (biblio_auth_number ,server,operation) values(?,?,?)");
+ $sth->execute($biblionumber,$server,$op);
+ $sth->finish;
+ }
+}
-#
-# my @Zconnbiblio;
-# my $tried = 0;
-# my $recon = 0;
-# my $reconnect = 0;
-# my $record;
-# my $shadow;
-#
-# reconnect:
-# $Zconnbiblio[0] = C4::Context->Zconn( $server, 0, 1 );
-#
-# if ( $server eq "biblioserver" ) {
-#
-# # it's unclear to me whether this should be in xml or MARC format
-# # but it is clear it should be nabbed from zebra rather than from
-# # the koha tables
-# $record = GetMarcBiblio($biblionumber);
-# $record = $record->as_xml_record() if $record;
-# # warn "RECORD $biblionumber => ".$record;
-# $shadow="biblioservershadow";
-#
-# # warn "RECORD $biblionumber => ".$record;
-# $shadow = "biblioservershadow";
-#
-# }
-# elsif ( $server eq "authorityserver" ) {
-# $record = C4::AuthoritiesMarc::XMLgetauthority( $dbh, $biblionumber );
-# $shadow = "authorityservershadow";
-# } ## Add other servers as necessary
-#
-# my $Zpackage = $Zconnbiblio[0]->package();
-# $Zpackage->option( action => $op );
-# $Zpackage->option( record => $record );
-#
-# retry:
-# $Zpackage->send("update");
-# my $i;
-# my $event;
-#
-# while ( ( $i = ZOOM::event( \@Zconnbiblio ) ) != 0 ) {
-# $event = $Zconnbiblio[0]->last_event();
-# last if $event == ZOOM::Event::ZEND;
-# }
-#
-# my ( $error, $errmsg, $addinfo, $diagset ) = $Zconnbiblio[0]->error_x();
-# if ( $error == 10000 && $reconnect == 0 )
-# { ## This is serious ZEBRA server is not available -reconnect
-# warn "problem with zebra server connection";
-# $reconnect = 1;
-# my $res = system('sc start "Z39.50 Server" >c:/zebraserver/error.log');
-#
-# #warn "Trying to restart ZEBRA Server";
-# #goto "reconnect";
-# }
-# elsif ( $error == 10007 && $tried < 2 )
-# { ## timeout --another 30 looonng seconds for this update
-# $tried = $tried + 1;
-# warn "warn: timeout, trying again";
-# goto "retry";
-# }
-# elsif ( $error == 10004 && $recon == 0 ) { ##Lost connection -reconnect
-# $recon = 1;
-# warn "error: reconnecting to zebra";
-# goto "reconnect";
-#
-# # as a last resort, we save the data to the filesystem to be indexed in batch
-# }
-# elsif ($error) {
-# warn
-# "Error-$server $op $biblionumber /errcode:, $error, /MSG:,$errmsg,$addinfo \n";
-# $Zpackage->destroy();
-# $Zconnbiblio[0]->destroy();
-# ModZebrafiles( $dbh, $biblionumber, $record, $op, $server );
-# return;
-# }
-# if ( C4::Context->$shadow ) {
-# $Zpackage->send('commit');
-# while ( ( $i = ZOOM::event( \@Zconnbiblio ) ) != 0 ) {
-#
-# #waiting zebra to finish;
-# }
-# }
-# $Zpackage->destroy();
+=head2 GetNoZebraIndexes
+
+ %indexes = GetNoZebraIndexes;
+
+ return the data from NoZebraIndexes syspref.
+
+=cut
+
+sub GetNoZebraIndexes {
+ my $index = C4::Context->preference('NoZebraIndexes');
+ my %indexes;
+ foreach my $line (split /('|"),/,$index) {
+ $line =~ /(.*)=>(.*)/;
+ my $index = substr($1,1); # get the index, don't forget to remove initial ' or "
+ my $fields = $2;
+ $index =~ s/'|"| //g;
+ $fields =~ s/'|"| //g;
+ $indexes{$index}=$fields;
+ }
+ return %indexes;
}
=head1 INTERNAL FUNCTIONS
+=head2 _DelBiblioNoZebra($biblionumber,$record,$server);
+
+ function to delete a biblio in NoZebra indexes
+ This function does NOT delete anything in database : it reads all the indexes entries
+ that have to be deleted & delete them in the hash
+ The SQL part is done either :
+ - after the Add if we are modifying a biblio (delete + add again)
+ - immediatly after this sub if we are doing a true deletion.
+ $server can be 'biblioserver' or 'authorityserver' : it indexes biblios or authorities (in the same table, $server being part of the table itself
+
+=cut
+
+
+sub _DelBiblioNoZebra {
+ my ($biblionumber, $record, $server)=@_;
+
+ # Get the indexes
+ my $dbh = C4::Context->dbh;
+ # Get the indexes
+ my %index;
+ my $title;
+ if ($server eq 'biblioserver') {
+ %index=GetNoZebraIndexes;
+ # get title of the record (to store the 10 first letters with the index)
+ my ($titletag,$titlesubfield) = GetMarcFromKohaField('biblio.title');
+ $title = lc($record->subfield($titletag,$titlesubfield));
+ } else {
+ # for authorities, the "title" is the $a mainentry
+ my $authref = C4::AuthoritiesMarc::GetAuthType($record->subfield(152,'b'));
+ warn "ERROR : authtype undefined for ".$record->as_formatted unless $authref;
+ $title = $record->subfield($authref->{auth_tag_to_report},'a');
+ $index{'mainmainentry'}= $authref->{'auth_tag_to_report'}.'a';
+ $index{'mainentry'} = $authref->{'auth_tag_to_report'}.'*';
+ $index{'auth_type'} = '152b';
+ }
+
+ my %result;
+ # remove blancks comma (that could cause problem when decoding the string for CQL retrieval) and regexp specific values
+ $title =~ s/ |,|;|\[|\]|\(|\)|\*|-|'|=//g;
+ # limit to 10 char, should be enough, and limit the DB size
+ $title = substr($title,0,10);
+ #parse each field
+ my $sth2=$dbh->prepare('SELECT biblionumbers FROM nozebra WHERE server=? AND indexname=? AND value=?');
+ foreach my $field ($record->fields()) {
+ #parse each subfield
+ next if $field->tag <10;
+ foreach my $subfield ($field->subfields()) {
+ my $tag = $field->tag();
+ my $subfieldcode = $subfield->[0];
+ my $indexed=0;
+ # check each index to see if the subfield is stored somewhere
+ # otherwise, store it in __RAW__ index
+ foreach my $key (keys %index) {
+# warn "examining $key index : ".$index{$key}." for $tag $subfieldcode";
+ if ($index{$key} =~ /$tag\*/ or $index{$key} =~ /$tag$subfieldcode/) {
+ $indexed=1;
+ my $line= lc $subfield->[1];
+ # remove meaningless value in the field...
+ $line =~ s/-|\.|\?|,|;|!|'|\(|\)|\[|\]|{|}|"|<|>|&|\+|\*|\/|=|:/ /g;
+ # ... and split in words
+ foreach (split / /,$line) {
+ next unless $_; # skip empty values (multiple spaces)
+ # if the entry is already here, do nothing, the biblionumber has already be removed
+ unless ($result{$key}->{$_} =~ /$biblionumber,$title\-(\d);/) {
+ # get the index value if it exist in the nozebra table and remove the entry, otherwise, do nothing
+ $sth2->execute($server,$key,$_);
+ my $existing_biblionumbers = $sth2->fetchrow;
+ # it exists
+ if ($existing_biblionumbers) {
+# warn " existing for $key $_: $existing_biblionumbers";
+ $result{$key}->{$_} =$existing_biblionumbers;
+ $result{$key}->{$_} =~ s/$biblionumber,$title\-(\d);//;
+ }
+ }
+ }
+ }
+ }
+ # the subfield is not indexed, store it in __RAW__ index anyway
+ unless ($indexed) {
+ my $line= lc $subfield->[1];
+ $line =~ s/-|\.|\?|,|;|!|'|\(|\)|\[|\]|{|}|"|<|>|&|\+|\*|\/|=|:/ /g;
+ # ... and split in words
+ foreach (split / /,$line) {
+ next unless $_; # skip empty values (multiple spaces)
+ # if the entry is already here, do nothing, the biblionumber has already be removed
+ unless ($result{'__RAW__'}->{$_} =~ /$biblionumber,$title\-(\d);/) {
+ # get the index value if it exist in the nozebra table and remove the entry, otherwise, do nothing
+ $sth2->execute($server,'__RAW__',$_);
+ my $existing_biblionumbers = $sth2->fetchrow;
+ # it exists
+ if ($existing_biblionumbers) {
+ $result{'__RAW__'}->{$_} =$existing_biblionumbers;
+ $result{'__RAW__'}->{$_} =~ s/$biblionumber,$title\-(\d);//;
+ }
+ }
+ }
+ }
+ }
+ }
+ return %result;
+}
+
+=head2 _AddBiblioNoZebra($biblionumber, $record, $server, %result);
+
+ function to add a biblio in NoZebra indexes
+
+=cut
+
+sub _AddBiblioNoZebra {
+ my ($biblionumber, $record, $server, %result)=@_;
+ my $dbh = C4::Context->dbh;
+ # Get the indexes
+ my %index;
+ my $title;
+ if ($server eq 'biblioserver') {
+ %index=GetNoZebraIndexes;
+ # get title of the record (to store the 10 first letters with the index)
+ my ($titletag,$titlesubfield) = GetMarcFromKohaField('biblio.title');
+ $title = lc($record->subfield($titletag,$titlesubfield));
+ } else {
+ # warn "server : $server";
+ # for authorities, the "title" is the $a mainentry
+ my $authref = C4::AuthoritiesMarc::GetAuthType($record->subfield(152,'b'));
+ warn "ERROR : authtype undefined for ".$record->as_formatted unless $authref;
+ $title = $record->subfield($authref->{auth_tag_to_report},'a');
+ $index{'mainmainentry'} = $authref->{auth_tag_to_report}.'a';
+ $index{'mainentry'} = $authref->{auth_tag_to_report}.'*';
+ $index{'auth_type'} = '152b';
+ }
+
+ # remove blancks comma (that could cause problem when decoding the string for CQL retrieval) and regexp specific values
+ $title =~ s/ |,|;|\[|\]|\(|\)|\*|-|'|=//g;
+ # limit to 10 char, should be enough, and limit the DB size
+ $title = substr($title,0,10);
+ #parse each field
+ my $sth2=$dbh->prepare('SELECT biblionumbers FROM nozebra WHERE server=? AND indexname=? AND value=?');
+ foreach my $field ($record->fields()) {
+ #parse each subfield
+ next if $field->tag <10;
+ foreach my $subfield ($field->subfields()) {
+ my $tag = $field->tag();
+ my $subfieldcode = $subfield->[0];
+ my $indexed=0;
+ # check each index to see if the subfield is stored somewhere
+ # otherwise, store it in __RAW__ index
+ foreach my $key (keys %index) {
+# warn "examining $key index : ".$index{$key}." for $tag $subfieldcode";
+ if ($index{$key} =~ /$tag\*/ or $index{$key} =~ /$tag$subfieldcode/) {
+ $indexed=1;
+ my $line= lc $subfield->[1];
+ # remove meaningless value in the field...
+ $line =~ s/-|\.|\?|,|;|!|'|\(|\)|\[|\]|{|}|"|<|>|&|\+|\*|\/|=|:/ /g;
+ # ... and split in words
+ foreach (split / /,$line) {
+ next unless $_; # skip empty values (multiple spaces)
+ # if the entry is already here, improve weight
+# warn "managing $_";
+ if ($result{$key}->{"$_"} =~ /$biblionumber,$title\-(\d);/) {
+ my $weight=$1+1;
+ $result{$key}->{"$_"} =~ s/$biblionumber,$title\-(\d);//;
+ $result{$key}->{"$_"} .= "$biblionumber,$title-$weight;";
+ } else {
+ # get the value if it exist in the nozebra table, otherwise, create it
+ $sth2->execute($server,$key,$_);
+ my $existing_biblionumbers = $sth2->fetchrow;
+ # it exists
+ if ($existing_biblionumbers) {
+ $result{$key}->{"$_"} =$existing_biblionumbers;
+ my $weight=$1+1;
+ $result{$key}->{"$_"} =~ s/$biblionumber,$title\-(\d);//;
+ $result{$key}->{"$_"} .= "$biblionumber,$title-$weight;";
+ # create a new ligne for this entry
+ } else {
+# warn "INSERT : $server / $key / $_";
+ $dbh->do('INSERT INTO nozebra SET server='.$dbh->quote($server).', indexname='.$dbh->quote($key).',value='.$dbh->quote($_));
+ $result{$key}->{"$_"}.="$biblionumber,$title-1;";
+ }
+ }
+ }
+ }
+ }
+ # the subfield is not indexed, store it in __RAW__ index anyway
+ unless ($indexed) {
+ my $line= lc $subfield->[1];
+ $line =~ s/-|\.|\?|,|;|!|'|\(|\)|\[|\]|{|}|"|<|>|&|\+|\*|\/|=|:/ /g;
+ # ... and split in words
+ foreach (split / /,$line) {
+ next unless $_; # skip empty values (multiple spaces)
+ # if the entry is already here, improve weight
+ if ($result{'__RAW__'}->{"$_"} =~ /$biblionumber,$title\-(\d);/) {
+ my $weight=$1+1;
+ $result{'__RAW__'}->{"$_"} =~ s/$biblionumber,$title\-(\d);//;
+ $result{'__RAW__'}->{"$_"} .= "$biblionumber,$title-$weight;";
+ } else {
+ # get the value if it exist in the nozebra table, otherwise, create it
+ $sth2->execute($server,'__RAW__',$_);
+ my $existing_biblionumbers = $sth2->fetchrow;
+ # it exists
+ if ($existing_biblionumbers) {
+ $result{'__RAW__'}->{"$_"} =$existing_biblionumbers;
+ my $weight=$1+1;
+ $result{'__RAW__'}->{"$_"} =~ s/$biblionumber,$title\-(\d);//;
+ $result{'__RAW__'}->{"$_"} .= "$biblionumber,$title-$weight;";
+ # create a new ligne for this entry
+ } else {
+ $dbh->do('INSERT INTO nozebra SET server='.$dbh->quote($server).', indexname="__RAW__",value='.$dbh->quote($_));
+ $result{'__RAW__'}->{"$_"}.="$biblionumber,$title-1;";
+ }
+ }
+ }
+ }
+ }
+ }
+ return %result;
+}
+
+
=head2 MARCitemchange
=over 4
my $dbh = C4::Context->dbh;
my ( $tagfield, $tagsubfield ) =
- GetMarcFromKohaField( $dbh, $itemfield, "" );
+ GetMarcFromKohaField( $itemfield, "" );
if ( ($tagfield) && ($tagsubfield) ) {
my $tag = $record->field($tagfield);
if ($tag) {
=over 4
+$biblionumber = _koha_modify_biblio($dbh,$biblio);
Internal function for updating the biblio table
=back
sub _koha_modify_biblio {
my ( $dbh, $biblio ) = @_;
-
-# FIXME: this code could be made more portable by not hard-coding the values that are supposed to be in biblio table
- my $sth =
- $dbh->prepare(
-"Update biblio set title = ?, author = ?, abstract = ?, copyrightdate = ?, seriestitle = ?, serial = ?, unititle = ?, notes = ? where biblionumber = ?"
- );
+ # FIXME: this code could be made more portable by not hard-coding
+ # the values that are supposed to be in biblio table
+ my $query = qq{
+ UPDATE biblio
+ SET title = ?,
+ author = ?,
+ abstract = ?,
+ copyrightdate = ?,
+ seriestitle = ?,
+ serial = ?,
+ unititle = ?,
+ notes = ?
+ WHERE biblionumber = ?
+ };
+ my $sth = $dbh->prepare($query);
+
$sth->execute(
- $biblio->{'title'}, $biblio->{'author'},
- $biblio->{'abstract'}, $biblio->{'copyrightdate'},
- $biblio->{'seriestitle'}, $biblio->{'serial'},
- $biblio->{'unititle'}, $biblio->{'notes'},
+ $biblio->{'title'},
+ $biblio->{'author'},
+ $biblio->{'abstract'},
+ $biblio->{'copyrightdate'},
+ $biblio->{'seriestitle'},
+ $biblio->{'serial'},
+ $biblio->{'unititle'},
+ $biblio->{'notes'},
$biblio->{'biblionumber'}
- );
- $sth->finish;
+ ) if $biblio->{'biblionumber'};
+
+ warn $sth->err if $sth->err;
+ warn "BIG ERROR :: No biblionumber for $biblio->{title}" if $biblio->{biblionumber} !~ /\d+/; # if it is not a number
return ( $biblio->{'biblionumber'} );
}
$biblioitem->{'bnotes'} = $dbh->quote( $biblioitem->{'bnotes'} );
$biblioitem->{'size'} = $dbh->quote( $biblioitem->{'size'} );
$biblioitem->{'place'} = $dbh->quote( $biblioitem->{'place'} );
+ $biblioitem->{'collectiontitle'} = $dbh->quote( $biblioitem->{'collectiontitle'} );
+ $biblioitem->{'collectionissn'} = $dbh->quote( $biblioitem->{'collectionissn'} );
+ $biblioitem->{'collectionvolume'} = $dbh->quote( $biblioitem->{'collectionvolume'} );
+ $biblioitem->{'editionstatement'} = $dbh->quote( $biblioitem->{'editionstatement'} );
+ $biblioitem->{'editionresponsibility'} = $dbh->quote( $biblioitem->{'editionresponsibility'} );
$biblioitem->{'ccode'} = $dbh->quote( $biblioitem->{'ccode'} );
$biblioitem->{'biblionumber'} =
$dbh->quote( $biblioitem->{'biblionumber'} );
- $query = "Update biblioitems set
+ $query = "UPDATE biblioitems SET
itemtype = $biblioitem->{'itemtype'},
url = $biblioitem->{'url'},
isbn = $biblioitem->{'isbn'},
notes = $biblioitem->{'bnotes'},
size = $biblioitem->{'size'},
place = $biblioitem->{'place'},
+ collectiontitle = $biblioitem->{'collectiontitle'},
+ collectionissn = $biblioitem->{'collectionissn'},
+ collectionvolume= $biblioitem->{'collectionvolume'},
+ editionstatement= $biblioitem->{'editionstatement'},
+ editionresponsibility= $biblioitem->{'editionresponsibility'},
ccode = $biblioitem->{'ccode'}
where biblionumber = $biblioitem->{'biblionumber'}";
$dbh->do($query);
if ( $dbh->errstr ) {
- warn "$query";
+ warn "ERROR in _koha_modify_biblioitem $query";
}
}
volumeddesc = ?, illus = ?,
pages = ?, notes = ?,
size = ?, lccn = ?,
- marc = ?, lcsort =?,
- place = ?, ccode = ?
+ marc = ?, lcsort = ?,
+ place = ?, ccode = ?,
+ collectiontitle = ?, collectionissn = ?,
+ collectionvolume = ?, editionstatement= ?,
+ editionresponsibility= ?
"
);
my ($lcsort) =
$biblioitem->{'pages'}, $biblioitem->{'bnotes'},
$biblioitem->{'size'}, $biblioitem->{'lccn'},
$biblioitem->{'marc'}, $biblioitem->{'place'},
- $lcsort, $biblioitem->{'ccode'}
+ $lcsort, $biblioitem->{'ccode'},
+ $biblioitem->{'collectiontitle'},$biblioitem->{'collectionissn'},
+ $biblioitem->{'collectionvolume'},$biblioitem->{'editionstatement'},
+ $biblioitem->{'editionresponsibility'}
);
$sth->finish;
return ($bibitemnum);
$sth = $dbh->prepare(
"Insert into items set
itemnumber = ?, biblionumber = ?,
- multivolumepart = ?,
biblioitemnumber = ?, barcode = ?,
booksellerid = ?, dateaccessioned = NOW(),
homebranch = ?, holdingbranch = ?,
price = ?, replacementprice = ?,
replacementpricedate = NOW(), datelastseen = NOW(),
- multivolume = ?, stack = ?,
+ stack = ?,
itemlost = ?, wthdrawn = ?,
paidfor = ?, itemnotes = ?,
itemcallnumber =?, notforloan = ?,
);
$sth->execute(
$itemnumber, $item->{'biblionumber'},
- $item->{'multivolumepart'}, $item->{'biblioitemnumber'},
+ $item->{'biblioitemnumber'},
$barcode, $item->{'booksellerid'},
$item->{'homebranch'}, $item->{'holdingbranch'},
$item->{'price'}, $item->{'replacementprice'},
- $item->{multivolume}, $item->{stack},
+ $item->{stack},
$item->{itemlost}, $item->{wthdrawn},
$item->{paidfor}, $item->{'itemnotes'},
$item->{'itemcallnumber'}, $item->{'notforloan'},
$sth = $dbh->prepare(
"INSERT INTO items SET
itemnumber = ?, biblionumber = ?,
- multivolumepart = ?,
biblioitemnumber = ?, barcode = ?,
booksellerid = ?, dateaccessioned = ?,
homebranch = ?, holdingbranch = ?,
price = ?, replacementprice = ?,
replacementpricedate = NOW(), datelastseen = NOW(),
- multivolume = ?, stack = ?,
+ stack = ?,
itemlost = ?, wthdrawn = ?,
paidfor = ?, itemnotes = ?,
itemcallnumber = ?, notforloan = ?,
);
$sth->execute(
$itemnumber, $item->{'biblionumber'},
- $item->{'multivolumepart'}, $item->{'biblioitemnumber'},
+ $item->{'biblioitemnumber'},
$barcode, $item->{'booksellerid'},
$item->{'dateaccessioned'}, $item->{'homebranch'},
$item->{'holdingbranch'}, $item->{'price'},
- $item->{'replacementprice'}, $item->{multivolume},
+ $item->{'replacementprice'},
$item->{stack}, $item->{itemlost},
$item->{wthdrawn}, $item->{paidfor},
$item->{'itemnotes'}, $item->{'itemcallnumber'},
# if all we're doing is setting statuses, just update those and get out
if ( $op eq "setstatus" ) {
my $query =
- "UPDATE items SET itemlost=?,wthdrawn=?,binding=? WHERE itemnumber=?";
+ "UPDATE items SET itemlost=?,wthdrawn=? WHERE itemnumber=?";
my @bind = (
$item->{'itemlost'}, $item->{'wthdrawn'},
- $item->{'binding'}, $item->{'itemnumber'}
+ $item->{'itemnumber'}
);
my $sth = $dbh->prepare($query);
$sth->execute(@bind);
itemcalculator( $dbh, $item->{'bibitemnum'}, $item->{'itemcallnumber'} );
my $query = "UPDATE items SET
-barcode=?,itemnotes=?,itemcallnumber=?,notforloan=?,location=?,multivolumepart=?,multivolume=?,stack=?,wthdrawn=?,holdingbranch=?,homebranch=?,cutterextra=?, onloan=?, binding=?";
+barcode=?,itemnotes=?,itemcallnumber=?,notforloan=?,location=?,stack=?,wthdrawn=?,holdingbranch=?,homebranch=?,cutterextra=?, onloan=?";
my @bind = (
$item->{'barcode'}, $item->{'notes'},
$item->{'itemcallnumber'}, $item->{'notforloan'},
- $item->{'location'}, $item->{multivolumepart},
- $item->{multivolume}, $item->{stack},
+ $item->{'location'}, $item->{stack},
$item->{wthdrawn}, $item->{holdingbranch},
$item->{homebranch}, $cutterextra,
- $item->{onloan}, $item->{binding}
+ $item->{onloan},
);
if ( $item->{'lost'} ne '' ) {
$query =
"update items set biblioitemnumber=?,barcode=?,itemnotes=?,homebranch=?,
itemlost=?,wthdrawn=?,itemcallnumber=?,notforloan=?,
- location=?,multivolumepart=?,multivolume=?,stack=?,wthdrawn=?,holdingbranch=?,cutterextra=?,onloan=?, binding=?";
+ location=?,stack=?,wthdrawn=?,holdingbranch=?,cutterextra=?,onloan=?";
@bind = (
$item->{'bibitemnum'}, $item->{'barcode'},
$item->{'notes'}, $item->{'homebranch'},
$item->{'lost'}, $item->{'wthdrawn'},
$item->{'itemcallnumber'}, $item->{'notforloan'},
- $item->{'location'}, $item->{multivolumepart},
- $item->{multivolume}, $item->{stack},
+ $item->{'location'}, $item->{stack},
$item->{wthdrawn}, $item->{holdingbranch},
- $cutterextra, $item->{onloan},
- $item->{binding}
+ $cutterextra, $item->{onloan}
);
if ( $item->{homebranch} ) {
$query .= ",homebranch=?";
=head2 ModBiblioMarc
-=over 4
-
-&ModBiblioMarc($newrec,$biblionumber,$frameworkcode);
-
-Add MARC data for a biblio to koha
-
-Function exported, but should NOT be used, unless you really know what you're doing
-
-=back
+ &ModBiblioMarc($newrec,$biblionumber,$frameworkcode);
+
+ Add MARC data for a biblio to koha
+
+ Function exported, but should NOT be used, unless you really know what you're doing
=cut
sub ModBiblioMarc {
-
-# pass the MARC::Record to this function, and it will create the records in the marc tables
+
+# pass the MARC::Record to this function, and it will create the records in the marc field
my ( $record, $biblionumber, $frameworkcode ) = @_;
my $dbh = C4::Context->dbh;
my @fields = $record->fields();
$sth->finish;
my $encoding = C4::Context->preference("marcflavour");
-# deal with UNIMARC field 100 (encoding) : create it if needed & set encoding to unicode
+ # deal with UNIMARC field 100 (encoding) : create it if needed & set encoding to unicode
if ( $encoding eq "UNIMARC" ) {
my $string;
- if ( $record->subfield( 100, "a" ) ) {
+ if ( length($record->subfield( 100, "a" )) == 35 ) {
$string = $record->subfield( 100, "a" );
my $f100 = $record->field(100);
$record->delete_field($f100);
MARC::Field->new( 100, "", "", "a" => $string ) );
}
}
-# warn "biblionumber : ".$biblionumber;
+ ModZebra($biblionumber,"specialUpdate","biblioserver",$record);
$sth =
$dbh->prepare(
"update biblioitems set marc=?,marcxml=? where biblionumber=?");
- $sth->execute( $record->as_usmarc(), $record->as_xml_record(),
+ $sth->execute( $record->as_usmarc(), $record->as_xml_record($encoding),
$biblionumber );
-# warn $record->as_xml_record();
$sth->finish;
- ModZebra($biblionumber,"specialUpdate","biblioserver");
return $biblionumber;
}
sub AddItemInMarc {
-# pass the MARC::Record to this function, and it will create the records in the marc tables
+ # pass the MARC::Record to this function, and it will create the records in the marc tables
my ( $record, $biblionumber, $frameworkcode ) = @_;
my $newrec = &GetMarcBiblio($biblionumber);
return $serviceOptions;
}
+=head2 GetItemsCount
+
+$count = &GetItemsCount( $biblionumber);
+this function return count of item with $biblionumber
+=cut
+
+sub GetItemsCount {
+ my ( $biblionumber ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = qq|SELECT count(*)
+ FROM items
+ WHERE biblionumber=?|;
+ my $sth = $dbh->prepare($query);
+ $sth->execute($biblionumber);
+ my $count = $sth->fetchrow;
+ $sth->finish;
+ return ($count);
+}
+
END { } # module clean-up code here (global destructor)
1;
# $Id$
# $Log$
+# Revision 1.221 2007/07/31 16:01:11 toins
+# Some new functions.
+# TransformHTMLtoMarc rewrited.
+#
+# Revision 1.220 2007/07/20 15:43:16 hdl
+# Bug Fixing GetMarcSubjects.
+# Links parameters were mixed.
+#
+# Revision 1.218 2007/07/19 07:40:08 hdl
+# Adding selection by location for inventory
+#
+# Revision 1.217 2007/07/03 13:47:44 tipaul
+# fixing some display bugs (itemtype not properly returned and a html table bug that makes items appear strangely
+#
+# Revision 1.216 2007/07/03 09:40:58 tipaul
+# return itemtype description properly
+#
+# Revision 1.215 2007/07/03 09:33:05 tipaul
+# if you just replace su by a space in subjects, you'll replace jesus by je s, which is strange for users. this fix solves the problem and introduces authoritysep systempref as separator of subfields, for a better identification of where the authority starts and end
+#
+# Revision 1.214 2007/07/02 09:13:22 tipaul
+# unimarc bugfix : the encoding is in field 100 in UNIMARC. when TransformHTMLtoXML on an item, you must not automatically add a 100 field in items, otherwise there will be 2 100 fields in the biblio, which is wrong
+#
+# Revision 1.213 2007/06/25 15:01:45 tipaul
+# bugfixes on unimarc 100 handling (the field used for encoding)
+#
+# Revision 1.212 2007/06/15 13:44:44 tipaul
+# some fixes (and only fixes)
+#
+# Revision 1.211 2007/06/15 09:40:06 toins
+# do not get $3 $4 and $5 on GetMarcSubjects GetMarcAuthors on unimarc flavour.
+#
+# Revision 1.210 2007/06/13 13:03:34 toins
+# removing warn compilation.
+#
+# Revision 1.209 2007/05/23 16:19:40 tipaul
+# various bugfixes (minor) and french translation updated
+#
+# Revision 1.208 2007/05/22 09:13:54 tipaul
+# Bugfixes & improvements (various and minor) :
+# - updating templates to have tmpl_process3.pl running without any errors
+# - adding a drupal-like css for prog templates (with 3 small images)
+# - fixing some bugs in circulation & other scripts
+# - updating french translation
+# - fixing some typos in templates
+#
+# Revision 1.207 2007/05/22 08:51:19 hdl
+# Changing GetMarcStructure signature.
+# Deleting first parameter $dbh
+#
+# Revision 1.206 2007/05/21 08:44:17 btoumi
+# add security when u delete biblio :
+# u must delete linked items before delete biblio
+#
+# Revision 1.205 2007/05/11 16:04:03 btoumi
+# bug fix:
+# problem in displayed label link with subject in detail.tmpl
+# ex: label random => rdom
+#
+# Revision 1.204 2007/05/10 14:45:15 tipaul
+# Koha NoZebra :
+# - support for authorities
+# - some bugfixes in ordering and "CCL" parsing
+# - support for authorities <=> biblios walking
+#
+# Seems I can do what I want now, so I consider its done, except for bugfixes that will be needed i m sure !
+#
+# Revision 1.203 2007/05/03 15:16:02 tipaul
+# BUGFIX for : NoZebra
+# - NoZebra features : seems they work fine now (adding, modifying, deleting)
+# - Biblio edition major bugfix : before this commit editing a biblio resulted in an item removal in marcxml field
+#
+# Revision 1.202 2007/05/02 16:44:31 tipaul
+# NoZebra SQL index management :
+# * adding 3 subs in Biblio.pm
+# - GetNoZebraIndexes, that get the index structure in a new systempreference (added with this commit)
+# - _DelBiblioNoZebra, that retrieve all index entries for a biblio and remove in a variable the biblio reference
+# - _AddBiblioNoZebra, that add index entries for a biblio.
+# Note that the 2 _Add and _Del subs work only in a hash variable, to speed up things in case of a modif (ie : delete+add). The effective SQL update is done in the ModZebra sub (that existed before, and dealed with zebra index).
+# I think the code has to be more deeply tested, but it works at least partially.
+#
+# Revision 1.201 2007/04/27 14:00:49 hdl
+# Removing $dbh from GetMarcFromKohaField (dbh is not used in this function.)
+#
+# Revision 1.200 2007/04/25 16:26:42 tipaul
+# Koha 3.0 nozebra 1st commit : the script misc/migration_tools/rebuild_nozebra.pl build the nozebra table, and, if you set NoZebra to Yes, queries will be done through zebra. TODO :
+# - add nozebra table management on biblio editing
+# - the index table content is hardcoded. I still have to add some specific systempref to let the library update it
+# - manage pagination (next/previous)
+# - manage facets
+# WHAT works :
+# - NZgetRecords : has exactly the same API & returns as zebra getQuery, except that some parameters are unused
+# - search & sort works quite good
+# - CQL parser is better that what I thought I could do : title="harry and sally" and publicationyear>2000 not itemtype=LIVR should work fine
+#
+# Revision 1.199 2007/04/24 09:07:53 tipaul
+# moving dotransfer to Biblio.pm::ModItemTransfer + some CheckReserves fixes
+#
+# Revision 1.198 2007/04/23 15:21:17 tipaul
+# renaming currenttransfers to transferstoreceive
+#
+# Revision 1.197 2007/04/18 17:00:14 tipaul
+# removing all useless %env / $env
+#
+# Revision 1.196 2007/04/17 08:48:00 tipaul
+# circulation cleaning continued: bufixing
+#
+# Revision 1.195 2007/04/04 16:46:22 tipaul
+# HUGE COMMIT : code cleaning circulation.
+#
+# some stuff to do, i'll write a mail on koha-devel NOW !
+#
# Revision 1.194 2007/03/30 12:00:42 tipaul
# why the hell do we need to explicitly utf8 decode this string ? I really don't know, but it seems it's mandatory, otherwise, tag descriptions are not properly encoded...
#