]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/SearchMsg.pm
search: remove unnecessary abstractions and functionality
[public-inbox.git] / lib / PublicInbox / SearchMsg.pm
index 477ffff773610d1faaa869187c860624b440e0a5..a19d45db8cc87531753eb0bc032bc38253e358d1 100644 (file)
@@ -7,20 +7,14 @@ package PublicInbox::SearchMsg;
 use strict;
 use warnings;
 use Search::Xapian;
-use Email::Address qw//;
-use POSIX qw//;
 use Date::Parse qw/str2time/;
 use PublicInbox::MID qw/mid_clean/;
-use Encode qw/find_encoding/;
-my $enc_utf8 = find_encoding('UTF-8');
-our $PFX2TERM_RE = undef;
-use constant EPOCH_822 => 'Thu, 01 Jan 1970 00:00:00 +0000';
-use POSIX qw(strftime);
+use PublicInbox::Address;
 
 sub new {
        my ($class, $mime) = @_;
        my $doc = Search::Xapian::Document->new;
-       $doc->add_term(PublicInbox::Search::xpfx('type') . 'mail');
+       $doc->add_term('T' . 'mail');
 
        bless { type => 'mail', doc => $doc, mime => $mime }, $class;
 }
@@ -37,10 +31,10 @@ sub get_val ($$) {
 
 sub load_doc {
        my ($class, $doc) = @_;
-       my $data = $doc->get_data;
+       my $data = $doc->get_data or return;
        my $ts = get_val($doc, &PublicInbox::Search::TS);
-       $data = $enc_utf8->decode($data);
-       my ($subj, $from, $refs, $to, $cc) = split(/\n/, $data);
+       utf8::decode($data);
+       my ($subj, $from, $refs, $to, $cc, $blob) = split(/\n/, $data);
        bless {
                doc => $doc,
                subject => $subj,
@@ -49,6 +43,7 @@ sub load_doc {
                references => $refs,
                to => $to,
                cc => $cc,
+               blob => $blob,
        }, $class;
 }
 
@@ -65,7 +60,8 @@ sub __hdr ($$) {
        my $mime = $self->{mime} or return;
        $val = $mime->header($field);
        $val = '' unless defined $val;
-       $val =~ tr/\t\r\n/ /;
+       $val =~ tr/\n/ /;
+       $val =~ tr/\r//d;
        $self->{$field} = $val;
 }
 
@@ -73,22 +69,26 @@ sub subject ($) { __hdr($_[0], 'subject') }
 sub to ($) { __hdr($_[0], 'to') }
 sub cc ($) { __hdr($_[0], 'cc') }
 
+# no strftime, that is locale-dependent and not for RFC822
+my @DoW = qw(Sun Mon Tue Wed Thu Fri Sat);
+my @MoY = qw(Jan Feb Mar Apr May Jun Jul Aug Sep Oct Nov Dec);
+
 sub date ($) {
        my ($self) = @_;
-       my $date = __hdr($self, 'date');
-       return $date if defined $date;
        my $ts = $self->{ts};
        return unless defined $ts;
-       $self->{date} = strftime('%a, %d %b %Y %T %z', gmtime($ts));
+       my ($sec, $min, $hour, $mday, $mon, $year, $wday) = gmtime($ts);
+       "$DoW[$wday], " . sprintf("%02d $MoY[$mon] %04d %02d:%02d:%02d +0000",
+                               $mday, $year+1900, $hour, $min, $sec);
+
 }
 
 sub from ($) {
        my ($self) = @_;
        my $from = __hdr($self, 'from');
        if (defined $from && !defined $self->{from_name}) {
-               $from =~ tr/\t\r\n/ /;
-               my @from = Email::Address->parse($from);
-               $self->{from_name} = $from[0]->name;
+               my @n = PublicInbox::Address::names($from);
+               $self->{from_name} = join(', ', @n);
        }
        $from;
 }
@@ -103,13 +103,15 @@ sub from_name {
 
 sub ts {
        my ($self) = @_;
-       $self->{ts} ||= eval { str2time($self->mime->header('Date')) } || 0;
+       $self->{ts} ||= eval { str2time($self->{mime}->header('Date')) } || 0;
 }
 
 sub to_doc_data {
-       my ($self) = @_;
-       join("\n", $self->subject, $self->from, $self->references,
-               $self->to, $self->cc);
+       my ($self, $blob) = @_;
+       my @rows = ($self->subject, $self->from, $self->references,
+                       $self->to, $self->cc);
+       push @rows, $blob if defined $blob;
+       join("\n", @rows);
 }
 
 sub references {
@@ -118,61 +120,17 @@ sub references {
        defined $x ? $x : '';
 }
 
-sub ensure_metadata {
-       my ($self) = @_;
+sub _get_term_val ($$$) {
+       my ($self, $pfx, $re) = @_;
        my $doc = $self->{doc};
        my $end = $doc->termlist_end;
-
-       unless (defined $PFX2TERM_RE) {
-               my $or = join('|', keys %PublicInbox::Search::PFX2TERM_RMAP);
-               $PFX2TERM_RE = qr/\A($or)/;
-       }
-
-       while (my ($pfx, $field) = each %PublicInbox::Search::PFX2TERM_RMAP) {
-               # ideally we'd move this out of the loop:
-               my $i = $doc->termlist_begin;
-
-               $i->skip_to($pfx);
-               if ($i != $end) {
-                       my $val = $i->get_termname;
-
-                       if ($val =~ s/$PFX2TERM_RE//o) {
-                               $self->{$field} = $val;
-                       }
-               }
+       my $i = $doc->termlist_begin;
+       $i->skip_to($pfx);
+       if ($i != $end) {
+               my $val = $i->get_termname;
+               $val =~ s/$re// and return $val;
        }
-}
-
-# for threading only
-sub mini_mime {
-       my ($self) = @_;
-       $self->ensure_metadata;
-       my @hs = (
-               'Subject' => $self->subject,
-               'X-PI-From' => $self->from_name,
-       );
-
-       my @h = (
-               # prevent Email::Simple::Creator from running,
-               # this header is useless for threading as we use X-PI-TS
-               # for sorting and display:
-               'Date' => EPOCH_822,
-               'Message-ID' => "<$self->{mid}>",
-               'X-PI-TS' => $self->ts,
-       );
-       if (my $refs = $self->{references}) {
-               push @h, References => $refs;
-       }
-       my $mime = Email::MIME->create(header_str => \@hs, header => \@h);
-       my $h = $mime->header_obj;
-
-       # set these headers manually since Encode::encode('MIME-Q', ...)
-       # will add spaces to long values when using header_str above.
-
-       # drop useless headers Email::MIME set for us
-       $h->header_set('Date');
-       $h->header_set('MIME-Version');
-       $mime;
+       undef;
 }
 
 sub mid ($;$) {
@@ -183,47 +141,26 @@ sub mid ($;$) {
        } elsif (my $rv = $self->{mid}) {
                $rv;
        } else {
-               $self->ensure_metadata; # needed for ghosts
-               $self->{mid} ||= $self->_extract_mid;
+               $self->{mid} = _get_term_val($self, 'Q', qr/\AQ/) ||
+                               $self->_extract_mid;
        }
 }
 
-sub _extract_mid { mid_clean(mid_mime($_[0]->mime)) }
-
-sub mime {
-       my ($self, $mime) = @_;
-       if (defined $mime) {
-               $self->{mime} = $mime;
-       } else {
-               # TODO load from git
-               $self->{mime};
-       }
-}
-
-sub doc_id {
-       my ($self, $doc_id) = @_;
-       if (defined $doc_id) {
-               $self->{doc_id} = $doc_id;
-       } else {
-               # TODO load from xapian
-               $self->{doc_id};
-       }
-}
+sub _extract_mid { mid_clean(mid_mime($_[0]->{mime})) }
 
 sub thread_id {
        my ($self) = @_;
        my $tid = $self->{thread};
        return $tid if defined $tid;
-       $self->ensure_metadata;
-       $self->{thread};
+       $self->{thread} = _get_term_val($self, 'G', qr/\AG/); # *G*roup
 }
 
+# XXX: consider removing this, we can phrase match subject
 sub path {
        my ($self) = @_;
        my $path = $self->{path};
        return $path if defined $path;
-       $self->ensure_metadata;
-       $self->{path};
+       $self->{path} = _get_term_val($self, 'XPATH', qr/\AXPATH/); # path
 }
 
 1;