work on Koha search with webpac
[webpac2] / vhost / webpac2.cgi
index f09b0e4..2abdb4c 100755 (executable)
@@ -11,9 +11,11 @@ use Data::Page;
 use Data::Dump qw/dump/;
 use SWISH::API;
 use JSON;
+use Text::Unaccent::PurePerl qw/unac_string/;
+use HTML::FillInForm::Lite;
 
 my $range_around = 5;
-my $entries_per_page = 30;
+my @entries_per_page = ( 30, 50, 100, 500 );
 my $debug = param('debug');
 
 print header(
@@ -109,7 +111,12 @@ my $db = $config->{databases}->{$database};
 sub read_config_txt {
        my ( $file ) = @_;
        my $input;
-       foreach ( split(/[\n\r]+/, read_file( "$dir/$path/$path-$file.txt" ) ) ) {
+       my $path ="$dir/$path/$path-$file.txt";
+       if ( ! -e $path ) {
+               warn "missing $path";
+               return;
+       }
+       foreach ( split(/[\n\r]+/, read_file( $path ) ) ) {
                my ( $val,$label ) = split(/\s*\t\s*/,$_,2);
                push @{ $input->{ '-values' } }, $val;
                                $input->{ '-labels' }->{$val} = $label;
@@ -120,19 +127,22 @@ sub read_config_txt {
 my $attr_labels    = read_config_txt 'labels';
 my $attr_operators = read_config_txt 'operators';
 
-my @attr = @{ $attr_labels->{'-values'} };
+my @attr = @{ $attr_labels->{'-values'} } if $attr_labels;
 @attr = keys %{ $stats->{attr} } unless @attr;
 
 
 warn dump( $attr_labels, $attr_operators );
 
 my $only_input;
+my $inputs_available = 0;
 
 foreach ( @{ $db->{input} } ) {
        my $input = $_->{name} || die "no name in ",dump( $_ );
+       next unless defined $stats->{input}->{$input}; # skip inputs without data
        if ( ! $only_input->{'-labels'}->{$input} ) {
                push @{ $only_input->{'-values'} }, $input;
                        $only_input->{'-labels'}->{$input} = $_->{description} || $input;
+               $inputs_available++;
        }
 }
 
@@ -143,27 +153,38 @@ push @style, "../../$path/$path.css" if -e "$dir/$path/$path.css";
 dump_yaml( 'style', \@style );
 
 sub search_form {
+
+       my $form_html = "$dir/$path/$path-search.html";
+       if ( -e $form_html ) {
+               my $html = read_file( $form_html );
+               my $q = CGI->new();
+               my $h = HTML::FillInForm::Lite->new();
+               return $h->fill(\$html, $q);
+       }
+
        qq|<a name="form"></a>|,
        start_form( -action => self_url( query => 0 ) ),
-               radio_group(
+               checkbox_group(
                        -name => 'attr',
                        %$attr_labels,
 #                      -linebreak => 0,
                ),
                textfield( -name => 'search' ),
-               popup_menu( -name => 'attr_operator', %$attr_operators ),
-               submit,
-               hidden( -name => 'entries_per_page', -default => $entries_per_page ),
+               $attr_operators ? popup_menu( -name => 'attr_operator', %$attr_operators ) : '',
+               submit( -value => 'Search' ),
+#              hidden( -name => 'entries_per_page', -default => $entries_per_page ),
+               popup_menu( -name => 'entries_per_page', -values => [ @entries_per_page ], -title => 'entries per page' ),
                # we need current_page fixed at 1 so that every submit through form will reset it
                qq|<input type=hidden name=current_page value=1 >|,
                checkbox( -name => 'debug', -default => 0 ), # FIXME hidden?
                qq|<div id=inputs>|,
-               h2( 'Select input' ),
+               $inputs_available > 1 ?
+               h2( 'Select input' ) .
                checkbox_group(
                        -name => 'only_input',
                        %$only_input,
                        -linebreak=> 'true',
-               ),
+               ) : '',
                qq|</div>|,
                end_form,
        ;
@@ -181,6 +202,8 @@ print
 
 if ( my $search = param('search') ) {
 
+       $search = unac_string( $search );
+
        print qq|
                <a href="#form" class="skip" title="skip to search form">#</a>
                <div id="results">
@@ -190,43 +213,53 @@ if ( my $search = param('search') ) {
        $swish->abort_last_error if $swish->Error;
 
        my @search = (); 
+       my @attrs = param('attr');
+       my $op = param('attr_operator');
+
        if ( $search =~ m{(=|"|AND|OR)} ) {
                push @search, $search;
-       } elsif ( my $op = param('attr_operator') ) {
-               my $attr = param('attr');
-               my $v = $search;
-               $v =~ s/^\s+//;
-               warn "-- v: $v\n";
-               sub rewrite {
-                       my ( $whitespace, $v ) = @_;
-                       warn "## filter $op $whitespace $v\n";
-                       my $template = $op;
-                          $template =~ s{Q}{$v};
-                       $whitespace = " AND " if $whitespace;
-
-                       return
-                               $whitespace .
-                               $attr . '="' . $template . '"';
-                               ;
-               };
-               if ( $op =~ m{\s} ) {
-                       my $template = $op;
-                          $template =~ s{Q}{$v};
-                       $v = $attr . '="' . $template . '"';
-               } else {
-                       $v =~ s{(\s*)(\S+)}{rewrite($1,$2)}ge;
+       } elsif ( @attrs ) {
+
+               $op ||= 'Q*';
+               my @or;
+               foreach my $attr ( @attrs ) {
+                       my $v = $search;
+                       $v =~ s/^\s+//;
+                       warn "-- v: $v\n";
+                       sub rewrite {
+                               my ( $attr, $whitespace, $v ) = @_;
+                               warn "## filter $op $whitespace $v\n";
+                               my $template = $op;
+                                  $template =~ s{Q}{$v};
+                               $whitespace = " AND " if $whitespace;
+
+                               return
+                                       $whitespace .
+                                       $attr . '="' . $template . '"';
+                                       ;
+                       };
+                       if ( $op =~ m{\s} ) {
+                               my $template = $op;
+                                  $template =~ s{Q}{$v};
+                               $v = $attr . '="' . $template . '"';
+                       } else {
+                               $v =~ s{(\s*)(\S+)}{rewrite($attr,$1,$2)}ge;
+                       }
+
+                       push @or, $v;
+               
                }
+               push @search, '(' . join(') OR (', @or) . ')';
 
-               push @search, $v;
-       
-               my @only_input = param('only_input');
-               push @search, '((' . join(') OR (', map { "input=\"$_\"" } @only_input) . '))' if @only_input;
        } else {
                push @search, "all=\"$search\"";
        }
 
        my $q = '(' . join(') AND (', @search) . ')';
-       $q =~ s{\(\((.+)\)\)}{($1)};
+
+       my @only_input = param('only_input');
+       $q .= ' AND ((' . join(') OR (', map { "input=\"$_\"" } @only_input) . '))' if @only_input;
+
        warn "# query: $q\n";
        my $swish_results = $swish->query( $q );
 
@@ -262,9 +295,14 @@ if ( my $search = param('search') ) {
 
                while ( my $result = $swish_results->next_result ) {
 
-                       my $data = from_json $result->property('data');
-
+                       my $data = $result->property('data');
                        dump_yaml( 'data', $data );
+                       # FIXME if we produce valid json we shouldn't need eval here!
+                       eval { $data = from_json( $data, {utf8 => 1} ); };
+                       if ( $@ ) {
+                               warn "ERROR: $@ from ",dump( $data );
+                               next;
+                       }
 
                        my $li_class = '';
                        $li_class = qq| class="z"| if $nr % 2 == 0;
@@ -273,11 +311,13 @@ if ( my $search = param('search') ) {
                                next unless defined $data->{$attr};
                                my $v = $data->{$attr};
                                if ( $html_markup && ! $html_markup_skip->{$attr} ) {
-                                       eval "\$v = $html_markup->$attr( \$v );";
+                                       eval "\$v = $html_markup->$attr( \$v, \$data );";
                                        if ( $@ ) {
                                                warn "disable html markup for $attr: $@";
                                                $html_markup_skip->{$attr} = $@;
                                        }
+                               } else {
+                                       $v =~ s{(http://\S+)}{<a href="$1">$1</a>};
                                }
                                my $label = $attr_labels->{'-labels'}->{$attr} || $attr;
                                print qq|<div><label>$label</label><span class=$attr>$v</span></div>\n|;