use C4::Context;
use Koha::SimpleMARC;
+use Koha::MoreUtils;
-use vars qw($VERSION @ISA @EXPORT);
+use vars qw(@ISA @EXPORT);
use constant DEBUG => 0;
BEGIN {
- $VERSION = 1.00; # set the version for version checking
@ISA = qw(Exporter);
@EXPORT = qw(
&GetModificationTemplates
=head2 GetModificationTemplates
- my @templates = GetModificationTemplates( [ $template_id ] );
+ my @templates = GetModificationTemplates( $template_id );
+
+ Passing optional $template_id marks it as the selected template.
- Passing a $template_id will mark the given id as the selected template.
=cut
sub GetModificationTemplates {
warn("C4::MarcModificationTemplates::GetModificationTemplates( $template_id )") if DEBUG;
my $dbh = C4::Context->dbh;
- my $sth = $dbh->prepare("SELECT * FROM marc_modification_templates");
+ my $sth = $dbh->prepare("SELECT * FROM marc_modification_templates ORDER BY name");
$sth->execute();
my @templates;
while ( my $template = $sth->fetchrow_hashref() ) {
- $template->{'selected'} = 1 if ( $template->{'template_id'} eq $template_id );
+ $template->{'selected'} = 1
+ if $template_id && $template->{'template_id'} eq $template_id;
push( @templates, $template );
}
$to_regex_search, $to_regex_replace, $to_regex_modifiers, $conditional, $conditional_field, $conditional_subfield, $conditional_comparison,
$conditional_value, $conditional_regex, $description )" ) if DEBUG;
+ $conditional ||= undef;
+ $conditional_comparison ||= undef;
$conditional_regex ||= '0';
my $dbh = C4::Context->dbh;
) = @_;
my $dbh = C4::Context->dbh;
+ $conditional ||= undef;
+ $conditional_comparison ||= undef;
+ $conditional_regex ||= '0';
my $query = "
UPDATE marc_modification_template_actions SET
foreach my $a ( @actions ) {
my $action = $a->{'action'};
- my $field_number = $a->{'field_number'};
+ my $field_number = $a->{'field_number'} // 1;
my $from_field = $a->{'from_field'};
my $from_subfield = $a->{'from_subfield'};
my $field_value = $a->{'field_value'};
}
my $do = 1;
+ my $field_numbers = [];
if ( $conditional ) {
if ( $conditional_comparison eq 'exists' ) {
- my $exists = field_exists({
- record =>$record,
+ $field_numbers = field_exists({
+ record => $record,
field => $conditional_field,
subfield => $conditional_subfield,
});
$do = $conditional eq 'if'
- ? $exists
- : not $exists;
+ ? @$field_numbers
+ : not @$field_numbers;
}
elsif ( $conditional_comparison eq 'not_exists' ) {
- my $exists = field_exists({
+ $field_numbers = field_exists({
record => $record,
field => $conditional_field,
subfield => $conditional_subfield
});
$do = $conditional eq 'if'
- ? not $exists
- : $exists;
+ ? not @$field_numbers
+ : @$field_numbers;
}
elsif ( $conditional_comparison eq 'equals' ) {
- my $equals = field_equals({
+ $field_numbers = field_equals({
record => $record,
value => $conditional_value,
field => $conditional_field,
subfield => $conditional_subfield,
- regex => $conditional_regex
+ is_regex => $conditional_regex,
});
$do = $conditional eq 'if'
- ? $equals
- : not $equals;
+ ? @$field_numbers
+ : not @$field_numbers;
}
elsif ( $conditional_comparison eq 'not_equals' ) {
- my $equals = field_equals({
+ $field_numbers = field_equals({
record => $record,
value => $conditional_value,
field => $conditional_field,
subfield => $conditional_subfield,
- regex => $conditional_regex
+ is_regex => $conditional_regex,
});
+ my $all_fields = [
+ 1 .. scalar @{
+ field_exists(
+ {
+ record => $record,
+ field => $conditional_field,
+ subfield => $conditional_subfield
+ }
+ )
+ }
+ ];
+ $field_numbers = [Koha::MoreUtils::singleton ( @$field_numbers, @$all_fields ) ];
$do = $conditional eq 'if'
- ? not $equals
- : $equals;
+ ? @$field_numbers
+ : not @$field_numbers;
}
}
if ( $do ) {
+
+ # field_number == 0 if all field need to be updated
+ # or 1 if only the first field need to be updated
+
+ # A condition has been given
+ if ( @$field_numbers > 0 ) {
+ if ( $field_number == 1 ) {
+ # We want only the first matching
+ $field_numbers = [ $field_numbers->[0] ];
+ }
+ }
+ # There was no condition
+ else {
+ if ( $field_number == 1 ) {
+ # We want to process the first field
+ $field_numbers = [ 1 ];
+ } elsif ( $to_field and $from_field ne $to_field ) {
+ # If the from and to fields are not the same, we only process the first field.
+ $field_numbers = [ 1 ];
+ }
+ }
+
if ( $action eq 'copy_field' ) {
copy_field({
record => $record,
replace => $to_regex_replace,
modifiers => $to_regex_modifiers
},
- n => $field_number,
+ field_numbers => $field_numbers,
+ });
+ }
+ elsif ( $action eq 'copy_and_replace_field' ) {
+ copy_and_replace_field({
+ record => $record,
+ from_field => $from_field,
+ from_subfield => $from_subfield,
+ to_field => $to_field,
+ to_subfield => $to_subfield,
+ regex => {
+ search => $to_regex_search,
+ replace => $to_regex_replace,
+ modifiers => $to_regex_modifiers
+ },
+ field_numbers => $field_numbers,
+ });
+ }
+ elsif ( $action eq 'add_field' ) {
+ add_field({
+ record => $record,
+ field => $from_field,
+ subfield => $from_subfield,
+ values => [ $field_value ],
+ field_numbers => $field_numbers,
});
}
-
elsif ( $action eq 'update_field' ) {
update_field({
record => $record,
field => $from_field,
subfield => $from_subfield,
values => [ $field_value ],
+ field_numbers => $field_numbers,
});
}
elsif ( $action eq 'move_field' ) {
replace => $to_regex_replace,
modifiers => $to_regex_modifiers
},
- n => $field_number,
+ field_numbers => $field_numbers,
});
}
elsif ( $action eq 'delete_field' ) {
record => $record,
field => $from_field,
subfield => $from_subfield,
- n => $field_number,
+ field_numbers => $field_numbers,
});
}
}
warn( $record->as_formatted() ) if DEBUG >= 10;
}
+
+ return;
}
1;
__END__