--- trunk/lib/WebPAC/Normalize.pm	2005/12/19 15:34:47	295
+++ trunk/lib/WebPAC/Normalize.pm	2006/01/07 02:05:55	344
@@ -11,11 +11,11 @@
 
 =head1 VERSION
 
-Version 0.07
+Version 0.08
 
 =cut
 
-our $VERSION = '0.07';
+our $VERSION = '0.08';
 
 =head1 SYNOPSIS
 
@@ -157,7 +157,7 @@
 
 	$log->debug("data_structure rec = ", sub { Dumper($rec) });
 
-	$log->logdie("need unique ID (mfn) in field 000 of record ", sub { Dumper($rec) } ) unless (defined($rec->{'000'}));
+	$log->logdie("need unique ID (mfn) in field 000 of record " . Dumper($rec) ) unless (defined($rec->{'000'}));
 
 	my $id = $rec->{'000'}->[0] || $log->logdie("field 000 isn't array!");
 
@@ -317,7 +317,7 @@
 
 	my @out;
 
-	$log->debug("format: $format");
+	$log->debug("format: $format [$i]");
 
 	my $eval_code;
 	# remove eval{...} from beginning
@@ -327,29 +327,45 @@
 	# remove filter{...} from beginning
 	$filter_name = $1 if ($format =~ s/^filter{([^}]+)}//s);
 
+	# did we found any (att all) field from format in row?
+	my $found_any;
+	# prefix before first field which we preserve it $found_any
 	my $prefix;
-	my $all_found=0;
+
+	my $f_step = 1;
 
 	while ($format =~ s/^(.*?)(v|s)(\d+)(?:\^(\w))?//s) {
 
 		my $del = $1 || '';
-		$prefix ||= $del if ($all_found == 0);
+		$prefix = $del if ($f_step == 1);
+
+		my $fld_type = lc($2);
 
 		# repeatable index
 		my $r = $i;
-		$r = 0 if (lc("$2") eq 's');
+		if ($fld_type eq 's') {
+			if ($found_any->{'v'}) {
+				$r = 0;
+			} else {
+				return;
+			}
+		}
 
 		my $found = 0;
 		my $tmp = $self->get_data(\$rec,$3,$4,$r,\$found);
 
 		if ($found) {
-			push @out, $del;
+			$found_any->{$fld_type} += $found;
+
+			# we will skip delimiter before first occurence of field!
+			push @out, $del unless($found_any->{$fld_type} == 1);
 			push @out, $tmp;
-			$all_found += $found;
 		}
+		$f_step++;
 	}
 
-	return if (! $all_found);
+	# test if any fields found?
+	return if (! $found_any->{'v'} && ! $found_any->{'s'});
 
 	my $out = join('',@out);
 
@@ -463,6 +479,7 @@
 	}
 
 	my $found = 0;
+	my $just_single = 1;
 
 	my $eval_code;
 	# remove eval{...} from beginning
@@ -474,9 +491,14 @@
 
 	# do actual replacement of placeholders
 	# repeatable fields
-	$format =~ s/v(\d+)(?:\^(\w))?/$self->get_data(\$rec,$1,$2,$i,\$found)/ges;
+	if ($format =~ s/v(\d+)(?:\^(\w))?/$self->get_data(\$rec,$1,$2,$i,\$found)/ges) {
+		$just_single = 0;
+	}
+	
 	# non-repeatable fields
-	$format =~ s/s(\d+)(?:\^(\w))?/$self->get_data(\$rec,$1,$2,0,\$found)/ges;
+	if ($format =~ s/s(\d+)(?:\^(\w))?/$self->get_data(\$rec,$1,$2,0,\$found)/ges) {
+		return if ($i > 0 && $just_single);
+	}
 
 	if ($found) {
 		$log->debug("format: $format");