--- trunk/isis_sf.pm	2003/01/16 17:35:54	10
+++ trunk/hash_sf.pm	2005/02/27 23:07:35	678
@@ -1,23 +1,26 @@
 #
-# isis_sf($isis_row,'isis_field'[,'subfield'])
+# hash_sf($row_data,'field'[,'subfield'])
 #
-# e.g. isis_sf($row,'700','a')
+# e.g. hash_sf($row,'700','a')
 #
-sub isis_sf {
+sub hash_sf {
 	my $row = shift @_;
-	my $isis_id = shift @_;
+	my $field = shift @_;
 	my $subfield = shift @_;
 
-	if ($row->{$isis_id}->[0]) {
-		my $sf = OpenIsis::subfields($row->{$isis_id}->[0]);
-		if (! defined $subfield || length($subfield) == 0) {
+	my $i = shift @_ || 0;
+
+	my $out;
+
+	if ($row->{$field}->[$i]) {
+		if (! $subfield) {
 			# subfield list undef, empty or no defined subfields for this record
-			my $all_sf = $row->{$isis_id}->[0];
-			$all_sf =~ s/\^./ /g;	nuke definirions
-			return $all_sf;
-		} elsif ($sf->{$subfield}) {
-			return $sf->{$subfield};
+			my $all_sf = $row->{record}->{$field}->[$i] || confess "can't find field $field:$i",Dumper($row);
+			$all_sf =~ s/[\^\$]./ /g;   # nuke definitions
+			return $all_sf;	
 		}
+		my $sf = $row->{$field}->[$i]->{$subfield};
+		return $sf if ($sf);
 	}
 }