Attaching files from Invoice and Memos screen to Email Activity screen

There may be a better way to do this, but this is what I would do:

public delegate IEnumerable SendEmailDelegate(PXAdapter adapter);
[PXOverride]
public IEnumerable SendEmail(PXAdapter adapter, SendEmailDelegate baseMethod)
{
    var invoice = Base.Document.Current;

    // Captures when the email graph is initialized
    PXGraph.InstanceCreated.AddHandler<CREmailActivityMaint>((g) =>
    {
        // Captures when the email is inserted
        g.RowInserting.AddHandler<CRSMEmail>((cache, e) =>
        {
            CRSMEmail row = (CRSMEmail)e.Row;
            if (row == null) return;

            var fileIDs = PXNoteAttribute.GetFileNotes(Base.Document.Cache, invoice);
            foreach (Guid fileID in fileIDs)
            {
                PXTrace.WriteInformation($"{row?.NoteID}");
                PXNoteAttribute.SetFileNotes(cache, row, fileID);
            }
        });
    });

    return baseMethod(adapter);
}

(ignore the weird code formatting)

Oky so I went ahead and debugged the process to see what the problem might be and gathered the following conclusion: 

It seems like the NoteID field is different in the Email Activity screen when the Send Email action is triggered. Here is the NoteID that is referenced by the Email Activity screen : 

(I created a datafield that retrieves the NoteID of this email)

Here is the NoteID that is retrieved in the customization’s code where the SendEmail action is overridden as mentioned in the main topic: 

What could cause the discrepancy ? 

Here is my Graph Extension code for the ARInvoiceEntry graph: 

@dewaldblaauw12 - If I’m reading this right, this will attach all files for any entity related to an email that is created. This could potentially cause a significant bloat to your database. I would suggest at least checking the type of the related entity.

Alternatively, did you try using a RowInserted event handler in my code? I can’t say what the difference would be. I usually use RowPersisting, but that won’t work in this case because the email isn’t being saved during the method.