--- trunk/WebPac.pm	2003/03/15 21:34:36	41
+++ trunk/WebPac.pm	2003/07/08 08:24:13	80
@@ -8,24 +8,31 @@
 use SWISH;
 use Text::Iconv;
 use DBI;
+use Config::IniFiles;
+use Text::Unaccent;
 
 use lib '..';
 use index_DBI;
 use back2html;
 
-# configuration options
-# FIX: they really should go in configuration file!
-my $TEMPLATE_PATH = '/data/webpac/template_html';
-my $CHARSET = 'ISO-8859-2';
-my $SWISH = '/usr/bin/swish-e';
-my $INDEX = '/data/webpac/index/isis.index';
-my $MAX_HITS = 500;
-my $ON_PAGE = 10;
+
+# read global.conf configuration
+my $cfg_global = new Config::IniFiles( -file => '../global.conf' ) || die "can't open 'global.conf'";
+
+# configuration options from global.conf
+my $TEMPLATE_PATH = $cfg_global->val('webpac', 'template_html') || die "need template_html in global.conf, section webpac";
+my $CHARSET = $cfg_global->val('webpac', 'charset') || 'ISO-8859-1';
+my $SWISH = $cfg_global->val('webpac', 'swish') || '/usr/bin/swish-e';
+my $INDEX = $cfg_global->val('webpac', 'index') || die "need index in global.conf, section webpac";
+my $MAX_HITS = $cfg_global->val('webpac', 'max_hits') || 0;
+my $ON_PAGE =$cfg_global->val('webpac', 'on_page') || 10;
+
 
 Text::Iconv->raise_error(0);     # Conversion errors raise exceptions
 
 my $from_utf8 = Text::Iconv->new('UTF8', $CHARSET);
 
+
 sub setup {
 	my $self = shift;
 	$self->tmpl_path($TEMPLATE_PATH);
@@ -67,37 +74,56 @@
 
 	my @s_arr;	# all queries are located here
 
-	for(my $i = 1; $i <=10; $i++) {
+	my @path_arr = $q->param('path');
+	my $full = $q->param('full');
+
+	for(my $i = 1; $i <=30; $i++) {
 
 		return show_index($self, $i) if ($q->param("f".$i."_index"));
-		next if (! $q->param("f$i"));
 		next if (! $q->param("v$i"));
+		next if (! $q->param("f$i"));
 
 		# re-write query from +/- to and/and not
-		my $s;
-		my $search = $q->param("v$i");
-		while ($search =~ s/\s*("[^"]+")\s*/ /) {
-			$s .= "$1 ";
-		}
-		$search =~ s/^\s+//;
-		$search =~ s/\s+$//;
+		my @param_vals = $q->param("v$i");
+		my @swish_q;
+		while (my $search = shift @param_vals) {
+			my $s;
+			# remove accents
+			$search = unac_string($CHARSET,$search);
+			while ($search =~ s/\s*("[^"]+")\s*/ /) {
+				$s .= "$1 ";
+			}
+			$search =~ s/^\s+//;
+			$search =~ s/\s+$//;
 
-		foreach (split(/\s+/,$search)) {
-			if (m/^([+-])(\S+)/) {
-				$s.= ($s) ? "and " : "";
-				$s.="not " if ($1 eq "-");
-				$s.="$2* ";
-			} else {
-				$s.="$_* ";
+			foreach (split(/\s+/,$search)) {
+				if (m/^([+-])(\S+)/) {
+					$s.= ($s) ? "and " : "";
+					$s.="not " if ($1 eq "-");
+					$s.="$2* ";
+				} elsif (m/(and|or|not)/i) {
+					$s.="$_ ";
+				} else {
+					$s.="$_* ";
+				}
 			}
+			$s =~ s/\*+/*/g;
+			push @swish_q,$s;
 		}
-		$s =~ s/\*+/*/g;
-
-		push @s_arr,$q->param("f$i")."_swish=($s)";
+		# FIXME default operator for multi-value fields is or. There is
+		# no way to change it, except here for now. Is there need?
+		push @s_arr, $q->param("f$i")."_swish=(".join(" or ",@swish_q).")";
 	}
 
 	my $tmpl = $self->load_tmpl('results.html');
 
+	sub esc_html {
+		my $html = shift;
+		$html =~ s/</&lt;/g;
+		$html =~ s/>/&gt;/g;
+		return $html;
+	}
+
 	# call swish
 	my $sh = SWISH->connect('Fork',
 		prog     => $SWISH,
@@ -109,21 +135,32 @@
 			push @swish_results, {
 				nr => ($#swish_results + 2),
 				path => $hit->swishdocpath,
-				headline => $from_utf8->convert($hit->headline),
+				headline => esc_html($from_utf8->convert($hit->headline)),
 				html => back2html($from_utf8->convert($hit->html)),
 				rank => $hit->swishrank };
 
 		},
 		#startnum => 0,
-		maxhits => $MAX_HITS,
+		maxhits => $MAX_HITS
 	);
 
 	die $SWISH::errstr unless $sh;
+	# construct swish query
+	my $sw_q = join(" and ",@s_arr);
+	if (@path_arr) {
+		$sw_q .= "and (swishdocpath=\"";
+		$sw_q .= join("\" or swishdocpath=\"",@path_arr);
+		$sw_q .= "\")";
+		$tmpl->param('full',1);	# show full records
+	}
 
-	my $hits = $sh->query(join(" and ",@s_arr)) || 0;	# FIX: and/or
+	my $hits = $sh->query($sw_q);
 
 	$tmpl->param('hits',$hits);
-	$tmpl->param('search',join(" and ",@s_arr));
+	$tmpl->param('search',$sw_q);
+
+	$tmpl->param('PAGER_offset',$q->param("PAGER_offset") || 0);
+	$tmpl->param('last_PAGER_offset',$q->param("last_PAGER_offset") || 0);
 
 	# create a Pager object
 	my $pager = HTML::Pager->new(
@@ -177,7 +214,12 @@
 
 	my $html;
 
-	my $index = new index_DBI();
+	my $index = new index_DBI(
+		$cfg_global->val('global', 'dbi_dbd'),
+		$cfg_global->val('global', 'dbi_dsn'),
+		$cfg_global->val('global', 'dbi_user'),
+		$cfg_global->val('global', 'dbi_passwd') || ''
+	);
 
 	my $total = $index->check($field);
 	if (! $total) {
@@ -192,7 +234,7 @@
 	$tmpl->param('limit',$limit);
 	$tmpl->param('total',$total);
 
-# FIX: I should set offset and leave out limit from fetch!!
+# FIXME I should set offset and leave out limit from fetch!!
 #	if (! $q->param("PAGER_offset") {
 #		$q->param("Pager_offet)
 #	}