Error always logged on sync complete about quickbooks-sync

[jsgoupil]

@bzbetty Sorry but this seems to be your code logic? Is this coming from your LogMessage ?

[bzbetty]

I'm looking at line 406 of the Sync Manager

quickbooks-sync/src/WebConnector/SyncManager.cs

Line 406 in 1f79516

result = "Sync Completed";

[jsgoupil]

I'm not sure what the problem is? LogMessage does not mean an error happened. LogMessage is called for anything going in or out the API.

[bzbetty]

The LogMessage is called with LogMessageType.Error, just means every completion is coming up in my logs as an error and not an info/debug message.

LogMessage(authenticatedTicket, LogMessageType.Error, LogDirection.Out, ticket, result);

[jsgoupil]

Ah you are right. I misread that... sorry! It should definitely not be logged as an error. Thanks for pointing this out!

[jsgoupil]

I think this would fix it:

try
{
    LogMessage(authenticatedTicket, LogMessageType.Close, LogDirection.In, ticket);

    var logMessageType = LogMessageType.Error;
    var result = "Invalid Ticket";

    if (authenticatedTicket != null)
    {
        result = "Sync Completed";
        logMessageType = LogMessageType.Close;
    }

    LogMessage(authenticatedTicket, logMessageType, LogDirection.Out, ticket, result);

    return result;
}

[bzbetty]

yeah that looks much better. thanks.