couldn't we just call mmsAttachmentClick without this if block ? mmsAttachmentClick returns if it finds nothing in the weak map anyway;

And so we don't need the class at all.