--- trunk/lib/WebPAC/Normalize.pm	2006/09/06 14:25:16	631
+++ trunk/lib/WebPAC/Normalize.pm	2006/09/06 20:54:47	641
@@ -4,6 +4,7 @@
 	_set_rec _set_lookup
 	_get_ds _clean_ds
 	_debug
+	_pack_subfields_hash
 
 	tag search display
 	marc marc_indicators marc_repeatable_subfield
@@ -722,6 +723,41 @@
 This function should be used inside functions to create C<data_structure> described
 above.
 
+=head2 _pack_subfields_hash
+
+ @values = _pack_subfields_hash( $h, $include_subfields )
+
+=cut
+
+sub _pack_subfields_hash {
+
+	warn "## _pack_subfields_hash( ",dump(@_), " )\n";
+
+	my ($h,$include_subfields) = @_;
+
+
+	if ( defined($h->{subfields}) ) {
+		my $sfs = delete $h->{subfields} || die "no subfields?";
+		my @out;
+		while (@$sfs) {
+			my $sf = shift @$sfs;
+			push @out, '^' . $sf if ($include_subfields);
+			my $o = shift @$sfs;
+			if ($o == 0 && ref( $h->{$sf} ) ne 'ARRAY' ) {
+				# single element subfields are not arrays
+				push @out, $h->{$sf};
+			} else {
+#warn "====> $f $sf $o $#$sfs ", dump( $sfs ), "\n";
+				push @out, $h->{$sf}->[$o];
+			}
+		}
+		return @out;
+	} else {
+		# FIXME this should probably be in alphabetical order instead of hash order
+		values %{$h};
+	}
+}
+
 =head2 rec1
 
 Return all values in some field
@@ -738,32 +774,16 @@
 	return unless (defined($rec) && defined($rec->{$f}));
 	warn "rec1($f) = ", dump( $rec->{$f} ), $/ if ($debug > 1);
 	if (ref($rec->{$f}) eq 'ARRAY') {
-		return map { 
-			if (ref($_) eq 'HASH') {
-				my $h = $_;
-				if ( defined($h->{subfields}) ) {
-					my $sfs = delete $h->{subfields} || die "no subfields?";
-					my @out;
-					while (@$sfs) {
-						my $sf = shift @$sfs;
-						my $o = shift @$sfs;
-						if ($o == 0 && ref( $h->{$sf} ) ne 'ARRAY' ) {
-							# single element subfields are not arrays
-							push @out, $h->{$sf};
-						} else {
-warn "====> $f $sf $o $#$sfs ", dump( $sfs ), "\n";
-							push @out, $h->{$sf}->[$o];
-						}
-					}
-					return @out;
-				} else {
-					# FIXME this should probably be in alphabetical order instead of hash order
-					values %{$h};
-				}
+		my @out;
+		foreach my $h ( @{ $rec->{$f} } ) {
+			if (ref($h) eq 'HASH') {
+warn "rec1 hash: ",dump($h),"\n";
+				push @out, ( _pack_subfields_hash( $h ) );
 			} else {
-				$_;
+				push @out, $h;
 			}
-		} @{ $rec->{$f} };
+		}
+		return @out;
 	} elsif( defined($rec->{$f}) ) {
 		return $rec->{$f};
 	}