Changeset 550ef2a for apps


Ignore:
Timestamp:
May 27, 2018 1:46:13 PM (2 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
d128bab
Parents:
5feeffa
Message:

SusiMail?: Remove disk null checks, always non-null

File:
1 edited

Legend:

Unmodified
Added
Removed
  • apps/susimail/src/src/i2p/susi/webmail/MailCache.java

    r5feeffa r550ef2a  
    9090                mails = new Hashtable<String, Mail>();
    9191                disk = new PersistentMailCache(ctx, host, port, user, pass, folderName);
    92                 // TODO Drafts, Sent, Trash
    9392                _context = ctx;
    9493                Folder<String> folder = new Folder<String>();   
     
    374373                                if (rb != null) {
    375374                                        mail.setBody(rb);
    376                                         if (disk != null && disk.saveMail(mail) &&
     375                                        if (disk.saveMail(mail) &&
    377376                                            !Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
    378377                                                mailbox.queueForDeletion(mail.uidl);
     
    440439                        if( headerOnly ) {
    441440                                if(!mail.hasHeader()) {
    442                                         if (disk != null) {
    443                                                 if (disk.getMail(mail, true)) {
    444                                                         if (_log.shouldDebug()) _log.debug("Loaded header from disk cache: " + uidl);
    445                                                         // note that disk loaded the full body if it had it
    446                                                         if (mail.hasBody() &&
    447                                                                 !Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
    448                                                                 // we already have it, send delete
    449                                                                 mailbox.queueForDeletion(mail.uidl);
    450                                                         }
    451                                                         continue;  // found on disk, woo
     441                                        if (disk.getMail(mail, true)) {
     442                                                if (_log.shouldDebug()) _log.debug("Loaded header from disk cache: " + uidl);
     443                                                // note that disk loaded the full body if it had it
     444                                                if (mail.hasBody() &&
     445                                                        !Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
     446                                                        // we already have it, send delete
     447                                                        mailbox.queueForDeletion(mail.uidl);
    452448                                                }
     449                                                continue;  // found on disk, woo
    453450                                        }
    454451                                        POP3Request pr = new POP3Request(mail, true, new MemoryBuffer(1024));
     
    463460                        } else {
    464461                                if(!mail.hasBody()) {
    465                                         if (disk != null) {
    466                                                 if (disk.getMail(mail, false)) {
    467                                                         if (_log.shouldDebug()) _log.debug("Loaded body from disk cache: " + uidl);
    468                                                         // note that disk loaded the full body if it had it
    469                                                         if (!Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
    470                                                                 // we already have it, send delete
    471                                                                 mailbox.queueForDeletion(mail.uidl);
    472                                                         }
    473                                                         continue;  // found on disk, woo
     462                                        if (disk.getMail(mail, false)) {
     463                                                if (_log.shouldDebug()) _log.debug("Loaded body from disk cache: " + uidl);
     464                                                // note that disk loaded the full body if it had it
     465                                                if (!Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
     466                                                        // we already have it, send delete
     467                                                        mailbox.queueForDeletion(mail.uidl);
    474468                                                }
     469                                                continue;  // found on disk, woo
    475470                                        }
    476471                                        File file = new File(_context.getTempDir(), "susimail-new-" + _context.random().nextLong());
     
    505500                                        }
    506501                                        rv = true;
    507                                         if (disk != null) {
    508                                                 if (disk.saveMail(mail) && mail.hasBody() &&
    509                                                     !Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
    510                                                         mailbox.queueForDeletion(mail.uidl);
    511                                                 }
     502                                        if (disk.saveMail(mail) && mail.hasBody() &&
     503                                            !Boolean.parseBoolean(Config.getProperty(WebMail.CONFIG_LEAVE_ON_SERVER))) {
     504                                                mailbox.queueForDeletion(mail.uidl);
    512505                                        }
    513506                                }
     
    540533                List<String> toDelete = new ArrayList<String>(uidls.size());
    541534                for (String uidl : uidls) {
    542                         if (disk != null)
    543                                 disk.deleteMail(uidl);
     535                        disk.deleteMail(uidl);
    544536                        synchronized(mails) {
    545537                                Mail mail = mails.get(uidl);
Note: See TracChangeset for help on using the changeset viewer.