|
Lines 127-167
use Koha::ItemTypes;
Link Here
|
| 127 |
my $frameworks = Koha::BiblioFrameworks->search; |
127 |
my $frameworks = Koha::BiblioFrameworks->search; |
| 128 |
my $invalid_locations_per_framework; |
128 |
my $invalid_locations_per_framework; |
| 129 |
while ( my $framework = $frameworks->next ) { |
129 |
while ( my $framework = $frameworks->next ) { |
| 130 |
my $avs = Koha::AuthorisedValues->search_by_koha_field( |
130 |
my $msss = Koha::MarcSubfieldStructures->search({ frameworkcode => $framework->frameworkcode, authorised_value => { '!=' => [ -and => ( undef, '' ) ]} }); |
| 131 |
{ |
131 |
while ( my $mss = $msss->next ) { |
| 132 |
frameworkcode => $framework->frameworkcode, |
132 |
my $kohafield = $mss->kohafield; |
| 133 |
kohafield => 'items.location' |
133 |
next if grep {/$kohafield/} qw( branches itemtypes cn_source ); # internal categories |
|
|
134 |
my $avs = Koha::AuthorisedValues->search_by_koha_field( |
| 135 |
{ |
| 136 |
frameworkcode => $framework->frameworkcode, |
| 137 |
kohafield => $kohafield, |
| 138 |
} |
| 139 |
); |
| 140 |
(my $tmp_kohafield = $kohafield) =~ s|items|me|; # replace items.attr with me.attr |
| 141 |
my $items = Koha::Items->search( |
| 142 |
{ |
| 143 |
$tmp_kohafield => |
| 144 |
{ -not_in => [ $avs->get_column('authorised_value') ] }, |
| 145 |
'biblio.frameworkcode' => $framework->frameworkcode |
| 146 |
}, |
| 147 |
{ join => [ 'biblioitem', 'biblio' ] } |
| 148 |
); |
| 149 |
if ( $items->count ) { |
| 150 |
$invalid_locations_per_framework->{ $framework->frameworkcode } = |
| 151 |
{ items => $items, av_category => $mss->authorised_value, kohafield => $kohafield }; |
| 134 |
} |
152 |
} |
| 135 |
); |
|
|
| 136 |
my $items = Koha::Items->search( |
| 137 |
{ |
| 138 |
location => |
| 139 |
{ -not_in => [ $avs->get_column('authorised_value') ] }, |
| 140 |
'biblio.frameworkcode' => $framework->frameworkcode |
| 141 |
}, |
| 142 |
{ join => [ 'biblioitem', 'biblio' ] } |
| 143 |
); |
| 144 |
if ( $items->count ) { |
| 145 |
$invalid_locations_per_framework->{ $framework->frameworkcode } = |
| 146 |
{ items => $items, av_category => $avs->next->category, }; |
| 147 |
} |
153 |
} |
| 148 |
} |
154 |
} |
| 149 |
if (%$invalid_locations_per_framework) { |
155 |
if (%$invalid_locations_per_framework) { |
| 150 |
new_section('Wrong value dor items.location'); |
156 |
new_section('Wrong values linked to authorised values'); |
| 151 |
for my $frameworkcode ( keys %$invalid_locations_per_framework ) { |
157 |
for my $frameworkcode ( keys %$invalid_locations_per_framework ) { |
| 152 |
my $output; |
158 |
my $output; |
| 153 |
my $items = |
159 |
my $items = |
| 154 |
$invalid_locations_per_framework->{$frameworkcode}->{items}; |
160 |
$invalid_locations_per_framework->{$frameworkcode}->{items}; |
| 155 |
my $av_category = |
161 |
my $av_category = |
| 156 |
$invalid_locations_per_framework->{$frameworkcode}->{av_category}; |
162 |
$invalid_locations_per_framework->{$frameworkcode}->{av_category}; |
|
|
163 |
my $kohafield = |
| 164 |
$invalid_locations_per_framework->{$frameworkcode}->{kohafield}; |
| 165 |
my ( $table, $column ) = split '\.', $kohafield; |
| 157 |
while ( my $i = $items->next ) { |
166 |
while ( my $i = $items->next ) { |
| 158 |
$output .= " {" . $i->itemnumber . " => " . $i->location . "}"; |
167 |
my $value = $table eq 'items' ? $i->$column : $i->biblioitem->$column; |
|
|
168 |
$output .= " {" . $i->itemnumber . " => " . $value . "}"; |
| 159 |
} |
169 |
} |
| 160 |
new_item( |
170 |
new_item( |
| 161 |
sprintf( |
171 |
sprintf( |
| 162 |
"The Framework *%s* is using the authorised value's category *%s*, " |
172 |
"The Framework *%s* is using the authorised value's category *%s*, " |
| 163 |
. "but the following items.location do not have a value defined ({itemnumber => value }):\n%s", |
173 |
. "but the following %s do not have a value defined ({itemnumber => value }):\n%s", |
| 164 |
$frameworkcode, $av_category, $output |
174 |
$frameworkcode, $av_category, $kohafield, $output |
| 165 |
) |
175 |
) |
| 166 |
); |
176 |
); |
| 167 |
} |
177 |
} |
| 168 |
- |
|
|