]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiMailSync.pm
www_stream: note existence of IMAP and NNTP URLs
[public-inbox.git] / lib / PublicInbox / LeiMailSync.pm
index ec05404ad004fc9740e7aec781fd9ddc96efd539..5a10c127c1f8e2db1c080f20530ed90385c3d461 100644 (file)
@@ -32,9 +32,7 @@ sub new {
        bless { filename => $f, fmap => {} }, $cls;
 }
 
-sub lms_commit { delete($_[0]->{dbh})->commit }
-
-sub lms_begin { ($_[0]->{dbh} //= dbh_new($_[0], 1))->begin_work };
+sub lms_write_prepare { ($_[0]->{dbh} //= dbh_new($_[0], 1)) };
 
 sub create_tables {
        my ($dbh) = @_;
@@ -79,8 +77,8 @@ sub fid_for {
        my ($fid) = $dbh->selectrow_array($sel, undef, $folder);
        return $fid if defined $fid;
 
+       # caller had trailing slash (LeiToMail)
        if ($folder =~ s!\A((?:maildir|mh):.*?)/+\z!$1!i) {
-               warn "folder: $folder/ had trailing slash in arg\n";
                ($fid) = $dbh->selectrow_array($sel, undef, $folder);
                if (defined $fid) {
                        $dbh->do(<<EOM, undef, $folder, $fid) if $rw;
@@ -118,7 +116,7 @@ EOM
 }
 
 sub set_src {
-       my ($self, $oidhex, $folder, $id) = @_;
+       my ($self, $oidbin, $folder, $id) = @_;
        my $fid = $self->{fmap}->{$folder} //= fid_for($self, $folder, 1);
        my $sth;
        if (ref($id)) { # scalar name
@@ -131,7 +129,7 @@ INSERT OR IGNORE INTO blob2name (oidbin, fid, name) VALUES (?, ?, ?)
 INSERT OR IGNORE INTO blob2num (oidbin, fid, uid) VALUES (?, ?, ?)
 
        }
-       $sth->execute(pack('H*', $oidhex), $fid, $id);
+       $sth->execute($oidbin, $fid, $id);
 }
 
 sub clear_src {
@@ -158,7 +156,13 @@ sub mv_src {
        my $sth = $self->{dbh}->prepare_cached(<<'');
 UPDATE blob2name SET name = ? WHERE fid = ? AND oidbin = ? AND name = ?
 
-       $sth->execute($newbn, $fid, $oidbin, $$id);
+       my $nr = $sth->execute($newbn, $fid, $oidbin, $$id);
+       if ($nr == 0) { # may race with a clear_src, ensure new value exists
+               $sth = $self->{dbh}->prepare_cached(<<'');
+INSERT OR IGNORE INTO blob2name (oidbin, fid, name) VALUES (?, ?, ?)
+
+               $sth->execute($oidbin, $fid, $newbn);
+       }
 }
 
 # read-only, iterates every oidbin + UID or name for a given folder
@@ -206,16 +210,16 @@ SELECT $op(uid) FROM blob2num WHERE fid = ?
 
 # returns a { location => [ list-of-ids-or-names ] } mapping
 sub locations_for {
-       my ($self, $oidhex) = @_;
+       my ($self, $oidbin) = @_;
        my ($fid, $sth, $id, %fid2id);
        my $dbh = $self->{dbh} //= dbh_new($self);
        $sth = $dbh->prepare('SELECT fid,uid FROM blob2num WHERE oidbin = ?');
-       $sth->execute(pack('H*', $oidhex));
+       $sth->execute($oidbin);
        while (my ($fid, $uid) = $sth->fetchrow_array) {
                push @{$fid2id{$fid}}, $uid;
        }
        $sth = $dbh->prepare('SELECT fid,name FROM blob2name WHERE oidbin = ?');
-       $sth->execute(pack('H*', $oidhex));
+       $sth->execute($oidbin);
        while (my ($fid, $name) = $sth->fetchrow_array) {
                push @{$fid2id{$fid}}, $name;
        }
@@ -225,6 +229,7 @@ sub locations_for {
                $sth->execute($fid);
                my ($loc) = $sth->fetchrow_array;
                unless (defined $loc) {
+                       my $oidhex = unpack('H*', $oidbin);
                        warn "E: fid=$fid for $oidhex unknown:\n", map {
                                        'E: '.(ref() ? $$_ : "#$_")."\n";
                                } @$ids;
@@ -315,6 +320,41 @@ sub match_imap_url {
                        "E: `$url' is ambiguous:\n\t".join("\n\t", @match)."\n";
 }
 
+# returns undef on failure, number on success
+sub group2folders {
+       my ($self, $lei, $all, $folders) = @_;
+       return $lei->fail(<<EOM) if @$folders;
+--all= not compatible with @$folders on command-line
+EOM
+       my %x = map { $_ => $_ } split(/,/, $all);
+       my @ok = grep(defined, delete(@x{qw(local remote), ''}));
+       push(@ok, '') if $all eq '';
+       my @no = keys %x;
+       if (@no) {
+               @no = (join(',', @no));
+               return $lei->fail(<<EOM);
+--all=@no not accepted (must be `local' and/or `remote')
+EOM
+       }
+       my (%seen, @inc);
+       my @all = $self->folders;
+       for my $ok (@ok) {
+               if ($ok eq 'local') {
+                       @inc = grep(!m!\A[a-z0-9\+]+://!i, @all);
+               } elsif ($ok eq 'remote') {
+                       @inc = grep(m!\A[a-z0-9\+]+://!i, @all);
+               } elsif ($ok ne '') {
+                       return $lei->fail("--all=$all not understood");
+               } else {
+                       @inc = @all;
+               }
+               push(@$folders, (grep { !$seen{$_}++ } @inc));
+       }
+       scalar(@$folders) || $lei->fail(<<EOM);
+no --mail-sync folders known to lei
+EOM
+}
+
 # map CLI args to folder table entries, returns undef on failure
 sub arg2folder {
        my ($self, $lei, $folders) = @_;
@@ -360,13 +400,34 @@ EOF
 
 sub forget_folder {
        my ($self, $folder) = @_;
-       my ($fid, $sth);
-       $fid = delete($self->{fmap}->{$folder}) //
+       my $fid = delete($self->{fmap}->{$folder}) //
                fid_for($self, $folder) // return;
-       my $dbh = $self->{dbh};
-       $dbh->do('DELETE FROM blob2name WHERE fid = ?', undef, $fid);
-       $dbh->do('DELETE FROM blob2num WHERE fid = ?', undef, $fid);
-       $dbh->do('DELETE FROM folders WHERE fid = ?', undef, $fid);
+       for my $t (qw(blob2name blob2num folders)) {
+               $self->{dbh}->do("DELETE FROM $t WHERE fid = ?", undef, $fid);
+       }
+}
+
+# only used for changing canonicalization errors
+sub rename_folder {
+       my ($self, $old, $new) = @_;
+       my $ofid = delete($self->{fmap}->{$old}) //
+               fid_for($self, $old) // return;
+       eval {
+               $self->{dbh}->do(<<EOM, undef, $new, $ofid);
+UPDATE folders SET loc = ? WHERE fid = ?
+EOM
+       };
+       if ($@ =~ /\bunique\b/i) {
+               my $nfid = $self->{fmap}->{$new} // fid_for($self, $new);
+               for my $t (qw(blob2name blob2num)) {
+                       $self->{dbh}->do(<<EOM, undef, $nfid, $ofid);
+UPDATE OR REPLACE $t SET fid = ? WHERE fid = ?
+EOM
+               }
+               $self->{dbh}->do(<<EOM, undef, $ofid);
+DELETE FROM folders WHERE fid = ?
+EOM
+       }
 }
 
 sub imap_oidbin ($$$) {
@@ -405,15 +466,4 @@ sub imap_oid {
        $oidbin ? unpack('H*', $oidbin) : undef;
 }
 
-# FIXME: something with "lei <up|q>" is causing uncommitted transaction
-# warnings, not sure what...
-sub DESTROY {
-       my ($self) = @_;
-       my $dbh = delete($self->{dbh}) or return;
-       return if $dbh->{ReadOnly};
-       use Carp;
-       undef $dbh;
-       warn "BUG $$ $0 $self {dbh} OPEN ppid=".getppid.' '.Carp::longmess();
-}
-
 1;