--- trunk/run.pl	2007/10/30 20:32:16	916
+++ trunk/run.pl	2007/11/02 10:54:42	956
@@ -299,20 +299,34 @@
 		debug => $debug,
 	});
 
+
 	#
 	# prepare output
 	#
-	my @outputs;
-	if (defined( $db_config->{output} )) {
-		my $module = $db_config->{output}->{module} || $log->logdie("need module in output section of $database");
+	my @outputs = force_array( $db_config->{output}, sub {
+		$log->error("Database $database doesn't have any outputs defined. Do you want to remove it from configuration?" );
+	} );
+
+	my @output_modules;
+
+	foreach my $output ( @outputs ) {
+
+warn '## output = ',dump( $output );
+
+		my $module = $output->{module} || $log->logdie("need module in output section of $database");
 		$module = 'WebPAC::Output::' . $module unless $module =~ m/::/;
+	
 		$log->debug("loading output module $module");
 		eval "require $module";
-		$db_config->{output}->{database} = $database;
-		$log->debug("calling $module->new(",dump( $db_config->{output} ),")");
-		my $out = new $module->new( $db_config->{output} );
+
+		# add database to arugemnts for output filter
+		$output->{database} = $database;
+
+		$log->debug("calling $module->new(",dump( $output ),")");
+		my $out = new $module->new( $output );
 		$out->init;
-		push @outputs, $out;
+
+		push @output_modules, $out;
 	}
 
 
@@ -320,14 +334,10 @@
 	# now, iterate through input formats
 	#
 
-	my @inputs;
-	if (ref($db_config->{input}) eq 'ARRAY') {
-		@inputs = @{ $db_config->{input} };
-	} elsif ($db_config->{input}) {
-		push @inputs, $db_config->{input};
-	} else {
+
+	my @inputs = force_array( $db_config->{input}, sub {
 		$log->info("database $database doesn't have inputs defined");
-	}
+	} );
 
 	foreach my $input (@inputs) {
 
@@ -499,6 +509,16 @@
 		}
 
 
+		# setup input name for all output filters
+		foreach my $out ( @output_modules ) {
+			if ( $out->can('input') ) {
+				$out->input( $input_name );
+			} else {
+				$log->warn("output filter ",ref($out)," doesn't support input name");
+			}
+		}
+
+
 		foreach my $pos ( 0 ... $input_db->size ) {
 
 			my $row = $input_db->fetch || next;
@@ -523,7 +543,7 @@
 				next;	# validation doesn't create any output
 			}
 
-			my $ds = WebPAC::Normalize::data_structure(
+			if ( my $ds = WebPAC::Normalize::data_structure(
 				row => $row,
 				rules => $rules,
 				lookup => $lookup_hash,
@@ -537,41 +557,45 @@
 						id => $mfn,
 					);
 				},
-			);
+			) ) { # if
 
-			$log->debug("ds = ", sub { dump($ds) }) if ($ds);
+				$log->debug("ds = ", sub { dump($ds) });
 
-			$store->save_ds(
-				database => $database,
-				input => $input_name,
-				id => $mfn,
-				ds => $ds,
-			) if ($ds && !$stats);
-
-			$indexer->add(
-				id => "${input_name}/${mfn}",
-				ds => $ds,
-				type => $config->get($indexer_config)->{type},
-			) if ($indexer && $ds);
-
-			if ($marc) {
-				my $i = 0;
-
-				while (my $fields = WebPAC::Normalize::_get_marc_fields( fetch_next => 1 ) ) {
-					$marc->add(
-						id => $mfn . ( $i ? "/$i" : '' ),
-						fields => $fields,
-						leader => WebPAC::Normalize::_get_marc_leader(),
-						row => $row,
-					);
-					$i++;
+				$store->save_ds(
+					database => $database,
+					input => $input_name,
+					id => $mfn,
+					ds => $ds,
+				) if !$stats;
+
+				$indexer->add(
+					id => "${input_name}/${mfn}",
+					ds => $ds,
+					type => $config->get($indexer_config)->{type},
+				) if $indexer;
+
+				if ($marc) {
+					my $i = 0;
+
+					while (my $fields = WebPAC::Normalize::_get_marc_fields( fetch_next => 1 ) ) {
+						$marc->add(
+							id => $mfn . ( $i ? "/$i" : '' ),
+							fields => $fields,
+							leader => WebPAC::Normalize::_get_marc_leader(),
+							row => $row,
+						);
+						$i++;
+					}
+
+					$log->info("Created $i instances of MFN $mfn\n") if ($i > 1);
 				}
 
-				$log->info("Created $i instances of MFN $mfn\n") if ($i > 1);
-			}
+				foreach my $out ( @output_modules ) {
+					$out->add( $mfn, $ds ) if $out->can('add');
+				}
 
-			foreach my $out ( @outputs ) {
-				$out->add( $mfn, $ds ) if $out->can('add');
+			} else {
+				$log->warn("record $pos didn't produce any output after normalization rules!");
 			}
 
 		}
@@ -604,7 +628,7 @@
 
 	eval { $indexer->finish } if ($indexer && $indexer->can('finish'));
 
-	foreach my $out ( @outputs ) {
+	foreach my $out ( @output_modules ) {
 		$out->finish if $out->can('finish');
 	}