]> Sergey Matveev's repositories - public-inbox.git/commitdiff
lei: fix read/write IMAP access
authorEric Wong <e@80x24.org>
Fri, 3 Sep 2021 08:54:27 +0000 (08:54 +0000)
committerEric Wong <e@80x24.org>
Fri, 3 Sep 2021 08:57:36 +0000 (08:57 +0000)
xt/net_writer-imap.t was completely broken in recent months and
I completely forgot this test.  net->add_url still only accepts
bare scalars (and not scalar refs), so we must set that up
properly.  Furthermore, our changes to do FLAGS-only
synchronization in lei of old messages was causing us to not
handle FLAGS properly for the test.

lib/PublicInbox/LeiToMail.pm
lib/PublicInbox/NetReader.pm
lib/PublicInbox/NetWriter.pm
lib/PublicInbox/Watch.pm

index be6e178fe2c3019575afa815ba4faeac3a61c80f..6e102a1de0d4de4b5dc144e48307521e390cd393 100644 (file)
@@ -406,7 +406,7 @@ sub new {
                my $net = PublicInbox::NetWriter->new;
                $net->{quiet} = $lei->{opt}->{quiet};
                my $uri = PublicInbox::URIimap->new($dst)->canonical;
-               $net->add_url($uri);
+               $net->add_url($$uri);
                my $err = $net->errors($lei);
                return $lei->fail($err) if $err;
                $uri->mailbox or return $lei->fail("No mailbox: $dst");
index 23445e7a49f2200834ae5571464c97d6c00b560d..c050c60ffb87192c1c4a3130186c547ac3182878 100644 (file)
@@ -493,6 +493,9 @@ sub perm_fl_ok ($) {
        undef;
 }
 
+# may be overridden in NetWriter or Watch
+sub folder_select { $_[0]->{each_old} ? 'select' : 'examine' }
+
 sub _imap_fetch_all ($$$) {
        my ($self, $mic, $orig_uri) = @_;
        my $sec = uri_section($orig_uri);
@@ -501,7 +504,7 @@ sub _imap_fetch_all ($$$) {
 
        # we need to check for mailbox writability to see if we care about
        # FLAGS from already-imported messages.
-       my $cmd = $self->{each_old} ? 'select' : 'examine';
+       my $cmd = $self->folder_select;
        $mic->$cmd($mbx) or return "E: \U$cmd\E $mbx ($sec) failed: $!";
 
        my ($r_uidval, $r_uidnext, $perm_fl);
index 82288e6bc6391106c0d2123ee7bc1e03c24af489..629a752ae79a3b3b1c1ea8ed91e029f925801268 100644 (file)
@@ -26,6 +26,8 @@ sub imap_append {
                die "APPEND $folder: $@";
 }
 
+sub folder_select { 'select' } # for PublicInbox::NetReader
+
 sub imap_delete_all {
        my ($self, $uri) = @_;
        my $mic = $self->mic_for_folder($uri) or return;
index 86dae91fd65149849966f3b6e7c968be3930289e..482d35c44687c583a6e3726b9bea4a1dc7f978ef 100644 (file)
@@ -682,4 +682,6 @@ EOF
        undef;
 }
 
+sub folder_select { 'select' } # for PublicInbox::NetReader
+
 1;