Bug 32797: Check if fields have a value, not if true
When saving OAI set mappgins we are checking for truth, this prevents saving subfield 0. We should check for content To test: 1 - Go to Admin->OAI sets 2 - Create a set 3 - Define mappings for the set 4 - Attempt to save a rule on field 952 subfield 0 (withdrawn) 5 - It does not save 6 - Apply patch 7 - Try again 8 - Success! Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
3bf31ae4d4
commit
6f606e00c5
|
|
@ -60,7 +60,7 @@ if($op && $op eq "save") {
|
|||
my @mappings;
|
||||
my $i = 0;
|
||||
while($i < @marcfields and $i < @marcsubfields and $i < @marcvalues) {
|
||||
if($marcfields[$i] and $marcsubfields[$i]) {
|
||||
if($marcfields[$i] ne '' and $marcsubfields[$i] ne '' ) {
|
||||
push @mappings, {
|
||||
marcfield => $marcfields[$i],
|
||||
marcsubfield => $marcsubfields[$i],
|
||||
|
|
|
|||
Loading…
Reference in New Issue