]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/IMAP.pm
imap: remove non-UID SEARCH for now
[public-inbox.git] / lib / PublicInbox / IMAP.pm
index 1ba3a3ff1bb7cbb883cb1b85f462ab77ee94aa9c..7efe538764683deaec7400a9b9e2eff892d80a6c 100644 (file)
@@ -986,21 +986,14 @@ sub parse_date ($) { # 02-Oct-1993
        timegm(0, 0, 0, $dd, $mm, $yyyy);
 }
 
-sub msn_convert ($$) {
-       my ($self, $uids) = @_;
-       my $adj = $self->{uid_base};
-       $_ -= $adj for @$uids;
-}
-
 sub search_uid_range { # long_response
-       my ($self, $tag, $sql, $range_info, $want_msn) = @_;
+       my ($self, $tag, $sql, $range_info) = @_;
        my $uids = [];
        if (defined(my $err = refill_uids($self, $uids, $range_info, $sql))) {
                $err ||= 'OK Search done';
                $self->write("\r\n$tag $err\r\n");
                return;
        }
-       msn_convert($self, $uids) if $want_msn;
        $self->msg_more(join(' ', '', @$uids));
        1; # more
 }
@@ -1162,46 +1155,38 @@ sub refill_xap ($$$$) {
 }
 
 sub search_xap_range { # long_response
-       my ($self, $tag, $q, $range_info, $want_msn) = @_;
+       my ($self, $tag, $q, $range_info) = @_;
        my $uids = [];
        if (defined(my $err = refill_xap($self, $uids, $range_info, $q))) {
                $err ||= 'OK Search done';
                $self->write("\r\n$tag $err\r\n");
                return;
        }
-       msn_convert($self, $uids) if $want_msn;
        $self->msg_more(join(' ', '', @$uids));
        1; # more
 }
 
-sub search_common {
-       my ($self, $tag, $rest, $want_msn) = @_;
+sub cmd_uid_search ($$$;) {
+       my ($self, $tag) = splice(@_, 0, 2);
        my $ibx = $self->{ibx} or return "$tag BAD No mailbox selected\r\n";
-       my $q = parse_query($self, $rest);
+       my $q = parse_query($self, \@_);
        return "$tag $q\r\n" if !ref($q);
        my ($sql, $range_info) = delete @$q{qw(sql range_info)};
        if (!scalar(keys %$q)) { # overview.sqlite3
                $self->msg_more('* SEARCH');
                long_response($self, \&search_uid_range,
-                               $tag, $sql, $range_info, $want_msn);
+                               $tag, $sql, $range_info);
        } elsif ($q = $q->{xap}) {
                $self->msg_more('* SEARCH');
                long_response($self, \&search_xap_range,
-                               $tag, $q, $range_info, $want_msn);
+                               $tag, $q, $range_info);
        } else {
                "$tag BAD Error\r\n";
        }
 }
 
-sub cmd_uid_search ($$$;) {
-       my ($self, $tag) = splice(@_, 0, 2);
-       search_common($self, $tag, \@_);
-}
-
-sub cmd_search ($$$;) {
-       my ($self, $tag) = splice(@_, 0, 2);
-       search_common($self, $tag, \@_, 1);
-}
+# note: MSN SEARCH is NOT supported.  Do any widely-used MUAs
+# rely on MSNs from SEARCH results?  Let us know at meta@public-inbox.org
 
 sub args_ok ($$) { # duplicated from PublicInbox::NNTP
        my ($cb, $argc) = @_;