peterhaneve / evemon

A lightweight, easy-to-use standalone Windows application designed to assist you in keeping track of your EVE Online character progression.
311 stars 70 forks source link

Sticky Industry Notifications #298

Open wvdvegt opened 3 years ago

wvdvegt commented 3 years ago

The current version accumulates industry notifications without ever removing them when jobs are delivered.

My fix for this is to add this method to GlobalNotificationCollection.cs:

internal void InvalidateIndustryCompleted(CCPCharacter character)
{
        Invalidate(new NotificationInvalidationEventArgs(character, NotificationCategory.IndustryJobsCompletion));
}

and call it from CharacterDataQuerying.cs as most other types (PI etc) do by adding a line:

// Invalidate previous notifications
EveMonClient.Notifications.InvalidateIndustryCompleted(target);

to the OnIndustryJobsUpdated() method (directly after the 'if (target != null)' statement.

wvdvegt commented 3 years ago

After testing this change does not seem to work properly