dgets / DOCshell

Damo's version of the DOC (Citadel-esque) shell, in JavaScript, for use with Synchronet 3.14+ systems
https://www.pivotaltracker.com/n/projects/1206002
4 stars 1 forks source link

Mail> isn't displaying the last message #212

Closed dgets closed 9 years ago

dgets commented 9 years ago

I believe this is probably a bug that I just introduced by improper usage of openNewMBase(), but I'm not totally sure yet. After sending a new message from my primary account, here's what I get when I jump to Mail> and try to scan:

bbs-general-scan-mail-broken-last-msg

dgets commented 9 years ago

It appears that the following bit of code was at the root of this:

        try {
          mmBase.open();
        } catch (e) {
          console.putmsg(red + high_intensity + "Unable to open mmBase\n");
          throw new dDocException("readMail() exception",
            "The cave is too dark to read yr scroll", 1);
        }
        //fuck it, let's give the generic a chance here too
        //msg_base.util.openNewMBase("mail");

at the very start of uMail.readMail(). Not sure why the generic opening handler isn't working for Mail>. This is something that should be able to be fixed with an extra case there, if nothing else. For now I'm just going to leave the commenting in the code as it is above, as that has fixed the problem.